From b4673c5026a345c81b859e9e9489204eb772fd8f Mon Sep 17 00:00:00 2001 From: Nathan Heskew Date: Mon, 29 Jun 2026 07:32:31 -0600 Subject: [PATCH 1/2] chore(ci): bump ai-review-prompts to 67d7611 (prompt-ref tracking) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Forward bump 1bbc562 -> 67d7611 (ai-review-prompts #71): the log step now records which prompt version produced each ai-review-log entry — a `**Prompt ref:**` body field plus a `prompt:` label — so calibration can attribute verdicts to a specific prompt version instead of a date bucket. No prompt-content change vs 1bbc562 (which already carried the #67 calibration + #69 log-count fix); this turns on per-version tracking. `uses:` and `ai-review-prompts-ref:` move in lockstep. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/claude-review.yml | 4 ++-- .github/workflows/gemini-review.yml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index d06d542..f7b2b35 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -33,7 +33,7 @@ jobs: # Note: the `claude-review` label name is matched there too — # `_claude-review.yml`'s authorize `if:`, not in this caller. if: ${{ vars.CLAUDE_ALWAYS_ON == 'true' || github.event.action == 'labeled' }} - uses: HarperFast/ai-review-prompts/.github/workflows/_claude-review.yml@1bbc562d57d131363586fdde3e893380381f455e # main 2026-06-24 (#67 week-of-06-15 calibration: error/abort-path rule + throwaway/inert-CI-config ignore; #69 log finding-count fix) + uses: HarperFast/ai-review-prompts/.github/workflows/_claude-review.yml@67d7611de07c5ed728c1606d51b8a1523ea094c4 # main 2026-06-29 (#71 record ai-review-prompts ref per log entry; on 1bbc562 = #67 calibration + #69 log-count fix) # Caller-side permissions, scoped at the calling-job level (NOT # workflow-level — that placement caps the reusable's per-job # grants below what they need and breaks the workflow at startup; @@ -58,7 +58,7 @@ jobs: # introspect their own ref (`github.workflow_ref` resolves to the # CALLER's ref in `workflow_call` context), and `uses: …@` # is parsed literally so we can't interpolate a variable. - ai-review-prompts-ref: 1bbc562d57d131363586fdde3e893380381f455e + ai-review-prompts-ref: 67d7611de07c5ed728c1606d51b8a1523ea094c4 review-layers: | universal harper/common diff --git a/.github/workflows/gemini-review.yml b/.github/workflows/gemini-review.yml index 16ab3a6..69fbe30 100644 --- a/.github/workflows/gemini-review.yml +++ b/.github/workflows/gemini-review.yml @@ -45,7 +45,7 @@ jobs: # Note: the `gemini-review` label name is matched there too — # `_gemini-review.yml`'s authorize `if:`, not in this caller. if: ${{ vars.GEMINI_ALWAYS_ON == 'true' || github.event.action == 'labeled' }} - uses: HarperFast/ai-review-prompts/.github/workflows/_gemini-review.yml@1bbc562d57d131363586fdde3e893380381f455e # main 2026-06-24 (#67 week-of-06-15 calibration: error/abort-path rule + throwaway/inert-CI-config ignore; #69 log finding-count fix) + uses: HarperFast/ai-review-prompts/.github/workflows/_gemini-review.yml@67d7611de07c5ed728c1606d51b8a1523ea094c4 # main 2026-06-29 (#71 record ai-review-prompts ref per log entry; on 1bbc562 = #67 calibration + #69 log-count fix) # Caller-side permissions, scoped at the calling-job level (NOT # workflow-level — that placement caps the reusable's per-job # grants below what they need and breaks the workflow at startup; @@ -67,7 +67,7 @@ jobs: # in this repo for why the duplication is unavoidable # (reusable workflows can't introspect their own ref in # workflow_call context). - ai-review-prompts-ref: 1bbc562d57d131363586fdde3e893380381f455e + ai-review-prompts-ref: 67d7611de07c5ed728c1606d51b8a1523ea094c4 review-layers: | universal harper/common From 41f60503f1c30f5ffceb64e8581b3be7f3773b1b Mon Sep 17 00:00:00 2001 From: Nathan Heskew Date: Mon, 29 Jun 2026 07:49:18 -0600 Subject: [PATCH 2/2] chore(ci): re-point bump to 9cf49d2 (add #70 week-of-06-22 calibration) #70 (week-of-06-22 calibration) merged after this bump was opened, so re-point 67d7611 -> 9cf49d2 to land the calibration AND the #71 prompt-ref tracking together in a single bump rather than forcing a second one. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/claude-review.yml | 4 ++-- .github/workflows/gemini-review.yml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index f7b2b35..654d7a3 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -33,7 +33,7 @@ jobs: # Note: the `claude-review` label name is matched there too — # `_claude-review.yml`'s authorize `if:`, not in this caller. if: ${{ vars.CLAUDE_ALWAYS_ON == 'true' || github.event.action == 'labeled' }} - uses: HarperFast/ai-review-prompts/.github/workflows/_claude-review.yml@67d7611de07c5ed728c1606d51b8a1523ea094c4 # main 2026-06-29 (#71 record ai-review-prompts ref per log entry; on 1bbc562 = #67 calibration + #69 log-count fix) + uses: HarperFast/ai-review-prompts/.github/workflows/_claude-review.yml@9cf49d23cb046ecd6c87f3bba86dc3338d6998fb # main 2026-06-29 (#70 week-of-06-22 calibration + #71 prompt-ref tracking; on 1bbc562 = #67 + #69) # Caller-side permissions, scoped at the calling-job level (NOT # workflow-level — that placement caps the reusable's per-job # grants below what they need and breaks the workflow at startup; @@ -58,7 +58,7 @@ jobs: # introspect their own ref (`github.workflow_ref` resolves to the # CALLER's ref in `workflow_call` context), and `uses: …@` # is parsed literally so we can't interpolate a variable. - ai-review-prompts-ref: 67d7611de07c5ed728c1606d51b8a1523ea094c4 + ai-review-prompts-ref: 9cf49d23cb046ecd6c87f3bba86dc3338d6998fb review-layers: | universal harper/common diff --git a/.github/workflows/gemini-review.yml b/.github/workflows/gemini-review.yml index 69fbe30..c860774 100644 --- a/.github/workflows/gemini-review.yml +++ b/.github/workflows/gemini-review.yml @@ -45,7 +45,7 @@ jobs: # Note: the `gemini-review` label name is matched there too — # `_gemini-review.yml`'s authorize `if:`, not in this caller. if: ${{ vars.GEMINI_ALWAYS_ON == 'true' || github.event.action == 'labeled' }} - uses: HarperFast/ai-review-prompts/.github/workflows/_gemini-review.yml@67d7611de07c5ed728c1606d51b8a1523ea094c4 # main 2026-06-29 (#71 record ai-review-prompts ref per log entry; on 1bbc562 = #67 calibration + #69 log-count fix) + uses: HarperFast/ai-review-prompts/.github/workflows/_gemini-review.yml@9cf49d23cb046ecd6c87f3bba86dc3338d6998fb # main 2026-06-29 (#70 week-of-06-22 calibration + #71 prompt-ref tracking; on 1bbc562 = #67 + #69) # Caller-side permissions, scoped at the calling-job level (NOT # workflow-level — that placement caps the reusable's per-job # grants below what they need and breaks the workflow at startup; @@ -67,7 +67,7 @@ jobs: # in this repo for why the duplication is unavoidable # (reusable workflows can't introspect their own ref in # workflow_call context). - ai-review-prompts-ref: 67d7611de07c5ed728c1606d51b8a1523ea094c4 + ai-review-prompts-ref: 9cf49d23cb046ecd6c87f3bba86dc3338d6998fb review-layers: | universal harper/common