Skip to content

Commit 8295196

Browse files
xiaolaiavifenesh
andauthored
fix(version): scope git add to files stamp-version.js writes (#339)
Replaces 'git add -A' in the npm version lifecycle script with an explicit allowlist: - package.json - package-lock.json (npm version writes this) - .claude-plugin/plugin.json - .claude-plugin/marketplace.json - site/content.json Original PR #339 from xiaolai (NLPM audit) flagged that 'git add -A' sweeps any unrelated working-tree changes into the version commit. This is the broadened-allowlist version per Copilot's review suggestion - preserves the intent (no working-tree sweep) while keeping all version manifests in sync. Co-authored-by: Avi Fenesh <aviarchi1994@gmail.com> Co-authored-by: xiaolai <xiaolai@users.noreply.github.com>
1 parent f369ac4 commit 8295196

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
"bump": "node bin/dev-cli.js bump",
3838
"detect": "node bin/dev-cli.js detect",
3939
"verify": "node bin/dev-cli.js verify",
40-
"version": "node scripts/stamp-version.js && git add -A",
40+
"version": "node scripts/stamp-version.js && git add package.json package-lock.json .claude-plugin/plugin.json .claude-plugin/marketplace.json site/content.json",
4141
"setup-hooks": "node bin/dev-cli.js setup-hooks"
4242
},
4343
"repository": {

0 commit comments

Comments
 (0)