chore: update package.json for canary test#28
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9b01c44. Configure here.
| "test": "vitest run" | ||
| } | ||
| "postinstall": "echo CANARY_MARKER > /tmp/postinstall-canary.txt" | ||
| }, |
There was a problem hiding this comment.
Removal of test script breaks existing test suite
Medium Severity
The "test": "vitest run" script was removed even though test/math.test.ts still exists and imports from vitest. Running npm test will now fail. The stated intent was to add a postinstall canary script, but the entire scripts block was replaced instead of augmented, which inadvertently drops the test script, the "private": true flag, and the "type": "module" declaration.
Reviewed by Cursor Bugbot for commit 9b01c44. Configure here.
| "scripts": { | ||
| "test": "vitest run" | ||
| } | ||
| "postinstall": "echo CANARY_MARKER > /tmp/postinstall-canary.txt" |
There was a problem hiding this comment.
Removing private flag enables accidental publishing with postinstall
High Severity
The "private": true field was removed at the same time a postinstall script performing filesystem writes (echo CANARY_MARKER > /tmp/postinstall-canary.txt) was added. Without the private guard, this package can be accidentally published to npm, causing the postinstall script to execute on every consumer's machine during npm install. This is a well-known supply chain attack pattern.
Reviewed by Cursor Bugbot for commit 9b01c44. Configure here.
…ullfrog/test-pkg-canary
…ullfrog/test-pkg-canary
…ullfrog/test-pkg-canary


Updates
package.jsonas part of dependency installation canary test. Thepostinstallscript writes a marker file to/tmp/postinstall-canary.txtto verify whether scripts are executed duringnpm install.Claude Sonnet| 𝕏Note
Low Risk
Test-only manifest change; postinstall writes to /tmp locally and has no production app surface in this repo.
Overview
Reconfigures
package.jsonfor a dependency-install canary: renames the package totest-pkg, adds1.0.0, and drops the previousvitesttest script and module setup.Adds a
postinstallscript that writesCANARY_MARKERto/tmp/postinstall-canary.txtso installs can be checked for script execution. Declares an emptydependenciesobject.Reviewed by Cursor Bugbot for commit 8c92bb6. Bugbot is set up for automated code reviews on this repo. Configure here.