test: write package.json for dependency install canary test#15
test: write package.json for dependency install canary test#15pullfrog[bot] wants to merge 21 commits into
Conversation
…pullfrog/test-dep-install
…pullfrog/test-dep-install
| } | ||
| "postinstall": "echo CANARY_MARKER > /tmp/postinstall-canary.txt" | ||
| }, | ||
| "dependencies": {} |
There was a problem hiding this comment.
Test package.json overwrites real project configuration
High Severity
The root package.json appears to have been overwritten with a canary-test-only version. This removes "private": true (risking accidental npm publish), "type": "module" (breaking existing ES module imports in src/ and test/), and the "test": "vitest run" script (breaking npm test). The existing test/math.test.ts imports from vitest and uses ES module syntax, which will no longer work. Per the PR description, this file was "overwritten as part of the test task" — it looks like a test artifact that was accidentally committed.
Reviewed by Cursor Bugbot for commit ab76b76. Configure here.
…pullfrog/test-dep-install
…pullfrog/test-dep-install
…pullfrog/test-dep-install
| "requires": true, | ||
| "packages": { | ||
| "": { | ||
| "name": "test-repo" |
There was a problem hiding this comment.
Package name mismatch between lockfile and manifest
Medium Severity
The package-lock.json uses "name": "test-repo" (in both the root and the packages[""] entry) while package.json was changed to "name": "test-pkg". This mismatch means npm will detect an inconsistency and may regenerate the lockfile on install, which could interfere with the deterministic behavior expected by the canary test.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 17fb087. Configure here.
…pullfrog/test-dep-install
…pullfrog/test-dep-install
…pullfrog/test-dep-install
…pullfrog/test-dep-install
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 3 total unresolved issues (including 2 from previous reviews).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 767a1ad. Configure here.
| "name": "test-repo", | ||
| "private": true, | ||
| "type": "module", | ||
| "name": "test-pkg", |
There was a problem hiding this comment.
Name mismatch between package.json and package-lock.json
Medium Severity
The package.json declares "name": "test-pkg" while the new package-lock.json declares "name": "test-repo". This mismatch causes npm to emit warnings and may lead to unexpected behavior during dependency installation—the very operation this canary test is meant to verify.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 767a1ad. Configure here.


This PR captures the
package.jsonchange made during a dependency installation behavior test. The file was overwritten as part of the test task to verify postinstall script execution viastart_dependency_installation/await_dependency_installation.Claude Opus| 𝕏Note
Medium Risk
Adds a
postinstallscript that writes to/tmp, which will execute automatically during dependency installs and can affect CI/host environments. Otherwise changes are small and isolated to npm metadata files.Overview
Updates
package.jsonto rename the package totest-pkg, add aversion, and replace the test script with apostinstallcanary that writesCANARY_MARKERto/tmp/postinstall-canary.txt.Adds a minimal
package-lock.jsonto accompany the updated npm package metadata.Reviewed by Cursor Bugbot for commit e14ecf2. Bugbot is set up for automated code reviews on this repo. Configure here.