Skip to content

ci(claude): bump ai-review-prompts pin to 6463b3da#492

Merged
heskew merged 1 commit intomainfrom
ci/bump-ai-review-prompts-6463b3da
May 7, 2026
Merged

ci(claude): bump ai-review-prompts pin to 6463b3da#492
heskew merged 1 commit intomainfrom
ci/bump-ai-review-prompts-6463b3da

Conversation

@heskew
Copy link
Copy Markdown
Member

@heskew heskew commented May 7, 2026

Summary

Bumps all four caller workflow files to ai-review-prompts `6463b3da` (post-#19). Picks up:

  • `:test` validation now runs `npm run format:check` (parity with the other label scopes — `:test` was the only one missing it). The first `:test` PR on this repo (Add sinon/rewire unit tests for OCSP verification #490) hit a Format Check failure the agent didn't catch before pushing; this fixes that gap for the retry.

ai-review-prompts `#18` (App-token push so bot-authored commits retrigger downstream CI) intentionally not included — that's gated on adding `Contents: Write` + `Pull requests: Write` to the `HarperFast AI Workflows` App's installation permissions, and we're parking it on the side. Without that fix, bot commits still won't retrigger CI; close + reopen remains the workaround.

Test plan

  • Caller-validator script (from ai-review-prompts main) dry-runs cleanly.
  • All four caller files reference `6463b3da6326f0ca4646fb6b5139c2ecf92130f6` in both `uses:` and `with.ai-review-prompts-ref`.
  • After merge: re-label the test issue with `claude-fix:test` and verify the agent format-checks before push.

🤖 Generated with Claude Code

Picks up `claude-fix:test` validation now running `npm run format:check`
(parity with the other label scopes — :test was the outlier). Surfaced
when the first :test PR on this repo (#490) hit a Format Check failure
the agent didn't catch before pushing.

ai-review-prompts #18 (App-token push so bot commits retrigger CI)
intentionally not in this bump — gated on the HarperFast AI Workflows
App needing `Contents: Write` + `Pull requests: Write` added.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@heskew heskew requested review from a team as code owners May 7, 2026 17:34
@heskew heskew merged commit a44d3cc into main May 7, 2026
13 of 14 checks passed
@heskew heskew deleted the ci/bump-ai-review-prompts-6463b3da branch May 7, 2026 17:35
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