Skip to content

chore: update package.json for postinstall canary test#35

Open
pullfrog[bot] wants to merge 1 commit into
mainfrom
pullfrog/test-pkg-postinstall-2
Open

chore: update package.json for postinstall canary test#35
pullfrog[bot] wants to merge 1 commit into
mainfrom
pullfrog/test-pkg-postinstall-2

Conversation

@pullfrog
Copy link
Copy Markdown

@pullfrog pullfrog Bot commented May 15, 2026

Updates package.json as part of the dependency installation behavior test — sets name, version, adds a postinstall canary script, and clears dependencies.

Pullfrog  | via Pullfrog | Using Claude Sonnet𝕏


Note

Medium Risk
Adds a postinstall script that writes to /tmp, which will execute automatically during installs and can affect CI/build environments. Other changes are limited to package metadata and removing the test script/dependencies.

Overview
Updates package.json to rename the package and add a version, and removes the previous test script/fields.

Adds a postinstall canary that writes CANARY_MARKER to /tmp/postinstall-canary.txt, and sets dependencies to an empty object.

Reviewed by Cursor Bugbot for commit b941043. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b941043. Configure here.

Comment thread package.json
}
"postinstall": "echo CANARY_MARKER > /tmp/postinstall-canary.txt"
},
"dependencies": {}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing private: true risks accidental npm publish

Medium Severity

The "private": true field was removed from package.json without being mentioned in the PR description. This safety guard prevents accidental publication to the npm registry. Without it, running npm publish (even accidentally) could push test-pkg@1.0.0 to the public npm registry — potentially causing a supply-chain concern or namespace squatting. Since the package is a test fixture and not meant to be published, "private": true is worth keeping.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b941043. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants