peaks-loop 3.1.2 → 4.0.0-beta

package/dist/src/services/skills/skill-conformance-service.js

@@ -28,10 +28,10 @@ const SKILL_NAMES = [
     'peaks-rd',
     'peaks-qa',
     'peaks-sc',
-    'peaks-solo',
-    'peaks-solo-resume',
-    'peaks-solo-status',
-    'peaks-solo-test',
+    'peaks-code',
+    'peaks-resume',
+    'peaks-status',
+    'peaks-test',
     'peaks-sop',
     'peaks-txt',
     'peaks-doctor'
@@ -85,7 +85,10 @@ export function auditSkillConformance(input) {
     const skillsDir = join(input.projectRoot, 'skills');
     const checks = [];
     for (const skillName of SKILL_NAMES) {
-        const skillPath = join(skillsDir, skillName, 'SKILL.md');
+        // LLM-internal skills live under `skills/bee/<name>/` per spec 2026-07-05.
+        const topPath = join(skillsDir, skillName, 'SKILL.md');
+        const beePath = join(skillsDir, 'bee', skillName, 'SKILL.md');
+        const skillPath = existsSync(topPath) ? topPath : beePath;
         const fm = readSkillFrontmatter(skillPath);
         // 1. task-level frontmatter
         if (fm === null) {

package/dist/src/services/skills/skill-presence-service.d.ts

@@ -28,7 +28,7 @@ export type SkillPresence = {
      * observability consumer that an outer-session swap was observed
      * on the previous heartbeat. The actual binding rotation is
      * performed by `ensureSessionWithRotation` (slice 018), not by
-     * `setSkillPresence`. `peaks-solo`'s Step 0 used to read this
+     * `setSkillPresence`. `peaks-code`'s Step 0 used to read this
      * field and turn it into an AskUserQuestion; that ask is no
      * longer needed because the rotation already happened by the time
      * the skill is invoked.
@@ -81,11 +81,11 @@ export declare function getSkillPresence(projectRootOverride?: string): SkillPre
  *     different outer (Claude / harness) session than the one that
  *     is now driving peaks. Most common cause: the LLM closed the
  *     previous Claude session, the next session boots and finds a
- *     presence leftover from the old one. peaks-solo Step 1 must
+ *     presence leftover from the old one. peaks-code Step 1 must
  *     AskUserQuestion to confirm the user wants the old mode.
  *
  *   - `'no-presence'`            — there is no presence file on disk.
- *     Not strictly "stale", but callers (peaks-solo Step 1, `peaks
+ *     Not strictly "stale", but callers (peaks-code Step 1, `peaks
  *     solo should-pause --step step-1-mode-select`) want to treat
  *     this case as "no opinion yet — must ask". Surfaced as
  *     `stale: true` with reason `'no-presence'` so the ask path is
@@ -130,7 +130,7 @@ export declare function checkStalePresence(opts?: {
  * Called from `peaks workspace init` immediately after a successful
  * `outer-session-mismatch` rotation. When the previous session's
  * presence is still on disk under the OLD outer session id,
- * clearing it prevents peaks-solo Step 1 from picking up a stale
+ * clearing it prevents peaks-code Step 1 from picking up a stale
  * `mode` field that the user never explicitly chose.
  *
  * Only clears when:

package/dist/src/services/skills/skill-presence-service.js

@@ -143,7 +143,7 @@ function getBoundOuterSessionId(projectRootOverride) {
  * the presence envelope (useful for the statusline to render a stale
  * marker and for the QA / log consumers to know an outer-session swap
  * happened), but it no longer carries the implicit "ask the user"
- * promise — `peaks-solo`'s Step 0 no longer needs to surface an
+ * promise — `peaks-code`'s Step 0 no longer needs to surface an
  * AskUserQuestion, because the rotation already fired by the time the
  * skill is invoked.
  *
@@ -377,7 +377,7 @@ export function checkStalePresence(opts) {
  * Called from `peaks workspace init` immediately after a successful
  * `outer-session-mismatch` rotation. When the previous session's
  * presence is still on disk under the OLD outer session id,
- * clearing it prevents peaks-solo Step 1 from picking up a stale
+ * clearing it prevents peaks-code Step 1 from picking up a stale
  * `mode` field that the user never explicitly chose.
  *
  * Only clears when:

package/dist/src/services/skills/skill-registry.js

@@ -13,6 +13,33 @@ export async function loadSkillRegistry(baseDir = skillsDir) {
     const skills = [];
     const failures = [];
     for (const directory of directories) {
+        // LLM-internal skills (bees) live under `bee/` per spec 2026-07-05.
+        // Recurse one level into `bee/`; treat each child as a separate skill.
+        if (directory === 'bee') {
+            const subEntries = await listDirectories(join(baseDir, 'bee'));
+            for (const subDir of subEntries) {
+                const skillPath = join(baseDir, 'bee', subDir, 'SKILL.md');
+                if (!(await pathExists(skillPath))) {
+                    continue;
+                }
+                try {
+                    const frontmatter = parseFrontmatter(await readText(skillPath));
+                    // `directory` is the on-disk skill folder name; for bees that is the
+                    // basename of the bee child, NOT `bee/<name>` (keeps the
+                    // doctor skill-name-matches-directory invariant working).
+                    skills.push({
+                        name: frontmatter.name,
+                        description: frontmatter.description,
+                        directory: subDir,
+                        skillPath
+                    });
+                }
+                catch (error) {
+                    failures.push({ directory: subDir, skillPath, message: getLoadFailureMessage(error) });
+                }
+            }
+            continue;
+        }
         const skillPath = join(baseDir, directory, 'SKILL.md');
         if (!(await pathExists(skillPath))) {
             continue;

package/dist/src/services/skills/skill-runbook-service.js

@@ -36,7 +36,7 @@ function extractRunbookSection(body) {
 /**
  * Load the runbook section, falling back to `references/runbook.md` if the
  * SKILL.md only has a pointer section. This supports skills (notably
- * `peaks-solo`) that extracted their 150-line bash runbook to a sibling
+ * `peaks-code`) that extracted their 150-line bash runbook to a sibling
  * reference to keep SKILL.md under the 800-line cap. The CLI
  * `peaks skill runbook` command uses the same fallback so a human
  * reviewer sees the full runbook regardless of where it lives.
@@ -52,7 +52,7 @@ async function loadRunbookSection(skillPath, body) {
     const inline = extractRunbookSection(body);
     const skillDir = dirname(skillPath);
     // Try multiple reference filenames. Convention: each skill may name its
-    // runbook file `runbook.md` (the generic name, used by peaks-solo), or
+    // runbook file `runbook.md` (the generic name, used by peaks-code), or
     // `<role>-runbook.md` (the role-suffixed name, used by peaks-rd → rd-runbook.md,
     // peaks-qa → qa-runbook.md, etc.). Prefer the longest extracted section.
     const refCandidates = [

package/dist/src/services/slice/slice-check-service.js

@@ -297,7 +297,7 @@ export async function sliceCheck(options) {
             description: 'vitest run (skipped per --skip-tests)',
             status: 'skipped',
             durationMs: 0,
-            detail: 'Skipped: --skip-tests was set. Use the peaks-solo-test skill to run the full suite manually.'
+            detail: 'Skipped: --skip-tests was set. Use the peaks-test skill to run the full suite manually.'
         });
         unitTestsRunMode = 'skipped';
     }

package/dist/src/services/slice/slice-check-types.d.ts

@@ -2,7 +2,7 @@
  * Type envelope for the `peaks slice check` CLI command.
  *
  * `peaks slice check` is the boundary check for the RD micro-cycle
- * (see `skills/peaks-solo/references/micro-cycle.md`). It bundles the
+ * (see `skills/peaks-code/references/micro-cycle.md`). It bundles the
  * 4 self-checks that must pass at slice end before the slice is handed
  * off to peaks-qa:
  *

package/dist/src/services/slice/slice-check-types.js

@@ -2,7 +2,7 @@
  * Type envelope for the `peaks slice check` CLI command.
  *
  * `peaks slice check` is the boundary check for the RD micro-cycle
- * (see `skills/peaks-solo/references/micro-cycle.md`). It bundles the
+ * (see `skills/peaks-code/references/micro-cycle.md`). It bundles the
  * 4 self-checks that must pass at slice end before the slice is handed
  * off to peaks-qa:
  *

package/dist/src/services/slice/slice-decompose-types.d.ts

@@ -2,7 +2,7 @@
  * Type envelope for `peaks slice decompose` and its outputs.
  *
  * The slice-decomposition algorithm is a 6-stage deterministic function
- * (see `peaks-solo/references/slice-algorithm.md` for the full spec):
+ * (see `peaks-code/references/slice-algorithm.md` for the full spec):
  *
  *   1. workUnitResolution  -- codegraph.query + knowledge_graph.grep
  *   2. buildDependencyDAG  -- real import edges + codegraph.affected fallback

package/dist/src/services/slice/slice-decompose-types.js

@@ -2,7 +2,7 @@
  * Type envelope for `peaks slice decompose` and its outputs.
  *
  * The slice-decomposition algorithm is a 6-stage deterministic function
- * (see `peaks-solo/references/slice-algorithm.md` for the full spec):
+ * (see `peaks-code/references/slice-algorithm.md` for the full spec):
  *
  *   1. workUnitResolution  -- codegraph.query + knowledge_graph.grep
  *   2. buildDependencyDAG  -- real import edges + codegraph.affected fallback

package/dist/src/services/solo/auto-compact-orchestrator.d.ts

@@ -22,7 +22,7 @@ export interface AutoCompactInput {
     /**
      * Slice 2026-06-28-solo-mode-bypass-fix (defect #4): which session
      * the compact should target. Default `'main'` — the orchestrator
-     * (peaks-solo body) runs in the main-session Claude Code window and
+     * (peaks-code body) runs in the main-session Claude Code window and
      * wants to compress *its* context. Sub-agent shells pass
      * `'sub-agent'` to preserve the legacy shell-spawn behaviour.
      */

package/dist/src/services/solo/auto-compact-orchestrator.js

@@ -139,7 +139,7 @@ function appendAutoDecisionLog(input) {
         ''
     ].join('\n');
     if (!existsSync(logPath)) {
-        writeFileSync(logPath, `# peaks-solo auto-decisions log\n${row}`, 'utf8');
+        writeFileSync(logPath, `# peaks-code auto-decisions log\n${row}`, 'utf8');
         return;
     }
     const existing = readFileSync(logPath, 'utf8');
@@ -200,7 +200,7 @@ function writePreCompactCheckpoint(input) {
         recentDecisions: [],
         recentArtifactPaths: [],
         gitStatus: '',
-        skillsActive: ['peaks-solo'],
+        skillsActive: ['peaks-code'],
         todoState: []
     };
     writeFileSync(path, JSON.stringify(content, null, 2), 'utf8');

package/dist/src/services/solo/dag-orchestrator.d.ts

@@ -8,7 +8,7 @@
  * Design:
  *  - **Pure dispatcher contract**: the orchestrator never calls an LLM
  *    directly; it returns `DispatchSpec[]` per topological level and lets
- *    the caller (the CLI / peaks-solo LLM) execute the per-IDE tool calls.
+ *    the caller (the CLI / peaks-code LLM) execute the per-IDE tool calls.
  *    This keeps the orchestrator testable end-to-end with mock sub-agents.
  *  - **Join barrier**: between levels, the orchestrator awaits the
  *    results of all dispatched leaves and refuses to advance if any
@@ -21,7 +21,7 @@
  *    and the next dispatch's prompt is built with `formatContractInjection`.
  *
  * Why this lives in `services/solo/`:
- *  `peaks-solo` is the natural caller. The CLI surface (`peaks sub-agent
+ *  `peaks-code` is the natural caller. The CLI surface (`peaks sub-agent
  *  dispatch --from-dag`) consumes the planner's output but the planner
  *  itself is a library.
  */
@@ -123,6 +123,6 @@ export declare function runDag(dag: SliceDag, opts: RunDagOptions): Promise<DagR
  *      cancels in-flight siblings and breaks the loop.
  *
  * Pure planner; same I/O contract as `runDag`. The caller (CLI /
- * peaks-solo LLM) executes the per-IDE tool calls.
+ * peaks-code LLM) executes the per-IDE tool calls.
  */
 export declare function runLayeredDag(dag: SliceDag, opts: RunDagOptions): Promise<DagRunResult>;

package/dist/src/services/solo/dag-orchestrator.js

@@ -8,7 +8,7 @@
  * Design:
  *  - **Pure dispatcher contract**: the orchestrator never calls an LLM
  *    directly; it returns `DispatchSpec[]` per topological level and lets
- *    the caller (the CLI / peaks-solo LLM) execute the per-IDE tool calls.
+ *    the caller (the CLI / peaks-code LLM) execute the per-IDE tool calls.
  *    This keeps the orchestrator testable end-to-end with mock sub-agents.
  *  - **Join barrier**: between levels, the orchestrator awaits the
  *    results of all dispatched leaves and refuses to advance if any
@@ -21,7 +21,7 @@
  *    and the next dispatch's prompt is built with `formatContractInjection`.
  *
  * Why this lives in `services/solo/`:
- *  `peaks-solo` is the natural caller. The CLI surface (`peaks sub-agent
+ *  `peaks-code` is the natural caller. The CLI surface (`peaks sub-agent
  *  dispatch --from-dag`) consumes the planner's output but the planner
  *  itself is a library.
  */
@@ -110,7 +110,7 @@ export function buildDispatchSpec(dag, sliceId, contracts, defaultPrompt) {
         'Handoff protocol (REQUIRED — orchestrator depends on this):',
         '  1. Execute the slice to completion.',
         `  2. Write your public contract via \`peaks contract write --project <root> --session-id <sid> --slice-id ${sliceId} --exports <...> --types <...> --signatures <...>\`. The orchestrator will pick it up on the next dispatch run.`,
-        '  3. Do NOT re-invoke `peaks sub-agent dispatch --from-dag` yourself — the parent orchestrator (peaks-solo) drives level advancement.',
+        '  3. Do NOT re-invoke `peaks sub-agent dispatch --from-dag` yourself — the parent orchestrator (peaks-code) drives level advancement.',
         '',
         `After your contract is on disk, the orchestrator will auto-advance to the next topological level (if any) by re-invoking \`peaks sub-agent dispatch --from-dag\` with the same batch-id. Your slice will appear in its ancestors via \`formatContractInjection\`.${ancestorFragment}`
     ].join('\n');
@@ -214,7 +214,7 @@ export async function runDag(dag, opts) {
  *      cancels in-flight siblings and breaks the loop.
  *
  * Pure planner; same I/O contract as `runDag`. The caller (CLI /
- * peaks-solo LLM) executes the per-IDE tool calls.
+ * peaks-code LLM) executes the per-IDE tool calls.
  */
 export async function runLayeredDag(dag, opts) {
     try {

package/dist/src/services/solo/mode-gate.d.ts

@@ -43,7 +43,7 @@ export type GatedStepId = 'step-0.5-openspec-opt-in' | 'step-0.6-audit-goal' | '
 export declare const GATED_STEPS: readonly GatedStepId[];
 /**
  * Slice 2026-06-28-solo-mode-bypass-fix: the kind of gate that
- * produced this decision. The LLM-side caller (peaks-solo body) reads
+ * produced this decision. The LLM-side caller (peaks-code body) reads
  * this to distinguish "you paused because the user must choose the
  * mode" (`mode-selection-itself`) from "you paused because the mode
  * you already chose is assisted/strict" (`mode-driven`) from "you
@@ -98,7 +98,7 @@ export declare function shouldAutoProceed(mode: SoloMode): boolean;
 /**
  * v2.15.0 slice 002 AC-4: the 5 action identifiers that qualify as
  * commit-boundary side effects. Matched by the LLM-side caller
- * (peaks-solo body / peaks-rd fork agent) when it is about to run a
+ * (peaks-code body / peaks-rd fork agent) when it is about to run a
  * Bash command that maps to one of these. Set `commitBoundaryAction:
  * true` on the `shouldPauseAtGate` call to trigger the hard-floor
  * override.

package/dist/src/services/solo/post-compact-detector.js

@@ -146,7 +146,7 @@ export async function detectPostCompactResume(opts) {
         };
     }
     // Active skill check (D7.a.4). Defaults to reading from skill presence.
-    const expectedActiveSkill = 'peaks-solo';
+    const expectedActiveSkill = 'peaks-code';
     const observedActiveSkill = opts.activeSkill ?? readActiveSkillName(opts.projectRoot);
     if (observedActiveSkill !== expectedActiveSkill) {
         return {
@@ -154,7 +154,7 @@ export async function detectPostCompactResume(opts) {
             reason: 'active-skill-mismatch',
             checkpointPath: file.path,
             checkpointMtime: file.mtime.toISOString(),
-            warnings: [`active skill "${observedActiveSkill ?? 'unset'}" is not peaks-solo`]
+            warnings: [`active skill "${observedActiveSkill ?? 'unset'}" is not peaks-code`]
         };
     }
     // Mode: prefer the checkpoint's mode; fall back to the presence override

package/dist/src/services/solo/status-line-renderer.d.ts

@@ -1,13 +1,13 @@
 /**
- * G6.5 — status line renderer for the peaks-solo batch-sync wait period.
+ * G6.5 — status line renderer for the peaks-code batch-sync wait period.
  *
  * Single line, 80-120 chars, status-line-friendly. The shape is
  * documented in PRD §G6.5:
  *
- *   [peaks-solo] swarm 3/3 running | rd-planning 45% (12s ago) | qa-test-cases 30% (5s ago) | ui-design 20% (2s ago)
- *   [peaks-solo] swarm 3/3 running | rd-planning 70% (8s ago) | qa-test-cases 50% (3s ago) | ui-design 30% (6s ago)
+ *   [peaks-code] swarm 3/3 running | rd-planning 45% (12s ago) | qa-test-cases 30% (5s ago) | ui-design 20% (2s ago)
+ *   [peaks-code] swarm 3/3 running | rd-planning 70% (8s ago) | qa-test-cases 50% (3s ago) | ui-design 30% (6s ago)
  *   ...
- *   [peaks-solo] swarm 3/3 done in 47.3s
+ *   [peaks-code] swarm 3/3 done in 47.3s
  *
  * Pure helper; the poller calls it once per tick. No IO.
  */

package/dist/src/services/standards/missing-standards-detector.d.ts

@@ -1,5 +1,5 @@
 /**
- * Slice 2026-06-16-peaks-solo-auto-scaffold (RD#7) — missing-standards-detector.
+ * Slice 2026-06-16-peaks-code-auto-scaffold (RD#7) — missing-standards-detector.
  *
  * Read-side companion to the `peaks standards init` / `peaks standards update`
  * writers in `./project-standards-service.ts`. The writer KNOWS how to scaffold

package/dist/src/services/standards/missing-standards-detector.js

@@ -1,5 +1,5 @@
 /**
- * Slice 2026-06-16-peaks-solo-auto-scaffold (RD#7) — missing-standards-detector.
+ * Slice 2026-06-16-peaks-code-auto-scaffold (RD#7) — missing-standards-detector.
  *
  * Read-side companion to the `peaks standards init` / `peaks standards update`
  * writers in `./project-standards-service.ts`. The writer KNOWS how to scaffold

package/dist/src/services/standards/project-standards-service.d.ts

@@ -30,7 +30,7 @@ export type ProjectStandardsSource = {
     readonly usage: 'curated-baseline-reference';
 };
 export type StandardsSkillPreflight = {
-    readonly appliesTo: readonly ['peaks-rd', 'peaks-qa', 'peaks-solo'];
+    readonly appliesTo: readonly ['peaks-rd', 'peaks-qa', 'peaks-code'];
     readonly summary: string;
 };
 export type ProjectStandardsInitPlan = {

package/dist/src/services/standards/project-standards-service.js

@@ -23,8 +23,8 @@ const SOURCE = {
     usage: 'curated-baseline-reference'
 };
 const SKILL_PREFLIGHT = {
-    appliesTo: ['peaks-rd', 'peaks-qa', 'peaks-solo'],
-    summary: 'peaks-rd、peaks-qa、peaks-solo 进入代码仓工作流时自动 preflight 项目规范。'
+    appliesTo: ['peaks-rd', 'peaks-qa', 'peaks-code'],
+    summary: 'peaks-rd、peaks-qa、peaks-code 进入代码仓工作流时自动 preflight 项目规范。'
 };
 const SUPPORTED_LANGUAGES = new Set(['generic', 'typescript', 'javascript', 'python', 'go', 'rust']);
 function normalizeRoot(path) {
@@ -104,7 +104,7 @@ function renderHeader(title) {
         `# ${title}`,
         '',
         'Source: Peaks curated baseline; everything-claude-code reference: https://github.com/affaan-m/everything-claude-code',
-        'Scope: project-local standards for peaks-rd, peaks-qa, and peaks-solo workflow preflight.',
+        'Scope: project-local standards for peaks-rd, peaks-qa, and peaks-code workflow preflight.',
         ''
     ].join('\n');
 }
@@ -151,7 +151,7 @@ function renderClaudeMd(language, ctx) {
         'Peaks workflow automation:',
         '- peaks-rd checks these standards before RD planning or implementation work.',
         '- peaks-qa checks code review and security guidance before verification work.',
-        '- peaks-solo summarizes RD and QA standards preflight before end-to-end code workflows.',
+        '- peaks-code summarizes RD and QA standards preflight before end-to-end code workflows.',
         '',
         'Rules:',
         '- Read `.claude/rules/common/coding-style.md` before editing code.',

package/dist/src/services/understand/understand-types.d.ts

@@ -33,7 +33,7 @@ export type UnderstandScanReport = {
 };
 /**
  * Slice L3.1 — opt-in prompt payload. When uaPrompt === 'unset' and UA is
- * absent, the peaks-solo / peaks-ide layer surfaces this to the user via
+ * absent, the peaks-code / peaks-ide layer surfaces this to the user via
  * AskUserQuestion. The CLI does not prompt directly; it returns this
  * payload so the LLM-side UX layer can decide.
  */

package/dist/src/services/upgrade/1x-detector-service.js

@@ -6,7 +6,7 @@
  *
  * The canonical implementation lives in the .mjs postinstall
  * (because the postinstall runs before any TS compile step).
- * This TS mirror exists so the peaks-solo skill can call
+ * This TS mirror exists so the peaks-code skill can call
  * `peaks upgrade --detect-1x --project <root> --json` and
  * read a structured JSON envelope to gate the
  * AskUserQuestion that prompts the 1.x → 2.0 upgrade.

package/dist/src/services/upgrade/upgrade-service.d.ts

@@ -1,3 +1,30 @@
+/**
+ * Pluggable executor for sub-step command spawns. Production
+ * callers omit `executor` on `UpgradeInput`; the default
+ * (`defaultSubstepExecutor`) wraps `node:child_process` `spawnSync`.
+ * Tests pass a synchronous fake executor to avoid the spawn cost.
+ *
+ * Contract:
+ *  - Return `status: 0` for success, non-zero for failure, `null`
+ *    for synchronous spawn errors (binary not found, EACCES, …).
+ *  - `stdout` / `stderr` are captured as UTF-8 strings.
+ *  - `durationMs` is the wall-clock duration of the spawn
+ *    (caller-facing field on `SubStepResult`).
+ */
+export type SubstepExecutor = (command: string, args: readonly string[], timeoutMs: number) => {
+    status: number | null;
+    stdout: string;
+    stderr: string;
+    durationMs: number;
+};
+/**
+ * Default substep executor — wraps `spawnSync` with the same
+ * stdio / encoding / timeout config the umbrella has used since
+ * v1. Exported for test coverage: the seam-preservation test
+ * imports this directly to verify the normalized return shape
+ * on a spawn failure (e.g. a non-existent command).
+ */
+export declare const defaultSubstepExecutor: SubstepExecutor;
 export interface UpgradeInput {
     readonly projectRoot: string;
     /**
@@ -14,6 +41,13 @@ export interface UpgradeInput {
      * install hint in `nextActions` instead.
      */
     readonly peaksBin?: string;
+    /**
+     * Optional injectable executor for sub-step command execution.
+     * When omitted, the umbrella uses `defaultSubstepExecutor`
+     * (real `spawnSync`). Tests pass a synchronous fake to avoid
+     * the per-sub-step `node peaks.js …` spawn cost.
+     */
+    readonly executor?: SubstepExecutor;
 }
 export interface SubStepResult {
     readonly name: string;

package/dist/src/services/upgrade/upgrade-service.js

@@ -33,6 +33,38 @@ import { fileURLToPath } from 'node:url';
 import { runRedLinesAudit } from '../audit/red-lines-service.js';
 import { savePreferences } from '../preferences/preferences-service.js';
 import { migrateGitignoreFile } from './gitignore-migrate-service.js';
+/**
+ * Default substep executor — wraps `spawnSync` with the same
+ * stdio / encoding / timeout config the umbrella has used since
+ * v1. Exported for test coverage: the seam-preservation test
+ * imports this directly to verify the normalized return shape
+ * on a spawn failure (e.g. a non-existent command).
+ */
+export const defaultSubstepExecutor = (command, args, timeoutMs) => {
+    const start = Date.now();
+    try {
+        const result = spawnSync(command, args, {
+            encoding: 'utf8',
+            stdio: ['ignore', 'pipe', 'pipe'],
+            timeout: timeoutMs,
+        });
+        return {
+            status: result.status,
+            stdout: result.stdout ?? '',
+            stderr: result.stderr ?? '',
+            durationMs: Date.now() - start,
+        };
+    }
+    catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        return {
+            status: null,
+            stdout: '',
+            stderr: message,
+            durationMs: Date.now() - start,
+        };
+    }
+};
 const STEPS = [
     { name: 'config-migrate', args: (p) => ['config', 'migrate', '--project', p, '--apply', '--json'] },
     { name: 'standards-migrate', args: (p) => ['standards', 'migrate', '--from-claude-rules', '--project', p, '--apply', '--json'] },
@@ -145,8 +177,7 @@ function read1xVersion(cwd) {
     }
     return null;
 }
-function runStep(peaksBin, name, args, timeoutMs = 60_000) {
-    const start = Date.now();
+function runStep(peaksBin, name, args, timeoutMs = 60_000, executor = defaultSubstepExecutor) {
     // The global `peaks` shim is a `/bin/sh` symlink script (the
     // npm install postinstall creates `peaks` → `peaks.sh` on
     // Windows). cmd.exe (the default Windows shell) cannot run
@@ -168,32 +199,18 @@ function runStep(peaksBin, name, args, timeoutMs = 60_000) {
         command = peaksBin;
         spawnArgs = args;
     }
-    try {
-        const result = spawnSync(command, spawnArgs, {
-            encoding: 'utf8',
-            stdio: ['ignore', 'pipe', 'pipe'],
-            timeout: timeoutMs,
-        });
-        return {
-            name,
-            status: result.status === 0 ? 'pass' : 'fail',
-            exitCode: result.status,
-            stdout: result.stdout ?? '',
-            stderr: result.stderr ?? '',
-            durationMs: Date.now() - start,
-        };
-    }
-    catch (err) {
-        const message = err instanceof Error ? err.message : String(err);
-        return {
-            name,
-            status: 'fail',
-            exitCode: null,
-            stdout: '',
-            stderr: message,
-            durationMs: Date.now() - start,
-        };
-    }
+    // The executor normalizes spawn errors to { status: null, stderr: ... }
+    // so no try/catch wrapper is needed here — the call always returns
+    // a shaped record. See defaultSubstepExecutor for the reference impl.
+    const result = executor(command, spawnArgs, timeoutMs);
+    return {
+        name,
+        status: result.status === 0 ? 'pass' : 'fail',
+        exitCode: result.status,
+        stdout: result.stdout,
+        stderr: result.stderr,
+        durationMs: result.durationMs,
+    };
 }
 function writeUpgradeRecord(projectRoot, result) {
     try {
@@ -260,6 +277,11 @@ export function runUpgrade(input) {
         resolve(here, '..', '..', '..', '..', 'bin', 'peaks.js');
     const fallbackPeaks = 'peaks';
     const resolvedPeaksBin = existsSync(peaksBin) ? peaksBin : fallbackPeaks;
+    // Resolve the executor once at the top of runUpgrade so every
+    // sub-step call shares the same instance. Tests pass a fake
+    // executor via UpgradeInput.executor; production callers omit
+    // it and get the default spawnSync wrapper.
+    const executor = input.executor ?? defaultSubstepExecutor;
     const fromVersion = read1xVersion(input.projectRoot);
     const steps = [];
     const warnings = [];
@@ -325,7 +347,7 @@ export function runUpgrade(input) {
                 continue;
             }
             const args = ['memory', 'extract', '--project', input.projectRoot, '--artifact', ...artifacts, '--apply', '--json'];
-            const r = runStep(resolvedPeaksBin, 'memory-extract', args);
+            const r = runStep(resolvedPeaksBin, 'memory-extract', args, 60_000, executor);
             steps.push(r);
             if (r.status === 'fail') {
                 warnings.push(`memory-extract failed: ${r.stderr.slice(0, 200)}`);
@@ -333,7 +355,7 @@ export function runUpgrade(input) {
             continue;
         }
         const args = step.args(input.projectRoot);
-        const r = runStep(resolvedPeaksBin, step.name, args);
+        const r = runStep(resolvedPeaksBin, step.name, args, 60_000, executor);
         steps.push(r);
         if (r.status === 'fail') {
             warnings.push(`${step.name} failed: ${r.stderr.slice(0, 200)}`);

package/dist/src/services/verdict/verdict-aggregator.d.ts

@@ -1,5 +1,5 @@
 /**
- * peaks-solo multi-signal verdict aggregator (v2.13.1 Group A).
+ * peaks-code multi-signal verdict aggregator (v2.13.1 Group A).
  *
  * Implements the 4 aggregation rules documented in
  * `.peaks/project-scan/audit-output-schema.md:66-78` plus 3rd-party

package/dist/src/services/workflow/workflow-router-service.d.ts

@@ -7,7 +7,7 @@ export type ModelTier = 'top-tier' | 'mid-tier';
 export type ModelRole = 'strongest' | 'execution';
 export type WorkflowRoutePolicy = 'solo-broad-multi-model' | 'team-rd-limited-multi-model';
 export type WorkflowStepStage = 'product-direction' | 'design-direction' | 'tech-direction' | 'tech-review' | 'rd-planning' | 'coding-execution' | 'unit-test-execution' | 'quality-review';
-export type WorkflowStepOwner = 'peaks-solo' | 'peaks-rd' | 'peaks-tech' | 'human';
+export type WorkflowStepOwner = 'peaks-code' | 'peaks-rd' | 'peaks-tech' | 'human';
 export type WorkflowRouterRequest = {
     sessionId: string;
     goal: string;

package/dist/src/services/workflow/workflow-router-service.js

@@ -80,14 +80,14 @@ function annotateSteps(steps, soloMode) {
 }
 function createSoloSteps(executionModelId) {
     return [
-        step({ id: 'solo-product-direction', stage: 'product-direction', owner: 'peaks-solo', modelTier: 'top-tier', reason: 'Product direction needs strong judgment before execution work is delegated.', dependsOn: [] }, executionModelId),
-        step({ id: 'solo-design-direction', stage: 'design-direction', owner: 'peaks-solo', modelTier: 'top-tier', reason: 'Design direction uses the recommended default before cheaper implementation work.', dependsOn: ['solo-product-direction'] }, executionModelId),
+        step({ id: 'solo-product-direction', stage: 'product-direction', owner: 'peaks-code', modelTier: 'top-tier', reason: 'Product direction needs strong judgment before execution work is delegated.', dependsOn: [] }, executionModelId),
+        step({ id: 'solo-design-direction', stage: 'design-direction', owner: 'peaks-code', modelTier: 'top-tier', reason: 'Design direction uses the recommended default before cheaper implementation work.', dependsOn: ['solo-product-direction'] }, executionModelId),
         step({ id: 'solo-tech-direction', stage: 'tech-direction', owner: 'peaks-tech', modelTier: 'top-tier', reason: 'Technical boundaries and approval gates use the recommended default with high-confidence planning.', dependsOn: ['solo-design-direction'] }, executionModelId),
         step({ id: 'solo-tech-review', stage: 'tech-review', owner: 'peaks-tech', modelTier: 'top-tier', reason: 'Tech artifacts and gate decisions require strong review and a recommended default path.', dependsOn: ['solo-tech-direction'] }, executionModelId),
         step({ id: 'solo-rd-planning', stage: 'rd-planning', owner: 'peaks-rd', modelTier: 'top-tier', reason: 'RD task decomposition and acceptance criteria use the recommended default before execution delegation.', dependsOn: ['solo-tech-review'] }, executionModelId),
         step({ id: 'solo-coding-execution', stage: 'coding-execution', owner: 'peaks-rd', modelTier: executionModelId === STRONGEST_MODEL_ID ? 'top-tier' : 'mid-tier', reason: `Coding and routine refactoring must use the configured execution worker model ${executionModelId}.`, dependsOn: ['solo-rd-planning'] }, executionModelId),
         step({ id: 'solo-unit-test-execution', stage: 'unit-test-execution', owner: 'peaks-rd', modelTier: executionModelId === STRONGEST_MODEL_ID ? 'top-tier' : 'mid-tier', reason: `Unit test authoring and focused test runs must use the configured execution worker model ${executionModelId}.`, dependsOn: ['solo-coding-execution'] }, executionModelId),
-        step({ id: 'solo-quality-review', stage: 'quality-review', owner: 'peaks-solo', modelTier: 'top-tier', reason: 'Reducer and final quality gates need strong synthesis and risk review.', dependsOn: ['solo-unit-test-execution'] }, executionModelId)
+        step({ id: 'solo-quality-review', stage: 'quality-review', owner: 'peaks-code', modelTier: 'top-tier', reason: 'Reducer and final quality gates need strong synthesis and risk review.', dependsOn: ['solo-unit-test-execution'] }, executionModelId)
     ];
 }
 function createSoloStepsForMode(soloMode, executionModelId) {

package/dist/src/services/workspace/claude-settings-template.d.ts

@@ -5,7 +5,7 @@
  * The template is a PreToolUse hook allow-list that bypasses the
  * Claude Code [Fact-Forcing Gate] for tool calls whose paths target
  * the peaks-managed `.peaks/` workspace. Without this bypass,
- * `peaks workspace init` (Step 0 of every peaks-solo session) is
+ * `peaks workspace init` (Step 0 of every peaks-code session) is
  * unrunnable in a consumer project because the gate blocks the very
  * first Write.
  *