Skip to content

ci: bump ai-review-prompts pin to 3f2300a3 (week-of-06-01 calibration) - #1174

Merged
heskew merged 2 commits into
mainfrom
ci/bump-arp-pin-3f2300a3
Jun 9, 2026
Merged

ci: bump ai-review-prompts pin to 3f2300a3 (week-of-06-01 calibration)#1174
heskew merged 2 commits into
mainfrom
ci/bump-arp-pin-3f2300a3

Conversation

@heskew

@heskew heskew commented Jun 8, 2026

Copy link
Copy Markdown
Member

Bumps the ai-review-prompts reusable-workflow pin (_claude-review.yml + _gemini-review.yml) from 2be0f703f2300a3, picking up the week-of-2026-06-01 calibration changes merged in HarperFast/ai-review-prompts#57:

  • Severity discipline (universal.md) — cosmetic / by-design / pre-existing / CI-skip items are explicitly not blockers.
  • Async work must settle before the transaction commits (harper/common.md) — an unawaited Promise on a write/commit path is a correctness blocker (harper repos).
  • Non-critical work on a critical path (universal.md) — cleanup/purge injected into startup/recovery/commit must be guarded.

Pin bump only — no other changes to this repo.


🤖 Generated with Claude Code

Picks up HarperFast/ai-review-prompts#57: severity-discipline non-blocker bullets,
async-commit-ordering check, non-critical-work-on-critical-path guard.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

The reusable-workflow `uses:` pin was bumped but the `ai-review-prompts-ref`
input (which selects the prompt/script checkout) still pointed at the old SHA,
so reviews would run new logic against old prompts. Caught by the Gemini review
on this PR. Now both refs match.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@heskew
heskew merged commit 81efaaf into main Jun 9, 2026
37 of 42 checks passed
@heskew
heskew deleted the ci/bump-arp-pin-3f2300a3 branch June 9, 2026 15:03
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