@productbrain/cli 0.1.0-beta.15 → 0.1.0-beta.1522

package/dist/commands/__tests__/setup-detect-surfaces.test.js

@@ -0,0 +1,149 @@
+/**
+ * WP-433 S5 — `formatDetectSurfacesSummary` pure unit tests.
+ *
+ * Per INS-1420: runSetupDetectSurfaces makes live server calls via getConfigSafe()
+ * and kernelCall. These tests ONLY exercise the pure formatter on hand-crafted
+ * SetupDetectSurfacesResult objects — runSetupDetectSurfaces is never called.
+ *
+ * Test cases:
+ *  1. Bare-repo (zero detected, zero existing) → appends locked guidance question
+ *  2. Normal detect (new surfaces detected) → does NOT append
+ *  3. Idempotent re-run (existing surfaces, zero new) → does NOT append
+ *  4. Server-stamp-error case → surfaces error in summary, does NOT append extra question
+ */
+import { describe, expect, it } from 'vitest';
+import { formatDetectSurfacesSummary, } from '../setup-detect-surfaces.js';
+const LOCKED_QUESTION = "I don't see Claude Code or Cursor — which AI tool do you use?";
+/** Helper: build a minimal SetupDetectSurfacesResult */
+function makeResult(overrides = {}) {
+    return {
+        detected: [],
+        existing: [],
+        newlyAdded: [],
+        union: [],
+        manifestUpdated: false,
+        manifestPath: '/repo/.productbrain/manifest.json',
+        serverStamp: null,
+        serverStampError: null,
+        ...overrides,
+    };
+}
+describe('formatDetectSurfacesSummary — WP-433 S5 bare-repo guidance', () => {
+    it('bare-repo (zero detected, zero existing) appends locked guidance question', () => {
+        const result = makeResult({
+            detected: [],
+            existing: [],
+            newlyAdded: [],
+            union: [],
+            manifestUpdated: false,
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        expect(summary).toContain(LOCKED_QUESTION);
+        // Confirm the question is on a separate line
+        expect(summary).toContain('\n' + LOCKED_QUESTION);
+    });
+    it('normal detect (new surfaces detected) does NOT append locked question', () => {
+        const result = makeResult({
+            detected: ['claude'],
+            existing: [],
+            newlyAdded: ['claude'],
+            union: ['claude'],
+            manifestUpdated: true,
+            serverStamp: { stamped: true, surfaces: ['claude'] },
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        expect(summary).not.toContain(LOCKED_QUESTION);
+        // Confirm surfaces are mentioned
+        expect(summary).toContain('claude');
+    });
+    it('STD-235 remnant (zero detected, existing surfaces remain in manifest) does NOT append locked question', () => {
+        // Marker dir deleted from disk; STD-235 forward-only keeps the surface in manifest.
+        // Detected is empty but existing is not — guard must NOT trigger bare-repo guidance.
+        const result = makeResult({
+            detected: [],
+            existing: ['claude'],
+            newlyAdded: [],
+            union: ['claude'],
+            manifestUpdated: false,
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        expect(summary).not.toContain(LOCKED_QUESTION);
+        expect(summary).toContain('Detected: none');
+        expect(summary).toContain('Existing: claude');
+    });
+    it('idempotent re-run (existing surfaces, zero new detected) does NOT append locked question', () => {
+        const result = makeResult({
+            detected: ['claude'],
+            existing: ['claude'],
+            newlyAdded: [],
+            union: ['claude'],
+            manifestUpdated: false,
+            serverStamp: { stamped: false, surfaces: ['claude'] },
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        expect(summary).not.toContain(LOCKED_QUESTION);
+        // Existing should appear in summary
+        expect(summary).toContain('claude');
+    });
+    it('server-stamp-error case surfaces the error and does NOT append the locked question', () => {
+        // If detect-surfaces fails at the server stamp but DID find surfaces locally,
+        // the error should appear but the bare-repo question should not.
+        const result = makeResult({
+            detected: ['cursor'],
+            existing: [],
+            newlyAdded: ['cursor'],
+            union: ['cursor'],
+            manifestUpdated: true,
+            serverStamp: null,
+            serverStampError: 'Network timeout connecting to workspace',
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        // Error is surfaced
+        expect(summary).toContain('Server stamp failed: Network timeout connecting to workspace');
+        // Bare-repo question is NOT appended (surfaces were found)
+        expect(summary).not.toContain(LOCKED_QUESTION);
+    });
+    it('bare-repo with server-stamp-error (edge case: zero surfaces + server error) appends locked question', () => {
+        // Zero detected + zero existing → bare-repo. Server stamp would not have been
+        // called (union is empty), but if serverStampError somehow populated, the
+        // bare-repo signal still dominates and the question should appear.
+        const result = makeResult({
+            detected: [],
+            existing: [],
+            newlyAdded: [],
+            union: [],
+            manifestUpdated: false,
+            serverStamp: null,
+            serverStampError: 'Config missing',
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        // Server stamp error is surfaced
+        expect(summary).toContain('Server stamp failed: Config missing');
+        // Bare-repo guidance still applies
+        expect(summary).toContain(LOCKED_QUESTION);
+    });
+    it('summary has correct shape for bare-repo result', () => {
+        const result = makeResult();
+        const summary = formatDetectSurfacesSummary(result);
+        // Base line before the question
+        expect(summary).toMatch(/^Detected: none \(new: none\)\. Existing: none\. Manifest unchanged\./);
+    });
+    it('summary has correct shape for normal detect', () => {
+        const result = makeResult({
+            detected: ['claude', 'cursor'],
+            existing: ['claude'],
+            newlyAdded: ['cursor'],
+            union: ['claude', 'cursor'],
+            manifestUpdated: true,
+            serverStamp: { stamped: true, surfaces: ['claude', 'cursor'] },
+        });
+        const summary = formatDetectSurfacesSummary(result);
+        expect(summary).toContain('Detected: claude, cursor');
+        expect(summary).toContain('new: cursor');
+        expect(summary).toContain('Existing: claude');
+        expect(summary).toContain('Manifest updated.');
+        expect(summary).toContain('Server stamped.');
+        expect(summary).not.toContain(LOCKED_QUESTION);
+    });
+});
+//# sourceMappingURL=setup-detect-surfaces.test.js.map

package/dist/commands/__tests__/setup-detect-surfaces.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"setup-detect-surfaces.test.js","sourceRoot":"","sources":["../../../src/commands/__tests__/setup-detect-surfaces.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,2BAA2B,GAE5B,MAAM,6BAA6B,CAAC;AAErC,MAAM,eAAe,GAAG,+DAA+D,CAAC;AAExF,wDAAwD;AACxD,SAAS,UAAU,CACjB,YAAgD,EAAE;IAElD,OAAO;QACL,QAAQ,EAAE,EAAE;QACZ,QAAQ,EAAE,EAAE;QACZ,UAAU,EAAE,EAAE;QACd,KAAK,EAAE,EAAE;QACT,eAAe,EAAE,KAAK;QACtB,YAAY,EAAE,mCAAmC;QACjD,WAAW,EAAE,IAAI;QACjB,gBAAgB,EAAE,IAAI;QACtB,GAAG,SAAS;KACb,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,4DAA4D,EAAE,GAAG,EAAE;IAC1E,EAAE,CAAC,2EAA2E,EAAE,GAAG,EAAE;QACnF,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,EAAE;YACZ,QAAQ,EAAE,EAAE;YACZ,UAAU,EAAE,EAAE;YACd,KAAK,EAAE,EAAE;YACT,eAAe,EAAE,KAAK;SACvB,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC3C,6CAA6C;QAC7C,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,IAAI,GAAG,eAAe,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,QAAQ,EAAE,EAAE;YACZ,UAAU,EAAE,CAAC,QAAQ,CAAC;YACtB,KAAK,EAAE,CAAC,QAAQ,CAAC;YACjB,eAAe,EAAE,IAAI;YACrB,WAAW,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,QAAQ,CAAC,EAAE;SACrD,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC/C,iCAAiC;QACjC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uGAAuG,EAAE,GAAG,EAAE;QAC/G,oFAAoF;QACpF,qFAAqF;QACrF,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,EAAE;YACZ,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,UAAU,EAAE,EAAE;YACd,KAAK,EAAE,CAAC,QAAQ,CAAC;YACjB,eAAe,EAAE,KAAK;SACvB,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC/C,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC5C,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0FAA0F,EAAE,GAAG,EAAE;QAClG,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,UAAU,EAAE,EAAE;YACd,KAAK,EAAE,CAAC,QAAQ,CAAC;YACjB,eAAe,EAAE,KAAK;YACtB,WAAW,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,QAAQ,CAAC,EAAE;SACtD,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QAC/C,oCAAoC;QACpC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oFAAoF,EAAE,GAAG,EAAE;QAC5F,8EAA8E;QAC9E,iEAAiE;QACjE,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,QAAQ,EAAE,EAAE;YACZ,UAAU,EAAE,CAAC,QAAQ,CAAC;YACtB,KAAK,EAAE,CAAC,QAAQ,CAAC;YACjB,eAAe,EAAE,IAAI;YACrB,WAAW,EAAE,IAAI;YACjB,gBAAgB,EAAE,yCAAyC;SAC5D,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,oBAAoB;QACpB,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,8DAA8D,CAAC,CAAC;QAC1F,2DAA2D;QAC3D,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qGAAqG,EAAE,GAAG,EAAE;QAC7G,8EAA8E;QAC9E,0EAA0E;QAC1E,mEAAmE;QACnE,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,EAAE;YACZ,QAAQ,EAAE,EAAE;YACZ,UAAU,EAAE,EAAE;YACd,KAAK,EAAE,EAAE;YACT,eAAe,EAAE,KAAK;YACtB,WAAW,EAAE,IAAI;YACjB,gBAAgB,EAAE,gBAAgB;SACnC,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,iCAAiC;QACjC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,qCAAqC,CAAC,CAAC;QACjE,mCAAmC;QACnC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,UAAU,EAAE,CAAC;QAC5B,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,gCAAgC;QAChC,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,uEAAuE,CAAC,CAAC;IACnG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,MAAM,GAAG,UAAU,CAAC;YACxB,QAAQ,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC;YAC9B,QAAQ,EAAE,CAAC,QAAQ,CAAC;YACpB,UAAU,EAAE,CAAC,QAAQ,CAAC;YACtB,KAAK,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC;YAC3B,eAAe,EAAE,IAAI;YACrB,WAAW,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,QAAQ,EAAE,QAAQ,CAAC,EAAE;SAC/D,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAC;QACtD,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;QACzC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;QAC9C,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAC/C,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAC7C,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/commands/__tests__/setup-state.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=setup-state.test.d.ts.map

package/dist/commands/__tests__/setup-state.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"setup-state.test.d.ts","sourceRoot":"","sources":["../../../src/commands/__tests__/setup-state.test.ts"],"names":[],"mappings":""}

package/dist/commands/__tests__/setup-state.test.js

@@ -0,0 +1,194 @@
+/**
+ * WP-431 S3 — `pb setup-state` and `pb setup` alias tests.
+ *
+ * Mocks kernelCall (gateway boundary) so error/auth/network paths and the
+ * recordInvocation idempotence behaviour are exercised without a real Convex
+ * deployment.
+ */
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { mkdirSync, mkdtempSync, rmSync } from 'fs';
+import { dirname, join } from 'path';
+import { fileURLToPath } from 'url';
+import { tmpdir } from 'os';
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+vi.mock('../../lib/client.js', async () => {
+    class FakeMcpError extends Error {
+        code;
+        constructor(message, code) {
+            super(message);
+            this.name = 'McpError';
+            this.code = code;
+        }
+    }
+    return {
+        kernelCall: vi.fn(),
+        McpError: FakeMcpError,
+    };
+});
+import { runSetupState, fetchSetupState } from '../setup-state.js';
+import { runSetup } from '../setup.js';
+import { kernelCall, McpError } from '../../lib/client.js';
+const mockedKernelCall = vi.mocked(kernelCall);
+let scratch;
+let originalCwd;
+let exitSpy;
+let stdoutSpy;
+let stderrSpy;
+beforeEach(() => {
+    originalCwd = process.cwd();
+    scratch = mkdtempSync(join(tmpdir(), 'pb-setup-state-'));
+    process.chdir(scratch);
+    mockedKernelCall.mockReset();
+    exitSpy = vi.spyOn(process, 'exit').mockImplementation(((_code) => {
+        throw new Error(`__exit:${_code ?? 0}`);
+    }));
+    stdoutSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
+});
+afterEach(() => {
+    process.chdir(originalCwd);
+    rmSync(scratch, { recursive: true, force: true });
+    exitSpy.mockRestore();
+    stdoutSpy.mockRestore();
+    stderrSpy.mockRestore();
+});
+const SAMPLE = {
+    phase: 1,
+    role: 'creator',
+    expectedAction: 'declare a surface',
+    reasonForPhase: 'manifest.surfaces empty',
+    isComplete: false,
+    surfaceCapability: 'cli',
+};
+describe('fetchSetupState()', () => {
+    it('returns the typed result from setup.getPbSetupState', async () => {
+        mockedKernelCall.mockResolvedValueOnce(SAMPLE);
+        const result = await fetchSetupState();
+        expect(result).toEqual(SAMPLE);
+        expect(mockedKernelCall).toHaveBeenCalledWith('setup.getPbSetupState', {});
+    });
+});
+describe('runSetupState()', () => {
+    it('writes compact JSON to stdout on success', async () => {
+        mockedKernelCall.mockResolvedValueOnce(SAMPLE);
+        await runSetupState();
+        const written = stdoutSpy.mock.calls.map((c) => String(c[0])).join('');
+        const parsed = JSON.parse(written.trim());
+        expect(parsed).toEqual(SAMPLE);
+    });
+    it('exits 2 on AUTH_REQUIRED', async () => {
+        mockedKernelCall.mockRejectedValueOnce(new McpError('not member', 'AUTHZ_NOT_MEMBER'));
+        await expect(runSetupState()).rejects.toThrow('__exit:2');
+        const stderr = stderrSpy.mock.calls.map((c) => String(c[0])).join('');
+        expect(stderr).toContain('AUTH_DENIED');
+    });
+    it('exits 3 on network error (CLIError category=network)', async () => {
+        const { CLIError, ErrorCode } = await import('../../lib/errors.js');
+        mockedKernelCall.mockRejectedValueOnce(new CLIError('Could not reach host.', {
+            code: ErrorCode.NETWORK_UNREACHABLE,
+            category: 'network',
+        }));
+        await expect(runSetupState()).rejects.toThrow('__exit:3');
+    });
+    it('emits JSON to stdout regardless of tty (non-tty path)', async () => {
+        Object.defineProperty(process.stdout, 'isTTY', { value: false, configurable: true });
+        mockedKernelCall.mockResolvedValueOnce(SAMPLE);
+        await runSetupState();
+        const written = stdoutSpy.mock.calls.map((c) => String(c[0])).join('');
+        expect(written.trim().startsWith('{')).toBe(true);
+    });
+});
+describe('pb start ≠ pb setup (DEC-995 invariant)', () => {
+    it('runSessionStart is the binding for `pb start`, not runSetup', async () => {
+        const fs = await import('fs');
+        const indexSrc = fs.readFileSync(join(__dirname, '..', '..', 'index.ts'), 'utf8');
+        // pb start dispatches to runSessionStart
+        expect(indexSrc).toContain("args[0] === 'start'");
+        const startSection = indexSrc.split("args[0] === 'start'")[1] ?? '';
+        const closeSection = startSection.split("args[0] === 'close'")[0] ?? '';
+        expect(closeSection).toContain('runSessionStart');
+        expect(closeSection).not.toContain('runSetup(');
+    });
+});
+describe('runSetup() (pb setup alias)', () => {
+    beforeEach(() => {
+        // Default to TTY=true so the human summary is exercised
+        Object.defineProperty(process.stdout, 'isTTY', { value: true, configurable: true });
+    });
+    it('Phase 1 → triggers detect-surfaces, re-reads state, prints final state', async () => {
+        // WP-433 S2: .claude dir triggers Claude surface detection → stampDetectedSurfaces
+        // is called as a 3rd kernelCall (between recordSetupInvocation and the re-read).
+        mkdirSync(join(scratch, '.claude'));
+        mockedKernelCall
+            .mockResolvedValueOnce({ ...SAMPLE, phase: 1 }) // initial getPbSetupState
+            .mockResolvedValueOnce({ firstSetupInvocationAt: 1, alreadyRecorded: false }) // recordSetupInvocation
+            .mockResolvedValueOnce({ stamped: true, surfaces: ['claude'] }) // stampDetectedSurfaces (WP-433 S2)
+            .mockResolvedValueOnce({ ...SAMPLE, phase: 2 }); // re-read after detect
+        await runSetup({ cwd: scratch });
+        const stdout = stdoutSpy.mock.calls.map((c) => String(c[0])).join('');
+        expect(stdout).toContain('"phase":2');
+    });
+    it('Phase 2+ → no auto-detect, prints initial state', async () => {
+        const phase2 = { ...SAMPLE, phase: 2 };
+        mockedKernelCall
+            .mockResolvedValueOnce(phase2)
+            .mockResolvedValueOnce({ firstSetupInvocationAt: 1, alreadyRecorded: true });
+        await runSetup({ cwd: scratch });
+        const calls = mockedKernelCall.mock.calls.map((c) => c[0]);
+        expect(calls.filter((c) => c === 'setup.getPbSetupState')).toHaveLength(1);
+    });
+    it('records invocation exactly once per pb-setup invocation', async () => {
+        const phase2 = { ...SAMPLE, phase: 2 };
+        mockedKernelCall
+            .mockResolvedValueOnce(phase2)
+            .mockResolvedValueOnce({ firstSetupInvocationAt: 1, alreadyRecorded: false });
+        await runSetup({ cwd: scratch });
+        const invocationCalls = mockedKernelCall.mock.calls.filter((c) => c[0] === 'setup.recordSetupInvocation');
+        expect(invocationCalls).toHaveLength(1);
+        expect(invocationCalls[0][1]).toMatchObject({ surfaceCapability: 'cli' });
+    });
+    it('records invocation with chat-only when state.surfaceCapability=chat-only (WP-431 S3 review fix #1)', async () => {
+        // Pre-revision: this test fabricated a `reasonForPhase` string the producer
+        // never emits and asserted that the (now-deleted) `inferSurfaceCapability`
+        // string-greppy heuristic returned 'chat-only'. The heuristic was broken
+        // (producer never emits 'chat-only' in reasonForPhase), so every chat-only
+        // user was silently tagged 'cli'. Post-revision, the resolver surfaces
+        // `surfaceCapability` as a structural field — the CLI reads it directly.
+        const chatOnly = {
+            ...SAMPLE,
+            phase: 3,
+            role: 'creator',
+            reasonForPhase: 'Phase 2 done (surface-skipped per DEC-993); catalogAdoptionCount=0 → Phase 3.',
+            surfaceCapability: 'chat-only',
+        };
+        mockedKernelCall
+            .mockResolvedValueOnce(chatOnly)
+            .mockResolvedValueOnce({ firstSetupInvocationAt: 1, alreadyRecorded: false });
+        await runSetup({ cwd: scratch });
+        const invocationCalls = mockedKernelCall.mock.calls.filter((c) => c[0] === 'setup.recordSetupInvocation');
+        expect(invocationCalls[0][1]).toMatchObject({ surfaceCapability: 'chat-only' });
+    });
+    it('chat-only resolver path: empty manifest + chat-only state → state.surfaceCapability="chat-only" (live resolver)', async () => {
+        // WP-431 S3 review fix #1: assert the actual resolver emits surfaceCapability
+        // structurally. This is a unit-level pin against the *resolver* (not the
+        // CLI handler), so we exercise resolveSetupState directly with a
+        // chat-only input shape that DOES occur in production
+        // (manifest.surfaces=['mcp-only'] is the upstream stamp for chat-only
+        // surfaces per DEC-993).
+        const { resolveSetupState } = await import('../../setup/state-machine.js');
+        const out = resolveSetupState({
+            manifest: { surfaces: ['mcp-only'] },
+            assetCounts: { totalAssets: 0, totalUnimported: 0, hasMore: false },
+            catalogAdoptionCount: 0,
+            confirmedEntryCount: 0,
+            entriesAcrossCollections: 0,
+            retrievalDemoConfirmed: false,
+            workspaceMaterialized: false,
+            userActivated: false,
+            surfaceCapability: 'chat-only',
+        });
+        expect(out.surfaceCapability).toBe('chat-only');
+    });
+});
+//# sourceMappingURL=setup-state.test.js.map

package/dist/commands/__tests__/setup-state.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"setup-state.test.js","sourceRoot":"","sources":["../../../src/commands/__tests__/setup-state.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AACpD,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AACrC,OAAO,EAAE,aAAa,EAAE,MAAM,KAAK,CAAC;AACpC,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAE5B,MAAM,UAAU,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AAClD,MAAM,SAAS,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC;AAEtC,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,KAAK,IAAI,EAAE;IACxC,MAAM,YAAa,SAAQ,KAAK;QAC9B,IAAI,CAAU;QACd,YAAY,OAAe,EAAE,IAAa;YACxC,KAAK,CAAC,OAAO,CAAC,CAAC;YACf,IAAI,CAAC,IAAI,GAAG,UAAU,CAAC;YACvB,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC;QACnB,CAAC;KACF;IACD,OAAO;QACL,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;QACnB,QAAQ,EAAE,YAAY;KACvB,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,OAAO,EAAE,aAAa,EAAE,eAAe,EAAyB,MAAM,mBAAmB,CAAC;AAC1F,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AACvC,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAC;AAE3D,MAAM,gBAAgB,GAAG,EAAE,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;AAE/C,IAAI,OAAe,CAAC;AACpB,IAAI,WAAmB,CAAC;AACxB,IAAI,OAAoC,CAAC;AACzC,IAAI,SAAsC,CAAC;AAC3C,IAAI,SAAsC,CAAC;AAE3C,UAAU,CAAC,GAAG,EAAE;IACd,WAAW,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAC5B,OAAO,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,iBAAiB,CAAC,CAAC,CAAC;IACzD,OAAO,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACvB,gBAAgB,CAAC,SAAS,EAAE,CAAC;IAC7B,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,KAAc,EAAE,EAAE;QACzE,MAAM,IAAI,KAAK,CAAC,UAAU,KAAK,IAAI,CAAC,EAAE,CAAC,CAAC;IAC1C,CAAC,CAAU,CAAC,CAAC;IACb,SAAS,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;IAC7E,SAAS,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;AAC/E,CAAC,CAAC,CAAC;AAEH,SAAS,CAAC,GAAG,EAAE;IACb,OAAO,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;IAC3B,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IAClD,OAAO,CAAC,WAAW,EAAE,CAAC;IACtB,SAAS,CAAC,WAAW,EAAE,CAAC;IACxB,SAAS,CAAC,WAAW,EAAE,CAAC;AAC1B,CAAC,CAAC,CAAC;AAEH,MAAM,MAAM,GAAqB;IAC/B,KAAK,EAAE,CAAC;IACR,IAAI,EAAE,SAAS;IACf,cAAc,EAAE,mBAAmB;IACnC,cAAc,EAAE,yBAAyB;IACzC,UAAU,EAAE,KAAK;IACjB,iBAAiB,EAAE,KAAK;CACzB,CAAC;AAEF,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,gBAAgB,CAAC,qBAAqB,CAAC,MAAM,CAAC,CAAC;QAC/C,MAAM,MAAM,GAAG,MAAM,eAAe,EAAE,CAAC;QACvC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAC/B,MAAM,CAAC,gBAAgB,CAAC,CAAC,oBAAoB,CAAC,uBAAuB,EAAE,EAAE,CAAC,CAAC;IAC7E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;QACxD,gBAAgB,CAAC,qBAAqB,CAAC,MAAM,CAAC,CAAC;QAC/C,MAAM,aAAa,EAAE,CAAC;QACtB,MAAM,OAAO,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClF,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;IACjC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0BAA0B,EAAE,KAAK,IAAI,EAAE;QACxC,gBAAgB,CAAC,qBAAqB,CAAC,IAAI,QAAQ,CAAC,YAAY,EAAE,kBAAkB,CAAC,CAAC,CAAC;QACvF,MAAM,MAAM,CAAC,aAAa,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;QAC1D,MAAM,MAAM,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,EAAE,QAAQ,EAAE,SAAS,EAAE,GAAG,MAAM,MAAM,CAAC,qBAAqB,CAAC,CAAC;QACpE,gBAAgB,CAAC,qBAAqB,CACpC,IAAI,QAAQ,CAAC,uBAAuB,EAAE;YACpC,IAAI,EAAE,SAAS,CAAC,mBAAmB;YACnC,QAAQ,EAAE,SAAS;SACpB,CAAC,CACH,CAAC;QACF,MAAM,MAAM,CAAC,aAAa,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,EAAE,KAAK,EAAE,KAAK,EAAE,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;QACrF,gBAAgB,CAAC,qBAAqB,CAAC,MAAM,CAAC,CAAC;QAC/C,MAAM,aAAa,EAAE,CAAC;QACtB,MAAM,OAAO,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAClF,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;IACvD,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,EAAE,GAAG,MAAM,MAAM,CAAC,IAAI,CAAC,CAAC;QAC9B,MAAM,QAAQ,GAAG,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,UAAU,CAAC,EAAE,MAAM,CAAC,CAAC;QAClF,yCAAyC;QACzC,MAAM,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QAClD,MAAM,YAAY,GAAG,QAAQ,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACpE,MAAM,YAAY,GAAG,YAAY,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACxE,MAAM,CAAC,YAAY,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAClD,MAAM,CAAC,YAAY,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;IAC3C,UAAU,CAAC,GAAG,EAAE;QACd,wDAAwD;QACxD,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,YAAY,EAAE,IAAI,EAAE,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;QACtF,mFAAmF;QACnF,iFAAiF;QACjF,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC,CAAC;QACpC,gBAAgB;aACb,qBAAqB,CAAC,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,0BAA0B;aACzE,qBAAqB,CAAC,EAAE,sBAAsB,EAAE,CAAC,EAAE,eAAe,EAAE,KAAK,EAAE,CAAC,CAAC,wBAAwB;aACrG,qBAAqB,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,oCAAoC;aACnG,qBAAqB,CAAC,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,uBAAuB;QAC1E,MAAM,QAAQ,CAAC,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACjC,MAAM,MAAM,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,MAAM,GAAqB,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;QACzD,gBAAgB;aACb,qBAAqB,CAAC,MAAM,CAAC;aAC7B,qBAAqB,CAAC,EAAE,sBAAsB,EAAE,CAAC,EAAE,eAAe,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/E,MAAM,QAAQ,CAAC,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACjC,MAAM,KAAK,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3D,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,uBAAuB,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC7E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,MAAM,GAAqB,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;QACzD,gBAAgB;aACb,qBAAqB,CAAC,MAAM,CAAC;aAC7B,qBAAqB,CAAC,EAAE,sBAAsB,EAAE,CAAC,EAAE,eAAe,EAAE,KAAK,EAAE,CAAC,CAAC;QAChF,MAAM,QAAQ,CAAC,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACjC,MAAM,eAAe,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CACxD,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,6BAA6B,CAC9C,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,EAAE,iBAAiB,EAAE,KAAK,EAAE,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oGAAoG,EAAE,KAAK,IAAI,EAAE;QAClH,4EAA4E;QAC5E,2EAA2E;QAC3E,yEAAyE;QACzE,2EAA2E;QAC3E,uEAAuE;QACvE,yEAAyE;QACzE,MAAM,QAAQ,GAAqB;YACjC,GAAG,MAAM;YACT,KAAK,EAAE,CAAC;YACR,IAAI,EAAE,SAAS;YACf,cAAc,EAAE,+EAA+E;YAC/F,iBAAiB,EAAE,WAAW;SAC/B,CAAC;QACF,gBAAgB;aACb,qBAAqB,CAAC,QAAQ,CAAC;aAC/B,qBAAqB,CAAC,EAAE,sBAAsB,EAAE,CAAC,EAAE,eAAe,EAAE,KAAK,EAAE,CAAC,CAAC;QAChF,MAAM,QAAQ,CAAC,EAAE,GAAG,EAAE,OAAO,EAAE,CAAC,CAAC;QACjC,MAAM,eAAe,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CACxD,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,6BAA6B,CAC9C,CAAC;QACF,MAAM,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,EAAE,iBAAiB,EAAE,WAAW,EAAE,CAAC,CAAC;IAClF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iHAAiH,EAAE,KAAK,IAAI,EAAE;QAC/H,8EAA8E;QAC9E,yEAAyE;QACzE,iEAAiE;QACjE,sDAAsD;QACtD,sEAAsE;QACtE,yBAAyB;QACzB,MAAM,EAAE,iBAAiB,EAAE,GAAG,MAAM,MAAM,CAAC,8BAA8B,CAAC,CAAC;QAC3E,MAAM,GAAG,GAAG,iBAAiB,CAAC;YAC5B,QAAQ,EAAE,EAAE,QAAQ,EAAE,CAAC,UAAU,CAAC,EAAE;YACpC,WAAW,EAAE,EAAE,WAAW,EAAE,CAAC,EAAE,eAAe,EAAE,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE;YACnE,oBAAoB,EAAE,CAAC;YACvB,mBAAmB,EAAE,CAAC;YACtB,wBAAwB,EAAE,CAAC;YAC3B,sBAAsB,EAAE,KAAK;YAC7B,qBAAqB,EAAE,KAAK;YAC5B,aAAa,EAAE,KAAK;YACpB,iBAAiB,EAAE,WAAW;SAC/B,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,CAAC,iBAAiB,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/commands/accept.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"accept.d.ts","sourceRoot":"","sources":["../../src/commands/accept.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAaH,UAAU,aAAa;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,OAAO,CAAC;CAChB;AAED,wBAAsB,SAAS,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,IAAI,CAAC,CA8DrE"}
+{"version":3,"file":"accept.d.ts","sourceRoot":"","sources":["../../src/commands/accept.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAcH,UAAU,aAAa;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,OAAO,CAAC;CAChB;AAED,wBAAsB,SAAS,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,IAAI,CAAC,CAiErE"}

package/dist/commands/accept.js

@@ -10,8 +10,9 @@
  * skipped for manual review. Reports what was approved and what was skipped.
  */
 import { getConfigOrGuide } from '../lib/config.js';
-import { mcpCall } from '../lib/client.js';
+import { kernelCall } from '../lib/client.js';
 import { runCliCommand } from '../lib/runner.js';
+import { CLIError, ErrorCode } from '../lib/errors.js';
 import { formatAcceptReceipt, formatAutoAcceptReport, } from '../formatters/proposals.js';
 export async function runAccept(options) {
     const config = await getConfigOrGuide(() => runAccept(options));
@@ -21,7 +22,7 @@ export async function runAccept(options) {
     if (options.auto) {
         await runCliCommand({
             fn: async () => {
-                const proposals = await mcpCall('governance.listProposals', {});
+                const proposals = await kernelCall('governance.listProposals', {});
                 const approved = [];
                 const skipped = [];
                 for (const p of proposals) {
@@ -37,7 +38,7 @@ export async function runAccept(options) {
                         continue;
                     }
                     try {
-                        await mcpCall('governance.respondToProposal', {
+                        await kernelCall('governance.respondToProposal', {
                             proposalId: p._id,
                             verdict: 'approve',
                         });
@@ -56,12 +57,15 @@ export async function runAccept(options) {
     }
     // ── Single proposal approve ────────────────────────────────────────
     if (!options.proposalId) {
-        process.stderr.write('Proposal ID is required. Usage: pb accept <id> or pb accept --auto\n');
-        process.exit(1);
+        throw new CLIError('Proposal ID is required.', {
+            code: ErrorCode.VALIDATION_FAILED,
+            category: 'validation',
+            guidance: 'Usage: pb accept <id> or pb accept --auto',
+        });
     }
     await runCliCommand({
         fn: async () => {
-            return await mcpCall('governance.respondToProposal', {
+            return await kernelCall('governance.respondToProposal', {
                 proposalId: options.proposalId,
                 verdict: 'approve',
             });

package/dist/commands/accept.js.map

@@ -1 +1 @@
-{"version":3,"file":"accept.js","sourceRoot":"","sources":["../../src/commands/accept.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AACpD,OAAO,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AAC3C,OAAO,EAAE,aAAa,EAAE,MAAM,kBAAkB,CAAC;AACjD,OAAO,EACL,mBAAmB,EACnB,sBAAsB,GAIvB,MAAM,4BAA4B,CAAC;AAOpC,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,OAAsB;IACpD,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC;IAChE,IAAI,CAAC,MAAM;QAAE,OAAO;IAEpB,sEAAsE;IACtE,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC;QACjB,MAAM,aAAa,CAAmB;YACpC,EAAE,EAAE,KAAK,IAAI,EAAE;gBACb,MAAM,SAAS,GAAG,MAAM,OAAO,CAAiB,0BAA0B,EAAE,EAAE,CAAC,CAAC;gBAEhF,MAAM,QAAQ,GAAiC,EAAE,CAAC;gBAClD,MAAM,OAAO,GAAgC,EAAE,CAAC;gBAEhD,KAAK,MAAM,CAAC,IAAI,SAAS,EAAE,CAAC;oBAC1B,MAAM,KAAK,GAAG,GAAG,CAAC,CAAC,kBAAkB,IAAI,CAAC,CAAC,eAAe,YAAY,CAAC,CAAC,cAAc,IAAI,CAAC,CAAC,WAAW,IAAI,SAAS,EAAE,CAAC;oBAEvH,yBAAyB;oBACzB,IAAI,CAAC,CAAC,SAAS,EAAE,CAAC;wBAChB,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;wBACtD,SAAS;oBACX,CAAC;oBAED,2FAA2F;oBAC3F,IAAI,CAAC,CAAC,eAAe,EAAE,CAAC;wBACtB,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,oBAAoB,CAAC,CAAC,eAAe,EAAE,EAAE,CAAC,CAAC;wBACpF,SAAS;oBACX,CAAC;oBAED,IAAI,CAAC;wBACH,MAAM,OAAO,CAAoB,8BAA8B,EAAE;4BAC/D,UAAU,EAAE,CAAC,CAAC,GAAG;4BACjB,OAAO,EAAE,SAAS;yBACnB,CAAC,CAAC;wBACH,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,CAAC,CAAC;oBACtC,CAAC;oBAAC,OAAO,GAAG,EAAE,CAAC;wBACb,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;wBAC7D,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;oBAClD,CAAC;gBACH,CAAC;gBAED,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;YAC/B,CAAC;YACD,YAAY,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,sBAAsB,CAAC,IAAI,CAAC;SACrD,CAAC,CAAC;QACH,OAAO;IACT,CAAC;IAED,sEAAsE;IACtE,IAAI,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC;QACxB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,sEAAsE,CAAC,CAAC;QAC7F,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAClB,CAAC;IAED,MAAM,aAAa,CAAoB;QACrC,EAAE,EAAE,KAAK,IAAI,EAAE;YACb,OAAO,MAAM,OAAO,CAAoB,8BAA8B,EAAE;gBACtE,UAAU,EAAE,OAAO,CAAC,UAAU;gBAC9B,OAAO,EAAE,SAAS;aACnB,CAAC,CAAC;QACL,CAAC;QACD,YAAY,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,mBAAmB,CAAC,IAAI,CAAC;KAClD,CAAC,CAAC;AACL,CAAC"}
+{"version":3,"file":"accept.js","sourceRoot":"","sources":["../../src/commands/accept.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AACpD,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAC9C,OAAO,EAAE,aAAa,EAAE,MAAM,kBAAkB,CAAC;AACjD,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EACL,mBAAmB,EACnB,sBAAsB,GAIvB,MAAM,4BAA4B,CAAC;AAOpC,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,OAAsB;IACpD,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC;IAChE,IAAI,CAAC,MAAM;QAAE,OAAO;IAEpB,sEAAsE;IACtE,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC;QACjB,MAAM,aAAa,CAAmB;YACpC,EAAE,EAAE,KAAK,IAAI,EAAE;gBACb,MAAM,SAAS,GAAG,MAAM,UAAU,CAAiB,0BAA0B,EAAE,EAAE,CAAC,CAAC;gBAEnF,MAAM,QAAQ,GAAiC,EAAE,CAAC;gBAClD,MAAM,OAAO,GAAgC,EAAE,CAAC;gBAEhD,KAAK,MAAM,CAAC,IAAI,SAAS,EAAE,CAAC;oBAC1B,MAAM,KAAK,GAAG,GAAG,CAAC,CAAC,kBAAkB,IAAI,CAAC,CAAC,eAAe,YAAY,CAAC,CAAC,cAAc,IAAI,CAAC,CAAC,WAAW,IAAI,SAAS,EAAE,CAAC;oBAEvH,yBAAyB;oBACzB,IAAI,CAAC,CAAC,SAAS,EAAE,CAAC;wBAChB,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;wBACtD,SAAS;oBACX,CAAC;oBAED,2FAA2F;oBAC3F,IAAI,CAAC,CAAC,eAAe,EAAE,CAAC;wBACtB,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,oBAAoB,CAAC,CAAC,eAAe,EAAE,EAAE,CAAC,CAAC;wBACpF,SAAS;oBACX,CAAC;oBAED,IAAI,CAAC;wBACH,MAAM,UAAU,CAAoB,8BAA8B,EAAE;4BAClE,UAAU,EAAE,CAAC,CAAC,GAAG;4BACjB,OAAO,EAAE,SAAS;yBACnB,CAAC,CAAC;wBACH,QAAQ,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,CAAC,CAAC;oBACtC,CAAC;oBAAC,OAAO,GAAG,EAAE,CAAC;wBACb,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;wBAC7D,OAAO,CAAC,IAAI,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;oBAClD,CAAC;gBACH,CAAC;gBAED,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;YAC/B,CAAC;YACD,YAAY,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,sBAAsB,CAAC,IAAI,CAAC;SACrD,CAAC,CAAC;QACH,OAAO;IACT,CAAC;IAED,sEAAsE;IACtE,IAAI,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC;QACxB,MAAM,IAAI,QAAQ,CAAC,0BAA0B,EAAE;YAC7C,IAAI,EAAE,SAAS,CAAC,iBAAiB;YACjC,QAAQ,EAAE,YAAY;YACtB,QAAQ,EAAE,2CAA2C;SACtD,CAAC,CAAC;IACL,CAAC;IAED,MAAM,aAAa,CAAoB;QACrC,EAAE,EAAE,KAAK,IAAI,EAAE;YACb,OAAO,MAAM,UAAU,CAAoB,8BAA8B,EAAE;gBACzE,UAAU,EAAE,OAAO,CAAC,UAAU;gBAC9B,OAAO,EAAE,SAAS;aACnB,CAAC,CAAC;QACL,CAAC;QACD,YAAY,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,mBAAmB,CAAC,IAAI,CAAC;KAClD,CAAC,CAAC;AACL,CAAC"}

package/dist/commands/admin/cockpit.d.ts

@@ -0,0 +1,90 @@
+/**
+ * Ink TUI cockpit for pb admin (WP-314 S1, WP-317 S3).
+ *
+ * WP-317 S3 adds:
+ *   - Persistent env banner (DEV/PROD) on every tab
+ *   - Tab navigation: Workspaces | Seeds | Changelog
+ *   - Tab/Shift+Tab keyboard switching; mouse enable/disable sequences (best-effort)
+ *   - Seeds tab: shows SEED_COMMAND_ENTRIES with static status (real runtime data in S4)
+ *
+ * Keyboard bindings:
+ *   Tab          Next tab
+ *   Shift+Tab    Previous tab
+ *   /            Enter filter mode (Workspaces + Seeds tabs)
+ *   Escape       Exit filter mode / clear filter, or exit cockpit
+ *   Up/Down      Navigate rows
+ *   Enter        Drill into selected workspace (Workspaces tab only)
+ *   q            Quit cockpit
+ *   ?            Toggle help overlay
+ */
+/** Mirrors NextCommand from adminBootstrap.ts (no cross-import). WP-314 S2. */
+export type NextCommand = {
+    label: string;
+    cmd: string;
+    safety: 'safe' | 'destructive';
+    estSeconds: number;
+    args?: Record<string, unknown>;
+};
+export type WorkspaceRow = {
+    _id: string;
+    name: string;
+    slug: string;
+    createdAt: number;
+    entryCount: number;
+    collectionCount: number;
+    systemCollectionVersion: number | null;
+    onboardingCompletedAt: number | null;
+    topIssue?: {
+        label: string;
+        nextCommand: NextCommand;
+    };
+};
+/** WP-317 S4 — seed row for Seeds tab, wired to real system_migrations data. */
+export type SeedRow = {
+    name: string;
+    description: string;
+    requiresWorkspaceId: boolean;
+    dryRunSupported: boolean;
+    migrationId: string;
+    argsHash: string;
+    appliedAt: number | null;
+    lastRunAt: number | null;
+    status: 'applied' | 'failed' | 'no-op' | 'unknown';
+};
+/** WP-317 E8 — changelog row for heatmap tab. */
+export type ChangelogRow = {
+    timestamp: number;
+    definitionSlug: string;
+};
+export type CockpitTab = 'workspaces' | 'seeds' | 'changelog' | 'manage';
+export type CockpitProps = {
+    workspaces: WorkspaceRow[];
+    currentKeyWorkspaceId: string | null;
+    onExit: () => void;
+    onDrillIn: (ws: WorkspaceRow) => void;
+    /** Called when user confirms 'f' fix for a row. Parent runs the command. */
+    onRunFix: (ws: WorkspaceRow, cmd: NextCommand) => Promise<void>;
+    /** Gateway siteUrl — used for env banner and envLabel derivation. WP-317 S3. */
+    siteUrl: string;
+    /** Seed rows for Seeds tab. WP-317 S3. */
+    seedRows: SeedRow[];
+    /** Changelog rows for heatmap. WP-317 E8 / S4. */
+    changelogRows: ChangelogRow[];
+    /** Called when ManageTab destructive flow becomes active/inactive. WP-324 S2. */
+    onFlowActive?: (active: boolean) => void;
+};
+export declare function Cockpit({ workspaces, currentKeyWorkspaceId, onExit, onDrillIn, onRunFix, siteUrl, seedRows, changelogRows, onFlowActive }: CockpitProps): import("react/jsx-runtime").JSX.Element;
+export type WorkspaceHealthData = {
+    readiness: unknown;
+    collections: unknown;
+    consistency: unknown;
+    drafts: unknown;
+};
+export type WorkspaceDetailProps = {
+    workspace: WorkspaceRow;
+    healthData: WorkspaceHealthData;
+    onExit: () => void;
+    onBack: () => void;
+};
+export declare function WorkspaceDetail({ workspace, healthData, onExit, onBack }: WorkspaceDetailProps): import("react/jsx-runtime").JSX.Element;
+//# sourceMappingURL=cockpit.d.ts.map

package/dist/commands/admin/cockpit.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"cockpit.d.ts","sourceRoot":"","sources":["../../../src/commands/admin/cockpit.tsx"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAaH,+EAA+E;AAC/E,MAAM,MAAM,WAAW,GAAG;IACxB,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,GAAG,aAAa,CAAC;IAC/B,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CAChC,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IACzB,GAAG,EAAE,MAAM,CAAC;IACZ,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,eAAe,EAAE,MAAM,CAAC;IACxB,uBAAuB,EAAE,MAAM,GAAG,IAAI,CAAC;IACvC,qBAAqB,EAAE,MAAM,GAAG,IAAI,CAAC;IACrC,QAAQ,CAAC,EAAE;QACT,KAAK,EAAE,MAAM,CAAC;QACd,WAAW,EAAE,WAAW,CAAC;KAC1B,CAAC;CACH,CAAC;AAEF,gFAAgF;AAChF,MAAM,MAAM,OAAO,GAAG;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,mBAAmB,EAAE,OAAO,CAAC;IAC7B,eAAe,EAAE,OAAO,CAAC;IACzB,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,MAAM,EAAE,SAAS,GAAG,QAAQ,GAAG,OAAO,GAAG,SAAS,CAAC;CACpD,CAAC;AAEF,iDAAiD;AACjD,MAAM,MAAM,YAAY,GAAG;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC;AAEF,MAAM,MAAM,UAAU,GAAG,YAAY,GAAG,OAAO,GAAG,WAAW,GAAG,QAAQ,CAAC;AAEzE,MAAM,MAAM,YAAY,GAAG;IACzB,UAAU,EAAE,YAAY,EAAE,CAAC;IAC3B,qBAAqB,EAAE,MAAM,GAAG,IAAI,CAAC;IACrC,MAAM,EAAE,MAAM,IAAI,CAAC;IACnB,SAAS,EAAE,CAAC,EAAE,EAAE,YAAY,KAAK,IAAI,CAAC;IACtC,4EAA4E;IAC5E,QAAQ,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,GAAG,EAAE,WAAW,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,gFAAgF;IAChF,OAAO,EAAE,MAAM,CAAC;IAChB,0CAA0C;IAC1C,QAAQ,EAAE,OAAO,EAAE,CAAC;IACpB,kDAAkD;IAClD,aAAa,EAAE,YAAY,EAAE,CAAC;IAC9B,iFAAiF;IACjF,YAAY,CAAC,EAAE,CAAC,MAAM,EAAE,OAAO,KAAK,IAAI,CAAC;CAC1C,CAAC;AAumBF,wBAAgB,OAAO,CAAC,EAAE,UAAU,EAAE,qBAAqB,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,OAAO,EAAE,QAAQ,EAAE,aAAa,EAAE,YAAY,EAAE,EAAE,YAAY,2CA+UvJ;AAuBD,MAAM,MAAM,mBAAmB,GAAG;IAChC,SAAS,EAAE,OAAO,CAAC;IACnB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,OAAO,CAAC;IACrB,MAAM,EAAE,OAAO,CAAC;CACjB,CAAC;AAEF,MAAM,MAAM,oBAAoB,GAAG;IACjC,SAAS,EAAE,YAAY,CAAC;IACxB,UAAU,EAAE,mBAAmB,CAAC;IAChC,MAAM,EAAE,MAAM,IAAI,CAAC;IACnB,MAAM,EAAE,MAAM,IAAI,CAAC;CACpB,CAAC;AAQF,wBAAgB,eAAe,CAAC,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,oBAAoB,2CAwH9F"}