peaks-cli 1.3.3 → 1.3.5

package/dist/src/cli/commands/core-artifact-commands.js

@@ -1,7 +1,8 @@
 import { createArtifactInitPlan, getArtifactStatus, createGuidedArtifactSetup } from '../../services/artifacts/artifact-service.js';
 import { getArtifactWorkspaceStatus, planArtifactSync } from '../../services/artifacts/workspace-service.js';
 import { executeProjectMemoryBackup, executeProjectMemoryExtract, summarizeProjectMemoryBackupResult, summarizeProjectMemoryExtractResult } from '../../services/memory/project-memory-service.js';
-import { executeProjectStandardsInit, executeProjectStandardsUpdate, summarizeProjectStandardsInitResult, summarizeProjectStandardsUpdateResult } from '../../services/standards/project-standards-service.js';
+import { summarizeProjectStandardsInitResult, summarizeProjectStandardsUpdateResult } from '../../services/standards/project-standards-service.js';
+import { executeProjectStandardsInitIdeAware, executeProjectStandardsUpdateIdeAware } from '../../services/standards/ide-aware-standards-service.js';
 import { listProfiles } from '../../services/profiles/profile-service.js';
 import { planProxyTest } from '../../services/proxy/proxy-service.js';
 import { runDoctor } from '../../services/doctor/doctor-service.js';
@@ -297,6 +298,7 @@ export function registerCoreAndArtifactCommands(program, io) {
         .description('Initialize project-local coding standards for Peaks skill preflight')
         .requiredOption('--project <path>', 'target project root')
         .option('--language <language>', 'standards language pack')
+        .option('--ide <id>', 'override IDE detection (e.g. claude-code, trae)')
         .option('--dry-run', 'preview writes without changing files')
         .option('--apply', 'write missing standards into the target project')).action((options) => {
         if (options.dryRun === true && options.apply === true) {
@@ -305,7 +307,7 @@ export function registerCoreAndArtifactCommands(program, io) {
             return;
         }
         try {
-            const result = executeProjectStandardsInit({ projectRoot: options.project, ...(options.language !== undefined ? { language: options.language } : {}), apply: options.apply === true });
+            const result = executeProjectStandardsInitIdeAware({ projectRoot: options.project, ...(options.language !== undefined ? { language: options.language } : {}), ...(options.ide !== undefined ? { ideId: options.ide } : {}), apply: options.apply === true });
             printResult(io, ok('standards.init', summarizeProjectStandardsInitResult(result)), options.json);
         }
         catch (error) {
@@ -318,6 +320,7 @@ export function registerCoreAndArtifactCommands(program, io) {
         .description('Append managed standards metadata to an existing CLAUDE.md without rewriting the body')
         .requiredOption('--project <path>', 'target project root')
         .option('--language <language>', 'standards language pack')
+        .option('--ide <id>', 'override IDE detection (e.g. claude-code, trae)')
         .option('--dry-run', 'preview writes without changing files')
         .option('--apply', 'append managed metadata to the target project')).action((options) => {
         if (options.dryRun === true && options.apply === true) {
@@ -326,7 +329,7 @@ export function registerCoreAndArtifactCommands(program, io) {
             return;
         }
         try {
-            const result = executeProjectStandardsUpdate({ projectRoot: options.project, ...(options.language !== undefined ? { language: options.language } : {}), apply: options.apply === true });
+            const result = executeProjectStandardsUpdateIdeAware({ projectRoot: options.project, ...(options.language !== undefined ? { language: options.language } : {}), ...(options.ide !== undefined ? { ideId: options.ide } : {}), apply: options.apply === true });
             const summary = summarizeProjectStandardsUpdateResult(result);
             const response = summary.reviewSuggestions.length > 0
                 ? fail('standards.update', 'STANDARDS_UPDATE_REVIEW_REQUIRED', 'Standards update requires manual review', summary, summary.reviewSuggestions)

package/dist/src/cli/commands/hook-handle.d.ts

@@ -7,10 +7,10 @@ import { type ProgramIO } from '../cli-helpers.js';
  *   1. 读 stdin
  *   2. auto-detect 来源 IDE(env / stdin shape / cwd)
  *   3. 归一化到 peaks canonical schema
- *   4. dispatch 到内部 peaks 逻辑(目前:gate enforce 或 progress start)
+ *   4. dispatch 到内部 peaks 逻辑(目前:gate enforce)
  *   5. 用 IDE 期望的格式发回决策
  *
- * Slice #1 阶段:peaks hook handle 与 peaks gate enforce / peaks progress start
+ * Slice #1 阶段:peaks hook handle 与 peaks gate enforce
  * 并存(后者内部走 hook-translator)。Slice #2 把 IDE settings 改成调用
  * peaks hook handle 即可。Slice #3 删除旧命令。
  */

package/dist/src/cli/commands/hook-handle.js

@@ -30,18 +30,18 @@ async function readStdin() {
  *   1. 读 stdin
  *   2. auto-detect 来源 IDE(env / stdin shape / cwd)
  *   3. 归一化到 peaks canonical schema
- *   4. dispatch 到内部 peaks 逻辑(目前:gate enforce 或 progress start)
+ *   4. dispatch 到内部 peaks 逻辑(目前:gate enforce)
  *   5. 用 IDE 期望的格式发回决策
  *
- * Slice #1 阶段:peaks hook handle 与 peaks gate enforce / peaks progress start
+ * Slice #1 阶段:peaks hook handle 与 peaks gate enforce
  * 并存(后者内部走 hook-translator)。Slice #2 把 IDE settings 改成调用
  * peaks hook handle 即可。Slice #3 删除旧命令。
  */
 export function registerHookHandleCommand(program, io) {
-    const hook = program.command('hook').description('Peaks 自有 hook 协议单一入口（slice #1 新增；后续 slice 将逐步替代 gate enforce / progress start）');
+    const hook = program.command('hook').description('Peaks 自有 hook 协议单一入口（slice #1 新增；后续 slice 将逐步替代 gate enforce）');
     addJsonOption(hook
         .command('handle')
-        .description('Read stdin hook payload, auto-detect IDE, dispatch to peaks gate/progress logic, output IDE-formatted decision')
+        .description('Read stdin hook payload, auto-detect IDE, dispatch to peaks gate-enforce logic, output IDE-formatted decision')
         .option('--project <path>', 'project the gates evaluate against (default: current directory)', '.')).action(async (options) => {
         try {
             const raw = await readStdin();
@@ -74,7 +74,7 @@ export function registerHookHandleCommand(program, io) {
                 rawIdeFormat: ide,
                 rawPayload: parsed
             });
-            // Dispatch by toolName. For slice #1, we only handle Bash and Task.
+            // Dispatch by toolName. For slice #1+, we only handle Bash. Task tool sub-agent dispatch goes through `peaks sub-agent dispatch` (slice #009) and does not need a hook entry.
             // Other tools: allow (no-op; future events will be added here).
             if (hook.toolName === 'Bash' && typeof fallbackCommand === 'string' && fallbackCommand.trim().length > 0) {
                 // Lazy import to avoid circular: peaks gate enforce logic
@@ -89,11 +89,6 @@ export function registerHookHandleCommand(program, io) {
                     return;
                 }
             }
-            else if (hook.toolName === 'Task') {
-                // peaks progress start is a fire-and-forget; do not block hook.handle.
-                // Slice #1: simply acknowledge (no terminal spawn from hook handle itself;
-                // the legacy `peaks progress start` command still does that).
-            }
             const allow = formatDecisionResponse(ide, 'allow');
             io.stdout(allow.stdout);
             if (options.json === true) {

package/dist/src/cli/commands/hooks-commands.js

@@ -1,7 +1,9 @@
+import { existsSync } from 'node:fs';
 import { fail, ok } from '../../shared/result.js';
 import { addJsonOption, printResult, getErrorMessage } from '../cli-helpers.js';
 import { findProjectRoot } from '../../services/config/config-safety.js';
-import { applyHookInstall, planHookInstall, readHookStatus, removeHookInstall } from '../../services/skills/hooks-settings-service.js';
+import { applyHookInstall, planHookInstall, readHookStatus, readInstalledEntriesFromSettings, removeHookInstall } from '../../services/skills/hooks-settings-service.js';
+import { readJsonObjectFile } from '../../services/ide/shared/atomic-json.js';
 import { detectIdeFromContext } from '../../services/ide/hook-translator.js';
 import { getAdapter } from '../../services/ide/ide-registry.js';
 function resolveScope(options) {
@@ -23,60 +25,63 @@ function resolveIdeForCommand(options, projectRoot) {
     }
     return detectIdeFromContext({ env: process.env, cwd: projectRoot ?? process.cwd(), parsedStdin: null });
 }
-// Slice #3: compute the per-IDE peaks hook entries for the CLI response
-// summary. Replaces the slice #1 PEAKS_HOOK_ENTRIES constant which was
-// hardcoded to claude-code values. Slice 2026-06-06-sub-agent-spawn-bug-
-// and-decouple: the sub-agent progress matcher now reads from
-// `adapter.subAgentToolMatcher` instead of being hardcoded to 'Task', so
-// every IDE self-reports its sub-agent tool name (claude-code: 'Task',
-// future adapters: whatever the adapter declares).
-function listInstalledEntriesForIde(ide) {
+/**
+ * Slice #014: compute the per-IDE peaks hook entries for the install /
+ * dry-run RESPONSE SUMMARY. This is the *desired* shape (what the install
+ * WOULD write), not what is on disk. The status command uses a different
+ * helper (`readInstalledEntriesFromSettings`) that reads the actual
+ * settings.json.
+ *
+ * After slice #014 only the gate-enforce entry is ever installed
+ * (the legacy progress-start surface is gone). The summary mirrors
+ * the install shape so the JSON envelope doesn't claim a hook the
+ * service did not write.
+ */
+function listExpectedEntriesForIde(ide, _skipProgress = false) {
     const adapter = getAdapter(ide);
     if (ide === 'trae') {
-        return [
-            { matcher: adapter.toolMatcher, sentinel: 'peaks hook handle' },
-            { matcher: adapter.subAgentToolMatcher, sentinel: 'peaks progress start' }
-        ];
+        return [{ matcher: adapter.toolMatcher, sentinel: 'peaks hook handle' }];
     }
-    // Default (claude-code) and any future registered adapters.
-    return [
-        { matcher: adapter.toolMatcher, sentinel: 'peaks gate enforce' },
-        { matcher: adapter.subAgentToolMatcher, sentinel: 'peaks progress start' }
-    ];
+    return [{ matcher: adapter.toolMatcher, sentinel: 'peaks gate enforce' }];
 }
 export function registerHooksCommands(program, io) {
     const hooks = program
         .command('hooks')
-        .description("Manage the Peaks-managed hook entries in the adapter's settings.json (default: .claude/settings.json for Claude, .trae/settings.json for Trae): (1) gate-enforce hook (SOP gate), (2) progress-start hook (auto-spawn sub-agent progress terminal). Both are installed / removed together. The IDE is auto-detected from env / cwd; override with --ide <id>.");
+        .description("Manage the Peaks-managed hook entry in the adapter's settings.json (default: .claude/settings.json for Claude, .trae/settings.json for Trae). Slice #014: the only installed entry is the gate-enforce hook (SOP gate). The legacy progress-start hook (auto-spawn sub-agent progress terminal) is no longer installed — sub-agent progress is now surfaced via the dispatch + heartbeat flow (`peaks sub-agent dispatch` / `peaks sub-agent heartbeat`). The IDE is auto-detected from env / cwd; override with --ide <id>.");
     addJsonOption(hooks
         .command('install')
-        .description(`Install all peaks-managed hook entries into the adapter's settings.json. Idempotent: re-runs are no-ops. Project scope by default.`)
+        .description(`Install the peaks-managed gate-enforce hook entry into the adapter's settings.json. Slice #014: only the gate-enforce entry is installed; the legacy progress-start entry is no longer installed. Idempotent: re-runs are no-ops. Project scope by default.`)
         .option('--global', 'install into the user-level ~/.claude/settings.json instead of the project')
         .option('--project <path>', 'project root path (auto-detected from cwd when omitted)')
         .option('--ide <id>', "target adapter id (claude-code | trae); default: auto-detect from env/cwd")
-        .option('--dry-run', 'show what would change without writing')).action((options) => {
+        .option('--dry-run', 'show what would change without writing')
+        .option('--no-progress', 'skip the progress-start PreToolUse hook entry; install ONLY the gate-enforce entry')).action((options) => {
         const scope = resolveScope(options);
         const projectRoot = resolveProjectRoot(scope, options.project);
         const ide = resolveIdeForCommand(options, projectRoot);
+        const skipProgress = options.progress === false;
         try {
             if (options.dryRun === true) {
-                const plan = planHookInstall(scope, projectRoot, { ide });
-                const dryRunEntries = listInstalledEntriesForIde(ide);
+                const plan = planHookInstall(scope, projectRoot, { ide, skipProgress });
+                const dryRunEntries = listExpectedEntriesForIde(ide, skipProgress);
                 printResult(io, ok('hooks.install', {
                     ...plan,
                     ide,
                     applied: false,
                     dryRun: true,
+                    skipProgress,
                     entries: dryRunEntries
                 }, [], [`would install ${dryRunEntries.length} peaks-managed hook entries`]), options.json);
                 return;
             }
-            const result = applyHookInstall(scope, projectRoot, { ide });
+            const result = applyHookInstall(scope, projectRoot, { ide, skipProgress });
             // Slice #3: build the per-IDE entries summary from the actual installed
             // entries, not the slice #1 PEAKS_HOOK_ENTRIES constant (which is the
             // claude-code default). The user's JSON envelope must reflect the IDE
-            // they targeted.
-            const installedEntries = listInstalledEntriesForIde(ide);
+            // they targeted. Slice #014: the install only emits the gate-enforce
+            // entry; the summary mirrors the install shape, NOT a hardcoded
+            // expected list.
+            const installedEntries = listExpectedEntriesForIde(ide, skipProgress);
             const nextActions = result.applied
                 ? [
                     'Restart the IDE (or reload the workspace) so the hook entries take effect',
@@ -87,18 +92,19 @@ export function registerHooksCommands(program, io) {
                 ...result,
                 ide,
                 dryRun: false,
+                skipProgress,
                 entries: installedEntries.map((e) => ({ matcher: e.matcher, sentinel: e.sentinel }))
             }, [], nextActions), options.json);
         }
         catch (error) {
             const message = getErrorMessage(error);
-            printResult(io, fail('hooks.install', 'HOOKS_INSTALL_FAILED', message, { scope, ide, applied: false }, [message]), options.json);
+            printResult(io, fail('hooks.install', 'HOOKS_INSTALL_FAILED', message, { scope, ide, applied: false, skipProgress }, [message]), options.json);
             process.exitCode = 1;
         }
     });
     addJsonOption(hooks
         .command('uninstall')
-        .description("Remove all peaks-managed hook entries (gate-enforce + progress-start) from the target settings.json. Third-party hooks are preserved.")
+        .description("Remove the peaks-managed gate-enforce hook entry from the target settings.json. Any legacy progress-start entry that a pre-#014 install left behind is also removed (sentinel-based scan). Third-party hooks are preserved.")
         .option('--global', 'remove from the user-level ~/.claude/settings.json instead of the project')
         .option('--project <path>', 'project root path (auto-detected from cwd when omitted)')
         .option('--ide <id>', 'target adapter id (claude-code | trae); default: auto-detect from env/cwd')).action((options) => {
@@ -126,10 +132,19 @@ export function registerHooksCommands(program, io) {
         const ide = resolveIdeForCommand(options, projectRoot);
         try {
             const status = readHookStatus(scope, projectRoot, { ide });
+            // Slice #014: read the ACTUAL on-disk entries (post-install shape),
+            // not the IDE-EXPECTED list. Pre-#014 `listInstalledEntriesForIde`
+            // returned the expected list and reported `entries: [Bash, Task]`
+            // even when the file only had `Bash`. The new helper reads the
+            // file and reports whatever peaks-managed entries are present,
+            // including any legacy progress-start entry that a pre-#014
+            // install left behind.
+            const settingsPath = status.settingsPath;
+            const settings = existsSync(settingsPath) ? readJsonObjectFile(settingsPath) : {};
             printResult(io, ok('hooks.status', {
                 ...status,
                 ide,
-                entries: listInstalledEntriesForIde(ide)
+                entries: readInstalledEntriesFromSettings(settings, ide)
             }), options.json);
         }
         catch (error) {

package/dist/src/cli/commands/project-commands.js

@@ -8,9 +8,13 @@ export function registerProjectCommands(program, io) {
     addJsonOption(project
         .command('dashboard')
         .description('One-call snapshot of doctor / MCP / OpenSpec / requests / Understand Anything / capabilities for a project')
-        .requiredOption('--project <path>', 'target project root')).action(async (options) => {
+        .requiredOption('--project <path>', 'target project root')
+        .option('--strict', 'ok follows the doctor aggregate (legacy semantics). Default: workspace-only (ok tracks the runbook health)', false)).action(async (options) => {
         try {
-            const dashboard = await loadProjectDashboard({ projectRoot: options.project });
+            const dashboard = await loadProjectDashboard({
+                projectRoot: options.project,
+                okPolicy: options.strict === true ? 'strict' : 'workspace-only'
+            });
             if (!dashboard.runbookHealth.ok) {
                 const suggestions = [
                     dashboard.runbookHealth.missingRunbook.length > 0
@@ -29,8 +33,8 @@ export function registerProjectCommands(program, io) {
                 process.exitCode = 1;
                 return;
             }
-            if (!dashboard.doctor.ok) {
-                printResult(io, fail('project.dashboard', 'PROJECT_DASHBOARD_DOCTOR_FAILED', `Doctor reports ${dashboard.doctor.failed} failed check(s) (${dashboard.doctor.passed} passed)`, dashboard, ['Run `peaks doctor --json` and resolve the failing checks before re-running the dashboard']), options.json);
+            if (!dashboard.doctor.ok && options.strict === true) {
+                printResult(io, fail('project.dashboard', 'PROJECT_DASHBOARD_DOCTOR_STRICT_FAIL', `Doctor reports ${dashboard.doctor.failed} failed check(s) (${dashboard.doctor.passed} passed) — --strict mode requires the doctor aggregate to pass`, dashboard, ['Run `peaks doctor --json` and resolve the failing checks, or drop --strict to use the workspace-only policy']), options.json);
                 process.exitCode = 1;
                 return;
             }
@@ -74,7 +78,13 @@ export function registerProjectCommands(program, io) {
         .description('Scan a session artifact directory and extract <!-- peaks-memory:start --> blocks into .peaks/memory/')
         .requiredOption('--session-id <id>', 'session id (e.g. 2026-05-29-session-89ff35)')
         .requiredOption('--project <path>', 'target project root')
-        .option('--dry-run', 'preview writes without changing files', true)
+        // Slice #015: drop the `--dry-run true` default. With the default
+        // set to true, `options.dryRun === true && options.apply === true`
+        // fired on every `--apply` call (because dryRun was true by
+        // default), permanently breaking `--apply`. `--dry-run` is now
+        // opt-in; the mutual-exclusion check below is correct without a
+        // special-case.
+        .option('--dry-run', 'preview writes without changing files')
         .option('--apply', 'write extracted memories into .peaks/memory/')).action((options) => {
         if (options.dryRun === true && options.apply === true) {
             printResult(io, fail('project.memories:extract', 'INVALID_MEMORY_EXTRACT_FLAGS', 'Use either --dry-run or --apply, not both', { sessionId: options.sessionId, projectRoot: options.project }, ['Run without --apply to preview writes, or pass --apply to write memories']), options.json);

package/dist/src/cli/commands/workflow-commands.js

@@ -10,6 +10,7 @@ import { validateChangeIdOrThrow } from '../../shared/change-id.js';
 import { getEconomyAwareExecutionModelId } from '../../services/config/model-routing.js';
 import { getLocalArtifactPath } from '../../services/artifacts/workspace-service.js';
 import { getSessionId } from '../../services/session/session-manager.js';
+import { getSessionDir } from '../../services/session/getSessionDir.js';
 import { findProjectRoot } from '../../services/config/config-safety.js';
 import { verifyPipeline } from '../../services/workflow/pipeline-verify-service.js';
 import { fail, ok } from '../../shared/result.js';
@@ -18,7 +19,7 @@ function getCurrentWorkspaceContext() {
     try {
         const projectRoot = findProjectRoot(process.cwd()) ?? process.cwd();
         const sessionId = getSessionId(projectRoot);
-        return sessionId ? { sessionId, sessionDir: `.peaks/${sessionId}` } : {};
+        return sessionId ? { sessionId, sessionDir: getSessionDir(projectRoot, sessionId) } : {};
     }
     catch {
         return {};

package/dist/src/cli/commands/workspace-commands.js

@@ -449,8 +449,7 @@ export async function resolveFirstTimeHooksInstall(options) {
     // effectiveMode === 'ask' AND TTY: prompt once.
     process.stderr.write('\nPeaks-Cli: install the PreToolUse hooks for this project now?\n' +
         '  → Bash matcher: `peaks gate enforce` (SOP gate enforcement)\n' +
-        '  → Task matcher: `peaks progress start` (auto-spawn sub-agent progress terminal)\n' +
-        'Both run on every Claude Code tool call without further prompting. The decision is sticky\n' +
+        'The gate-enforce hook runs on every Claude Code tool call without further prompting. The decision is sticky\n' +
         '(recorded in .peaks/.peaks-init-hooks-decision.json) and re-runs of `workspace init` will\n' +
         'honour it. Re-run with --install-hooks=skip or --install-hooks=auto to override.\n\n' +
         'Install now? [Y/n]: ');

package/dist/src/cli/program.js

@@ -10,7 +10,9 @@ import { registerCodegraphCommands } from './commands/codegraph-commands.js';
 import { registerMcpCommands } from './commands/mcp-commands.js';
 import { registerOpenSpecCommands } from './commands/openspec-commands.js';
 import { registerPerfCommands } from './commands/perf-commands.js';
-import { registerProgressCommands } from './commands/progress-commands.js';
+// Slice #014: peaks progress * CLI surface deleted (replaced by sub-agent
+// dispatch + heartbeat, slice #009 + #010). Sub-agent progress is
+// surfaced via `peaks sub-agent dispatch|heartbeat|share`.
 import { registerProjectCommands } from './commands/project-commands.js';
 import { registerRequestCommands } from './commands/request-commands.js';
 import { registerScanCommands } from './commands/scan-commands.js';
@@ -88,7 +90,6 @@ Run peaks (no arguments) for a quickstart. You likely want one of:
     registerMcpCommands(program, io);
     registerOpenSpecCommands(program, io);
     registerPerfCommands(program, io);
-    registerProgressCommands(program, io);
     registerProjectCommands(program, io);
     registerRequestCommands(program, io);
     registerScanCommands(program, io);

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

@@ -27,6 +27,26 @@ export type ProjectDashboardDoctor = {
     ok: boolean;
     passed: number;
     failed: number;
+    okCount?: number;
+    failCount?: number;
+    lastRunAt?: string;
+    checkIds?: string[];
+};
+export type DashboardOkPolicy = 'workspace-only' | 'strict';
+/**
+ * Resolves the user-facing `ok` field. `workspace-only` (default) returns true
+ * when the runbook / workspace layout is healthy, even if 1-2 non-blocking
+ * doctor checks fail. `strict` returns false when the doctor aggregate fails.
+ * The CLI default is `workspace-only`; `peaks project dashboard --strict`
+ * restores the legacy aggregate semantics.
+ */
+export declare function resolveDashboardOk(args: {
+    okPolicy: DashboardOkPolicy;
+    doctor: ProjectDashboardDoctor;
+    runbookHealth: ProjectDashboardRunbookHealth;
+}): {
+    ok: boolean;
+    okPolicy: DashboardOkPolicy;
 };
 export type ProjectDashboardRunbookHealth = {
     ok: boolean;
@@ -51,6 +71,8 @@ export type ProjectDashboardSkillPresence = {
 export type ProjectDashboard = {
     generatedAt: string;
     projectRoot: string;
+    ok: boolean;
+    okPolicy: DashboardOkPolicy;
     requests: ProjectDashboardRequests;
     openspec: ProjectDashboardOpenSpec;
     understand: ProjectDashboardUnderstand;
@@ -71,5 +93,6 @@ export type LoadProjectDashboardOptions = {
     };
     runbookHealth?: ProjectDashboardRunbookHealth;
     skillPresence?: SkillPresence | null;
+    okPolicy?: DashboardOkPolicy;
 };
 export declare function loadProjectDashboard(options: LoadProjectDashboardOptions): Promise<ProjectDashboard>;

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

@@ -6,6 +6,19 @@ import { seedCapabilityItems } from '../recommendations/capability-seed-items.js
 import { requiredSkillNames } from '../../shared/paths.js';
 import { getSkillPresence } from '../skills/skill-presence-service.js';
 const SKILL_PRESENCE_FRESHNESS_THRESHOLD_MS = 24 * 60 * 60 * 1000;
+/**
+ * Resolves the user-facing `ok` field. `workspace-only` (default) returns true
+ * when the runbook / workspace layout is healthy, even if 1-2 non-blocking
+ * doctor checks fail. `strict` returns false when the doctor aggregate fails.
+ * The CLI default is `workspace-only`; `peaks project dashboard --strict`
+ * restores the legacy aggregate semantics.
+ */
+export function resolveDashboardOk(args) {
+    if (args.okPolicy === 'strict') {
+        return { ok: args.doctor.ok && args.runbookHealth.ok, okPolicy: 'strict' };
+    }
+    return { ok: args.runbookHealth.ok, okPolicy: 'workspace-only' };
+}
 function defaultClock() {
     return new Date().toISOString();
 }
@@ -95,6 +108,7 @@ function buildSkillPresenceSummary(presence, projectRoot) {
 export async function loadProjectDashboard(options) {
     const clock = options.clock ?? defaultClock;
     const sampleSize = options.sampleCapabilities ?? 8;
+    const okPolicy = options.okPolicy ?? 'workspace-only';
     const [items, openspecReport, mcpReport, understandReport, doctorAndRunbook] = await Promise.all([
         listRequestArtifacts({ projectRoot: options.projectRoot }),
         scanOpenSpec({ openspecRoot: `${options.projectRoot}/openspec` }),
@@ -102,9 +116,16 @@ export async function loadProjectDashboard(options) {
         scanUnderstandAnything({ projectRoot: options.projectRoot }),
         loadDoctorAndRunbookHealth(options.doctorReport, options.runbookHealth)
     ]);
+    const okVerdict = resolveDashboardOk({
+        okPolicy,
+        doctor: doctorAndRunbook.doctor,
+        runbookHealth: doctorAndRunbook.runbookHealth
+    });
     return {
         generatedAt: clock(),
         projectRoot: options.projectRoot,
+        ok: okVerdict.ok,
+        okPolicy: okVerdict.okPolicy,
         requests: {
             count: items.length,
             byRole: groupRequestsByRole(items),

package/dist/src/services/dispatch/sub-agent-dispatcher.d.ts

@@ -8,29 +8,29 @@
  * here, never leaked to SKILL.md.
  *
  * Why this exists:
- *   - Prior SKILL.md hardcoded `Task(subagent_type="general-purpose", ...)`
- *     which made peaks-cli depend on Claude Code's specific sub-agent
- *     tool name. Adding a new IDE (Trae, future Cursor, etc.) required
- *     editing every SKILL.md that mentioned sub-agent dispatch.
- *   - This file (plus the per-IDE adapter wiring) collapses all
- *     per-IDE sub-agent specifics to a single `SubAgentDispatcher`
- *     instance per adapter. SKILL.md now only references
- *     `peaks sub-agent dispatch <role>`, and the IDE-private tool
- *     name flows through the returned `data.toolCall` at runtime.
+ * - Prior SKILL.md hardcoded `Task(subagent_type="general-purpose", ...)`
+ * which made peaks-cli depend on Claude Code's specific sub-agent
+ * tool name. Adding a new IDE (Trae, future Cursor, etc.) required
+ * editing every SKILL.md that mentioned sub-agent dispatch.
+ * - This file (plus the per-IDE adapter wiring) collapses all
+ * per-IDE sub-agent specifics to a single `SubAgentDispatcher`
+ * instance per adapter. SKILL.md now only references
+ * `peaks sub-agent dispatch <role>`, and the IDE-private tool
+ * name flows through the returned `data.toolCall` at runtime.
  *
  * Cross-reference: PRD #002 G1 (AC-1..AC-5); RD tech-doc-002 §2.
  */
 /**
  * Role string namespace. Soft whitelist — the CLI does NOT hard-validate
- * specific role names. Empirically observed (peaks-qa SKILL.md): 3 top
- * roles + 3 sub-roles + arbitrary business subdivisions:
+ * specific role names. Empirically observed (peaks-qa SKILL.md):3 top
+ * roles +3 sub-roles + arbitrary business subdivisions:
  *
- *   - top:           rd | qa | ui | txt | general-purpose
- *   - qa sub-roles:  qa-business | qa-perf | qa-security
- *   - business细分:  qa-business-regression | qa-business-api
- *                    | qa-business-frontend | ...
- *   - promotable:    prd-business | prd-technical | prd-ux |
- *                    ui-visual | ui-flow | ui-component | ...
+ * - top: rd | qa | ui | txt | general-purpose
+ * - qa sub-roles: qa-business | qa-perf | qa-security
+ * - business细分: qa-business-regression | qa-business-api
+ * | qa-business-frontend | ...
+ * - promotable: prd-business | prd-technical | prd-ux |
+ * ui-visual | ui-flow | ui-component | ...
  *
  * Any non-empty string is a valid role. CLI emits a "soft whitelist"
  * hint in --help but does not reject unknown values.
@@ -64,43 +64,48 @@ export interface SubAgentDispatchInput {
  */
 export interface SubAgentDispatcher {
     /**
-     * Short label used in envelope `ide` field and CLI help text.
-     * e.g. "claude-code" / "trae" / "null".
-     */
+    * Short label used in envelope `ide` field and CLI help text.
+    * e.g. "claude-code" / "trae" / "null".
+    */
     readonly label: string;
     /**
-     * Whether this dispatcher supports dispatching a given role.
-     * claude-code returns true for all non-empty strings; trae is
-     * byte-identical (UNVERIFIED pending real Trae dogfood);
-     * null-dispatcher always returns false.
-     */
+    * Whether this dispatcher supports dispatching a given role.
+    * claude-code returns true for all non-empty strings; trae is
+    * byte-identical (UNVERIFIED pending real Trae dogfood);
+    * null-dispatcher always returns false.
+    */
     supportsRole(role: SubAgentRole): boolean;
     /**
-     * Build the IDE-specific tool call descriptor for a dispatch.
-     * Must be pure: no I/O, no side effects. The CLI wraps the
-     * returned descriptor in its JSON envelope.
-     */
+    * Build the IDE-specific tool call descriptor for a dispatch.
+    * Must be pure: no I/O, no side effects. The CLI wraps the
+    * returned descriptor in its JSON envelope.
+    */
     buildToolCall(input: SubAgentDispatchInput): SubAgentToolCall;
 }
 /**
  * Claude Code dispatcher. Real, byte-level implementation.
  *
- *   - `supportsRole`: any non-empty string (Claude Code's
- *     `general-purpose` sub-agent accepts any prompt).
- *   - `buildToolCall`: returns `{name: 'Task', args: {subagent_type,
- *     description, prompt}}` — the exact shape the `Task` tool
- *     in Claude Code expects.
+ * - `supportsRole`: any non-empty string (Claude Code's
+ * `general-purpose` sub-agent accepts any prompt).
+ * - `buildToolCall`: returns `{name: 'Task', args: {subagent_type,
+ * description, prompt}}` — the exact shape the `Task` tool
+ * in Claude Code expects.
  */
 export declare const claudeCodeSubAgentDispatcher: SubAgentDispatcher;
 /**
  * Trae dispatcher. UNVERIFIED — Trae sub-agent tool name TBD on real
  * dogfood. Byte-level identical to claude-code by design so:
- *   1. The slice #008 `subAgentToolMatcher: 'Task'` install entry
- *      stays byte-stable (the generated `.trae/settings.json` does
- *      not change).
- *   2. The dispatcher's return shape is uniform across both
- *      adapters — a single byte-equality test can verify the
- *      placeholder contract.
+ * - The dispatcher's return shape is uniform across both adapters
+ * — a single byte-equality test can verify the placeholder
+ * contract.
+ * - Future real Trae dogfood can replace the body of
+ * `buildToolCall` without breaking the adapter contract.
+ *
+ * Slice #014: the legacy `subAgentToolMatcher: 'Task'` install entry
+ * is gone — the field is removed from `IdeAdapter`. Slice #009+
+ * dispatched sub-agents directly, not via a PreToolUse hook. The Trae
+ * dispatcher remains a placeholder so the dispatch surface is uniform
+ * across adapters.
  *
  * When real Trae dogfood lands, replace the body of `buildToolCall`
  * with Trae's actual sub-agent tool name + args shape. The interface

package/dist/src/services/dispatch/sub-agent-dispatcher.js

@@ -8,26 +8,26 @@
  * here, never leaked to SKILL.md.
  *
  * Why this exists:
- *   - Prior SKILL.md hardcoded `Task(subagent_type="general-purpose", ...)`
- *     which made peaks-cli depend on Claude Code's specific sub-agent
- *     tool name. Adding a new IDE (Trae, future Cursor, etc.) required
- *     editing every SKILL.md that mentioned sub-agent dispatch.
- *   - This file (plus the per-IDE adapter wiring) collapses all
- *     per-IDE sub-agent specifics to a single `SubAgentDispatcher`
- *     instance per adapter. SKILL.md now only references
- *     `peaks sub-agent dispatch <role>`, and the IDE-private tool
- *     name flows through the returned `data.toolCall` at runtime.
+ * - Prior SKILL.md hardcoded `Task(subagent_type="general-purpose", ...)`
+ * which made peaks-cli depend on Claude Code's specific sub-agent
+ * tool name. Adding a new IDE (Trae, future Cursor, etc.) required
+ * editing every SKILL.md that mentioned sub-agent dispatch.
+ * - This file (plus the per-IDE adapter wiring) collapses all
+ * per-IDE sub-agent specifics to a single `SubAgentDispatcher`
+ * instance per adapter. SKILL.md now only references
+ * `peaks sub-agent dispatch <role>`, and the IDE-private tool
+ * name flows through the returned `data.toolCall` at runtime.
  *
  * Cross-reference: PRD #002 G1 (AC-1..AC-5); RD tech-doc-002 §2.
  */
 /**
  * Claude Code dispatcher. Real, byte-level implementation.
  *
- *   - `supportsRole`: any non-empty string (Claude Code's
- *     `general-purpose` sub-agent accepts any prompt).
- *   - `buildToolCall`: returns `{name: 'Task', args: {subagent_type,
- *     description, prompt}}` — the exact shape the `Task` tool
- *     in Claude Code expects.
+ * - `supportsRole`: any non-empty string (Claude Code's
+ * `general-purpose` sub-agent accepts any prompt).
+ * - `buildToolCall`: returns `{name: 'Task', args: {subagent_type,
+ * description, prompt}}` — the exact shape the `Task` tool
+ * in Claude Code expects.
  */
 export const claudeCodeSubAgentDispatcher = {
     label: 'claude-code',
@@ -44,12 +44,17 @@ export const claudeCodeSubAgentDispatcher = {
 /**
  * Trae dispatcher. UNVERIFIED — Trae sub-agent tool name TBD on real
  * dogfood. Byte-level identical to claude-code by design so:
- *   1. The slice #008 `subAgentToolMatcher: 'Task'` install entry
- *      stays byte-stable (the generated `.trae/settings.json` does
- *      not change).
- *   2. The dispatcher's return shape is uniform across both
- *      adapters — a single byte-equality test can verify the
- *      placeholder contract.
+ * - The dispatcher's return shape is uniform across both adapters
+ * — a single byte-equality test can verify the placeholder
+ * contract.
+ * - Future real Trae dogfood can replace the body of
+ * `buildToolCall` without breaking the adapter contract.
+ *
+ * Slice #014: the legacy `subAgentToolMatcher: 'Task'` install entry
+ * is gone — the field is removed from `IdeAdapter`. Slice #009+
+ * dispatched sub-agents directly, not via a PreToolUse hook. The Trae
+ * dispatcher remains a placeholder so the dispatch surface is uniform
+ * across adapters.
  *
  * When real Trae dogfood lands, replace the body of `buildToolCall`
  * with Trae's actual sub-agent tool name + args shape. The interface