Skip to content

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

Open
pullfrog[bot] wants to merge 15 commits into
mainfrom
pullfrog/test-pkg-postinstall-canary
Open

chore: update package.json for postinstall canary test#22
pullfrog[bot] wants to merge 15 commits into
mainfrom
pullfrog/test-pkg-postinstall-canary

Conversation

@pullfrog
Copy link
Copy Markdown

@pullfrog pullfrog Bot commented May 1, 2026

Captures the package.json modification made during the dependency installation behavior test. The file was updated to test-pkg with a postinstall script as part of the test task.

Pullfrog  | via Pullfrog | Using Claude Opus𝕏


Note

Medium Risk
Adds a postinstall script that writes to /tmp, which will execute on dependency install and can affect CI/build environments despite the small diff.

Overview
Updates package.json to rename the package to test-pkg, add a version, and replace the test script with a postinstall canary that writes CANARY_MARKER to /tmp/postinstall-canary.txt during installs.

Also removes private/type fields and leaves an empty dependencies block.

Reviewed by Cursor Bugbot for commit 1379474. 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 d26668d. 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.

Test artifact overwrites production package.json configuration

High Severity

The real package.json has been entirely replaced by a test fixture. This removes "private": true (risking accidental npm publish), removes "type": "module" (breaking ES module resolution for the existing .ts source files), and replaces the "test": "vitest run" script with a postinstall canary that writes to /tmp. The repository still contains real source and test files that depend on the original configuration.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit d26668d. 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