@codeam/shared 2.61.95 → 2.61.96

package/dist/index.d.mts

@@ -880,6 +880,16 @@ interface PackStageDef {
     /** The full role prompt sent (with the pack workflow article + task +
      *  previous handoff) as the stage's opening prompt. Read-only in the app. */
     prompt: string;
+    /**
+     * Whether this stage MUST end in a new commit to hand off. Defaults to `true`
+     * (undefined = true) — a Specifier/Coder/QA produces an artifact (spec, code,
+     * report) and a no-commit stage is a stall. Set `false` for review-style
+     * stages that legitimately approve clean with NO change: a Reviewer that finds
+     * nothing to fix should hand off "reviewed, no changes needed" against the
+     * commit it reviewed, not stall the pipeline. (Agent Packs live-run finding,
+     * 2026-08-08 — a clean Reviewer was mis-treated as "stage produced no commit".)
+     */
+    requiresCommit?: boolean;
 }
 interface PackDefinition {
     id: PackId;

package/dist/index.d.ts

@@ -880,6 +880,16 @@ interface PackStageDef {
     /** The full role prompt sent (with the pack workflow article + task +
      *  previous handoff) as the stage's opening prompt. Read-only in the app. */
     prompt: string;
+    /**
+     * Whether this stage MUST end in a new commit to hand off. Defaults to `true`
+     * (undefined = true) — a Specifier/Coder/QA produces an artifact (spec, code,
+     * report) and a no-commit stage is a stall. Set `false` for review-style
+     * stages that legitimately approve clean with NO change: a Reviewer that finds
+     * nothing to fix should hand off "reviewed, no changes needed" against the
+     * commit it reviewed, not stall the pipeline. (Agent Packs live-run finding,
+     * 2026-08-08 — a clean Reviewer was mis-treated as "stage produced no commit".)
+     */
+    requiresCommit?: boolean;
 }
 interface PackDefinition {
     id: PackId;

package/dist/index.js

@@ -2614,7 +2614,9 @@ var PACK_REGISTRY = {
         name: "Reviewer",
         description: "Skeptical review in a fresh context \u2014 finds and fixes what the coder missed.",
         skillIds: ["code-review", "code-naming"],
-        prompt: REVIEWER_PROMPT
+        prompt: REVIEWER_PROMPT,
+        // Review-only: approving clean with no change is success, not a stall.
+        requiresCommit: false
       }
     ]
   },
@@ -2643,7 +2645,9 @@ var PACK_REGISTRY = {
         name: "Reviewer",
         description: "Audits correctness, scope, design, and conventions with fresh eyes.",
         skillIds: ["code-review", "code-naming"],
-        prompt: REVIEWER_PROMPT
+        prompt: REVIEWER_PROMPT,
+        // Review-only: approving clean with no change is success, not a stall.
+        requiresCommit: false
       },
       {
         role: "qa",