@sublang/playbook 0.7.0 → 0.8.0

package/README.md

@@ -84,7 +84,7 @@ playbook-code --help
 
 The seed template runs each agent in cligent's protected auto mode
 (`permissions.mode: auto`), suppressing routine approval prompts. Its
-Codex Coder also grants `permissions.writablePaths: [.git]` so git
+Codex Reviewer also grants `permissions.writablePaths: [.git]` so git
 metadata writes stay available under auto mode without switching to
 bypass permissions.
 
@@ -108,40 +108,41 @@ the runtime binds to those host-configuration invariants per
 `<coder-llm>` / `<reviewer-llm>` substitution strings from each role's
 `model` when pinned and `adapter` otherwise — so the Committer's
 commit-message trailers can name the concrete model
-(e.g. `claude-opus-4-8`) rather than the adapter family (`claude`).
+(e.g. `claude-opus-4-8[1m]`) rather than the adapter family
+(`claude`).
 
 `players.committer` is an optional alias naming which role — `coder`
 or `reviewer` — runs the commit turn; the seeded overlay points it at
-the Reviewer. Absent the alias the Committer falls back to the Coder
+the Coder. Absent the alias the Committer falls back to the Coder
 ([PBRT-8](specs/user/playbook-runtime.md#pbrt-8)).
 
-For example, the seeded overlay runs the Coder on Codex and the
-Reviewer on Claude, with the Committer aliased to the Reviewer:
+For example, the seeded overlay runs the Coder on Claude Opus 4.8 1m
+and the Reviewer on GPT-5.5, with the Committer aliased to the Coder:
 
 ```yaml
 captain:
   adapter: claude
-  model: claude-sonnet-4-6
+  model: claude-opus-4-8
   reasoningEffort: high
   permissions:
     mode: auto
 
 players:
   coder:              # role key must stay `coder` — see PBRT-4
-    adapter: codex
-    model: gpt-5.5
+    adapter: claude
+    model: claude-opus-4-8[1m]
     reasoningEffort: xhigh
     permissions:
       mode: auto
-      writablePaths:
-        - .git          # allow git metadata writes under Codex auto mode
   reviewer:           # role key must stay `reviewer` — see PBRT-4
-    adapter: claude
-    model: claude-opus-4-8
+    adapter: codex
+    model: gpt-5.5
     reasoningEffort: xhigh
     permissions:
       mode: auto
-  committer: reviewer   # which role commits — `coder` or `reviewer`
+      writablePaths:
+        - .git          # allow git metadata writes under Codex auto mode
+  committer: coder      # which role commits — `coder` or `reviewer`
 ```
 
 Normal `playbook-code` runs use the seeded path above. If you need a
@@ -170,8 +171,8 @@ pnpm test
 in the checked-in `pnpm-lock.yaml` — the same version CI installs
 via `--frozen-lockfile`, so contributor checkouts and CI agree.
 The published `package.json` declares `@sublang/cligent` as
-`^0.12.0`, so an end-user install with no lockfile (e.g., `npm
-install -g @sublang/playbook`) resolves a compatible cligent 0.12.x
+`^0.13.0`, so an end-user install with no lockfile (e.g., `npm
+install -g @sublang/playbook`) resolves a compatible cligent 0.13.x
 release (see [RELEASE-14](specs/dev/release.md#release-14)). To
 refresh the contributor pin within that range, run
 `pnpm update @sublang/cligent` and commit the resulting

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sublang/playbook",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "type": "module",
   "description": "Reference CODE playbook — XState v5 FSM, host-agnostic runtime, and Playbook Captain shell for a coder/reviewer/committer loop driven by GEARS spec items.",
   "license": "Apache-2.0",
@@ -89,7 +89,7 @@
   "dependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.143",
     "@openai/codex-sdk": "^0.139.0",
-    "@sublang/cligent": "^0.12.0",
+    "@sublang/cligent": "^0.13.0",
     "xstate": "^5.19.4",
     "yaml": "^2.9.0"
   },

package/reference/sdlc/code.playbook/bin/playbook-code.js

@@ -264,8 +264,15 @@ export function composeRuntimeConfig(
   const composed = {};
   const theme = overlayConfig.theme ?? base?.theme;
   if (theme !== undefined) composed.theme = theme;
-  const layout = overlayConfig.layout ?? base?.layout;
-  if (layout !== undefined) composed.layout = layout;
+  // PBCODE-16/17: an overlay `layout` is the user's own tmux-play layout
+  // and is carried through verbatim; a base-inherited `layout` is
+  // reconciled so it round-trips onto the composed coder + reviewer roster
+  // (cligent normalizes the base layout to the base config's own roster).
+  if (overlayConfig.layout !== undefined) {
+    composed.layout = overlayConfig.layout;
+  } else if (base?.layout !== undefined) {
+    composed.layout = composeInheritedLayout(base.layout);
+  }
   const notifications = overlayConfig.notifications ?? base?.notifications;
   if (notifications !== undefined) composed.notifications = notifications;
   composed.captain = captain;
@@ -273,6 +280,33 @@ export function composeRuntimeConfig(
   return composed;
 }
 
+// PBCODE-17: reconcile a base-inherited `layout` so it round-trips onto
+// the composed `coder` + `reviewer` roster. cligent's loaded base `layout`
+// is normalized to the base config's own roster and shape: `initialVisible`
+// names the base config's players (cligent rejects ids outside the composed
+// roster) and the normalized output carries both the `columnWeights` alias
+// and the canonical `singlePlayerColumnWeights` / `multiPlayerColumnWeights`
+// (cligent emits the pair on load but rejects it on reload). Drop
+// `initialVisible` — CODE shows its full composed roster, which cligent
+// restores when the field is omitted — and drop the `columnWeights` alias
+// whenever the canonical field for the alias's shape is present, carrying
+// window and weights through otherwise. An overlay `layout` is the user's
+// own and is not reconciled.
+function composeInheritedLayout(layout) {
+  if (!isObject(layout)) return layout;
+  const result = { ...layout };
+  delete result.initialVisible;
+  const alias = result.columnWeights;
+  if (Array.isArray(alias)) {
+    const conflictsSingle =
+      alias.length === 2 && result.singlePlayerColumnWeights !== undefined;
+    const conflictsMulti =
+      alias.length === 3 && result.multiPlayerColumnWeights !== undefined;
+    if (conflictsSingle || conflictsMulti) delete result.columnWeights;
+  }
+  return result;
+}
+
 export function adaptersFromComposedConfig(config) {
   const adapters = new Set();
   const captainAdapter = config?.captain?.adapter;

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

@@ -27,7 +27,7 @@
 #   instead of requiring bypass permissions.
 # - players.committer optionally aliases the Committer to a role
 #   (coder or reviewer); that role's pane runs the commit turn. The
-#   seeded value is reviewer; omit it to fall back to the coder.
+#   seeded value is coder; omitting it uses the same Coder fallback.
 # - layout sizes the tmux window (layout.window: columns × rows) and
 #   sets the relative column widths (layout.columnWeights) for the
 #   Boss/Captain, Coder, and Reviewer columns.
@@ -49,24 +49,24 @@ notifications:
 
 captain:
   adapter: claude
-  model: claude-sonnet-4-6
+  model: claude-opus-4-8
   reasoningEffort: high
   permissions:
     mode: auto
 
 players:
   coder:
-    adapter: codex
-    model: gpt-5.5
+    adapter: claude
+    model: claude-opus-4-8[1m]
     reasoningEffort: xhigh
     permissions:
       mode: auto
-      writablePaths:
-        - .git
   reviewer:
-    adapter: claude
-    model: claude-opus-4-8
+    adapter: codex
+    model: gpt-5.5
     reasoningEffort: xhigh
     permissions:
       mode: auto
-  committer: reviewer
+      writablePaths:
+        - .git
+  committer: coder

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

@@ -19,12 +19,15 @@ captain:
   # shim that delegates to the same shell with CODE registered.
   from: ./playbook-captain.js
   adapter: claude
-  model: claude-sonnet-4-6
+  model: claude-opus-4-8
   reasoningEffort: high
   # Agents run in cligent's classifier/reviewer-protected auto mode
   # (cligent DR-005): claude → permissionMode auto, codex → auto_review.
   permissions:
     mode: auto
+  options:
+    code:
+      committer: coder
 
 # `players[].id` shall match the baked playerId strings the adapter
 # routes to. The CODE playbook bakes Coder→'coder' and
@@ -38,7 +41,7 @@ captain:
 players:
   - id: coder
     adapter: claude
-    model: claude-opus-4-7
+    model: claude-opus-4-8[1m]
     reasoningEffort: xhigh
     permissions:
       mode: auto
@@ -48,3 +51,5 @@ players:
     reasoningEffort: xhigh
     permissions:
       mode: auto
+      writablePaths:
+        - .git

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

@@ -13,15 +13,18 @@ notifications:
 captain:
   from: "@sublang/playbook/playbook-captain"
   adapter: claude
-  model: claude-sonnet-4-6
+  model: claude-opus-4-8
   reasoningEffort: high
   permissions:
     mode: auto
+  options:
+    code:
+      committer: coder
 
 players:
   - id: coder
     adapter: claude
-    model: claude-opus-4-7
+    model: claude-opus-4-8[1m]
     reasoningEffort: xhigh
     permissions:
       mode: auto
@@ -31,3 +34,5 @@ players:
     reasoningEffort: xhigh
     permissions:
       mode: auto
+      writablePaths:
+        - .git