Skip to content

chore(security): pin claude-code-action to known-good v1.0.174 - #3

Merged
abir-vim merged 1 commit into
mainfrom
chore/scanner-harden
Jul 20, 2026
Merged

chore(security): pin claude-code-action to known-good v1.0.174#3
abir-vim merged 1 commit into
mainfrom
chore/scanner-harden

Conversation

@abir-vim

Copy link
Copy Markdown
Collaborator

Automated fleet sync of the org security scanner. Pins claude-code-action off the floating @v1 tag (which regressed at v1.0.175+ and silently no-ops on Bedrock, failing the scan gate closed) to the known-good SHA. See scanner-org for the source template.

@github-actions

Copy link
Copy Markdown

Security Scanner Results

Critical High Medium Low Info Dep BLOCK Dep WARN
0 0 0 1 1 0 0

Findings

  • 🔵 Security scanner workflow now allows all bot-authored PRs to trigger it (allowed_bots: "*").github/workflows/security.yml:170
    • Remediation: Prefer an explicit allowlist (e.g. allowed_bots: "renovate[bot],dependabot[bot]") over "*" so only the intended automation can trigger the scanner. If a wildcard is intentional for org-wide reuse, keep the accompanying comment documenting the private-repo + fixed-prompt assumptions so the trust model stays visible.
  • Scanner LLM reads attacker-influenced PR diff/body via git diff and gh pr view (prompt-injection surface).github/workflows/security.yml:174
    • Remediation: No change required for this PR. Keep the model's tool allowlist read-only and its output confined to findings.json (already the case), and continue rendering PR comments deterministically rather than letting the model post. Optionally add a note in the prompt reminding the model that diff/PR-body text is untrusted data, not instructions.

Gate blocks at severity critical or a dependency marked BLOCK. Full history in scanner-stats.

@abir-vim
abir-vim merged commit 686e495 into main Jul 20, 2026
3 checks passed
@abir-vim
abir-vim deleted the chore/scanner-harden branch July 20, 2026 11:53
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.

1 participant