Skip to content

Commit 8550123

Browse files
BenBtgCopilot
andcommitted
fix(bug-fix): tighten bash allowlist and block protected files
Address Copilot review feedback on PR #3258: - Trim tools.bash to the inspect set plus a small test-runner set (pytest, npm, go, cargo, dotnet), dropping package-manager/build tools (pip, npx, pnpm, yarn, mvn, gradle, make, bundle, rake, ruby, node) to reduce blast radius under prompt injection. - Set create-pull-request.protected-files.policy: blocked so edits to sensitive files (dependency manifests, README/CHANGELOG/SECURITY, etc.) block PR creation, matching the stronger contract used by the other PR-creating workflows in this repo. Refs #3238 Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 5f02a86 commit 8550123

2 files changed

Lines changed: 9 additions & 18 deletions

File tree

.github/workflows/bug-fix.lock.yml

Lines changed: 6 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/bug-fix.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ on:
1010

1111
tools:
1212
edit:
13-
bash: ["echo", "cat", "head", "tail", "grep", "wc", "sort", "uniq", "python3", "jq", "date", "ls", "find", "pytest", "pip", "npm", "npx", "node", "pnpm", "yarn", "go", "cargo", "make", "mvn", "gradle", "dotnet", "bundle", "rake", "ruby"]
13+
bash: ["echo", "cat", "head", "tail", "grep", "wc", "sort", "uniq", "python3", "jq", "date", "ls", "find", "pytest", "npm", "go", "cargo", "dotnet"]
1414
github:
1515
toolsets: [issues, repos]
1616
min-integrity: none
@@ -31,6 +31,8 @@ safe-outputs:
3131
labels: [bug-fix, automated]
3232
draft: true
3333
max: 1
34+
protected-files:
35+
policy: blocked
3436
add-comment:
3537
max: 1
3638
add-labels:

0 commit comments

Comments
 (0)