Skip to content

test: dependency installation canary test#14

Open
pullfrog[bot] wants to merge 3 commits intomainfrom
pullfrog/dep-install-test
Open

test: dependency installation canary test#14
pullfrog[bot] wants to merge 3 commits intomainfrom
pullfrog/dep-install-test

Conversation

@pullfrog
Copy link
Copy Markdown

@pullfrog pullfrog Bot commented Apr 10, 2026

Commits the package.json written during the dependency installation behavior test. This file was modified as part of a canary test to verify postinstall script execution behavior.

Pullfrog  | Triggered by Pullfrog | Using Claude Opus𝕏


Note

Medium Risk
Adds a postinstall script that executes on dependency installation and writes a marker file to /tmp, which can affect CI/build environments and has supply-chain style implications despite the small scope.

Overview
Updates package.json to rename the package and make it publishable (name/version), removes the existing vitest test script, and adds a postinstall canary script that writes CANARY_MARKER to /tmp/postinstall-canary.txt during installs.

Reviewed by Cursor Bugbot for commit 4d9d5e9. 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 4d9d5e9. 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 real project package.json

High Severity

The canary test's package.json output has been committed over the project's real package.json. This removes "test": "vitest run" (breaking npm test for the existing test/math.test.ts), removes "type": "module" (breaking ES module imports used throughout src/ and test/), removes "private": true (making the package accidentally publishable), and adds a postinstall script that writes to /tmp on every npm install.

Fix in Cursor Fix in Web

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