@productbrain/cli 0.1.0-beta.14 → 0.1.0-beta.14010

package/dist/__tests__/gateway-timeout-body-phase.test.js

@@ -0,0 +1,116 @@
+/**
+ * gateway-timeout-body-phase.test.ts — WP-575 (TEN-2917, TEN-1126), PR #533 review round.
+ *
+ * THE DEFECT THIS PINS: `fetch` resolves as soon as HEADERS arrive, while the budget's
+ * AbortSignal is still live. A server that returns `200` and then stalls mid-body therefore
+ * aborts during `res.json()` — NOT during `fetch` — which put it outside the caller's
+ * timeout-classification `try`. The old blanket `.catch(() => ({}))` in `parseKernelEnvelope`
+ * swallowed that abort into an empty envelope, and a `200` status then read it as SUCCESS
+ * with `undefined` data. A timed-out write reported as completed: exactly the false-outcome
+ * this work package exists to remove, running in reverse.
+ *
+ * The mcp-server connector had the identical hole and it was fixed there earlier in this same
+ * PR (`throwClassifiedGatewayFailure`, mcp-server/src/gatewaySeam.ts). The CLI's
+ * kernel-call variants were the unswept siblings — so each is asserted here, not just
+ * the one the reviewer named.
+ */
+import { beforeEach, afterEach, describe, expect, it, vi } from 'vitest';
+import { kernelCall } from '../lib/client.js';
+import { CLIError, ErrorCode } from '../lib/errors.js';
+import { latencyBudgetMsForRoute } from '../generated/routeLatencyBudget.generated.js';
+const realFetch = global.fetch;
+/**
+ * A `Response` whose headers have already arrived but whose BODY rejects the way
+ * `AbortSignal.timeout` really does — a DOMException named "TimeoutError". The client
+ * discriminates on that `name`, so a plain `new Error()` would not exercise the branch.
+ */
+function responseThatStallsMidBody(status = 200, abortName = 'TimeoutError') {
+    return {
+        ok: status >= 200 && status < 300,
+        status,
+        statusText: 'OK',
+        headers: { get: () => null },
+        json: async () => {
+            const err = new Error('The operation was aborted due to timeout');
+            err.name = abortName;
+            throw err;
+        },
+    };
+}
+describe('CLI budget abort during the RESPONSE BODY (WP-575, PR #533 review)', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        process.env.CONVEX_SITE_URL = 'https://example.convex.site';
+        process.env.PRODUCTBRAIN_API_KEY = 'pb_sk_test';
+        global.fetch = vi.fn(async () => responseThatStallsMidBody());
+    });
+    afterEach(() => {
+        global.fetch = realFetch;
+    });
+    it('kernelCall surfaces the timeout instead of returning undefined as success', async () => {
+        // The regression in one line: this used to RESOLVE with `undefined`.
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err).toBeInstanceOf(CLIError);
+        expect(err.code).toBe(ErrorCode.NETWORK_TIMEOUT);
+    });
+    it('classifies an AbortError body abort too, matching the mcp-server twin', async () => {
+        // Depending on where in undici the budget signal is observed, a body-phase abort can
+        // surface as "AbortError" rather than "TimeoutError". Matching only the latter left that
+        // case falling through to an empty envelope, which a 200 reads as success with
+        // `undefined` data — this work package's own defect, surviving in one phase.
+        // `throwClassifiedGatewayFailure` (mcp-server/src/gatewaySeam.ts) has always accepted both.
+        global.fetch = vi.fn(async () => responseThatStallsMidBody(200, 'AbortError'));
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err).toBeInstanceOf(CLIError);
+        expect(err.code).toBe(ErrorCode.NETWORK_TIMEOUT);
+    });
+    it('a NON-OK status keeps its own error — a stalled error body is not a budget timeout', async () => {
+        // The server reached a verdict and said so; only its error body stalled. Reporting a
+        // budget timeout here would trade a real status for "may have completed anyway, search
+        // before retrying" about a call the server explicitly failed (PR #533 review round 5).
+        global.fetch = vi.fn(async () => responseThatStallsMidBody(429));
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err.code).not.toBe(ErrorCode.NETWORK_TIMEOUT);
+        expect(err.message).not.toMatch(/latency budget/i);
+    });
+    it('is a TIMEOUT, never NETWORK_UNREACHABLE — the host answered', async () => {
+        // The whole point of the distinction: the connection succeeded and the server began
+        // responding. Reporting "could not reach" would be a false statement about the server
+        // and would imply nothing ran.
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err.code).not.toBe(ErrorCode.NETWORK_UNREACHABLE);
+        expect(err.message).not.toMatch(/could not reach/i);
+    });
+    it('reports the budget the call was actually judged by, not a hardcoded number', async () => {
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err.message).toContain(String(latencyBudgetMsForRoute('chain.createEntry')));
+    });
+    it('says the write MAY HAVE COMPLETED — a body-phase abort leaves the outcome unknown', async () => {
+        // The server accepted the request and began responding; only the client stopped
+        // waiting. Telling the agent "it failed" is what invites the duplicating retry
+        // (TEN-1126's ghost entries).
+        const err = (await kernelCall('chain.createEntry', {}).catch((e) => e));
+        expect(err.guidance).toMatch(/may have\s+COMPLETED/i);
+        expect(err.guidance).toMatch(/duplicate/i);
+    });
+    it('a read route is told nothing was written — safe to retry, nothing to duplicate', async () => {
+        const err = (await kernelCall('resolveWorkspace', {}).catch((e) => e));
+        expect(err.guidance).toMatch(/nothing\s+was written/i);
+        expect(err.guidance).toMatch(/safe to retry/i);
+    });
+    it('a NON-timeout parse failure keeps the lenient empty-envelope path', async () => {
+        // Deliberately unchanged: a non-JSON or empty body is already handled by the ok/status
+        // checks every caller runs, and narrowing it here would change unrelated error paths.
+        global.fetch = vi.fn(async () => ({
+            ok: false,
+            status: 500,
+            statusText: 'Internal Server Error',
+            headers: { get: () => null },
+            json: async () => {
+                throw new SyntaxError('Unexpected token < in JSON at position 0');
+            },
+        }));
+        await expect(kernelCall('chain.createEntry', {})).rejects.toThrow(/Internal Server Error/);
+    });
+});
+//# sourceMappingURL=gateway-timeout-body-phase.test.js.map

package/dist/__tests__/gateway-timeout-body-phase.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"gateway-timeout-body-phase.test.js","sourceRoot":"","sources":["../../src/__tests__/gateway-timeout-body-phase.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AACH,OAAO,EAAE,UAAU,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AACzE,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAC9C,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EAAE,uBAAuB,EAAE,MAAM,8CAA8C,CAAC;AAEvF,MAAM,SAAS,GAAG,MAAM,CAAC,KAAK,CAAC;AAE/B;;;;GAIG;AACH,SAAS,yBAAyB,CAAC,MAAM,GAAG,GAAG,EAAE,SAAS,GAAG,cAAc;IAC1E,OAAO;QACN,EAAE,EAAE,MAAM,IAAI,GAAG,IAAI,MAAM,GAAG,GAAG;QACjC,MAAM;QACN,UAAU,EAAE,IAAI;QAChB,OAAO,EAAE,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;QAC5B,IAAI,EAAE,KAAK,IAAI,EAAE;YAChB,MAAM,GAAG,GAAG,IAAI,KAAK,CAAC,0CAA0C,CAAC,CAAC;YAClE,GAAG,CAAC,IAAI,GAAG,SAAS,CAAC;YACrB,MAAM,GAAG,CAAC;QACX,CAAC;KACsB,CAAC;AAC1B,CAAC;AAED,QAAQ,CAAC,oEAAoE,EAAE,GAAG,EAAE;IACnF,UAAU,CAAC,GAAG,EAAE;QACf,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,OAAO,CAAC,GAAG,CAAC,eAAe,GAAG,6BAA6B,CAAC;QAC5D,OAAO,CAAC,GAAG,CAAC,oBAAoB,GAAG,YAAY,CAAC;QAChD,MAAM,CAAC,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,yBAAyB,EAAE,CAA4B,CAAC;IAC1F,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACd,MAAM,CAAC,KAAK,GAAG,SAAS,CAAC;IAC1B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2EAA2E,EAAE,KAAK,IAAI,EAAE;QAC1F,qEAAqE;QACrE,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;QACtF,qFAAqF;QACrF,yFAAyF;QACzF,+EAA+E;QAC/E,6EAA6E;QAC7E,4FAA4F;QAC5F,MAAM,CAAC,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAC/B,yBAAyB,CAAC,GAAG,EAAE,YAAY,CAAC,CACjB,CAAC;QAE7B,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QACrC,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oFAAoF,EAAE,KAAK,IAAI,EAAE;QACnG,qFAAqF;QACrF,uFAAuF;QACvF,uFAAuF;QACvF,MAAM,CAAC,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAC/B,yBAAyB,CAAC,GAAG,CAAC,CACH,CAAC;QAE7B,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC;QACrD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC5E,oFAAoF;QACpF,sFAAsF;QACtF,+BAA+B;QAC/B,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QACzD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC3F,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,uBAAuB,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;IACrF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,KAAK,IAAI,EAAE;QAClG,gFAAgF;QAChF,+EAA+E;QAC/E,8BAA8B;QAC9B,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC7F,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAC;QACtD,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gFAAgF,EAAE,KAAK,IAAI,EAAE;QAC/F,MAAM,GAAG,GAAG,CAAC,MAAM,UAAU,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAU,EAAE,EAAE,CAAC,CAAC,CAAC,CAAa,CAAC;QAC5F,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAC;QACvD,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;QAClF,uFAAuF;QACvF,sFAAsF;QACtF,MAAM,CAAC,KAAK,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;YACjC,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,GAAG;YACX,UAAU,EAAE,uBAAuB;YACnC,OAAO,EAAE,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;YAC5B,IAAI,EAAE,KAAK,IAAI,EAAE;gBAChB,MAAM,IAAI,WAAW,CAAC,0CAA0C,CAAC,CAAC;YACnE,CAAC;SACD,CAAC,CAA4B,CAAC;QAE/B,MAAM,MAAM,CAAC,UAAU,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAC;IAC5F,CAAC,CAAC,CAAC;AACJ,CAAC,CAAC,CAAC"}

package/dist/__tests__/glossary.test.d.ts

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

package/dist/__tests__/glossary.test.d.ts.map

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

package/dist/__tests__/glossary.test.js

@@ -0,0 +1,15 @@
+import { describe, it, expect } from 'vitest';
+import { formatGlossary } from '../lib/glossary.js';
+import { PLATFORM_GLOSSARY_TERMS } from '../lib/platformGlossary.js';
+describe('pb glossary (TEN-4660)', () => {
+    it('prints the four platform terms from the kernel constant, one line each, and no hand-written CLI word', () => {
+        const output = formatGlossary();
+        expect(output.split('\n')).toHaveLength(PLATFORM_GLOSSARY_TERMS.length);
+        for (const term of PLATFORM_GLOSSARY_TERMS) {
+            expect(output).toContain(term.name);
+            expect(output).toContain(term.definition);
+        }
+        expect(output).not.toContain('One workspace = one product');
+    });
+});
+//# sourceMappingURL=glossary.test.js.map

package/dist/__tests__/glossary.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"glossary.test.js","sourceRoot":"","sources":["../../src/__tests__/glossary.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AACpD,OAAO,EAAE,uBAAuB,EAAE,MAAM,4BAA4B,CAAC;AAErE,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,EAAE,CAAC,sGAAsG,EAAE,GAAG,EAAE;QAC9G,MAAM,MAAM,GAAG,cAAc,EAAE,CAAC;QAChC,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,YAAY,CAAC,uBAAuB,CAAC,MAAM,CAAC,CAAC;QACxE,KAAK,MAAM,IAAI,IAAI,uBAAuB,EAAE,CAAC;YAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACpC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC5C,CAAC;QACD,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,6BAA6B,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/health-gap-kinds-drift.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=health-gap-kinds-drift.test.d.ts.map

package/dist/__tests__/health-gap-kinds-drift.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"health-gap-kinds-drift.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/health-gap-kinds-drift.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/health-gap-kinds-drift.test.js

@@ -0,0 +1,43 @@
+/**
+ * WP-480 S2 (FEAT-1359) — drift guard for the CLI HealthGapKind codegen.
+ *
+ * The canonical HealthGap union lives in convex/lib/healthGap.ts as 9
+ * discriminated type aliases (WP-776 S0 round-1 review: 12 → 11, starter-setup-missing
+ * deleted; TASK-409/R1a: 11 → 9, platform-domains-missing + authority-domain-degraded
+ * deleted); the CLI (Convex-free by BR-108/D13) consumes a
+ * codegen'd `HealthGapKind` union (src/generated/healthGapKinds.generated.ts,
+ * emitted by scripts/generate-health-gap-kinds.mjs).
+ *
+ * This test re-runs the generator's extraction RULE against the canonical
+ * source and compares the result to the committed generated file. Fingerprint
+ * is the rule, not a per-instance hash (INS-2177): a kind added, removed, or
+ * renamed upstream fails here until `npm run codegen` is re-run — no manual
+ * re-pinning.
+ *
+ * On failure: `npm run codegen` from packages/cli, commit the regenerated file.
+ *
+ * CLI tests run with the repo root as the vitest root; the canonical source is
+ * reached relative to this test file (same pattern as coaching-mirror-drift).
+ */
+import { describe, it, expect } from 'vitest';
+import { readFileSync } from 'node:fs';
+import { join, dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
+// The extraction rule is imported from the generator itself so test and codegen
+// can never disagree about what the rule is.
+// @ts-expect-error — plain .mjs module without type declarations
+import { extractHealthGapKinds } from '../../scripts/generate-health-gap-kinds.mjs';
+import { HEALTH_GAP_KINDS } from '../generated/healthGapKinds.generated.js';
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const CANONICAL_PATH = join(__dirname, '..', '..', '..', '..', 'convex', 'lib', 'healthGap.ts');
+describe('HealthGapKind codegen drift guard (WP-480 S2)', () => {
+    it('generated HEALTH_GAP_KINDS matches the kinds extracted from convex/lib/healthGap.ts — on failure, run `npm run codegen`', () => {
+        const source = readFileSync(CANONICAL_PATH, 'utf-8');
+        const canonicalKinds = extractHealthGapKinds(source);
+        // The rule itself failing to extract anything is drift too (the member
+        // declaration shape changed) — fail loudly rather than pass on [] === [].
+        expect(canonicalKinds.length).toBeGreaterThanOrEqual(4); // 4 since TASK-624 and TASK-627 deleted three kinds, WP-769 semantic-types-missing and TASK-644 occurredAt-backfill-pending
+        expect([...HEALTH_GAP_KINDS]).toEqual(canonicalKinds);
+    });
+});
+//# sourceMappingURL=health-gap-kinds-drift.test.js.map

package/dist/__tests__/health-gap-kinds-drift.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"health-gap-kinds-drift.test.js","sourceRoot":"","sources":["../../src/__tests__/health-gap-kinds-drift.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACvC,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AACzC,gFAAgF;AAChF,6CAA6C;AAC7C,iEAAiE;AACjE,OAAO,EAAE,qBAAqB,EAAE,MAAM,6CAA6C,CAAC;AACpF,OAAO,EAAE,gBAAgB,EAAE,MAAM,0CAA0C,CAAC;AAE5E,MAAM,SAAS,GAAG,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;AAC1D,MAAM,cAAc,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,cAAc,CAAC,CAAC;AAEhG,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;IAC7D,EAAE,CAAC,yHAAyH,EAAE,GAAG,EAAE;QACjI,MAAM,MAAM,GAAG,YAAY,CAAC,cAAc,EAAE,OAAO,CAAC,CAAC;QACrD,MAAM,cAAc,GAAa,qBAAqB,CAAC,MAAM,CAAC,CAAC;QAE/D,uEAAuE;QACvE,0EAA0E;QAC1E,MAAM,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAC,CAAC,4HAA4H;QAErL,MAAM,CAAC,CAAC,GAAG,gBAAgB,CAAC,CAAC,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/help-no-ids.test.d.ts

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

package/dist/__tests__/help-no-ids.test.d.ts.map

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

package/dist/__tests__/help-no-ids.test.js

@@ -0,0 +1,81 @@
+/**
+ * TASK-1034 (WP-863, RULE-705, RULE-686): every line pb help prints says what a command does for the
+ * person, never where it came from. Builds the REAL program from src/index.ts (parsing is stubbed so
+ * nothing runs), renders the full help of every command and subcommand (descriptions, options, arguments
+ * and added help text), and fails on an internal id, a metric code, a slice code, an error code, the word
+ * SSOT or a Convex function name on any line.
+ */
+import { Command } from 'commander';
+import { afterEach, describe, expect, it, vi } from 'vitest';
+// The last branch is a Convex function name (organisation.setOwnership, api.chain.commitEntry); a file path
+// or email the command writes or shows (.productbrain/config.local.json, jan@example.com) is what it does for
+// the person, so a word joined to a path, an @ or a file extension is not one.
+const INSIDER = /\b[A-Z]{2,6}-\d+\b|\bM\d\b|\bS\d+[a-z]?\b|\(E\d+\)|\bSSOT\b|(?<![\w/@-])[a-z]+\.(?!(?:json|md|com)\b)[a-z][A-Za-z]+\b(?!\.\w)/;
+afterEach(() => {
+    vi.restoreAllMocks();
+    vi.resetModules();
+});
+async function buildProgram() {
+    let root;
+    vi.spyOn(Command.prototype, 'parseAsync').mockImplementation(function () {
+        root = this;
+        return Promise.resolve(this);
+    });
+    await import('../index.js');
+    if (!root)
+        throw new Error('src/index.ts did not parse a program');
+    return root;
+}
+function helpLines(cmd, path = ['pb']) {
+    let help = '';
+    cmd.configureOutput({ writeOut: (s) => { help += s; }, writeErr: (s) => { help += s; } });
+    cmd.outputHelp();
+    const where = path.join(' ');
+    const lines = help.split('\n').map((text) => ({ where, text: text.trim() }));
+    for (const sub of cmd.commands)
+        lines.push(...helpLines(sub, [...path, sub.name()]));
+    return lines;
+}
+describe('pb help carries no internal ids or codes', () => {
+    it('no line of any command help matches an id, code or function name', async () => {
+        const program = await buildProgram();
+        const lines = helpLines(program);
+        expect(lines.length).toBeGreaterThan(300);
+        const offenders = lines.filter((l) => INSIDER.test(l.text)).map((l) => `${l.where}: ${l.text}`);
+        expect(offenders).toEqual([]);
+    }, 30_000);
+    it('no line of any command help writes a length as chars', async () => {
+        const lines = helpLines(await buildProgram());
+        expect(lines.filter((l) => /\bchars\b/i.test(l.text)).map((l) => `${l.where}: ${l.text}`)).toEqual([]);
+    }, 30_000);
+    it('--ack-reason on the update and entry accept commands reads the signed length form, and neither says ≥ or spine', async () => {
+        const program = await buildProgram();
+        const update = program.commands.find((c) => c.name() === 'update');
+        const accept = program.commands.find((c) => c.name() === 'entry').commands.find((c) => c.name() === 'accept');
+        for (const cmd of [update, accept]) {
+            expect(cmd.options.find((o) => o.long === '--ack-reason')?.description).toBe('Why you diverge, at least 12 characters; required with --acknowledge diverged');
+            expect(cmd.options.map((o) => o.description).filter((d) => /≥|\bspine\b/i.test(d))).toEqual([]);
+        }
+    }, 30_000);
+    it('no line of any command help says spine, the spine-check command name aside', async () => {
+        const lines = helpLines(await buildProgram());
+        const offenders = lines.filter((l) => /\bspine\b(?!-check)/i.test(l.text)).map((l) => `${l.where}: ${l.text}`);
+        expect(offenders).toEqual([]);
+    }, 30_000);
+    it('relate, entry preflight and quality spine-check help read the signed words', async () => {
+        const program = await buildProgram();
+        const relate = helpLines(program.commands.find((c) => c.name() === 'relate')).map((l) => l.text);
+        const preflight = helpLines(program.commands.find((c) => c.name() === 'entry').commands.find((c) => c.name() === 'preflight')).map((l) => l.text);
+        for (const lines of [relate, preflight])
+            expect(lines).toContain('Each level links to the one above:');
+        const check = program.commands.find((c) => c.name() === 'quality').commands.find((c) => c.name() === 'spine-check');
+        expect(check.description()).toBe('Check the direction, purpose to key results, for duplicates, unlinked entries and a strategy that refuses nothing; --story adds the one-story judgment');
+    }, 30_000);
+    it('the pattern catches the escapes a reviewer found and passes file paths and emails', () => {
+        for (const bad of ['judged by organisation.setOwnership.', 'chain.entries.create', 'api.chain.commitEntry', 'pre-S1b view'])
+            expect(INSIDER.test(bad), bad).toBe(true);
+        for (const ok of ['.productbrain/config.local.json', 'jan@example.com', 'discard session.json', '.codex/task-context.md', 'dev-system-collections.json', 'e.g. CI', 'i.e. this'])
+            expect(INSIDER.test(ok), ok).toBe(false);
+    });
+});
+//# sourceMappingURL=help-no-ids.test.js.map

package/dist/__tests__/help-no-ids.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"help-no-ids.test.js","sourceRoot":"","sources":["../../src/__tests__/help-no-ids.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE7D,4GAA4G;AAC5G,8GAA8G;AAC9G,+EAA+E;AAC/E,MAAM,OAAO,GACX,+HAA+H,CAAC;AAElI,SAAS,CAAC,GAAG,EAAE;IACb,EAAE,CAAC,eAAe,EAAE,CAAC;IACrB,EAAE,CAAC,YAAY,EAAE,CAAC;AACpB,CAAC,CAAC,CAAC;AAEH,KAAK,UAAU,YAAY;IACzB,IAAI,IAAyB,CAAC;IAC9B,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,kBAAkB,CAAC;QAC3D,IAAI,GAAG,IAAI,CAAC;QACZ,OAAO,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;IACH,MAAM,MAAM,CAAC,aAAa,CAAC,CAAC;IAC5B,IAAI,CAAC,IAAI;QAAE,MAAM,IAAI,KAAK,CAAC,sCAAsC,CAAC,CAAC;IACnE,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,SAAS,CAAC,GAAY,EAAE,OAAiB,CAAC,IAAI,CAAC;IACtD,IAAI,IAAI,GAAG,EAAE,CAAC;IACd,GAAG,CAAC,eAAe,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,EAAE,EAAE,GAAG,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,EAAE,EAAE,GAAG,IAAI,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;IAC1F,GAAG,CAAC,UAAU,EAAE,CAAC;IACjB,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC7B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,CAAC;IAC7E,KAAK,MAAM,GAAG,IAAI,GAAG,CAAC,QAAQ;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,SAAS,CAAC,GAAG,EAAE,CAAC,GAAG,IAAI,EAAE,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IACrF,OAAO,KAAK,CAAC;AACf,CAAC;AAED,QAAQ,CAAC,0CAA0C,EAAE,GAAG,EAAE;IACxD,EAAE,CAAC,kEAAkE,EAAE,KAAK,IAAI,EAAE;QAChF,MAAM,OAAO,GAAG,MAAM,YAAY,EAAE,CAAC;QACrC,MAAM,KAAK,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;QACjC,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC;QAC1C,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QAChG,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChC,CAAC,EAAE,MAAM,CAAC,CAAC;IAEX,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,KAAK,GAAG,SAAS,CAAC,MAAM,YAAY,EAAE,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IACzG,CAAC,EAAE,MAAM,CAAC,CAAC;IAEX,EAAE,CAAC,gHAAgH,EAAE,KAAK,IAAI,EAAE;QAC9H,MAAM,OAAO,GAAG,MAAM,YAAY,EAAE,CAAC;QACrC,MAAM,MAAM,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,QAAQ,CAAE,CAAC;QACpE,MAAM,MAAM,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,OAAO,CAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,QAAQ,CAAE,CAAC;QAChH,KAAK,MAAM,GAAG,IAAI,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,CAAC;YACnC,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,cAAc,CAAC,EAAE,WAAW,CAAC,CAAC,IAAI,CAC1E,+EAA+E,CAAC,CAAC;YACnF,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAClG,CAAC;IACH,CAAC,EAAE,MAAM,CAAC,CAAC;IAEX,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,KAAK,GAAG,SAAS,CAAC,MAAM,YAAY,EAAE,CAAC,CAAC;QAC9C,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,sBAAsB,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QAC/G,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAChC,CAAC,EAAE,MAAM,CAAC,CAAC;IAEX,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,OAAO,GAAG,MAAM,YAAY,EAAE,CAAC;QACrC,MAAM,MAAM,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,QAAQ,CAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAClG,MAAM,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,OAAO,CAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,WAAW,CAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACpJ,KAAK,MAAM,KAAK,IAAI,CAAC,MAAM,EAAE,SAAS,CAAC;YAAE,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,oCAAoC,CAAC,CAAC;QACvG,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,SAAS,CAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,KAAK,aAAa,CAAE,CAAC;QACtH,MAAM,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,CAAC,IAAI,CAAC,wJAAwJ,CAAC,CAAC;IAC7L,CAAC,EAAE,MAAM,CAAC,CAAC;IAEX,EAAE,CAAC,mFAAmF,EAAE,GAAG,EAAE;QAC3F,KAAK,MAAM,GAAG,IAAI,CAAC,sCAAsC,EAAE,sBAAsB,EAAE,uBAAuB,EAAE,cAAc,CAAC;YACzH,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC5C,KAAK,MAAM,EAAE,IAAI,CAAC,iCAAiC,EAAE,iBAAiB,EAAE,sBAAsB,EAAE,wBAAwB,EAAE,6BAA6B,EAAE,SAAS,EAAE,WAAW,CAAC;YAC9K,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/init.test.d.ts

@@ -0,0 +1,8 @@
+/**
+ * Tests for pb init — WP-303 Slice 4 (Keystone).
+ *
+ * Verifies detectState() returns correct states based on config/session availability,
+ * and that pb init installs the Claude Code session hooks (TASK-959, DEC-1926).
+ */
+export {};
+//# sourceMappingURL=init.test.d.ts.map

package/dist/__tests__/init.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"init.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/init.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG"}