@rightkit/git 0.2.22 → 0.2.24

package/manifest.test.mjs

@@ -32,7 +32,14 @@ test("rust-hybrid supports ci plus a configured unsigned release-candidate lane"
   const candidate = {
     ...valid,
     lanes: ["ci", "release-candidate"],
-    candidate: { buildScript: "ci:candidate", checkScript: "ci:candidate-check", artifactRoot: "legion-unsigned", os: ["ubuntu-latest"] },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "legion-unsigned",
+      os: ["ubuntu-latest"],
+    },
   };
   assert.equal(validateManifest(candidate).valid, true);
   assert.equal(validateManifest({ ...candidate, candidate: { ...candidate.candidate, signWindows: true } }).valid, false);
@@ -53,6 +60,7 @@ test("rust-hybrid supports ci plus a configured unsigned release-candidate lane"
     candidate: {
       ...windowsCandidate.candidate,
       releaseChain: {
+        evidenceVerificationScript: "release:verify-evidence",
         windowsFinalizeScript: "release:finalize:win",
         installedQualificationScript: "release:qualify:installed",
         publishScript: "release:publish:qualified",
@@ -68,6 +76,103 @@ test("rust-hybrid supports ci plus a configured unsigned release-candidate lane"
   assert.equal(validateManifest({ ...releaseChain, candidate: { ...releaseChain.candidate, releaseChain: { ...releaseChain.candidate.releaseChain, publishSecrets: ["TOKEN", "TOKEN"] } } }).valid, false);
 });
 
+// Part B fix: admissionScript & stageSummaryScript were moved OUT of the
+// optional candidate.releaseChain object and onto candidate directly,
+// because the admission job and the candidate job's stage-summary
+// init/finalize steps are unconditional parts of the rendered
+// release-candidate template — present for every release-candidate lane,
+// signed or not. This test proves each is independently required with a
+// specific, actionable error, and that the full releaseChain contract
+// (now minus those two promoted fields) is validated key-by-key.
+test("rust-hybrid release-candidate requires admissionScript & stageSummaryScript independent of signing", () => {
+  const base = {
+    ...VALID_MINIMAL,
+    profile: "rust-hybrid",
+    gate: { path: "scripts/gate.sh" },
+    lanes: ["ci", "release-candidate"],
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["ubuntu-latest"],
+    },
+  };
+  assert.equal(validateManifest(base).valid, true, JSON.stringify(validateManifest(base).errors));
+
+  const { admissionScript, ...withoutAdmission } = base.candidate;
+  const missingAdmission = validateManifest({ ...base, candidate: withoutAdmission });
+  assert.equal(missingAdmission.valid, false);
+  assert.ok(missingAdmission.errors.includes("candidate.admissionScript must be one safe package script name"));
+
+  const { stageSummaryScript, ...withoutStageSummary } = base.candidate;
+  const missingStageSummary = validateManifest({ ...base, candidate: withoutStageSummary });
+  assert.equal(missingStageSummary.valid, false);
+  assert.ok(missingStageSummary.errors.includes("candidate.stageSummaryScript must be one safe package script name"));
+
+  // Neither field is accepted inside releaseChain any more.
+  const windows = {
+    ...base,
+    matrix: { ...base.matrix, os: ["ubuntu-latest", "windows-2025"] },
+    candidate: { ...base.candidate, os: ["windows-2025"], signWindows: true },
+  };
+  const misplaced = validateManifest({
+    ...windows,
+    candidate: {
+      ...windows.candidate,
+      releaseChain: {
+        admissionScript: "release:admit",
+        evidenceVerificationScript: "release:verify-evidence",
+        windowsFinalizeScript: "release:finalize:win",
+        installedQualificationScript: "release:qualify:installed",
+        publishScript: "release:publish:qualified",
+        qualificationRunnerVariable: "QUALIFICATION_RUNNER_LABEL",
+      },
+    },
+  });
+  assert.equal(misplaced.valid, false);
+  assert.ok(misplaced.errors.includes("unknown candidate.releaseChain field: admissionScript"));
+});
+
+// Part B fix: evidenceVerificationScript is the one new script that DOES
+// stay inside releaseChain (it runs in the `publish` job, verifying
+// aggregated same-run evidence before the publish step) — required
+// whenever releaseChain is configured, with its own specific error.
+test("candidate.releaseChain requires evidenceVerificationScript with a specific error", () => {
+  const windows = {
+    ...VALID_MINIMAL,
+    profile: "rust-hybrid",
+    gate: { path: "scripts/gate.sh" },
+    lanes: ["ci", "release-candidate"],
+    matrix: { ...VALID_MINIMAL.matrix, os: ["ubuntu-latest", "windows-2025"] },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["windows-2025"],
+      signWindows: true,
+      releaseChain: {
+        windowsFinalizeScript: "release:finalize:win",
+        installedQualificationScript: "release:qualify:installed",
+        publishScript: "release:publish:qualified",
+        qualificationRunnerVariable: "QUALIFICATION_RUNNER_LABEL",
+      },
+    },
+  };
+  const result = validateManifest(windows);
+  assert.equal(result.valid, false);
+  assert.ok(result.errors.includes("candidate.releaseChain.evidenceVerificationScript must be one safe package script name"));
+
+  const fixed = {
+    ...windows,
+    candidate: { ...windows.candidate, releaseChain: { ...windows.candidate.releaseChain, evidenceVerificationScript: "release:verify-evidence" } },
+  };
+  assert.equal(validateManifest(fixed).valid, true, JSON.stringify(validateManifest(fixed).errors));
+});
+
 test("candidate targets permit native multi-architecture matrices independent of normal CI runners", () => {
   const multiArchitecture = {
     ...VALID_MINIMAL,
@@ -77,6 +182,8 @@ test("candidate targets permit native multi-architecture matrices independent of
     candidate: {
       buildScript: "ci:candidate",
       checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
       artifactRoot: "widgets-unsigned",
       os: ["windows-2025", "windows-11-arm", "macos-15", "macos-15-intel"],
       targets: [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rightkit/git",
-  "version": "0.2.22",
+  "version": "0.2.24",
   "description": "Public-repo-only GitHub Actions workflow layer: templates, visibility gate, sync/drift/adopt/uninstall for right-git.",
   "license": "MIT OR Apache-2.0",
   "type": "module",
@@ -15,6 +15,9 @@
     "*.json"
   ],
   "sideEffects": false,
+  "scripts": {
+    "test": "node --test *.test.mjs"
+  },
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"
@@ -24,7 +27,5 @@
     "url": "git+https://github.com/bogusyogi/claude.git",
     "directory": "tools/rightkit/packages/git"
   },
-  "scripts": {
-    "test": "node --test *.test.mjs"
-  }
-}
+  "packageManager": "pnpm@11.24.0"
+}

package/render.test.mjs

@@ -61,9 +61,10 @@ test("rust-hybrid ci renders its pinned package manager and gate", () => {
   assert.match(content, /--diff-filter=ACDMRTUXB/, "deleted Rust inputs must trigger Rust setup");
   assert.match(content, /\*\.rs\|Cargo\.toml[\s\S]*scripts\/gate\.sh/);
   assert.match(content, /\.rightgit\.json\|\.github\/workflows\/\*\.yml\|\.github\/workflows\/\*\.yaml/);
-  assert.equal((content.match(/if: steps\.rust-changes\.outputs\.changed == 'true'/g) ?? []).length, 3);
+  assert.equal((content.match(/if: steps\.rust-changes\.outputs\.required == 'true'/g) ?? []).length, 3);
   assert.match(content, /RIGHT_GIT_RUST_CHANGED: \$\{\{ steps\.rust-changes\.outputs\.changed \}\}/);
-  assert.match(content, /RUSTC_WRAPPER: \$\{\{ steps\.rust-changes\.outputs\.changed == 'true' && 'sccache' \|\| '' \}\}/);
+  assert.match(content, /RIGHT_GIT_RUST_REQUIRED: \$\{\{ steps\.rust-changes\.outputs\.required \}\}/);
+  assert.match(content, /RUSTC_WRAPPER: \$\{\{ steps\.rust-changes\.outputs\.required == 'true' && 'sccache' \|\| '' \}\}/);
   assert.match(content, /CARGO_INCREMENTAL: "0"/);
   assert.match(content, /CARGO_PROFILE_DEV_DEBUG: "0"/);
   assert.match(content, /CARGO_PROFILE_TEST_DEBUG: "0"/);
@@ -100,7 +101,14 @@ test("rust-hybrid release candidate renders product scripts, artifact handoff, a
     lanes: ["ci", "release-candidate"],
     profile: "rust-hybrid",
     gate: { path: "scripts/gate.sh" },
-    candidate: { buildScript: "ci:candidate", checkScript: "ci:candidate-check", artifactRoot: "widgets-unsigned", os: ["macos-latest", "windows-latest"] },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["macos-latest", "windows-latest"],
+    },
     templateVersion: "1.2.0",
   };
   const content = renderTemplate(
@@ -109,6 +117,8 @@ test("rust-hybrid release candidate renders product scripts, artifact handoff, a
   );
   assert.match(content, /pnpm run ci:candidate/);
   assert.match(content, /pnpm run ci:candidate-check/);
+  assert.match(content, /pnpm run release:admit/);
+  assert.match(content, /pnpm run release:stage-summary/);
   assert.match(content, /Mozilla-Actions\/sccache-action@fc920bf0ec8de6ee65d409111f7ec508035751ba/);
   assert.match(content, /shared-key: widgets-rust/);
   assert.doesNotMatch(content, /bash \.\/scripts\/gate\.sh/);
@@ -121,6 +131,12 @@ test("rust-hybrid release candidate renders product scripts, artifact handoff, a
   assert.doesNotMatch(content, /rust-changes|RIGHT_GIT_RUST_CHANGED/);
   assert.doesNotMatch(content, /if: steps\.rust-changes/);
   assert.match(content, /uses: dtolnay\/rust-toolchain@[a-f0-9]{40}\n        with:/);
+  // workflow_dispatch tag-last envelope: no windows-sign/macos-sign/release
+  // chain jobs render at all when signing is not configured.
+  assert.match(content, /^on:\n  workflow_dispatch:/m);
+  assert.doesNotMatch(content, /^ {2}(?:windows-sign|macos-sign|installed-qualification|publish):\s*$/m);
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
 });
 
 test("rust-hybrid cache contract renders every declared Cargo workspace", () => {
@@ -129,7 +145,7 @@ test("rust-hybrid cache contract renders every declared Cargo workspace", () =>
     lanes: ["ci"],
     profile: "rust-hybrid",
     gate: { path: "scripts/gate.sh" },
-    rustCache: { workspaces: [".", "apps/studio/src-tauri"] },
+    rustCache: { workspaces: [".", "apps/studio/src-tauri"], requiredForGate: true },
     matrix: { os: ["windows-2025"], node: ["22.23.2"] },
   };
   const content = renderTemplate(
@@ -140,7 +156,8 @@ test("rust-hybrid cache contract renders every declared Cargo workspace", () =>
   assert.match(content, /add-job-id-key: false/);
   assert.match(content, /cache-workspace-crates: false/);
   assert.match(content, /cache-on-failure: true/);
-  assert.match(content, /RUSTC_WRAPPER: \$\{\{ steps\.rust-changes\.outputs\.changed == 'true' && 'sccache' \|\| '' \}\}/);
+  assert.match(content, /if \[\[ "true" == "true" \]\]; then[\s\S]*required=true/);
+  assert.match(content, /RUSTC_WRAPPER: \$\{\{ steps\.rust-changes\.outputs\.required == 'true' && 'sccache' \|\| '' \}\}/);
 });
 
 test("rust-hybrid release candidate can opt into OIDC Windows signing without long-lived secrets", () => {
@@ -149,7 +166,15 @@ test("rust-hybrid release candidate can opt into OIDC Windows signing without lo
     lanes: ["ci", "release-candidate"],
     profile: "rust-hybrid",
     gate: { path: "scripts/gate.sh" },
-    candidate: { buildScript: "ci:candidate", checkScript: "ci:candidate-check", artifactRoot: "widgets-unsigned", os: ["windows-2025"], signWindows: true },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["windows-2025"],
+      signWindows: true,
+    },
     matrix: { os: ["ubuntu-24.04", "windows-2025"], node: ["22.23.2"] },
     templateVersion: "1.3.0",
   };
@@ -158,16 +183,26 @@ test("rust-hybrid release candidate can opt into OIDC Windows signing without lo
     buildRenderContext(manifest, "/repos/widgets"),
   );
   assert.match(content, /environment: release/);
-  assert.match(content, /windows-sign:\n    needs: \[candidate\]\n    if: \$\{\{ startsWith\(github\.ref, 'refs\/tags\/v'\) \|\| \(github\.event_name == 'workflow_dispatch' && inputs\.signed_qualification == true\) \}\}/);
+  // Tag-last contract: signing is gated on the admitted, same-run
+  // signed_qualification OUTPUT of the `admission` job — never on a ref/tag
+  // and never on the raw workflow_dispatch input directly.
+  assert.match(content, /windows-sign:\n    needs: \[admission, candidate\]\n    if: \$\{\{ needs\.admission\.outputs\.signed_qualification == 'true' \}\}/);
+  assert.doesNotMatch(content, /startsWith\(github\.ref, 'refs\/tags\/v'\)/);
   assert.match(content, /id-token: write/);
   assert.match(content, /azure\/login@[a-f0-9]{40}/);
   assert.match(content, /vars\.AZURE_CLIENT_ID/);
   assert.match(content, /right-release build --platform win --skip-checks/);
   assert.match(content, /Artifact Signing client hash mismatch/);
-  assert.match(content, /RIGHT_GIT_SOURCE_REVISION=\$\(git rev-parse HEAD\)/);
+  // Every downstream job (admission, candidate, and windows-sign here) checks
+  // out the exact admitted revision by SHA, never by resolving HEAD after a
+  // ref-triggered checkout.
+  assert.match(content, /windows-sign:[\s\S]*?ref: \$\{\{ needs\.admission\.outputs\.source_revision \}\}\n\s+persist-credentials: false/);
+  assert.doesNotMatch(content, /git rev-parse HEAD/);
   assert.ok(content.includes("LEGION_SOURCE_REVISION: ${{ env.RIGHT_GIT_SOURCE_REVISION }}"));
   assert.doesNotMatch(content, /LEGION_SOURCE_REVISION: \$\{\{ github\.sha \}\}/);
   assert.doesNotMatch(content, /secrets\.|client-secret|private.key/i);
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
 });
 
 test("rust-hybrid release candidate can opt into protected macOS signing & notarization", () => {
@@ -176,7 +211,15 @@ test("rust-hybrid release candidate can opt into protected macOS signing & notar
     lanes: ["ci", "release-candidate"],
     profile: "rust-hybrid",
     gate: { path: "scripts/gate.sh" },
-    candidate: { buildScript: "ci:candidate", checkScript: "ci:candidate-check", artifactRoot: "widgets-unsigned", os: ["macos-15"], signMacos: true },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["macos-15"],
+      signMacos: true,
+    },
     matrix: { os: ["ubuntu-24.04", "macos-15"], node: ["22.23.2"] },
     templateVersion: "1.3.0",
   };
@@ -185,17 +228,21 @@ test("rust-hybrid release candidate can opt into protected macOS signing & notar
     buildRenderContext(manifest, "/repos/widgets"),
   );
   assert.match(content, /macos-sign:/);
-  assert.match(content, /macos-sign:\n    needs: \[candidate\]\n    if: \$\{\{ startsWith\(github\.ref, 'refs\/tags\/v'\) \|\| \(github\.event_name == 'workflow_dispatch' && inputs\.signed_qualification == true\) \}\}/);
+  assert.match(content, /macos-sign:\n    needs: \[admission, candidate\]\n    if: \$\{\{ needs\.admission\.outputs\.signed_qualification == 'true' \}\}/);
+  assert.doesNotMatch(content, /startsWith\(github\.ref, 'refs\/tags\/v'\)/);
   assert.match(content, /environment: release/);
   assert.match(content, /secrets\.APPLE_CERTIFICATE_BASE64/);
   assert.match(content, /temporary keychain/i);
   assert.match(content, /right-release build --platform mac --skip-checks/);
   assert.match(content, /secrets\.APPLE_API_KEY_BASE64/);
-  assert.match(content, /RIGHT_GIT_SOURCE_REVISION=%s\\n.*git rev-parse HEAD/);
+  assert.match(content, /macos-sign:[\s\S]*?ref: \$\{\{ needs\.admission\.outputs\.source_revision \}\}\n\s+persist-credentials: false/);
+  assert.doesNotMatch(content, /git rev-parse HEAD/);
   assert.ok(content.includes("LEGION_SOURCE_REVISION: ${{ env.RIGHT_GIT_SOURCE_REVISION }}"));
   assert.doesNotMatch(content, /LEGION_SOURCE_REVISION: \$\{\{ github\.sha \}\}/);
   assert.match(content, /security delete-keychain/);
   assert.doesNotMatch(content, /pull_request_target/);
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
 });
 
 test("rust-hybrid multi-architecture candidates bind every native target to exact artifacts and signing", () => {
@@ -208,6 +255,8 @@ test("rust-hybrid multi-architecture candidates bind every native target to exac
     candidate: {
       buildScript: "ci:candidate",
       checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
       artifactRoot: "widgets-unsigned",
       os: ["windows-2025", "windows-11-arm", "macos-15", "macos-15-intel"],
       targets: [
@@ -233,10 +282,15 @@ test("rust-hybrid multi-architecture candidates bind every native target to exac
   assert.doesNotMatch(content, /bash \.\/scripts\/gate\.sh/);
   assert.match(content, /windows-sign:[\s\S]*matrix:[\s\S]*windows-11-arm/);
   assert.match(content, /macos-sign:[\s\S]*matrix:[\s\S]*macos-15-intel/);
-  assert.match(content, /name: widgets-unsigned-\$\{\{ matrix\.platform \}\}-\$\{\{ matrix\.architecture \}\}-22\.23\.2/);
-  assert.match(content, /name: widgets-signed-windows-\$\{\{ matrix\.architecture \}\}-22\.23\.2/);
-  assert.match(content, /name: widgets-signed-macos-\$\{\{ matrix\.architecture \}\}-22\.23\.2/);
-  assert.equal((content.match(/RIGHT_GIT_RELEASE_ARCHITECTURE: \$\{\{ matrix\.architecture \}\}/g) ?? []).length, 4);
+  // Every artifact name is run- and identity-bound: <prefix>-<run_id>-<run_attempt>-<kind>-...-<version>-<source_revision>.
+  assert.match(content, /name: widgets-\$\{\{ github\.run_id \}\}-\$\{\{ github\.run_attempt \}\}-candidate-\$\{\{ matrix\.platform \}\}-\$\{\{ matrix\.architecture \}\}-\$\{\{ needs\.admission\.outputs\.version \}\}-\$\{\{ needs\.admission\.outputs\.source_revision \}\}/);
+  assert.match(content, /name: widgets-\$\{\{ github\.run_id \}\}-\$\{\{ github\.run_attempt \}\}-signed-windows-\$\{\{ matrix\.architecture \}\}-\$\{\{ needs\.admission\.outputs\.version \}\}-\$\{\{ needs\.admission\.outputs\.source_revision \}\}/);
+  assert.match(content, /name: widgets-\$\{\{ github\.run_id \}\}-\$\{\{ github\.run_attempt \}\}-signed-macos-\$\{\{ matrix\.architecture \}\}-\$\{\{ needs\.admission\.outputs\.version \}\}-\$\{\{ needs\.admission\.outputs\.source_revision \}\}/);
+  // Exact-count regression guard across the candidate job's build/check/
+  // stage-summary steps and the windows-sign/macos-sign stage-summary steps.
+  assert.equal((content.match(/RIGHT_GIT_RELEASE_ARCHITECTURE: \$\{\{ matrix\.architecture \}\}/g) ?? []).length, 10);
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
 });
 
 test("rust-hybrid release chain preserves signed candidate, installed qualification, then publish", () => {
@@ -248,11 +302,14 @@ test("rust-hybrid release chain preserves signed candidate, installed qualificat
     candidate: {
       buildScript: "ci:candidate",
       checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
       artifactRoot: "widgets-unsigned",
       os: ["windows-2025", "macos-15"],
       signWindows: true,
       signMacos: true,
       releaseChain: {
+        evidenceVerificationScript: "release:verify-evidence",
         windowsFinalizeScript: "release:finalize:win",
         macosFinalizeScript: "release:finalize:mac",
         installedQualificationScript: "release:qualify:installed",
@@ -270,26 +327,202 @@ test("rust-hybrid release chain preserves signed candidate, installed qualificat
   );
   assert.match(content, /windows-sign:[\s\S]*pnpm run release:finalize:win/);
   assert.match(content, /macos-sign:[\s\S]*pnpm run release:finalize:mac/);
-  assert.match(content, /signed_qualification:\n        description: Run protected signing & installed qualification without publishing/);
-  assert.match(content, /windows-sign:[\s\S]*inputs\.signed_qualification == true/);
-  assert.match(content, /macos-sign:[\s\S]*inputs\.signed_qualification == true/);
-  assert.match(content, /installed-qualification:\n    needs: \[candidate, windows-sign\]/);
-  assert.match(content, /installed-qualification:[\s\S]*inputs\.signed_qualification == true/);
+  // Tag-last contract: every gate reads the ADMITTED, same-run outputs of
+  // the `admission` job — never the raw workflow_dispatch input directly,
+  // and never a ref/tag.
+  assert.match(content, /signed_qualification:\n        description: Run protected signing, installed qualification, & evidence verification/);
+  assert.doesNotMatch(content, /inputs\.signed_qualification == true/);
+  assert.doesNotMatch(content, /startsWith\(github\.ref, 'refs\/tags\/v'\)/);
+  assert.match(content, /windows-sign:\n    needs: \[admission, candidate\]\n    if: \$\{\{ needs\.admission\.outputs\.signed_qualification == 'true' \}\}/);
+  assert.match(content, /macos-sign:\n    needs: \[admission, candidate\]\n    if: \$\{\{ needs\.admission\.outputs\.signed_qualification == 'true' \}\}/);
+  assert.match(content, /installed-qualification:\n    needs: \[admission, candidate, windows-sign\]/);
+  assert.match(content, /installed-qualification:[\s\S]*?if: \$\{\{ always\(\) && needs\.admission\.outputs\.signed_qualification == 'true' && needs\.candidate\.result == 'success' \}\}/);
   assert.match(content, /installed_qualification_failed/);
   assert.match(content, /shell: pwsh\n        run: \|\n          throw 'installed_qualification_failed \{ reason: prerequisites_absent, detail: "same-run Windows signing\/finalization did not succeed" \}'/);
   assert.doesNotMatch(content, /^        run: throw 'installed_qualification_failed .*detail: /m);
   assert.match(content, /pnpm run release:qualify:installed/);
   assert.match(content, /installed-qualification:[\s\S]*GH_TOKEN: \$\{\{ github\.token \}\}/);
-  assert.match(content, /publish:\n    needs: \[installed-qualification, macos-sign\]/);
-  assert.match(content, /publish:[\s\S]*startsWith\(github\.ref, 'refs\/tags\/v'\)/);
-  assert.doesNotMatch(content.slice(content.indexOf("  publish:")), /inputs\.signed_qualification/);
-  assert.match(content, /needs\.macos-sign\.result == 'success'/);
+  assert.match(content, /publish:\n    needs: \[admission, installed-qualification, macos-sign\]/);
+  assert.match(content, /publish:[\s\S]*?if: \$\{\{ needs\.admission\.outputs\.publish == 'true' && needs\.installed-qualification\.result == 'success' && needs\.macos-sign\.result == 'success' \}\}/);
+  const publishJobIndex = content.search(/^ {2}publish:\n {4}needs:/m);
+  assert.ok(publishJobIndex >= 0, "publish job must be present");
+  assert.doesNotMatch(content.slice(publishJobIndex), /inputs\.signed_qualification|inputs\.publish/);
   assert.match(content, /contents: write/);
+  // Publish is the sole creator of the tag/release: it must not require a
+  // pre-existing tag anywhere in its gate.
+  assert.doesNotMatch(content.slice(publishJobIndex), /refs\/tags/);
+  assert.match(content, /pnpm run release:verify-evidence/);
+  assert.match(content, /Verify aggregated release evidence[\s\S]*?pnpm run release:verify-evidence/);
   assert.match(content, /pnpm run release:publish:qualified/);
   assert.match(content, /CLOUDFLARE_API_TOKEN: \$\{\{ secrets\.CLOUDFLARE_API_TOKEN \}\}/);
-  assert.equal((content.match(/RIGHT_GIT_SOURCE_REVISION=\$\(git rev-parse HEAD\)/g) ?? []).length, 3);
+  // The old ref-triggered "resolve HEAD after checkout" trick is gone
+  // entirely for the releaseChain path — every job instead checks out the
+  // exact admitted SHA from the admission job's own output.
+  assert.doesNotMatch(content, /git rev-parse HEAD/);
   assert.doesNotMatch(content, /RIGHT_GIT_SOURCE_REVISION: \$\{\{ github\.sha \}\}/);
+  // windows-sign, macos-sign, installed-qualification, publish all check out
+  // the admitted revision directly followed by persist-credentials: false;
+  // candidate does too but with a fetch-depth: 0 line between them (matched
+  // separately below), and admission checks out ${{ inputs.source_revision }}
+  // (it IS the admission job — there is no needs.admission yet).
+  assert.equal((content.match(/ref: \$\{\{ needs\.admission\.outputs\.source_revision \}\}\n\s+persist-credentials: false/g) ?? []).length, 4, "windows-sign, macos-sign, installed-qualification, publish should all pin the admitted revision");
+  assert.match(content, /ref: \$\{\{ needs\.admission\.outputs\.source_revision \}\}\n\s+fetch-depth: 0\n\s+persist-credentials: false/, "candidate must also pin the admitted revision");
+  assert.match(content, /ref: \$\{\{ inputs\.source_revision \}\}\n\s+fetch-depth: 0\n\s+persist-credentials: false/, "admission must check out the exact dispatch-supplied revision");
   assert.doesNotMatch(content.slice(0, content.indexOf("  windows-sign:")), /secrets\.|id-token: write|contents: write/);
+  // Downstream download-artifact names must match exactly what windows-sign/
+  // macos-sign uploaded (see the multi-architecture test) — this is the
+  // regression guard for the artifact-name mismatch bug.
+  const windowsSignedArtifact = "widgets-${{ github.run_id }}-${{ github.run_attempt }}-signed-windows-x86_64-${{ needs.admission.outputs.version }}-${{ needs.admission.outputs.source_revision }}";
+  const macosSignedArtifact = "widgets-${{ github.run_id }}-${{ github.run_attempt }}-signed-macos-arm64-${{ needs.admission.outputs.version }}-${{ needs.admission.outputs.source_revision }}";
+  assert.equal((content.match(new RegExp(windowsSignedArtifact.replace(/[.*+?^${}()|[\]\\]/g, "\\$&"), "g")) ?? []).length, 2, "windows-sign upload + installed-qualification/publish download must share one artifact name");
+  assert.ok(content.includes(macosSignedArtifact), "publish must download the exact macOS artifact macos-sign uploaded");
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
+});
+
+// Part E regression guard: every `run: <pm> run <script>` line in a
+// rendered full release-chain workflow must name a script the manifest
+// actually declares. This is the general form of the `pnpm run undefined`
+// class of bug — it would also catch a typo'd or stale script reference
+// even when it resolves to a real (but wrong) string, which the fail-closed
+// undefined/null check in buildRenderContext cannot.
+test("every 'run <script>' in a full release-chain workflow names a script the manifest declares", () => {
+  const manifest = {
+    ...FULL_MANIFEST,
+    lanes: ["ci", "release-candidate"],
+    profile: "rust-hybrid",
+    gate: { path: "scripts/gate.sh" },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["windows-2025", "macos-15"],
+      signWindows: true,
+      signMacos: true,
+      releaseChain: {
+        evidenceVerificationScript: "release:verify-evidence",
+        windowsFinalizeScript: "release:finalize:win",
+        macosFinalizeScript: "release:finalize:mac",
+        installedQualificationScript: "release:qualify:installed",
+        publishScript: "release:publish:qualified",
+        qualificationRunnerVariable: "QUALIFICATION_RUNNER_LABEL",
+        publishSecrets: ["CLOUDFLARE_API_TOKEN"],
+      },
+    },
+    matrix: { os: ["ubuntu-24.04", "windows-2025", "macos-15"], node: ["22.23.2"] },
+    templateVersion: "1.4.0",
+  };
+  const content = renderTemplate(
+    fs.readFileSync(path.join(templatesDir, "profiles/rust-hybrid/release-candidate.yml"), "utf8"),
+    buildRenderContext(manifest, "/repos/widgets"),
+  );
+  const declaredScripts = new Set([
+    manifest.candidate.buildScript,
+    manifest.candidate.checkScript,
+    manifest.candidate.admissionScript,
+    manifest.candidate.stageSummaryScript,
+    manifest.candidate.releaseChain.evidenceVerificationScript,
+    manifest.candidate.releaseChain.windowsFinalizeScript,
+    manifest.candidate.releaseChain.macosFinalizeScript,
+    manifest.candidate.releaseChain.installedQualificationScript,
+    manifest.candidate.releaseChain.publishScript,
+  ]);
+  const runLines = content.match(/run: (?:pnpm|npm|yarn) run \S+/g) ?? [];
+  assert.ok(runLines.length > 0, "expected at least one package-script run line");
+  for (const line of runLines) {
+    const script = line.split(" run ").pop();
+    assert.ok(declaredScripts.has(script), `rendered step runs undeclared script: ${line}`);
+  }
+});
+
+// Downstream-consumer gap fix: the evidence verifier can only prove a stage
+// ran and succeeded if that stage produces a stage summary AND the publish
+// job actually downloads it. This test proves all four authoritative
+// stages (candidate, windows-sign, macos-sign, installed-qualification)
+// produce a summary, that installed-qualification wraps itself in
+// init/finalize like every other stage, that publish downloads the
+// complete same-run set into distinct subdirectories (no collisions
+// between candidate's two platform legs), and that evidence is bound to
+// this exact run (not merely version+source_revision) via RIGHT_GIT_RUN_ID/
+// RIGHT_GIT_RUN_ATTEMPT on both the verification and publish steps.
+test("release chain: installed-qualification has a stage summary and publish downloads the complete same-run evidence set", () => {
+  const manifest = {
+    ...FULL_MANIFEST,
+    lanes: ["ci", "release-candidate"],
+    profile: "rust-hybrid",
+    gate: { path: "scripts/gate.sh" },
+    candidate: {
+      buildScript: "ci:candidate",
+      checkScript: "ci:candidate-check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
+      artifactRoot: "widgets-unsigned",
+      os: ["windows-2025", "macos-15"],
+      signWindows: true,
+      signMacos: true,
+      releaseChain: {
+        evidenceVerificationScript: "release:verify-evidence",
+        windowsFinalizeScript: "release:finalize:win",
+        macosFinalizeScript: "release:finalize:mac",
+        installedQualificationScript: "release:qualify:installed",
+        publishScript: "release:publish:qualified",
+        qualificationRunnerVariable: "QUALIFICATION_RUNNER_LABEL",
+        publishSecrets: ["CLOUDFLARE_API_TOKEN"],
+      },
+    },
+    matrix: { os: ["ubuntu-24.04", "windows-2025", "macos-15"], node: ["22.23.2"] },
+    templateVersion: "1.4.0",
+  };
+  const content = renderTemplate(
+    fs.readFileSync(path.join(templatesDir, "profiles/rust-hybrid/release-candidate.yml"), "utf8"),
+    buildRenderContext(manifest, "/repos/widgets"),
+  );
+  const installedQualificationJob = content.slice(
+    content.search(/^ {2}installed-qualification:\n {4}needs:/m),
+    content.search(/^ {2}publish:\n {4}needs:/m),
+  );
+
+  // Point 1: installed-qualification wraps itself in stage-summary
+  // init/finalize like candidate/windows-sign/macos-sign, with the correct
+  // authoritative stage identifier, and finalize runs unconditionally.
+  assert.match(installedQualificationJob, /name: Initialize installed-qualification stage summary\n\s+run: pnpm run release:stage-summary\n\s+env:\n\s+RIGHT_GIT_STAGE_ACTION: init/);
+  assert.match(installedQualificationJob, /RIGHT_GIT_STAGE: installed-qualification/);
+  assert.match(installedQualificationJob, /name: Finalize installed-qualification stage summary\n\s+if: \$\{\{ always\(\) \}\}\n\s+run: pnpm run release:stage-summary\n\s+env:\n\s+RIGHT_GIT_STAGE_ACTION: finalize/);
+  assert.equal((installedQualificationJob.match(/RIGHT_GIT_STAGE: installed-qualification/g) ?? []).length, 2, "both init and finalize must carry the installed-qualification stage id");
+  assert.match(installedQualificationJob, /name: Upload installed-qualification stage summary\n\s+if: \$\{\{ always\(\) \}\}/);
+
+  const publishJob = content.slice(content.search(/^ {2}publish:\n {4}needs:/m));
+
+  // Point 2: publish downloads the complete same-run evidence set — both
+  // candidate legs (windows + macos) in distinct subdirectories, plus
+  // windows-sign, macos-sign, and installed-qualification summaries.
+  assert.match(publishJob, /Download same-run candidate \(Windows\) stage summary[\s\S]*?path: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/candidate-windows/);
+  assert.match(publishJob, /Download same-run candidate \(macOS\) stage summary[\s\S]*?path: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/candidate-macos/);
+  assert.match(publishJob, /Download same-run Windows signing stage summary[\s\S]*?path: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/windows-sign/);
+  assert.match(publishJob, /Download same-run macOS signing stage summary[\s\S]*?path: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/macos-sign/);
+  assert.match(publishJob, /Download same-run installed-qualification stage summary[\s\S]*?path: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/installed-qualification/);
+  // Distinct subdirectories: no two downloads share a path.
+  const downloadPaths = [...publishJob.matchAll(/path: (\$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries\/\S+)/g)].map((m) => m[1]);
+  assert.equal(downloadPaths.length, 5);
+  assert.equal(new Set(downloadPaths).size, 5, "every stage-summary download must use a distinct subdirectory");
+
+  // Point 3: both the evidence-verification step and the publish step carry
+  // the full run-identity contract (RIGHT_GIT_STAGE_SUMMARY_ROOT, run id,
+  // run attempt) alongside version/source_revision — evidence is bound to
+  // THIS run, not merely reusable across runs of the same version+SHA.
+  const verifyStep = publishJob.slice(publishJob.indexOf("Verify aggregated release evidence"), publishJob.indexOf("Publish qualified signed release"));
+  assert.match(verifyStep, /RIGHT_GIT_STAGE_SUMMARY_ROOT: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries/);
+  assert.match(verifyStep, /RIGHT_GIT_RUN_ID: \$\{\{ github\.run_id \}\}/);
+  assert.match(verifyStep, /RIGHT_GIT_RUN_ATTEMPT: \$\{\{ github\.run_attempt \}\}/);
+  const publishStep = publishJob.slice(publishJob.indexOf("Publish qualified signed release"));
+  assert.match(publishStep, /RIGHT_GIT_STAGE_SUMMARY_ROOT: \$\{\{ runner\.temp \}\}\/right-git-release\/stage-summaries/);
+  assert.match(publishStep, /RIGHT_GIT_RUN_ID: \$\{\{ github\.run_id \}\}/);
+  assert.match(publishStep, /RIGHT_GIT_RUN_ATTEMPT: \$\{\{ github\.run_attempt \}\}/);
+
+  assert.doesNotMatch(content, /run: (?:pnpm|npm|yarn) run undefined\b/);
+  assert.doesNotMatch(content, /\$\{\{\s*github\.output\s*\}\}/);
 });
 
 test("render determinism: same manifest renders byte-identical output every time", () => {

package/sync.test.mjs

@@ -67,11 +67,14 @@ test("sync rejects a Membrane-like release chain without root @rightkit/release"
     candidate: {
       buildScript: "release:candidate:build",
       checkScript: "release:candidate:check",
+      admissionScript: "release:admit",
+      stageSummaryScript: "release:stage-summary",
       artifactRoot: "membrane-release-candidate",
       os: ["windows-2025", "macos-15"],
       signWindows: true,
       signMacos: true,
       releaseChain: {
+        evidenceVerificationScript: "release:verify-evidence",
         windowsFinalizeScript: "release:finalize:windows",
         macosFinalizeScript: "release:finalize:macos",
         installedQualificationScript: "release:qualify:installed",

package/templates/profiles/rust-hybrid/ci.yml

@@ -54,17 +54,22 @@ jobs:
             fi
           fi
           printf 'changed=%s\n' "$changed" >> "$GITHUB_OUTPUT"
+          required="$changed"
+          if [[ "${RUST_REQUIRED_FOR_GATE}" == "true" ]]; then
+            required=true
+          fi
+          printf 'required=%s\n' "$required" >> "$GITHUB_OUTPUT"
       - uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ matrix.node }}
           package-manager-cache: false
 ${PACKAGE_MANAGER_SETUP}
       - uses: dtolnay/rust-toolchain@f8be11a05b1d4f3fcebe6410cc16743212b999b0
-        if: steps.rust-changes.outputs.changed == 'true'
+        if: steps.rust-changes.outputs.required == 'true'
         with:
           components: rustfmt, clippy
       - uses: Swatinem/rust-cache@6323deb102c322ba6fcbdcafc7e3dddab59af2b6 # v2.9.2
-        if: steps.rust-changes.outputs.changed == 'true'
+        if: steps.rust-changes.outputs.required == 'true'
         with:
           workspaces: |
 ${RUST_CACHE_WORKSPACES}
@@ -74,14 +79,15 @@ ${RUST_CACHE_WORKSPACES}
           cache-on-failure: true
           save-if: ${{ github.event_name != 'pull_request' }}
       - uses: Mozilla-Actions/sccache-action@fc920bf0ec8de6ee65d409111f7ec508035751ba # v0.0.11
-        if: steps.rust-changes.outputs.changed == 'true'
+        if: steps.rust-changes.outputs.required == 'true'
         with:
           version: "v0.16.0"
       - run: bash ./${GATE_PATH}
         env:
           RIGHT_GIT_RUST_CHANGED: ${{ steps.rust-changes.outputs.changed }}
+          RIGHT_GIT_RUST_REQUIRED: ${{ steps.rust-changes.outputs.required }}
           SCCACHE_GHA_ENABLED: "true"
-          RUSTC_WRAPPER: ${{ steps.rust-changes.outputs.changed == 'true' && 'sccache' || '' }}
+          RUSTC_WRAPPER: ${{ steps.rust-changes.outputs.required == 'true' && 'sccache' || '' }}
           CARGO_INCREMENTAL: "0"
           CARGO_PROFILE_DEV_DEBUG: "0"
           CARGO_PROFILE_TEST_DEBUG: "0"