Repository navigation
ci(v1.x): sync bot caller workflows with main - #217
Conversation
claude-code-action validates a PR's caller workflow against the default branch, so drifted copies on v1.x fail every review on v1.x-targeted PRs. Content matches the bump-ai-review-prompts-pin branch (oauth#216) — identical to main once that merges. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Reviewed; no blockers found. |
|
2 blockers found. Prior 2 finding(s) still unresolved. 1. Missing permissions for the auth gate and agent editsFile: .github/workflows/claude-mention.yml:35 permissions:
contents: write
pull-requests: write
issues: write
id-token: write2. Missing permissions for the auth gate and PR creationFile: .github/workflows/claude-issue-to-pr.yml:33 permissions:
contents: write
pull-requests: write
issues: write
id-token: write |
…review trigger Draft PRs skip review until flipped ready (label still opts one in), mechanical diffs skip pre-run, reasoning effort scales with diff size (60/high, 1500/xhigh, else max), synchronize runs debounce 120s. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt
|
Dispositions for the 2 review blockers (both the same class): Declined on this PR by construction. #217 is a byte-identity sync of main's caller files onto On the substance, the findings also over-call: the mention/issue-to-pr callers have never carried a The kernel worth keeping: making the grants caller-explicit (as the review callers already do) removes the dependence on the repo-default setting. Queued as a small fleet-wide follow-up against main. |
claude-code-action validates a PR's caller workflows against the default branch, so the drifted copies on
v1.xfail the review check on every v1.x-targeted PR. This syncs all five callers to the content of #218 (pinbe549ad+ready_for_reviewtriggers) — byte-identical to main once #218 merges, so merge #218 first, then this.Going forward: any pin bump on main re-drifts v1.x until re-synced.
🤖 Generated with Claude Code
https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt