Skip to content

ci: update Claude dependency review workflow - #27

Merged
Nsttt merged 1 commit into
mainfrom
ci/claude-dependency-approvals
Aug 28, 2026
Merged

Nsttt merged 1 commit into
mainfrom
ci/claude-dependency-approvals

Conversation

@Nsttt

@Nsttt Nsttt commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

Updates the organization Claude workflow to the current stable action and Sonnet model, replaces deprecated action inputs, and tightens both review prompts.

Claude may autonomously approve only Dependabot or Renovate PRs. The model returns a structured decision; fixed workflow steps enforce the bot identity, authenticated event actor, non-draft state, same-repository head, exact reviewed SHA, and verified bot-only commits before submitting approval.

Changes

  • Pin anthropics/claude-code-action v1.0.209 and use Claude Sonnet 5
  • Use action defaults for summary reviews; explicitly grant only inline-comment MCP access for full reviews
  • Keep workflow credentials and approval commands outside the model process
  • Route dependency bots through pull_request_target without checking out PR-head code
  • Dismiss earlier workflow approvals before reviewing a new bot PR state
  • Preserve one updated summary comment and include prior inline comments for deduplication

Security note

The approval decision intentionally remains model-driven, per the requested behavior. Before marking this PR ready, consuming repositories should enforce dismissal of stale approvals or require approval of the latest push. A workflow cannot make approval invalidation atomic with a head update by itself.

Testing

  • actionlint .github/workflows/claude.yml
  • git diff --check
  • Live GitHub API provenance checks against Zephyr Dependabot and Renovate PRs
  • Negative provenance fixture rejecting a non-bot commit author
  • Structured pre-ship review; actionable workflow defects fixed, model-authorization tradeoff documented above

@Nsttt Nsttt self-assigned this Aug 28, 2026
@Nsttt
Nsttt marked this pull request as ready for review August 28, 2026 18:05
@Nsttt
Nsttt merged commit 7ceb393 into main Aug 28, 2026
9 checks passed
@Nsttt
Nsttt deleted the ci/claude-dependency-approvals branch August 28, 2026 18:08
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.

2 participants