@sublang/playbook 4.0.0 → 6.0.0

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

@@ -1,24 +1,4 @@
 import { type CodePlaybookOptions, type PlaybookRuntime } from './code.playbook.js';
-export declare const codeCopyPasteGuardNames: readonly ["accepted", "approved", "challengeAccepted", "challengeRejected", "challengesRaised", "changesMadeCode", "changesMadeCodeAndChallenged", "changesMadeMixed", "changesMadeMixedAndChallenged", "changesMadeSpecs", "changesMadeSpecsAndChallenged", "hasFindings", "needsRevision", "noFindings", "noOpenItems"];
-export declare const codeStateCountLabels: {
-    readonly adjudicateChallenges: "rebuttal";
-    readonly reviewBossCommitSpecs: "review round";
-    readonly reviewBossCommitCode: "review round";
-    readonly reviewBossCommitMixed: "review round";
-    readonly reviewIrTaskCommitSpecs: "review round";
-    readonly reviewIrTaskCommitCode: "review round";
-    readonly reviewIrTaskCommitMixed: "review round";
-    readonly reviewChangesSpecs: "review round";
-    readonly reviewChangesCode: "review round";
-    readonly reviewChangesMixed: "review round";
-    readonly reviewChangesAndChallengesSpecs: "review round";
-    readonly reviewChangesAndChallengesCode: "review round";
-    readonly reviewChangesAndChallengesMixed: "review round";
-};
-export declare function codeSavedCountsLine(counts: {
-    interruptions: number;
-    copyPastes: number;
-}, rounds: number): string;
 export interface PlaybookSummaryPolicy {
     stateCountLabels: Readonly<Record<string, string>>;
     copyPasteGuardNames: readonly string[];
@@ -27,10 +7,6 @@ export interface PlaybookSummaryPolicy {
         copyPastes: number;
     }, rounds: number): string;
 }
-export declare const codeSummaryPolicy: PlaybookSummaryPolicy;
-export interface CodeOptions {
-    committer?: 'coder' | 'reviewer';
-}
 export interface RegistryPlayer {
     id: string;
     adapter?: string;
@@ -40,15 +16,23 @@ export interface CreateCodeRuntimeOptions {
     captainOptions: unknown;
     players: readonly RegistryPlayer[];
 }
+export type CodeOptions = Readonly<Record<string, never>>;
 export interface CodePlaybookRegistryEntry {
     id: 'code';
     command: 'code';
     intent: string;
-    requiredRoleIds: readonly string[];
+    requiredRoleIds: readonly ['coder'];
     summaryPolicy: PlaybookSummaryPolicy;
     validateOptions(captainOptions: unknown): CodeOptions;
     createRuntime(options: CreateCodeRuntimeOptions): PlaybookRuntime;
 }
+export declare const codeStateCountLabels: {};
+export declare const codeCopyPasteGuardNames: readonly ["directCommit", "irCommit", "moreTasks", "finalTask"];
+export declare function codeSavedCountsLine(counts: {
+    interruptions: number;
+    copyPastes: number;
+}, rounds: number): string;
+export declare const codeSummaryPolicy: PlaybookSummaryPolicy;
 export declare function validateCodeOptions(optionSlice: unknown): CodeOptions;
 export declare function createCodeRuntimeOptions({ captainOptions, players, }: CreateCodeRuntimeOptions): CodePlaybookOptions;
 export declare const codePlaybookRegistryEntry: CodePlaybookRegistryEntry;

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

@@ -1,55 +1,18 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: 2026 SubLang International <https://sublang.ai>
 import createPlaybookRuntime from './code.playbook.js';
-// PBRT-30: the CODE registry entry validates the option slice the shell
-// passes it (`captain.options.playbooks.code.options`), not a namespace
-// it extracts from the full Captain options bag. The schema defines one
-// key, `committer`: an optional Committer-alias player id, one of the
-// baked role ids `coder` / `reviewer`. A valid slice is absent, `{}`, or
-// `{ committer: 'coder' | 'reviewer' }`; every other key is unknown and
-// rejected with a path-named error. A further CODE option shall be
-// introduced as its own higher-numbered item that widens
-// `CODE_OPTION_KEYS`; the validator still fails closed on stray keys.
-const CODE_OPTION_KEYS = new Set(['committer']);
-const COMMITTER_PLAYER_IDS = new Set(['coder', 'reviewer']);
+// REVIEW owns and labels its real review rounds. CODE's two suspended wrapper
+// states only delegate to that child and must not double-count those rounds.
+export const codeStateCountLabels = {};
 export const codeCopyPasteGuardNames = [
-    'accepted',
-    'approved',
-    'challengeAccepted',
-    'challengeRejected',
-    'challengesRaised',
-    'changesMadeCode',
-    'changesMadeCodeAndChallenged',
-    'changesMadeMixed',
-    'changesMadeMixedAndChallenged',
-    'changesMadeSpecs',
-    'changesMadeSpecsAndChallenged',
-    'hasFindings',
-    'needsRevision',
-    'noFindings',
-    'noOpenItems',
+    'directCommit',
+    'irCommit',
+    'moreTasks',
+    'finalTask',
 ];
-export const codeStateCountLabels = {
-    adjudicateChallenges: 'rebuttal',
-    reviewBossCommitSpecs: 'review round',
-    reviewBossCommitCode: 'review round',
-    reviewBossCommitMixed: 'review round',
-    reviewIrTaskCommitSpecs: 'review round',
-    reviewIrTaskCommitCode: 'review round',
-    reviewIrTaskCommitMixed: 'review round',
-    reviewChangesSpecs: 'review round',
-    reviewChangesCode: 'review round',
-    reviewChangesMixed: 'review round',
-    reviewChangesAndChallengesSpecs: 'review round',
-    reviewChangesAndChallengesCode: 'review round',
-    reviewChangesAndChallengesMixed: 'review round',
-};
 function countNoun(count, singular, plural = `${singular}s`) {
     return `${count} ${count === 1 ? singular : plural}`;
 }
-// PBRT-15 / CAPTAIN-19: the CODE saved-counts line wording is
-// registry-owned. The shell renders this exact line through the active
-// entry's summary policy rather than hardcoding CODE phrasing.
 export function codeSavedCountsLine(counts, rounds) {
     return [
         'Saved you',
@@ -68,57 +31,36 @@ export const codeSummaryPolicy = {
 };
 export function validateCodeOptions(optionSlice) {
     if (optionSlice === undefined)
-        return {};
-    if (typeof optionSlice !== 'object' ||
-        optionSlice === null ||
+        return Object.freeze({});
+    if (optionSlice === null ||
+        typeof optionSlice !== 'object' ||
         Array.isArray(optionSlice)) {
         throw new Error('captain.options.playbooks.code.options must be an object');
     }
-    const slice = optionSlice;
-    for (const key of Object.keys(slice)) {
-        if (!CODE_OPTION_KEYS.has(key)) {
-            throw new Error(`Unknown config field captain.options.playbooks.code.options.${key}`);
-        }
-    }
-    const options = {};
-    const committer = slice.committer;
-    if (committer !== undefined) {
-        if (typeof committer !== 'string' || !COMMITTER_PLAYER_IDS.has(committer)) {
-            throw new Error("captain.options.playbooks.code.options.committer must be " +
-                "'coder' or 'reviewer'");
-        }
-        options.committer = committer;
+    const keys = Object.keys(optionSlice);
+    if (keys.length > 0) {
+        throw new Error(`Unknown config field captain.options.playbooks.code.options.${keys[0]}`);
     }
-    return options;
+    return Object.freeze({});
 }
 function playerIdentity(players, id) {
-    const entry = players.find((p) => p.id === id);
-    return entry?.model ?? entry?.adapter;
+    const player = players.find((entry) => entry.id === id);
+    return player?.model ?? player?.adapter;
 }
 export function createCodeRuntimeOptions({ captainOptions, players, }) {
-    const codeOptions = validateCodeOptions(captainOptions);
+    validateCodeOptions(captainOptions);
     const coderPlayer = playerIdentity(players, 'coder');
-    const reviewerPlayer = playerIdentity(players, 'reviewer');
-    return {
-        coderPlayer,
-        reviewerPlayer,
-        ...(codeOptions.committer !== undefined
-            ? { committerPlayer: codeOptions.committer }
-            : {}),
-    };
+    return coderPlayer === undefined ? {} : { coderPlayer };
 }
 export const codePlaybookRegistryEntry = {
     id: 'code',
     command: 'code',
-    intent: 'software development / SDLC coding workflow',
-    requiredRoleIds: ['coder', 'reviewer'],
+    intent: 'implement a coding intent in reviewed, one-commit phases, using an intent record when needed',
+    requiredRoleIds: ['coder'],
     summaryPolicy: codeSummaryPolicy,
     validateOptions: validateCodeOptions,
     createRuntime(options) {
         return createPlaybookRuntime(createCodeRuntimeOptions(options));
     },
 };
-// CAPTAIN-16 / PBRT-16: the published `@sublang/playbook/code/registry`
-// module's default export is the CODE registry entry the Playbook Captain
-// shell loads when a playbook block's `from` names this module.
 export default codePlaybookRegistryEntry;

package/reference/sdlc/code.playbook/code.registry.ts

@@ -6,77 +6,6 @@ import createPlaybookRuntime, {
   type PlaybookRuntime,
 } from './code.playbook.js';
 
-// PBRT-30: the CODE registry entry validates the option slice the shell
-// passes it (`captain.options.playbooks.code.options`), not a namespace
-// it extracts from the full Captain options bag. The schema defines one
-// key, `committer`: an optional Committer-alias player id, one of the
-// baked role ids `coder` / `reviewer`. A valid slice is absent, `{}`, or
-// `{ committer: 'coder' | 'reviewer' }`; every other key is unknown and
-// rejected with a path-named error. A further CODE option shall be
-// introduced as its own higher-numbered item that widens
-// `CODE_OPTION_KEYS`; the validator still fails closed on stray keys.
-const CODE_OPTION_KEYS = new Set<string>(['committer']);
-const COMMITTER_PLAYER_IDS = new Set<string>(['coder', 'reviewer']);
-export const codeCopyPasteGuardNames = [
-  'accepted',
-  'approved',
-  'challengeAccepted',
-  'challengeRejected',
-  'challengesRaised',
-  'changesMadeCode',
-  'changesMadeCodeAndChallenged',
-  'changesMadeMixed',
-  'changesMadeMixedAndChallenged',
-  'changesMadeSpecs',
-  'changesMadeSpecsAndChallenged',
-  'hasFindings',
-  'needsRevision',
-  'noFindings',
-  'noOpenItems',
-] as const;
-
-export const codeStateCountLabels = {
-  adjudicateChallenges: 'rebuttal',
-  reviewBossCommitSpecs: 'review round',
-  reviewBossCommitCode: 'review round',
-  reviewBossCommitMixed: 'review round',
-  reviewIrTaskCommitSpecs: 'review round',
-  reviewIrTaskCommitCode: 'review round',
-  reviewIrTaskCommitMixed: 'review round',
-  reviewChangesSpecs: 'review round',
-  reviewChangesCode: 'review round',
-  reviewChangesMixed: 'review round',
-  reviewChangesAndChallengesSpecs: 'review round',
-  reviewChangesAndChallengesCode: 'review round',
-  reviewChangesAndChallengesMixed: 'review round',
-} as const;
-
-function countNoun(count: number, singular: string, plural = `${singular}s`): string {
-  return `${count} ${count === 1 ? singular : plural}`;
-}
-
-// PBRT-15 / CAPTAIN-19: the CODE saved-counts line wording is
-// registry-owned. The shell renders this exact line through the active
-// entry's summary policy rather than hardcoding CODE phrasing.
-export function codeSavedCountsLine(
-  counts: { interruptions: number; copyPastes: number },
-  rounds: number,
-): string {
-  return [
-    'Saved you',
-    countNoun(counts.interruptions, 'interruption'),
-    'and',
-    countNoun(counts.copyPastes, 'copy-paste'),
-    'across',
-    countNoun(rounds, 'round'),
-    'of reviews/rebuttals.',
-  ].join(' ');
-}
-
-// CAPTAIN-20 summary policy: the counted state-id labels, the
-// copy-paste guard names, and the saved-counts line wording an entry
-// owns for the shell's turn-summary aggregation. An entry without a
-// summary policy opts out of visible turn summaries.
 export interface PlaybookSummaryPolicy {
   stateCountLabels: Readonly<Record<string, string>>;
   copyPasteGuardNames: readonly string[];
@@ -86,19 +15,6 @@ export interface PlaybookSummaryPolicy {
   ): string;
 }
 
-export const codeSummaryPolicy: PlaybookSummaryPolicy = {
-  stateCountLabels: codeStateCountLabels,
-  copyPasteGuardNames: codeCopyPasteGuardNames,
-  savedCountsLine: codeSavedCountsLine,
-};
-
-// The validated CODE options set. `committer`, when present, is the
-// resolved Committer-alias player id (PBRT-8 / PBRT-30); a future CODE
-// option widens both this type and `CODE_OPTION_KEYS`.
-export interface CodeOptions {
-  committer?: 'coder' | 'reviewer';
-}
-
 export interface RegistryPlayer {
   id: string;
   adapter?: string;
@@ -110,76 +26,99 @@ export interface CreateCodeRuntimeOptions {
   players: readonly RegistryPlayer[];
 }
 
+export type CodeOptions = Readonly<Record<string, never>>;
+
 export interface CodePlaybookRegistryEntry {
   id: 'code';
   command: 'code';
   intent: string;
-  requiredRoleIds: readonly string[];
+  requiredRoleIds: readonly ['coder'];
   summaryPolicy: PlaybookSummaryPolicy;
   validateOptions(captainOptions: unknown): CodeOptions;
   createRuntime(options: CreateCodeRuntimeOptions): PlaybookRuntime;
 }
 
+// REVIEW owns and labels its real review rounds. CODE's two suspended wrapper
+// states only delegate to that child and must not double-count those rounds.
+export const codeStateCountLabels = {} as const;
+
+export const codeCopyPasteGuardNames = [
+  'directCommit',
+  'irCommit',
+  'moreTasks',
+  'finalTask',
+] as const;
+
+function countNoun(
+  count: number,
+  singular: string,
+  plural = `${singular}s`,
+): string {
+  return `${count} ${count === 1 ? singular : plural}`;
+}
+
+export function codeSavedCountsLine(
+  counts: { interruptions: number; copyPastes: number },
+  rounds: number,
+): string {
+  return [
+    'Saved you',
+    countNoun(counts.interruptions, 'interruption'),
+    'and',
+    countNoun(counts.copyPastes, 'copy-paste'),
+    'across',
+    countNoun(rounds, 'round'),
+    'of reviews/rebuttals.',
+  ].join(' ');
+}
+
+export const codeSummaryPolicy: PlaybookSummaryPolicy = {
+  stateCountLabels: codeStateCountLabels,
+  copyPasteGuardNames: codeCopyPasteGuardNames,
+  savedCountsLine: codeSavedCountsLine,
+};
+
 export function validateCodeOptions(optionSlice: unknown): CodeOptions {
-  if (optionSlice === undefined) return {};
+  if (optionSlice === undefined) return Object.freeze({});
   if (
-    typeof optionSlice !== 'object' ||
     optionSlice === null ||
+    typeof optionSlice !== 'object' ||
     Array.isArray(optionSlice)
   ) {
     throw new Error('captain.options.playbooks.code.options must be an object');
   }
-  const slice = optionSlice as Record<string, unknown>;
-  for (const key of Object.keys(slice)) {
-    if (!CODE_OPTION_KEYS.has(key)) {
-      throw new Error(
-        `Unknown config field captain.options.playbooks.code.options.${key}`,
-      );
-    }
-  }
-  const options: CodeOptions = {};
-  const committer = slice.committer;
-  if (committer !== undefined) {
-    if (typeof committer !== 'string' || !COMMITTER_PLAYER_IDS.has(committer)) {
-      throw new Error(
-        "captain.options.playbooks.code.options.committer must be " +
-          "'coder' or 'reviewer'",
-      );
-    }
-    options.committer = committer as 'coder' | 'reviewer';
+  const keys = Object.keys(optionSlice);
+  if (keys.length > 0) {
+    throw new Error(
+      `Unknown config field captain.options.playbooks.code.options.${keys[0]}`,
+    );
   }
-  return options;
+  return Object.freeze({});
 }
 
 function playerIdentity(
   players: readonly RegistryPlayer[],
   id: string,
 ): string | undefined {
-  const entry = players.find((p) => p.id === id);
-  return entry?.model ?? entry?.adapter;
+  const player = players.find((entry) => entry.id === id);
+  return player?.model ?? player?.adapter;
 }
 
 export function createCodeRuntimeOptions({
   captainOptions,
   players,
 }: CreateCodeRuntimeOptions): CodePlaybookOptions {
-  const codeOptions = validateCodeOptions(captainOptions);
+  validateCodeOptions(captainOptions);
   const coderPlayer = playerIdentity(players, 'coder');
-  const reviewerPlayer = playerIdentity(players, 'reviewer');
-  return {
-    coderPlayer,
-    reviewerPlayer,
-    ...(codeOptions.committer !== undefined
-      ? { committerPlayer: codeOptions.committer }
-      : {}),
-  };
+  return coderPlayer === undefined ? {} : { coderPlayer };
 }
 
 export const codePlaybookRegistryEntry: CodePlaybookRegistryEntry = {
   id: 'code',
   command: 'code',
-  intent: 'software development / SDLC coding workflow',
-  requiredRoleIds: ['coder', 'reviewer'],
+  intent:
+    'implement a coding intent in reviewed, one-commit phases, using an intent record when needed',
+  requiredRoleIds: ['coder'],
   summaryPolicy: codeSummaryPolicy,
   validateOptions: validateCodeOptions,
   createRuntime(options) {
@@ -187,7 +126,4 @@ export const codePlaybookRegistryEntry: CodePlaybookRegistryEntry = {
   },
 };
 
-// CAPTAIN-16 / PBRT-16: the published `@sublang/playbook/code/registry`
-// module's default export is the CODE registry entry the Playbook Captain
-// shell loads when a playbook block's `from` names this module.
 export default codePlaybookRegistryEntry;

package/reference/sdlc/code.playbook/playbook-captain.d.ts

@@ -1,5 +1,6 @@
 import type { Captain } from '@sublang/cligent/tmux-play';
 import type { PlaybookRuntime } from '@sublang/playbook/runtime';
+import { type CaptainControllerPort } from '../captain.playbook/captain.playbook.js';
 import type { PlaybookSummaryPolicy, RegistryPlayer } from './code.registry.js';
 export interface CreatePlaybookRuntimeOptions {
     captainOptions: unknown;
@@ -14,6 +15,7 @@ export interface PlaybookCaptainDeps {
             readonly command: string;
             readonly intent: string;
         }[];
+        readonly controller: CaptainControllerPort;
     }) => PlaybookRuntime;
 }
 export interface PlaybookCaptainRegistryEntry {