-
Notifications
You must be signed in to change notification settings - Fork 0
test: add postinstall canary script to package.json #17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
dbb0899
899475e
5e99cc6
3237810
0ef09ee
ca0c746
0bf0d43
4ecd60b
c0d3ac5
cdeb2b6
35b11d4
91f633f
6743a95
b7dbd22
c9b9ab2
213038b
6e18958
a0bde76
5443ec1
474ed91
ede16b6
6e2fc90
7a7391a
ca7d2f5
47995e9
9e62c8e
d5701e3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| { | ||
| "name": "test-repo", | ||
| "private": true, | ||
| "type": "module", | ||
| "name": "test-pkg", | ||
| "version": "1.0.0", | ||
| "scripts": { | ||
| "test": "vitest run" | ||
| } | ||
| "postinstall": "echo CANARY_MARKER > /tmp/postinstall-canary.txt" | ||
| }, | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Test script removed while test files still existMedium Severity The Reviewed by Cursor Bugbot for commit 5e99cc6. Configure here. |
||
| "dependencies": {} | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removal of
|
||
| } | ||


There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Postinstall script is test code committed to production
High Severity
The
postinstallscript writing a canary marker file to/tmp/postinstall-canary.txtis test/debugging infrastructure that executes a shell command on everynpm install. This runs automatically for anyone installing this package or its dependents, writing to the filesystem as a side effect of installation. Even if created for "dependency installation behavior testing," this has no place in the committedpackage.json.Reviewed by Cursor Bugbot for commit 5e99cc6. Configure here.