@sublang/playbook 0.1.3 → 0.4.0

package/{code.tmux-play.js → reference/sdlc/code.playbook/code.tmux-play.js}

@@ -2,31 +2,40 @@
 // SPDX-FileCopyrightText: 2026 SubLang International <https://sublang.ai>
 import createPlaybookRuntime from './code.playbook.js';
 // Captain factory per TMUX-014: `(options: unknown) => Captain`.
-// `options` is whatever `captain.options` carries in the YAML config;
-// for the CODE playbook this is `CodePlaybookOptions` (coderPlayer,
-// reviewerPlayer, plus the rest of CodingInput).
+// `options` is whatever `captain.options` carries in the YAML config.
+// The per-run player identity strings (`coderPlayer`, `reviewerPlayer`)
+// are derived from `session.players` at init time per PBRT-4; any
+// same-named keys in `options` are ignored.
 export default function createCodeTmuxPlayCaptain(options) {
-    const runtime = createPlaybookRuntime(options);
     // CaptainSession is bound at init time and persists across turns;
     // CaptainContext is rebuilt per turn and carries the call
-    // primitives. PlaybookPorts is built once at init, so the per-turn
-    // context lives in a closure-scoped slot the port callbacks query
-    // lazily.
+    // primitives. The runtime is constructed in `init` so identity
+    // strings derived from `session.players` can flow into its options;
+    // PlaybookPorts is built once at init, so the per-turn context
+    // lives in a closure-scoped slot the port callbacks query lazily.
+    let runtime;
     let activeContext;
     return {
         async init(session) {
+            const coderPlayer = session.players.find((p) => p.id === 'coder')?.adapter;
+            const reviewerPlayer = session.players.find((p) => p.id === 'reviewer')?.adapter;
+            runtime = createPlaybookRuntime({
+                ...options,
+                coderPlayer,
+                reviewerPlayer,
+            });
             const ports = {
                 callPlayer: async (playerId, prompt, _signal) => {
                     if (!activeContext) {
                         throw new Error('callPlayer invoked outside a Boss turn');
                     }
-                    // RoleRunResult per TMUX-033 already matches PlayerResult
+                    // PlayerRunResult per TMUX-033 already matches PlayerResult
                     // (`status: 'ok' | 'aborted' | 'error'`, `finalText?`,
                     // `error?`). cligent honors context.signal internally;
                     // the runtime's signal is the same source forwarded
                     // through handleBossInput, so dropping `_signal` is
                     // safe.
-                    const r = await activeContext.callRole(playerId, prompt);
+                    const r = await activeContext.callPlayer(playerId, prompt);
                     return {
                         status: r.status,
                         finalText: r.finalText,
@@ -56,6 +65,9 @@ export default function createCodeTmuxPlayCaptain(options) {
             await runtime.init(ports);
         },
         async handleBossTurn(turn, context) {
+            if (!runtime) {
+                throw new Error('init must be called first');
+            }
             activeContext = context;
             try {
                 // Forward the Boss prompt + cligent's per-turn signal into
@@ -71,7 +83,7 @@ export default function createCodeTmuxPlayCaptain(options) {
             }
         },
         async dispose() {
-            await runtime.dispose();
+            await runtime?.dispose();
         },
     };
 }

package/{code.tmux-play.ts → reference/sdlc/code.playbook/code.tmux-play.ts}

@@ -18,37 +18,47 @@ import createPlaybookRuntime, {
 } from './code.playbook.js';
 
 // Captain factory per TMUX-014: `(options: unknown) => Captain`.
-// `options` is whatever `captain.options` carries in the YAML config;
-// for the CODE playbook this is `CodePlaybookOptions` (coderPlayer,
-// reviewerPlayer, plus the rest of CodingInput).
+// `options` is whatever `captain.options` carries in the YAML config.
+// The per-run player identity strings (`coderPlayer`, `reviewerPlayer`)
+// are derived from `session.players` at init time per PBRT-4; any
+// same-named keys in `options` are ignored.
 export default function createCodeTmuxPlayCaptain(
   options: unknown,
 ): Captain {
-  const runtime: PlaybookRuntime = createPlaybookRuntime(
-    options as CodePlaybookOptions,
-  );
-
   // CaptainSession is bound at init time and persists across turns;
   // CaptainContext is rebuilt per turn and carries the call
-  // primitives. PlaybookPorts is built once at init, so the per-turn
-  // context lives in a closure-scoped slot the port callbacks query
-  // lazily.
+  // primitives. The runtime is constructed in `init` so identity
+  // strings derived from `session.players` can flow into its options;
+  // PlaybookPorts is built once at init, so the per-turn context
+  // lives in a closure-scoped slot the port callbacks query lazily.
+  let runtime: PlaybookRuntime | undefined;
   let activeContext: CaptainContext | undefined;
 
   return {
     async init(session: CaptainSession): Promise<void> {
+      const coderPlayer = session.players.find(
+        (p) => p.id === 'coder',
+      )?.adapter;
+      const reviewerPlayer = session.players.find(
+        (p) => p.id === 'reviewer',
+      )?.adapter;
+      runtime = createPlaybookRuntime({
+        ...(options as CodePlaybookOptions),
+        coderPlayer,
+        reviewerPlayer,
+      });
       const ports: PlaybookPorts = {
         callPlayer: async (playerId, prompt, _signal) => {
           if (!activeContext) {
             throw new Error('callPlayer invoked outside a Boss turn');
           }
-          // RoleRunResult per TMUX-033 already matches PlayerResult
+          // PlayerRunResult per TMUX-033 already matches PlayerResult
           // (`status: 'ok' | 'aborted' | 'error'`, `finalText?`,
           // `error?`). cligent honors context.signal internally;
           // the runtime's signal is the same source forwarded
           // through handleBossInput, so dropping `_signal` is
           // safe.
-          const r = await activeContext.callRole(playerId, prompt);
+          const r = await activeContext.callPlayer(playerId, prompt);
           return {
             status: r.status,
             finalText: r.finalText,
@@ -89,6 +99,9 @@ export default function createCodeTmuxPlayCaptain(
       turn: BossTurn,
       context: CaptainContext,
     ): Promise<void> {
+      if (!runtime) {
+        throw new Error('init must be called first');
+      }
       activeContext = context;
       try {
         // Forward the Boss prompt + cligent's per-turn signal into
@@ -104,7 +117,7 @@ export default function createCodeTmuxPlayCaptain(
     },
 
     async dispose(): Promise<void> {
-      await runtime.dispose();
+      await runtime?.dispose();
     },
   };
 }

package/reference/sdlc/code.playbook/playbook-code.config.template.yaml

@@ -0,0 +1,38 @@
+# SPDX-License-Identifier: Apache-2.0
+# SPDX-FileCopyrightText: 2026 SubLang International <https://sublang.ai>
+
+# Seed template for the user-level playbook-code config.
+# First run copies this file to:
+#   ${XDG_CONFIG_HOME:-$HOME/.config}/playbook/playbook-code.config.yaml
+#
+# Safe tuning points:
+# - Change captain.adapter / captain.model to pick the judge/Captain agent.
+# - Change each player's adapter to pick the Coder and Reviewer agents.
+#
+# PBRT-4 host-configuration invariants:
+# - captain.from must keep pointing at the CODE tmux-play adapter module.
+# - players[].id must remain exactly "coder" and "reviewer"; the runtime routes
+#   to those baked ids and does not remap them. The adapter derives the
+#   per-run player identity strings (substituted into player prompts) from
+#   each entry's adapter.
+
+captain:
+  # PBRT-4 invariant: keep this adapter module path unchanged.
+  from: "@sublang/playbook/code/tmux-play"
+  # Tunable: the Captain/Judge adapter and model.
+  adapter: claude
+  model: claude-opus-4-7
+  permissions:
+    mode: auto
+
+players:
+  # PBRT-4 invariant: keep id: coder. Tune adapter/model if desired.
+  - id: coder
+    adapter: claude
+    permissions:
+      mode: auto
+  # PBRT-4 invariant: keep id: reviewer. Tune adapter/model if desired.
+  - id: reviewer
+    adapter: codex
+    permissions:
+      mode: auto

package/{tmux-play.config.yaml → reference/sdlc/code.playbook/tmux-play.config.yaml}

@@ -10,24 +10,28 @@ captain:
   # @sublang/playbook is published, swap this for the package
   # specifier — e.g.
   #   from: "@sublang/playbook/code/tmux-play"
-  # (final form confirmed at publish time per DR-004 §11). Roles,
+  # (final form confirmed at publish time per DR-004 §11). Players,
   # options, and the rest of the config are unchanged.
   from: ./code.tmux-play.js
   adapter: claude
   model: claude-opus-4-7
-  options:
-    # Substituted into the <coder-llm> placeholder in player prompts
-    # (DR-004 §6).
-    coderPlayer: claude
-    # Substituted into <reviewer-llm>.
-    reviewerPlayer: codex
+  # Agents run in cligent's classifier/reviewer-protected auto mode
+  # (cligent DR-005): claude → permissionMode auto, codex → auto_review.
+  permissions:
+    mode: auto
 
-# `roles[].id` shall match the baked playerId strings the adapter
+# `players[].id` shall match the baked playerId strings the adapter
 # routes to. The CODE playbook bakes Coder→'coder' and
 # Reviewer→'reviewer' at link time (DR-004 §2); the adapter does
-# not remap, so changing these ids would break callRole routing.
-roles:
+# not remap, so changing these ids would break callPlayer routing.
+# Each entry's `adapter` is also substituted into the <coder-llm> /
+# <reviewer-llm> placeholders in player prompts (DR-004 §6).
+players:
   - id: coder
     adapter: claude
+    permissions:
+      mode: auto
   - id: reviewer
     adapter: codex
+    permissions:
+      mode: auto

package/{tmux-play.production.config.yaml → reference/sdlc/code.playbook/tmux-play.production.config.yaml}

@@ -10,12 +10,15 @@ captain:
   from: "@sublang/playbook/code/tmux-play"
   adapter: claude
   model: claude-opus-4-7
-  options:
-    coderPlayer: claude
-    reviewerPlayer: codex
+  permissions:
+    mode: auto
 
-roles:
+players:
   - id: coder
     adapter: claude
+    permissions:
+      mode: auto
   - id: reviewer
     adapter: codex
+    permissions:
+      mode: auto

package/bin/playbook-code.js

@@ -1,43 +0,0 @@
-#!/usr/bin/env node
-// SPDX-License-Identifier: Apache-2.0
-// SPDX-FileCopyrightText: 2026 SubLang International <https://sublang.ai>
-
-// Thin shim that resolves the bundled tmux-play.production.config.yaml
-// relative to this script's own location and execs cligent's tmux-play
-// CLI from inside @sublang/playbook's own node_modules tree. We resolve
-// the CLI through cligent's `./tmux-play` subpath export instead of
-// relying on `tmux-play` being on PATH — npm global installs only link
-// bins from top-level packages, so a transitive cligent's bin won't
-// reach the user's PATH but its files are still resolvable here.
-
-import { spawn } from 'node:child_process';
-import { dirname, join, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const here = dirname(fileURLToPath(import.meta.url));
-const configPath = resolve(here, '..', 'tmux-play.production.config.yaml');
-
-// cligent's `./tmux-play` subpath export only declares "import"
-// (no "require"/"default"), so createRequire(...).resolve trips
-// ERR_PACKAGE_PATH_NOT_EXPORTED. import.meta.resolve uses ESM rules
-// and follows the "import" condition — works from Node 20.6+.
-const tmuxPlayIndexUrl = import.meta.resolve('@sublang/cligent/tmux-play');
-const tmuxPlayBin = join(dirname(fileURLToPath(tmuxPlayIndexUrl)), 'cli.js');
-
-const child = spawn(
-  process.execPath,
-  [tmuxPlayBin, '--config', configPath, ...process.argv.slice(2)],
-  { stdio: 'inherit' },
-);
-
-child.on('error', (err) => {
-  process.stderr.write(
-    `playbook-code: failed to launch tmux-play (${tmuxPlayBin}): ${err.message}\n`,
-  );
-  process.exit(127);
-});
-
-child.on('exit', (code, signal) => {
-  if (signal) process.kill(process.pid, signal);
-  else process.exit(code ?? 0);
-});