@sublang/playbook 11.0.0 → 12.0.0

package/docs/cli.md

@@ -105,8 +105,8 @@ its exit status or signal and exit `127` when it cannot be spawned
 
 The Boss pane starts at the Playbook Captain shell, where the session
 Captain runs for the whole session and sees every turn. Use `/code`,
-`/review`, or `/decide` followed by a task to select one of the bundled
-playbooks explicitly. A registered command resolves deterministically,
+`/review`, `/decide`, or `/dev` followed by a task to select one of the
+bundled playbooks explicitly. A registered command resolves deterministically,
 with no model call parsing it: at idle it starts that playbook, at its
 own leaf it delivers the rest of the line, an enabled command absent
 from the active path switches to it, and a bare command answers with
@@ -121,9 +121,11 @@ untouched
 ([[playbook-captain-1](https://github.com/sublang-ai/playbook/blob/main/specs/packages/playbook-captain.md#playbook-captain-1)],
 [[playbook-captain-2](https://github.com/sublang-ai/playbook/blob/main/specs/packages/playbook-captain.md#playbook-captain-2)]).
 
-The current CODE, REVIEW, and DECIDE workflows take their deterministic
+The current CODE, REVIEW, DECIDE, and DEV workflows take their deterministic
 initial event from the selecting Boss turn. CODE and DECIDE then call REVIEW
-as a nested playbook. Local role names do not imply continuity: each frame
+as a nested playbook, while DEV — the repository-aware planner behind
+`/dev` — analyzes a development request and itself calls CODE, or DECIDE and
+then CODE, as nested playbooks. Local role names do not imply continuity: each frame
 uses the exact stable player IDs configured under its `roles` map. Equal IDs
 share one pane and provider conversation across nested and later root
 engagements; distinct IDs remain isolated even when their agent settings are
@@ -321,8 +323,9 @@ to present the `incomplete` status it receives
 For each governed player call, Playbook records a Git baseline before the call
 and a durable receipt afterward. CODE and DECIDE commit arms, and REVIEW's
 Coder commit arm, accept a commit only when the receipt proves exactly one
-descendant commit with no residual repository change; REVIEW's Reviewer calls
-and DECIDE's proposal calls require the repository to remain exact. The
+descendant commit with no residual repository change; REVIEW's Reviewer calls,
+DECIDE's proposal calls, and every DEV Analyst planning call require the
+repository to remain exact. The
 player's prose, including any `Commit:` line, is presentation rather than
 proof.
 

package/docs/configuration.md

@@ -84,6 +84,13 @@ players:
     permissions:
       mode: auto # protected auto mode for the Claude Reviewer
 
+  dev.analyst:
+    adapter: claude
+    model: claude-opus-5
+    effort: xhigh
+    permissions:
+      mode: auto # protected auto mode for the Claude Analyst
+
 playbooks:
   code:
     from: '@sublang/playbook/code/registry'
@@ -101,6 +108,11 @@ playbooks:
     roles:
       coder: dev.coder
       reviewer: dev.reviewer
+
+  dev:
+    from: '@sublang/playbook/dev/registry'
+    roles:
+      analyst: dev.analyst
 ```
 
 The current bundled workflows accept no workflow-specific options.
@@ -146,7 +158,9 @@ Two bindings that name the same player ID deliberately share one sequential
 provider conversation throughout the logical Captain session — across nested
 calls, returns, and later root engagements. CODE's and REVIEW's `coder` roles
 therefore share `dev.coder` in the starter, and DECIDE and its nested REVIEW
-share both starter players. Disposal of one playbook frame does not clear that
+share both starter players. DEV's `analyst` instead binds the distinct
+`dev.analyst` player, so planning context does not bleed into the shared
+review conversation. Disposal of one playbook frame does not clear that
 session ledger.
 
 Two distinct player IDs stay isolated even when their agent blocks are

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@sublang/playbook",
-  "version": "11.0.0",
+  "version": "12.0.0",
   "type": "module",
-  "description": "Composable XState v5 playbook runtime with compiled Captain, CODE, REVIEW, and DECIDE workflows driven by GEARS specs.",
+  "description": "Composable XState v5 playbook runtime with compiled Captain, CODE, REVIEW, DECIDE, and DEV workflows driven by GEARS specs.",
   "license": "Apache-2.0",
   "homepage": "https://github.com/sublang-ai/playbook#readme",
   "repository": {
@@ -45,6 +45,7 @@
     "reference/sdlc/code.md",
     "reference/sdlc/review.md",
     "reference/sdlc/decide.md",
+    "reference/sdlc/dev.md",
     "reference/sdlc/captain.playbook/captain.gears.md",
     "reference/sdlc/captain.playbook/captain.fsm.ts",
     "reference/sdlc/captain.playbook/captain.fsm.js",
@@ -100,6 +101,16 @@
     "reference/sdlc/decide.playbook/decide.registry.ts",
     "reference/sdlc/decide.playbook/decide.registry.js",
     "reference/sdlc/decide.playbook/decide.registry.d.ts",
+    "reference/sdlc/dev.playbook/dev.gears.md",
+    "reference/sdlc/dev.playbook/dev.fsm.ts",
+    "reference/sdlc/dev.playbook/dev.fsm.js",
+    "reference/sdlc/dev.playbook/dev.fsm.d.ts",
+    "reference/sdlc/dev.playbook/dev.playbook.ts",
+    "reference/sdlc/dev.playbook/dev.playbook.js",
+    "reference/sdlc/dev.playbook/dev.playbook.d.ts",
+    "reference/sdlc/dev.playbook/dev.registry.ts",
+    "reference/sdlc/dev.playbook/dev.registry.js",
+    "reference/sdlc/dev.playbook/dev.registry.d.ts",
     "docs"
   ],
   "bin": {
@@ -150,6 +161,14 @@
     "./decide/registry": {
       "types": "./reference/sdlc/decide.playbook/decide.registry.d.ts",
       "default": "./reference/sdlc/decide.playbook/decide.registry.js"
+    },
+    "./dev/playbook": {
+      "types": "./reference/sdlc/dev.playbook/dev.playbook.d.ts",
+      "default": "./reference/sdlc/dev.playbook/dev.playbook.js"
+    },
+    "./dev/registry": {
+      "types": "./reference/sdlc/dev.playbook/dev.registry.d.ts",
+      "default": "./reference/sdlc/dev.playbook/dev.registry.js"
     }
   },
   "scripts": {
@@ -172,7 +191,7 @@
     "yaml": "^2.9.0"
   },
   "devDependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.3.221",
+    "@anthropic-ai/claude-agent-sdk": "^0.3.223",
     "@openai/codex-sdk": "^0.146.0",
     "@types/node": "^22.0.0",
     "typescript": "^5.8.0",

package/reference/sdlc/code.playbook/code.fsm.d.ts

@@ -23,7 +23,6 @@ export type PlayerInput = {
     readonly callerInput: string;
     readonly runResults: string;
     readonly irNumber?: string;
-    readonly irTask?: string;
     readonly pendingBossQuestion?: PendingBossQuestion;
     readonly bossReply?: string;
 };
@@ -36,16 +35,18 @@ export type PlayerOutput = {
     readonly coderOutput: string;
     readonly latestCommit: string;
     readonly irNumber: string;
-    readonly irTask: string;
 } | {
     readonly guard: 'moreTasks';
     readonly coderOutput: string;
     readonly latestCommit: string;
+    readonly irNumber: string;
     readonly irTask: string;
 } | {
     readonly guard: 'finalTask';
     readonly coderOutput: string;
     readonly latestCommit: string;
+    readonly irNumber: string;
+    readonly irTask: string;
 } | {
     readonly guard: 'needsBossReply';
     readonly question: string;
@@ -57,8 +58,13 @@ export type PlaybookInput = {
     readonly text: string;
 };
 export type ReviewOutput = {
-    readonly approvedCommit: 'latest';
     readonly noUnsettledFindings: true;
+    /**
+     * Exact repository revision at which the review scope was evaluated:
+     * the last review-fix commit when one landed, otherwise the clean
+     * round's receipt-observed HEAD (DR-045). Always present.
+     */
+    readonly evaluatedRevision: string;
 };
 export type CompactError = {
     readonly name: string;
@@ -66,16 +72,16 @@ export type CompactError = {
 };
 export type CodePlaybookOutput = {
     readonly status: 'complete';
-    /** Exact identity of the latest CODE-owned commit. */
+    /** Exact identity of the last CODE-owned commit. */
     readonly lastCodeCommit: string;
-    /** Exact Coder final text produced after that commit. */
-    readonly lastCodeOutput: string;
+    /** Exact repository revision the final passing review evaluated. */
+    readonly finalEvaluatedRevision: string;
+    /** Every phase's review passed with no unsettled findings. */
+    readonly allReviewsPassed: true;
 } | {
     readonly status: 'review-failed';
-    /** Exact identity of the latest CODE-owned commit. */
+    /** Exact identity of the last CODE-owned commit. */
     readonly lastCodeCommit: string;
-    /** Exact Coder final text produced after that commit. */
-    readonly lastCodeOutput: string;
     readonly error: CompactError;
 };
 export type CodingInput = {
@@ -88,7 +94,7 @@ export type CodingContext = {
     readonly latestCommit?: string;
     readonly irNumber?: string;
     readonly irTask?: string;
-    readonly nextIrTask?: string;
+    readonly evaluatedRevision?: string;
     readonly phase?: CodePhase;
     readonly completion?: 'complete' | 'review-failed';
     readonly reviewError?: CompactError;
@@ -146,9 +152,6 @@ export declare const codingMachine: import("xstate").StateMachine<CodingContext,
 } | {
     type: "rememberFinalTask";
     params: import("xstate").NonReducibleUnknown;
-} | {
-    type: "advanceToNextIrTask";
-    params: import("xstate").NonReducibleUnknown;
 } | {
     type: "rememberPendingQuestion";
     params: import("xstate").NonReducibleUnknown;
@@ -211,16 +214,16 @@ export declare const codingMachine: import("xstate").StateMachine<CodingContext,
     params: unknown;
 }, never, "done" | "failed" | "awaitBossReply" | "ready" | "runFirstPhase" | "reviewFirstCommit" | "runIrTask" | "reviewIrTask" | "reportedReviewFailure", string, CodingInput, {
     readonly status: "complete";
-    /** Exact identity of the latest CODE-owned commit. */
+    /** Exact identity of the last CODE-owned commit. */
     readonly lastCodeCommit: string;
-    /** Exact Coder final text produced after that commit. */
-    readonly lastCodeOutput: string;
+    /** Exact repository revision the final passing review evaluated. */
+    readonly finalEvaluatedRevision: string;
+    /** Every phase's review passed with no unsettled findings. */
+    readonly allReviewsPassed: true;
 } | {
     readonly status: "review-failed";
-    /** Exact identity of the latest CODE-owned commit. */
+    /** Exact identity of the last CODE-owned commit. */
     readonly lastCodeCommit: string;
-    /** Exact Coder final text produced after that commit. */
-    readonly lastCodeOutput: string;
     readonly error: CompactError;
 }, import("xstate").EventObject, import("xstate").MetaObject, {
     id: "code";

package/reference/sdlc/code.playbook/code.fsm.js

@@ -3,56 +3,70 @@
 //
 // Generated by slc/gears2fsm.md from ./code.gears.md.
 import { assign, fromPromise, setup } from 'xstate';
+const APPENDED_PHASE_PROMPT = [
+    'Keep to the original intent and follow what it asks.',
+    'Do not re-run tests or builds whose inputs have not changed since any previous reported run.',
+    "Make the phase's minimal changes and then one new commit, following @specs/packages/git.md; never amend an existing commit.",
+    'Make the commit message explain concisely what changed and why, including relevant verification.',
+    'Identify every new commit you make.',
+    'Coder is <coder-llm>.',
+];
 const FIRST_PHASE_PROMPT = [
     '> <caller-input>',
     '> <run-results>',
     '',
-    'Assess whether the coding intent can be completed well in one commit.',
-    'If yes, implement and test it, update the affected specs, and ensure @specs/map.md remains accurate.',
-    'Otherwise, decompose it into tasks sized to exactly one commit each, add a new IR under @specs/intents, and do not implement any IR task in this phase.',
+    'First determine whether the coding request starts a new coding intent or continues an existing IR with unfinished work.',
+    'If the request may continue an existing IR but does not identify it unambiguously, ask Boss before changing files.',
+    '',
+    'For a new coding intent, assess whether it can be completed well in one commit.',
+    'If it can, implement and test it, update the affected specs, and ensure @specs/map.md remains accurate.',
+    'If it cannot, decompose it into tasks sized to exactly one commit each, add a new IR under @specs/intents, and do not implement any IR task in this phase.',
     'Plan affected spec updates before, with, or after their corresponding code changes, either as standalone IR tasks or as explicit work within related tasks.',
     '',
+    'For an existing IR, read the identified IR and implement exactly its next unfinished task, including corresponding tests or specs if any.',
+    'Do not implement a later task in this phase.',
+    "Mark the IR's progress and deliverables when relevant.",
+    'If the IR will be finished after this phase, double-check that all acceptance criteria are met.',
+    '',
     'Consult @specs/map.md for relevant context and @specs/meta.md for spec requirements, if needed.',
     '',
-    'Do not re-run tests or builds whose inputs have not changed since any previous reported run.',
-    "Make the phase's minimal changes and then one new commit, following @specs/packages/git.md; never amend an existing commit.",
-    'Make the commit message explain concisely what changed and why, including relevant verification.',
-    'Coder is <coder-llm>; format the model token in conventional human form.',
+    ...APPENDED_PHASE_PROMPT,
 ].join('\n');
 const IR_TASK_PROMPT = [
-    '> <ir-task>',
+    '> <caller-input>',
+    '> <ir-number>',
     '> <run-results>',
     '',
-    'Read IR-<#> and implement exactly the next unfinished task, including corresponding tests or specs if any.',
+    'Read the identified IR and implement exactly its next unfinished task, including corresponding tests or specs if any.',
     'Do not implement a later task in this phase.',
     "Mark the IR's progress and deliverables when relevant.",
     'If the IR will be finished after this phase, double-check that all acceptance criteria are met.',
     '',
-    'Do not re-run tests or builds whose inputs have not changed since any previous reported run.',
-    "Make the phase's minimal changes and then one new commit, following @specs/packages/git.md; never amend an existing commit.",
-    'Make the commit message explain concisely what changed and why, including relevant verification.',
-    'Coder is <coder-llm>; format the model token in conventional human form.',
+    ...APPENDED_PHASE_PROMPT,
 ].join('\n');
+const NEEDS_BOSS_REPLY_RESULT = "The acting agent's prose surfaces a clarifying question for Boss that the agent cannot answer alone. Output shall include `question: <verbatim question text from the acting agent's prose>`.";
 const FIRST_PHASE_RESULTS = {
     directCommit: 'Coder completed and committed the direct implementation phase. Output shall include `coderOutput: <verbatim final text>` and `latestCommit: <commit identity>`.',
-    irCommit: 'Coder created and committed a new IR without implementing an IR task. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, `irNumber`, and `irTask` naming the exact next unfinished task.',
-    needsBossReply: "The acting agent's prose surfaces a clarifying question for Boss that the agent cannot answer alone. Output shall include `question: <verbatim question text from the acting agent's prose>`.",
+    irCommit: 'Coder created and committed a new IR without implementing an IR task. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, and `irNumber` identifying the created IR.',
+    moreTasks: 'Coder continued an existing IR, completed and committed exactly its next unfinished task, and at least one task remains. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, `irNumber` identifying the continued IR, and `irTask` naming the implemented task.',
+    finalTask: 'Coder continued an existing IR and completed and committed its final task. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, `irNumber` identifying the continued IR, and `irTask` naming the implemented task.',
+    needsBossReply: NEEDS_BOSS_REPLY_RESULT,
 };
 const IR_TASK_RESULTS = {
-    moreTasks: 'Coder completed and committed the current IR task and at least one task remains. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, and `irTask` naming the exact next unfinished task.',
-    finalTask: 'Coder completed and committed the final IR task. Output shall include `coderOutput: <verbatim final text>` and `latestCommit: <commit identity>`.',
-    needsBossReply: "The acting agent's prose surfaces a clarifying question for Boss that the agent cannot answer alone. Output shall include `question: <verbatim question text from the acting agent's prose>`.",
+    moreTasks: "Coder completed and committed exactly the IR's next unfinished task and at least one task remains. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, `irNumber` identifying the continued IR, and `irTask` naming the implemented task.",
+    finalTask: "Coder completed and committed the IR's final task. Output shall include `coderOutput: <verbatim final text>`, `latestCommit: <commit identity>`, `irNumber` identifying the continued IR, and `irTask` naming the implemented task.",
+    needsBossReply: NEEDS_BOSS_REPLY_RESULT,
 };
 const STATE_DESCRIPTIONS = {
     ready: 'Waiting for a coding intent.',
-    runFirstPhase: 'Coder is implementing one direct phase or committing a new intent record.',
-    reviewFirstCommit: 'The REVIEW playbook is checking the first phase commit.',
+    runFirstPhase: 'Coder is running the first coding phase: a direct implementation, a new intent record, or an existing intent-record task.',
+    reviewFirstCommit: 'The REVIEW playbook is checking the direct or new-IR phase commit.',
     runIrTask: 'Coder is implementing exactly one unfinished intent-record task.',
     reviewIrTask: 'The REVIEW playbook is checking the latest task commit.',
     awaitBossReply: 'Waiting for Boss to answer Coder.',
     failed: 'The coding workflow failed and is waiting for a new coding intent.',
     reportedReviewFailure: 'The coding workflow reported a REVIEW failure and the last code-owned commit.',
-    done: 'The coding workflow completed after REVIEW found no unsettled findings.',
+    done: "The coding workflow completed after every phase's REVIEW passed with no unsettled findings.",
 };
 function playbookMeta(stateId, role) {
     return {
@@ -89,6 +103,11 @@ function hasCommittedOutput(event) {
     return (outputString(event, 'coderOutput') !== undefined &&
         outputString(event, 'latestCommit') !== undefined);
 }
+function hasIrTaskOutput(event) {
+    return (hasCommittedOutput(event) &&
+        outputString(event, 'irNumber') !== undefined &&
+        outputString(event, 'irTask') !== undefined);
+}
 function isDirectCommit({ event }) {
     return (outputGuard(event, 'directCommit') &&
         hasCommittedOutput(event));
@@ -96,17 +115,15 @@ function isDirectCommit({ event }) {
 function isIrCommit({ event }) {
     return (outputGuard(event, 'irCommit') &&
         hasCommittedOutput(event) &&
-        outputString(event, 'irNumber') !== undefined &&
-        outputString(event, 'irTask') !== undefined);
+        outputString(event, 'irNumber') !== undefined);
 }
 function isMoreTasks({ event }) {
     return (outputGuard(event, 'moreTasks') &&
-        hasCommittedOutput(event) &&
-        outputString(event, 'irTask') !== undefined);
+        hasIrTaskOutput(event));
 }
 function isFinalTask({ event }) {
     return (outputGuard(event, 'finalTask') &&
-        hasCommittedOutput(event));
+        hasIrTaskOutput(event));
 }
 function needsBossReply({ event }) {
     return (outputGuard(event, 'needsBossReply') &&
@@ -116,11 +133,14 @@ function reviewOutput(event) {
     return isRecord(event) ? event.output : undefined;
 }
 function isReviewApprovedValue(value) {
-    return (isRecord(value) &&
-        Object.keys(value).sort().join('\0') ===
-            ['approvedCommit', 'noUnsettledFindings'].sort().join('\0') &&
-        value.approvedCommit === 'latest' &&
-        value.noUnsettledFindings === true);
+    if (!isRecord(value))
+        return false;
+    if (Reflect.ownKeys(value).some((key) => typeof key !== 'string' ||
+        (key !== 'noUnsettledFindings' && key !== 'evaluatedRevision'))) {
+        return false;
+    }
+    return (value.noUnsettledFindings === true &&
+        isNonEmptyString(value.evaluatedRevision));
 }
 function reviewApprovedFor(phase) {
     return ({ context, event }) => context.phase === phase && isReviewApprovedValue(reviewOutput(event));
@@ -265,16 +285,23 @@ function quoteRelay(label, value) {
     const lines = value.split('\n');
     return [`> ${label}: ${lines[0] ?? ''}`, ...lines.slice(1).map((line) => `> ${line}`)].join('\n');
 }
-function initialReviewText(context) {
+function reviewScopeLine(context) {
+    return (`> Review scope: the commit ${context.latestCommit ?? ''} ` +
+        'from this coding phase and its resulting repository state.');
+}
+function newIntentReviewText(context) {
     return [
-        quoteRelay('Initial intent', context.callerInput ?? ''),
+        quoteRelay('Original intent', context.callerInput ?? ''),
+        reviewScopeLine(context),
         quoteRelay('Coder output', context.coderOutput ?? ''),
     ].join('\n');
 }
 function irTaskReviewText(context) {
     return [
-        quoteRelay('IR task', context.irTask ?? ''),
+        quoteRelay('Original intent', context.callerInput ?? ''),
+        reviewScopeLine(context),
         quoteRelay('Coder output', context.coderOutput ?? ''),
+        quoteRelay('Current IR task', context.irTask ?? ''),
     ].join('\n');
 }
 const machineSetup = setup({
@@ -320,7 +347,7 @@ const machineSetup = setup({
                 latestCommit: undefined,
                 irNumber: undefined,
                 irTask: undefined,
-                nextIrTask: undefined,
+                evaluatedRevision: undefined,
                 phase: undefined,
                 completion: undefined,
                 reviewError: undefined,
@@ -340,8 +367,7 @@ const machineSetup = setup({
             coderOutput: outputString(event, 'coderOutput'),
             latestCommit: outputString(event, 'latestCommit'),
             irNumber: outputString(event, 'irNumber'),
-            irTask: outputString(event, 'irTask'),
-            nextIrTask: undefined,
+            irTask: undefined,
             phase: 'ir-created',
             pendingBossQuestion: undefined,
             bossReply: undefined,
@@ -349,7 +375,8 @@ const machineSetup = setup({
         rememberMoreTasks: assign(({ event }) => ({
             coderOutput: outputString(event, 'coderOutput'),
             latestCommit: outputString(event, 'latestCommit'),
-            nextIrTask: outputString(event, 'irTask'),
+            irNumber: outputString(event, 'irNumber'),
+            irTask: outputString(event, 'irTask'),
             phase: 'ir-task-more',
             pendingBossQuestion: undefined,
             bossReply: undefined,
@@ -357,15 +384,12 @@ const machineSetup = setup({
         rememberFinalTask: assign(({ event }) => ({
             coderOutput: outputString(event, 'coderOutput'),
             latestCommit: outputString(event, 'latestCommit'),
-            nextIrTask: undefined,
+            irNumber: outputString(event, 'irNumber'),
+            irTask: outputString(event, 'irTask'),
             phase: 'ir-task-final',
             pendingBossQuestion: undefined,
             bossReply: undefined,
         })),
-        advanceToNextIrTask: assign(({ context }) => ({
-            irTask: context.nextIrTask,
-            nextIrTask: undefined,
-        })),
         rememberPendingQuestion: assign(({ context, event }) => {
             const question = outputString(event, 'question');
             const output = playerOutput(event);
@@ -390,9 +414,14 @@ const machineSetup = setup({
             pendingBossQuestion: undefined,
             bossReply: undefined,
         }),
-        completeSuccessfully: assign({
-            completion: 'complete',
-            reviewError: undefined,
+        completeSuccessfully: assign(({ event }) => {
+            const output = reviewOutput(event);
+            const approved = isReviewApprovedValue(output) ? output : undefined;
+            return {
+                completion: 'complete',
+                evaluatedRevision: approved?.evaluatedRevision,
+                reviewError: undefined,
+            };
         }),
         completeWithReviewFailure: assign(({ event }) => ({
             completion: 'review-failed',
@@ -426,19 +455,26 @@ export const codingMachine = machineSetup.createMachine({
         if (!isNonEmptyString(lastCodeCommit)) {
             throw new Error('CODE reached a final state without a commit identity');
         }
-        const lastCodeOutput = context.coderOutput ?? '';
         if (context.completion === 'review-failed') {
             return {
                 status: 'review-failed',
                 lastCodeCommit,
-                lastCodeOutput,
                 error: context.reviewError ?? {
                     name: 'Error',
                     message: 'REVIEW failed without error detail.',
                 },
             };
         }
-        return { status: 'complete', lastCodeCommit, lastCodeOutput };
+        const finalEvaluatedRevision = context.evaluatedRevision;
+        if (!isNonEmptyString(finalEvaluatedRevision)) {
+            throw new Error('CODE completed without an evaluated repository revision');
+        }
+        return {
+            status: 'complete',
+            lastCodeCommit,
+            finalEvaluatedRevision,
+            allReviewsPassed: true,
+        };
     },
     states: {
         ready: {
@@ -503,6 +539,36 @@ export const codingMachine = machineSetup.createMachine({
                             'rememberIrCommit',
                         ],
                     },
+                    {
+                        guard: 'isMoreTasks',
+                        target: 'reviewIrTask',
+                        actions: [
+                            {
+                                type: 'playbook.acceptedOutcome',
+                                params: {
+                                    source: 'runFirstPhase',
+                                    target: 'reviewIrTask',
+                                    acceptedOutcome: 'moreTasks',
+                                },
+                            },
+                            'rememberMoreTasks',
+                        ],
+                    },
+                    {
+                        guard: 'isFinalTask',
+                        target: 'reviewIrTask',
+                        actions: [
+                            {
+                                type: 'playbook.acceptedOutcome',
+                                params: {
+                                    source: 'runFirstPhase',
+                                    target: 'reviewIrTask',
+                                    acceptedOutcome: 'finalTask',
+                                },
+                            },
+                            'rememberFinalTask',
+                        ],
+                    },
                     {
                         guard: 'needsBossReply',
                         target: 'awaitBossReply',
@@ -537,7 +603,7 @@ export const codingMachine = machineSetup.createMachine({
                     stateId: 'reviewFirstCommit',
                     sourceItem: 'CODE-2',
                     playbookId: 'review',
-                    text: initialReviewText(context),
+                    text: newIntentReviewText(context),
                 }),
                 onDone: [
                     {
@@ -582,7 +648,6 @@ export const codingMachine = machineSetup.createMachine({
                     ...(context.irNumber === undefined
                         ? {}
                         : { irNumber: context.irNumber }),
-                    ...(context.irTask === undefined ? {} : { irTask: context.irTask }),
                     ...(context.pendingBossQuestion === undefined
                         ? {}
                         : { pendingBossQuestion: context.pendingBossQuestion }),
@@ -661,7 +726,6 @@ export const codingMachine = machineSetup.createMachine({
                     {
                         guard: 'reviewApprovedMoreTasks',
                         target: 'runIrTask',
-                        actions: 'advanceToNextIrTask',
                     },
                     {
                         guard: 'reviewApprovedFinalTask',