From e1c7463e86d72af2ee51fd59485ab500074dfe25 Mon Sep 17 00:00:00 2001
From: Kyle Sexton <153232337+kyle-sexton@users.noreply.github.com>
Date: Sat, 24 Oct 2026 11:13:25 -0400
Subject: [PATCH] feat(runner-policy): register the ci-workflows v0.65.6
 pr-review-cli contract

Copies the v0.65.5 entry forward with a copy-forward test, as #871 did.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 components/runner-policy/policy.json          | 21 +++++++++++++++++++
 .../runner-policy/runner-policy.test.mjs      | 12 +++++++++++
 2 files changed, 33 insertions(+)

diff --git a/components/runner-policy/policy.json b/components/runner-policy/policy.json
index f62a92b9..24439b5a 100644
--- a/components/runner-policy/policy.json
+++ b/components/runner-policy/policy.json
@@ -1032,6 +1032,27 @@
         "CLAUDE_CODE_OAUTH_TOKEN": "${{ secrets.REVIEWS_CLAUDE_CODE_OAUTH_TOKEN }}"
       }
     },
+    "melodic-software/ci-workflows/.github/workflows/pr-review-cli.yml@914264587c35692c15e05c965103426eec18d1f0": {
+      "routing": "hosted-only",
+      "fixedRunsOn": ["ubuntu-24.04"],
+      "allowedInputs": [
+        "lane",
+        "publish",
+        "role",
+        "model-args",
+        "allowed-bot-sender-ids",
+        "allowed-author-ids",
+        "docs-only-paths",
+        "incremental-review",
+        "cli-version",
+        "hold-label",
+        "hold-paths"
+      ],
+      "allowedSecrets": {
+        "CLAUDE_API_KEY": "${{ secrets.REVIEWS_CLAUDE_API_KEY }}",
+        "CLAUDE_CODE_OAUTH_TOKEN": "${{ secrets.REVIEWS_CLAUDE_CODE_OAUTH_TOKEN }}"
+      }
+    },
     "melodic-software/ci-workflows/.github/workflows/intake-label-needs-triage.yml@ab83b01273026ab5c23c6f3b40e946d97a863fa2": {
       "routing": "runner-input",
       "runnerInput": "runner",
diff --git a/components/runner-policy/runner-policy.test.mjs b/components/runner-policy/runner-policy.test.mjs
index ea134f80..a1fbdb95 100644
--- a/components/runner-policy/runner-policy.test.mjs
+++ b/components/runner-policy/runner-policy.test.mjs
@@ -10020,6 +10020,18 @@ test("the pr-review-cli contract at the v0.65.5 tag copies its v0.65.4 entry for
   assert.equal(Object.keys(contracts).filter((k) => k.endsWith(`@${sha}`)).length, 1);
 });
 
+// v0.65.6 changes only the hold step's cost-decision carry-over
+// (ci-workflows#879): a decision carries no further once a pull request
+// comment was deleted since its hold marker. Inputs, secrets and runner are
+// unchanged, so the contract copies forward.
+test("the pr-review-cli contract at the v0.65.6 tag copies its v0.65.5 entry forward verbatim", () => {
+  const contracts = BASE_POLICY.approvedReusableWorkflowContracts;
+  const key = (sha) => `melodic-software/ci-workflows/.github/workflows/pr-review-cli.yml@${sha}`;
+  const sha = "914264587c35692c15e05c965103426eec18d1f0";
+  assert.deepEqual(contracts[key(sha)], contracts[key("f1096608192c54cfb3dc252e36670e6f86700b9b")]);
+  assert.equal(Object.keys(contracts).filter((k) => k.endsWith(`@${sha}`)).length, 1);
+});
+
 // Convergence as a property, never as a hardcoded SHA. The daily
 // claude-lanes-repin lane rewrites these components on every ci-workflows
 // release, and a test naming one revision would go red by construction on