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

package/dist/lib/spineVerdictRead.js

@@ -0,0 +1,76 @@
+// GENERATED by scripts/generate-mcp-embedded-copies.mjs from kernel/quality/spineVerdictRead.ts.
+// DO NOT EDIT. Run `npm run mcp:embedded-copies` to regenerate.
+// Local runtime copy for the isolated connector build.
+function isRecord(value) {
+    return typeof value === 'object' && value !== null && !Array.isArray(value);
+}
+function isStringArray(value) {
+    return Array.isArray(value) && value.every((item) => typeof item === 'string');
+}
+function isStructural(value) {
+    if (!isRecord(value) || typeof value.passed !== 'boolean' || !isStringArray(value.missing) ||
+        !isRecord(value.duplicateSingletons) || !isStringArray(value.unconnectedStrategyIds) ||
+        (value.visionConnectedToPurpose !== null && typeof value.visionConnectedToPurpose !== 'boolean'))
+        return false;
+    for (const key of ['purpose', 'vision']) {
+        const ids = value.duplicateSingletons[key];
+        if (ids !== null && !isStringArray(ids))
+            return false;
+    }
+    for (const key of ['strategiesWithUnknownConnectivity', 'unconnectedObjectiveIds',
+        'objectivesWithUnknownConnectivity', 'unconnectedKeyResultIds', 'keyResultsWithUnknownConnectivity']) {
+        if (value[key] != null && !isStringArray(value[key]))
+            return false;
+    }
+    const stranded = value.strandedWorkPackages;
+    if (stranded != null && !(Array.isArray(stranded) && stranded.every((w) => isRecord(w) && typeof w.id === 'string' && isStringArray(w.pastTargetIds))))
+        return false;
+    return value.visionPurposeConnectivityTruncated === undefined || typeof value.visionPurposeConnectivityTruncated === 'boolean';
+}
+function isStrategy(value) {
+    if (!isRecord(value) || typeof value.entryId !== 'string' || !isRecord(value.refusal) ||
+        typeof value.refusal.hasRefusalSignal !== 'boolean')
+        return false;
+    return value.rogerMartin === null || (isRecord(value.rogerMartin) &&
+        typeof value.rogerMartin.isStrategicChoice === 'boolean' && typeof value.rogerMartin.reasoning === 'string');
+}
+function isStory(value) {
+    return value === null || (isRecord(value) && typeof value.passed === 'boolean' &&
+        (value.failedCriterion === null || typeof value.failedCriterion === 'string') &&
+        Array.isArray(value.criteria) && value.criteria.every((item) => isRecord(item) &&
+        typeof item.id === 'string' && typeof item.passed === 'boolean' && typeof item.reasoning === 'string'));
+}
+/** Branch on the original discriminant before touching optional graded data. */
+export function readSpineVerdict(value) {
+    if (!isRecord(value) || typeof value.noSpineEntries !== 'boolean')
+        return { kind: 'malformed' };
+    // Empty checks have no grade to consume. Historical rows may store structural:null.
+    if (value.noSpineEntries)
+        return { kind: 'empty' };
+    if (!isStructural(value.structural) || !Array.isArray(value.perStrategy) ||
+        !value.perStrategy.every(isStrategy) || !isStory(value.story))
+        return { kind: 'malformed' };
+    for (const key of ['strategiesTruncated', 'perStrategyEvaluationTruncated', 'objectivesTruncated', 'keyResultsTruncated', 'workPackagesTruncated']) {
+        if (value[key] !== undefined && typeof value[key] !== 'boolean')
+            return { kind: 'malformed' };
+    }
+    // Keep the original payload (including newer fields); validation never strips information.
+    return { kind: 'graded', verdict: value };
+}
+/** A capped membership read preserves positive evidence, never infers removals by absence.
+ *  PRRT_kwDORS4ils6iIj0V: `relationsChanged` is a KNOWN, detected connectivity change (a
+ *  part_of edge added or removed — computeSpineVerdictStaleness's relations-fingerprint
+ *  mismatch), never an "unknown" scan outcome — it must render as its own reason, not fall into
+ *  the generic unknown-count branch below (which is reserved for a genuinely unresolved case:
+ *  a pre-migration row with no fingerprint to compare, or a legacy-rule regrade). */
+export function describeSpineStaleness(changedMemberCount, membershipTruncated = false, relationsChanged = false) {
+    if (changedMemberCount > 0) {
+        const reason = `${membershipTruncated ? 'at least ' : ''}${changedMemberCount} entr${changedMemberCount === 1 ? 'y' : 'ies'} changed, joined or left the direction since this was checked`;
+        return reason + (membershipTruncated ? '; membership scan incomplete' : '');
+    }
+    if (relationsChanged) {
+        return 'a part_of or validates link changed since this check ran' + (membershipTruncated ? '; membership scan incomplete' : '');
+    }
+    return 'freshness could not be verified; the changed-entry count is unknown' + (membershipTruncated ? '; membership scan incomplete' : '');
+}
+//# sourceMappingURL=spineVerdictRead.js.map

package/dist/lib/spineVerdictRead.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"spineVerdictRead.js","sourceRoot":"","sources":["../../src/lib/spineVerdictRead.ts"],"names":[],"mappings":"AAAA,iGAAiG;AACjG,gEAAgE;AAChE,uDAAuD;AAsDvD,SAAS,QAAQ,CAAC,KAAc;IAC/B,OAAO,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;AAC7E,CAAC;AAED,SAAS,aAAa,CAAC,KAAc;IACpC,OAAO,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,OAAO,IAAI,KAAK,QAAQ,CAAC,CAAC;AAChF,CAAC;AAED,SAAS,YAAY,CAAC,KAAc;IACnC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,OAAO,KAAK,CAAC,MAAM,KAAK,SAAS,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,OAAO,CAAC;QACzF,CAAC,QAAQ,CAAC,KAAK,CAAC,mBAAmB,CAAC,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,sBAAsB,CAAC;QACpF,CAAC,KAAK,CAAC,wBAAwB,KAAK,IAAI,IAAI,OAAO,KAAK,CAAC,wBAAwB,KAAK,SAAS,CAAC;QAAE,OAAO,KAAK,CAAC;IAChH,KAAK,MAAM,GAAG,IAAI,CAAC,SAAS,EAAE,QAAQ,CAAC,EAAE,CAAC;QACzC,MAAM,GAAG,GAAG,KAAK,CAAC,mBAAmB,CAAC,GAAG,CAAC,CAAC;QAC3C,IAAI,GAAG,KAAK,IAAI,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC;YAAE,OAAO,KAAK,CAAC;IACvD,CAAC;IACD,KAAK,MAAM,GAAG,IAAI,CAAC,mCAAmC,EAAE,yBAAyB;QAChF,mCAAmC,EAAE,yBAAyB,EAAE,mCAAmC,CAAC,EAAE,CAAC;QACvG,IAAI,KAAK,CAAC,GAAG,CAAC,IAAI,IAAI,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;YAAE,OAAO,KAAK,CAAC;IACpE,CAAC;IACD,MAAM,QAAQ,GAAG,KAAK,CAAC,oBAAoB,CAAC;IAC5C,IAAI,QAAQ,IAAI,IAAI,IAAI,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,OAAO,CAAC,CAAC,EAAE,KAAK,QAAQ,IAAI,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC;QAAE,OAAO,KAAK,CAAC;IACrK,OAAO,KAAK,CAAC,kCAAkC,KAAK,SAAS,IAAI,OAAO,KAAK,CAAC,kCAAkC,KAAK,SAAS,CAAC;AAChI,CAAC;AAED,SAAS,UAAU,CAAC,KAAc;IACjC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,OAAO,KAAK,CAAC,OAAO,KAAK,QAAQ,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,OAAO,CAAC;QACpF,OAAO,KAAK,CAAC,OAAO,CAAC,gBAAgB,KAAK,SAAS;QAAE,OAAO,KAAK,CAAC;IACnE,OAAO,KAAK,CAAC,WAAW,KAAK,IAAI,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,WAAW,CAAC;QAChE,OAAO,KAAK,CAAC,WAAW,CAAC,iBAAiB,KAAK,SAAS,IAAI,OAAO,KAAK,CAAC,WAAW,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC;AAC/G,CAAC;AAED,SAAS,OAAO,CAAC,KAAc;IAC9B,OAAO,KAAK,KAAK,IAAI,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,OAAO,KAAK,CAAC,MAAM,KAAK,SAAS;QAC7E,CAAC,KAAK,CAAC,eAAe,KAAK,IAAI,IAAI,OAAO,KAAK,CAAC,eAAe,KAAK,QAAQ,CAAC;QAC7E,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC;QAC7E,OAAO,IAAI,CAAC,EAAE,KAAK,QAAQ,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,SAAS,IAAI,OAAO,IAAI,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC;AAC3G,CAAC;AAED,gFAAgF;AAChF,MAAM,UAAU,gBAAgB,CAAC,KAAc;IAC9C,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,OAAO,KAAK,CAAC,cAAc,KAAK,SAAS;QAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC;IAChG,oFAAoF;IACpF,IAAI,KAAK,CAAC,cAAc;QAAE,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC;IACnD,IAAI,CAAC,YAAY,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,WAAW,CAAC;QACvE,CAAC,KAAK,CAAC,WAAW,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC;QAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC;IAC7F,KAAK,MAAM,GAAG,IAAI,CAAC,qBAAqB,EAAE,gCAAgC,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,uBAAuB,CAAC,EAAE,CAAC;QACpJ,IAAI,KAAK,CAAC,GAAG,CAAC,KAAK,SAAS,IAAI,OAAO,KAAK,CAAC,GAAG,CAAC,KAAK,SAAS;YAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC;IAC/F,CAAC;IACD,2FAA2F;IAC3F,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAsC,EAAE,CAAC;AAC5E,CAAC;AAGD;;;;;qFAKqF;AACrF,MAAM,UAAU,sBAAsB,CAAC,kBAA0B,EAAE,mBAAmB,GAAG,KAAK,EAAE,gBAAgB,GAAG,KAAK;IACvH,IAAI,kBAAkB,GAAG,CAAC,EAAE,CAAC;QAC5B,MAAM,MAAM,GAAG,GAAG,mBAAmB,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,GAAG,kBAAkB,QAAQ,kBAAkB,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,+DAA+D,CAAC;QAC3L,OAAO,MAAM,GAAG,CAAC,mBAAmB,CAAC,CAAC,CAAC,8BAA8B,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;IAC7E,CAAC;IACD,IAAI,gBAAgB,EAAE,CAAC;QACtB,OAAO,0DAA0D,GAAG,CAAC,mBAAmB,CAAC,CAAC,CAAC,8BAA8B,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;IACjI,CAAC;IACD,OAAO,qEAAqE,GAAG,CAAC,mBAAmB,CAAC,CAAC,CAAC,8BAA8B,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;AAC5I,CAAC"}

package/dist/lib/spinner.d.ts

@@ -0,0 +1,27 @@
+/**
+ * Contextual spinner for async operations.
+ * WP-303 Slice 2: Ambient Feedback (Spinners).
+ *
+ * - TTY: shows animated spinner with 200ms delayed start
+ * - Non-TTY / quiet / json: silent (no output)
+ * - On success: replaces spinner with success message
+ * - On failure: replaces spinner with failure message, rethrows
+ *
+ * Uses raw ANSI escape codes to stay consistent with style.ts (no @clack dependency).
+ */
+/**
+ * Run an async function with a contextual spinner.
+ *
+ * In interactive TTY mode, a spinner appears after 200ms (avoids flicker for
+ * fast operations). In non-TTY / quiet / JSON mode, the function runs silently.
+ *
+ * @param label - Text shown next to the spinner (e.g. "Loading orient data")
+ * @param fn - The async operation to run
+ * @param options.successMessage - Optional override for the completion message
+ * @returns The result of fn()
+ * @throws Rethrows any error from fn() after displaying failure
+ */
+export declare function withSpinner<T>(label: string, fn: () => Promise<T>, options?: {
+    successMessage?: string;
+}): Promise<T>;
+//# sourceMappingURL=spinner.d.ts.map

package/dist/lib/spinner.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"spinner.d.ts","sourceRoot":"","sources":["../../src/lib/spinner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAcH;;;;;;;;;;;GAWG;AACH,wBAAsB,WAAW,CAAC,CAAC,EACjC,KAAK,EAAE,MAAM,EACb,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,OAAO,CAAC,EAAE;IAAE,cAAc,CAAC,EAAE,MAAM,CAAA;CAAE,GACpC,OAAO,CAAC,CAAC,CAAC,CA4CZ"}

package/dist/lib/spinner.js

@@ -0,0 +1,76 @@
+/**
+ * Contextual spinner for async operations.
+ * WP-303 Slice 2: Ambient Feedback (Spinners).
+ *
+ * - TTY: shows animated spinner with 200ms delayed start
+ * - Non-TTY / quiet / json: silent (no output)
+ * - On success: replaces spinner with success message
+ * - On failure: replaces spinner with failure message, rethrows
+ *
+ * Uses raw ANSI escape codes to stay consistent with style.ts (no @clack dependency).
+ */
+import { isQuietMode, isJsonMode, areSpinnersSilenced } from './runner.js';
+import { green, red, icons } from './style.js';
+const SPINNER_FRAMES = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
+const FRAME_INTERVAL_MS = 80;
+const DELAYED_START_MS = 200;
+/** Returns true if spinner output should be suppressed. */
+function isSilent() {
+    return !process.stdout.isTTY || isQuietMode() || isJsonMode() || areSpinnersSilenced();
+}
+/**
+ * Run an async function with a contextual spinner.
+ *
+ * In interactive TTY mode, a spinner appears after 200ms (avoids flicker for
+ * fast operations). In non-TTY / quiet / JSON mode, the function runs silently.
+ *
+ * @param label - Text shown next to the spinner (e.g. "Loading orient data")
+ * @param fn - The async operation to run
+ * @param options.successMessage - Optional override for the completion message
+ * @returns The result of fn()
+ * @throws Rethrows any error from fn() after displaying failure
+ */
+export async function withSpinner(label, fn, options) {
+    // Non-interactive: just run the function
+    if (isSilent()) {
+        return fn();
+    }
+    let frameIndex = 0;
+    let spinnerInterval;
+    let started = false;
+    const write = (text) => {
+        // Clear line and write
+        process.stderr.write(`\r\x1b[K${text}`);
+    };
+    const startSpinner = () => {
+        started = true;
+        spinnerInterval = setInterval(() => {
+            const frame = SPINNER_FRAMES[frameIndex % SPINNER_FRAMES.length];
+            write(`${frame} ${label}`);
+            frameIndex++;
+        }, FRAME_INTERVAL_MS);
+    };
+    // Delayed start — avoids spinner flash for fast operations
+    const delayTimer = setTimeout(startSpinner, DELAYED_START_MS);
+    try {
+        const result = await fn();
+        clearTimeout(delayTimer);
+        if (spinnerInterval)
+            clearInterval(spinnerInterval);
+        if (started) {
+            const msg = options?.successMessage || label;
+            write(`${green(icons.pass)} ${msg}\n`);
+        }
+        return result;
+    }
+    catch (err) {
+        clearTimeout(delayTimer);
+        if (spinnerInterval)
+            clearInterval(spinnerInterval);
+        if (started) {
+            write(`${red(icons.fail)} ${label} \u2014 failed\n`);
+        }
+        throw err;
+    }
+}
+//# sourceMappingURL=spinner.js.map

package/dist/lib/spinner.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"spinner.js","sourceRoot":"","sources":["../../src/lib/spinner.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,WAAW,EAAE,UAAU,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AAC3E,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AAE/C,MAAM,cAAc,GAAG,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC;AAC1E,MAAM,iBAAiB,GAAG,EAAE,CAAC;AAC7B,MAAM,gBAAgB,GAAG,GAAG,CAAC;AAE7B,2DAA2D;AAC3D,SAAS,QAAQ;IACf,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,IAAI,WAAW,EAAE,IAAI,UAAU,EAAE,IAAI,mBAAmB,EAAE,CAAC;AACzF,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAC/B,KAAa,EACb,EAAoB,EACpB,OAAqC;IAErC,yCAAyC;IACzC,IAAI,QAAQ,EAAE,EAAE,CAAC;QACf,OAAO,EAAE,EAAE,CAAC;IACd,CAAC;IAED,IAAI,UAAU,GAAG,CAAC,CAAC;IACnB,IAAI,eAA2D,CAAC;IAChE,IAAI,OAAO,GAAG,KAAK,CAAC;IAEpB,MAAM,KAAK,GAAG,CAAC,IAAY,EAAE,EAAE;QAC7B,uBAAuB;QACvB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC;IAC1C,CAAC,CAAC;IAEF,MAAM,YAAY,GAAG,GAAG,EAAE;QACxB,OAAO,GAAG,IAAI,CAAC;QACf,eAAe,GAAG,WAAW,CAAC,GAAG,EAAE;YACjC,MAAM,KAAK,GAAG,cAAc,CAAC,UAAU,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC;YACjE,KAAK,CAAC,GAAG,KAAK,IAAI,KAAK,EAAE,CAAC,CAAC;YAC3B,UAAU,EAAE,CAAC;QACf,CAAC,EAAE,iBAAiB,CAAC,CAAC;IACxB,CAAC,CAAC;IAEF,2DAA2D;IAC3D,MAAM,UAAU,GAAG,UAAU,CAAC,YAAY,EAAE,gBAAgB,CAAC,CAAC;IAE9D,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,MAAM,EAAE,EAAE,CAAC;QAC1B,YAAY,CAAC,UAAU,CAAC,CAAC;QACzB,IAAI,eAAe;YAAE,aAAa,CAAC,eAAe,CAAC,CAAC;QACpD,IAAI,OAAO,EAAE,CAAC;YACZ,MAAM,GAAG,GAAG,OAAO,EAAE,cAAc,IAAI,KAAK,CAAC;YAC7C,KAAK,CAAC,GAAG,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC,CAAC;QACzC,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,YAAY,CAAC,UAAU,CAAC,CAAC;QACzB,IAAI,eAAe;YAAE,aAAa,CAAC,eAAe,CAAC,CAAC;QACpD,IAAI,OAAO,EAAE,CAAC;YACZ,KAAK,CAAC,GAAG,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,KAAK,kBAAkB,CAAC,CAAC;QACvD,CAAC;QACD,MAAM,GAAG,CAAC;IACZ,CAAC;AACH,CAAC"}

package/dist/lib/spinner.test.d.ts

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

package/dist/lib/spinner.test.d.ts.map

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

package/dist/lib/spinner.test.js

@@ -0,0 +1,39 @@
+/**
+ * withSpinner — unit tests.
+ * WP-303 Slice 2: Ambient Feedback (Spinners).
+ *
+ * Tests run in a non-TTY environment (vitest), so the spinner is always silent.
+ * This validates the critical contract: withSpinner is transparent in non-TTY mode.
+ */
+import { describe, expect, it } from 'vitest';
+import { withSpinner } from './spinner.js';
+describe('withSpinner', () => {
+    it('returns the function result in non-TTY mode', async () => {
+        const result = await withSpinner('Loading', async () => 42);
+        expect(result).toBe(42);
+    });
+    it('returns complex objects unchanged', async () => {
+        const data = { entries: [{ id: 'BET-1', name: 'Test' }], total: 1 };
+        const result = await withSpinner('Loading', async () => data);
+        expect(result).toEqual(data);
+    });
+    it('rethrows errors from the wrapped function', async () => {
+        const err = new Error('network failure');
+        await expect(withSpinner('Loading', async () => {
+            throw err;
+        })).rejects.toThrow('network failure');
+    });
+    it('passes through the success message option without affecting result', async () => {
+        const result = await withSpinner('Loading', async () => 'done', { successMessage: 'Loaded successfully' });
+        expect(result).toBe('done');
+    });
+    it('completes without hanging on fast operations', async () => {
+        // This validates the 200ms delay timer is properly cleaned up
+        const start = Date.now();
+        await withSpinner('Fast op', async () => 'quick');
+        const elapsed = Date.now() - start;
+        // Should complete well under the 200ms delay threshold
+        expect(elapsed).toBeLessThan(100);
+    });
+});
+//# sourceMappingURL=spinner.test.js.map

package/dist/lib/spinner.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"spinner.test.js","sourceRoot":"","sources":["../../src/lib/spinner.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAE3C,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,EAAE,CAAC,6CAA6C,EAAE,KAAK,IAAI,EAAE;QAC3D,MAAM,MAAM,GAAG,MAAM,WAAW,CAAC,SAAS,EAAE,KAAK,IAAI,EAAE,CAAC,EAAE,CAAC,CAAC;QAC5D,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAC1B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,KAAK,IAAI,EAAE;QACjD,MAAM,IAAI,GAAG,EAAE,OAAO,EAAE,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC;QACpE,MAAM,MAAM,GAAG,MAAM,WAAW,CAAC,SAAS,EAAE,KAAK,IAAI,EAAE,CAAC,IAAI,CAAC,CAAC;QAC9D,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;QACzD,MAAM,GAAG,GAAG,IAAI,KAAK,CAAC,iBAAiB,CAAC,CAAC;QACzC,MAAM,MAAM,CACV,WAAW,CAAC,SAAS,EAAE,KAAK,IAAI,EAAE;YAChC,MAAM,GAAG,CAAC;QACZ,CAAC,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;QAClF,MAAM,MAAM,GAAG,MAAM,WAAW,CAC9B,SAAS,EACT,KAAK,IAAI,EAAE,CAAC,MAAM,EAClB,EAAE,cAAc,EAAE,qBAAqB,EAAE,CAC1C,CAAC;QACF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC9B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,8DAA8D;QAC9D,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACzB,MAAM,WAAW,CAAC,SAAS,EAAE,KAAK,IAAI,EAAE,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK,CAAC;QACnC,uDAAuD;QACvD,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;IACpC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/lib/startup-signals.d.ts

@@ -0,0 +1,12 @@
+export interface CliStartupSignals {
+    changedPaths?: string[];
+    branchName?: string;
+    worktreeName?: string;
+}
+/**
+ * Collect the CLI's best-effort startup signal bundle for THIS invocation's cwd. Every field is
+ * omitted (not empty-stringed) when unavailable, matching the server validator's `v.optional`
+ * contract (`startupResolutionSignalsValidator`) — an absent key, not a fabricated empty one.
+ */
+export declare function collectCliStartupSignals(cwd?: string): CliStartupSignals;
+//# sourceMappingURL=startup-signals.d.ts.map

package/dist/lib/startup-signals.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"startup-signals.d.ts","sourceRoot":"","sources":["../../src/lib/startup-signals.ts"],"names":[],"mappings":"AAqCA,MAAM,WAAW,iBAAiB;IAChC,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAmGD;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,GAAG,GAAE,MAAsB,GAAG,iBAAiB,CAgCvF"}

package/dist/lib/startup-signals.js

@@ -0,0 +1,173 @@
+/**
+ * WP-486 Slice 1 (FEAT-1371) — CLI-side collection of the startup signals `pb orient --task`
+ * forwards as `changedPaths`, `branchName`, `worktreeName`.
+ *
+ * D1d (WP-754/TASK-204): originally collected to feed the server's `resolveStartupDomain`
+ * (deleted along with domain resolution — D1b/D1c). The sole surviving consumer is
+ * `context.served` activation telemetry (`convex/lib/contextServedInput.ts`'s `changedPaths` and
+ * the `branchNamePresent`/`worktreeNamePresent` presence booleans) — an unrelated purpose this
+ * module's output happens to satisfy unchanged, so it stays.
+ *
+ * Best-effort, fail-open: git absence, a non-repo cwd, or any shell error yields an empty signal
+ * set (never thrown), so a missing/broken git never blocks orient.
+ *
+ * `reviewedArtifactRefs` is deliberately NOT collected here — no CLI source exists (no "artifacts
+ * reviewed this session" tracking), per WP-486 spec §5 exclusions. Server-side sanitization
+ * (`convex/agentKnowledge/startupResolutionSignals.ts`) re-validates and caps every field this
+ * module produces — this collector does the best-effort gather; the server remains the trust
+ * boundary.
+ *
+ * `pb orient --task` is a per-task hot path (every orient call), so this collector reads git only:
+ * no `.productbrain` doc reads and no `gh` subprocess for "PR #N" refs. (The `pb handshake`
+ * deriver that did those reads, once per session-start/close, was deleted with the command in
+ * #1912.) `changedPaths` is the two-source
+ * union (dirty worktree + committed branch delta vs `origin/main`) — see
+ * `collectCliStartupSignals` below — because the spec's own Slice 1 text (WP-B §4) names "git diff
+ * against the worktree's base" as the source, not merely `git status`; PR-review (Codex, PR #369)
+ * caught that the first cut only covered the dirty-worktree half.
+ */
+import { execSync } from 'node:child_process';
+import { basename } from 'node:path';
+const GIT_TIMEOUT_MS = 3_000;
+const CHANGED_PATHS_CAP = 25;
+/** `origin/main` is the worktree's base per Product-OS's one-repo-one-remote convention
+ *  (constitution §2/§3). */
+const BASE_REF = 'origin/main';
+function runGit(cwd, command) {
+    try {
+        const value = execSync(command, {
+            cwd,
+            encoding: 'utf8',
+            stdio: ['ignore', 'pipe', 'ignore'],
+            timeout: GIT_TIMEOUT_MS,
+        }).trim();
+        return value.length > 0 ? value : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+/**
+ * Like `runGit`, but for `git status --porcelain`: does NOT `.trim()` the raw output. Porcelain
+ * v1's fixed-width status prefix (two status chars + one space) commonly STARTS with a space
+ * (unstaged-modified is `" M"`, unstaged-deleted is `" D"`); a blanket `.trim()` on the whole
+ * multi-line blob strips exactly that leading space off the FIRST line only, off-by-one-shifting
+ * `parseChangedPaths`'s positional `slice(3)` for that one entry (e.g. `" M convex/x.ts"` →
+ * trimmed → `"M convex/x.ts"` → sliced wrong → `"onvex/x.ts"`). Trims only the trailing newline.
+ */
+function runGitStatusRaw(cwd, command) {
+    try {
+        const value = execSync(command, {
+            cwd,
+            encoding: 'utf8',
+            stdio: ['ignore', 'pipe', 'ignore'],
+            timeout: GIT_TIMEOUT_MS,
+        }).replace(/\r?\n$/, '');
+        return value.length > 0 ? value : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+/**
+ * Parse `git status --porcelain --untracked-files=all` output into a flat list of changed paths.
+ * `--untracked-files=all` (not the porcelain default `normal`) is REQUIRED: the default mode
+ * collapses a wholly-untracked directory to one entry (`?? src/routes/new-feature/`), losing the
+ * filename/extension evidence the (now-deleted) domain path-glob rules used to key on; `all`
+ * expands it to individual files (`?? src/routes/new-feature/page.svelte`) — reproduced live and
+ * confirmed by PR review (Codex, PR #369) against `git status -h`'s documented
+ * `--untracked-files=<mode>`.
+ *
+ * Rename lines (`R  old -> new`) keep only the new path — consumers of this list (now
+ * `context.served` telemetry) care about the CURRENT file location, not its history. The leading
+ * two-character status code + one
+ * space (porcelain v1's fixed-width prefix) is stripped positionally rather than by
+ * regex-splitting on whitespace, so a path containing spaces survives intact. Callers MUST pass
+ * the UNTRIMMED output (`runGitStatusRaw`, not `runGit`) — see that function's doc for why a
+ * blanket outer trim breaks this positional slice on the first line.
+ */
+function parseChangedPaths(porcelainOutput) {
+    const paths = [];
+    for (const line of porcelainOutput.split(/\r?\n/)) {
+        if (line.length <= 3)
+            continue;
+        let entry = line.slice(3).trim();
+        if (!entry)
+            continue;
+        const renameSep = entry.indexOf(' -> ');
+        if (renameSep !== -1)
+            entry = entry.slice(renameSep + 4).trim();
+        // Porcelain quotes paths containing unusual characters in double quotes — strip them.
+        if (entry.startsWith('"') && entry.endsWith('"'))
+            entry = entry.slice(1, -1);
+        if (entry)
+            paths.push(entry);
+    }
+    return paths;
+}
+/**
+ * Parse `git diff --name-only -z` output — NUL-separated, bare paths, no status prefix. `-z` is
+ * REQUIRED on the caller's git invocation: it disables path quoting outright (regardless of
+ * `core.quotePath`), so a non-ASCII filename (e.g. `café.txt`) that porcelain would otherwise
+ * octal-escape into a quoted string (`"caf\303\251.txt"`) instead comes through as raw UTF-8
+ * bytes — no unescaping needed here, unlike `parseChangedPaths`'s newline-delimited quote strip.
+ */
+function parseNameOnlyPaths(diffOutput) {
+    return diffOutput.split('\0').map((entry) => entry.trim()).filter(Boolean);
+}
+/**
+ * Union two path lists, de-duped, order-stable (first-seen wins), capped at
+ * `CHANGED_PATHS_CAP`. The dirty-worktree and branch-delta lists overlap heavily in the common
+ * case (a file both dirty AND already differing from base), so de-dup keeps the cap from being
+ * spent on repeats.
+ */
+function unionCapped(a, b) {
+    const seen = new Set();
+    const out = [];
+    for (const p of [...a, ...b]) {
+        if (seen.has(p))
+            continue;
+        seen.add(p);
+        out.push(p);
+        if (out.length >= CHANGED_PATHS_CAP)
+            break;
+    }
+    return out;
+}
+/**
+ * Collect the CLI's best-effort startup signal bundle for THIS invocation's cwd. Every field is
+ * omitted (not empty-stringed) when unavailable, matching the server validator's `v.optional`
+ * contract (`startupResolutionSignalsValidator`) — an absent key, not a fabricated empty one.
+ */
+export function collectCliStartupSignals(cwd = process.cwd()) {
+    const signals = {};
+    // Two sources, unioned (the module doc says why PR-diff and local-doc reads stay out of this
+    // hot path):
+    //   1. Dirty worktree (local changes not yet in git) — what the ORIGINAL cut covered.
+    //   2. Committed branch delta vs BASE_REF — a builder mid-lane with a clean, committed
+    //      worktree (the normal `pb orient --task` re-orientation case) otherwise contributes
+    //      NO changedPaths at all, even though the branch differs meaningfully from main. Spec
+    //      text (WP-B §4) names "git diff against the worktree's base" as the source — this closes
+    //      the gap between that text and the first cut's git-status-only implementation (PR #369
+    //      review, Codex).
+    const statusRaw = runGitStatusRaw(cwd, 'git status --porcelain --untracked-files=all');
+    const dirtyPaths = statusRaw ? parseChangedPaths(statusRaw) : [];
+    let branchPaths = [];
+    const mergeBase = runGit(cwd, `git merge-base HEAD ${BASE_REF}`);
+    if (mergeBase) {
+        const branchDiffRaw = runGit(cwd, `git diff --name-only -z ${mergeBase} HEAD`);
+        if (branchDiffRaw)
+            branchPaths = parseNameOnlyPaths(branchDiffRaw);
+    }
+    const allPaths = unionCapped(dirtyPaths, branchPaths);
+    if (allPaths.length > 0)
+        signals.changedPaths = allPaths;
+    const branch = runGit(cwd, 'git branch --show-current');
+    if (branch)
+        signals.branchName = branch;
+    const toplevel = runGit(cwd, 'git rev-parse --show-toplevel');
+    if (toplevel)
+        signals.worktreeName = basename(toplevel);
+    return signals;
+}
+//# sourceMappingURL=startup-signals.js.map

package/dist/lib/startup-signals.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"startup-signals.js","sourceRoot":"","sources":["../../src/lib/startup-signals.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,QAAQ,EAAE,MAAM,WAAW,CAAC;AAErC,MAAM,cAAc,GAAG,KAAK,CAAC;AAC7B,MAAM,iBAAiB,GAAG,EAAE,CAAC;AAC7B;4BAC4B;AAC5B,MAAM,QAAQ,GAAG,aAAa,CAAC;AAQ/B,SAAS,MAAM,CAAC,GAAW,EAAE,OAAe;IAC1C,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,QAAQ,CAAC,OAAO,EAAE;YAC9B,GAAG;YACH,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC;YACnC,OAAO,EAAE,cAAc;SACxB,CAAC,CAAC,IAAI,EAAE,CAAC;QACV,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC;IAC9C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,eAAe,CAAC,GAAW,EAAE,OAAe;IACnD,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,QAAQ,CAAC,OAAO,EAAE;YAC9B,GAAG;YACH,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC;YACnC,OAAO,EAAE,cAAc;SACxB,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;QACzB,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC;IAC9C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,SAAS,iBAAiB,CAAC,eAAuB;IAChD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,IAAI,IAAI,eAAe,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC;QAClD,IAAI,IAAI,CAAC,MAAM,IAAI,CAAC;YAAE,SAAS;QAC/B,IAAI,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACjC,IAAI,CAAC,KAAK;YAAE,SAAS;QACrB,MAAM,SAAS,GAAG,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QACxC,IAAI,SAAS,KAAK,CAAC,CAAC;YAAE,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAChE,sFAAsF;QACtF,IAAI,KAAK,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC;YAAE,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;QAC7E,IAAI,KAAK;YAAE,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC/B,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;;GAMG;AACH,SAAS,kBAAkB,CAAC,UAAkB;IAC5C,OAAO,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;AAC7E,CAAC;AAED;;;;;GAKG;AACH,SAAS,WAAW,CAAC,CAAW,EAAE,CAAW;IAC3C,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,KAAK,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,GAAG,CAAC,CAAC,EAAE,CAAC;QAC7B,IAAI,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC;YAAE,SAAS;QAC1B,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QACZ,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACZ,IAAI,GAAG,CAAC,MAAM,IAAI,iBAAiB;YAAE,MAAM;IAC7C,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,wBAAwB,CAAC,MAAc,OAAO,CAAC,GAAG,EAAE;IAClE,MAAM,OAAO,GAAsB,EAAE,CAAC;IAEtC,6FAA6F;IAC7F,aAAa;IACb,sFAAsF;IACtF,uFAAuF;IACvF,0FAA0F;IAC1F,2FAA2F;IAC3F,+FAA+F;IAC/F,6FAA6F;IAC7F,uBAAuB;IACvB,MAAM,SAAS,GAAG,eAAe,CAAC,GAAG,EAAE,8CAA8C,CAAC,CAAC;IACvF,MAAM,UAAU,GAAG,SAAS,CAAC,CAAC,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IAEjE,IAAI,WAAW,GAAa,EAAE,CAAC;IAC/B,MAAM,SAAS,GAAG,MAAM,CAAC,GAAG,EAAE,uBAAuB,QAAQ,EAAE,CAAC,CAAC;IACjE,IAAI,SAAS,EAAE,CAAC;QACd,MAAM,aAAa,GAAG,MAAM,CAAC,GAAG,EAAE,2BAA2B,SAAS,OAAO,CAAC,CAAC;QAC/E,IAAI,aAAa;YAAE,WAAW,GAAG,kBAAkB,CAAC,aAAa,CAAC,CAAC;IACrE,CAAC;IAED,MAAM,QAAQ,GAAG,WAAW,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;IACtD,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,CAAC,YAAY,GAAG,QAAQ,CAAC;IAEzD,MAAM,MAAM,GAAG,MAAM,CAAC,GAAG,EAAE,2BAA2B,CAAC,CAAC;IACxD,IAAI,MAAM;QAAE,OAAO,CAAC,UAAU,GAAG,MAAM,CAAC;IAExC,MAAM,QAAQ,GAAG,MAAM,CAAC,GAAG,EAAE,+BAA+B,CAAC,CAAC;IAC9D,IAAI,QAAQ;QAAE,OAAO,CAAC,YAAY,GAAG,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAExD,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/dist/lib/startup-signals.test.d.ts

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

package/dist/lib/startup-signals.test.d.ts.map

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

package/dist/lib/startup-signals.test.js

@@ -0,0 +1,165 @@
+/**
+ * WP-486 Slice 1 (FEAT-1371, TEN-2724) — `collectCliStartupSignals` unit tests. Mocks
+ * `child_process.execSync` so these are deterministic and never shell out to real git.
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+const execSyncMock = vi.fn();
+vi.mock('node:child_process', () => ({
+    execSync: (...args) => execSyncMock(...args),
+}));
+import { collectCliStartupSignals } from './startup-signals.js';
+describe('collectCliStartupSignals', () => {
+    beforeEach(() => {
+        execSyncMock.mockReset();
+    });
+    function mockGitCommands(responses) {
+        execSyncMock.mockImplementation((command) => {
+            for (const [match, output] of Object.entries(responses)) {
+                if (command.includes(match))
+                    return output;
+            }
+            throw new Error(`unmocked git command: ${command}`);
+        });
+    }
+    it('collects changedPaths, branchName, worktreeName from git', () => {
+        mockGitCommands({
+            'git status --porcelain': ' M convex/agentKnowledge/orientView.ts\n?? packages/cli/src/lib/startup-signals.ts\n',
+            'git branch --show-current': 'ontology-r4-r5-signal-envelope\n',
+            'git rev-parse --show-toplevel': '/Users/randy/worktree/ontology-r4-r5-signal-envelope\n',
+        });
+        const signals = collectCliStartupSignals('/Users/randy/worktree/ontology-r4-r5-signal-envelope');
+        expect(signals).toEqual({
+            changedPaths: [
+                'convex/agentKnowledge/orientView.ts',
+                'packages/cli/src/lib/startup-signals.ts',
+            ],
+            branchName: 'ontology-r4-r5-signal-envelope',
+            worktreeName: 'ontology-r4-r5-signal-envelope',
+        });
+    });
+    it('resolves a rename line to the NEW path only', () => {
+        mockGitCommands({
+            'git status --porcelain': 'R  old-name.ts -> new-name.ts\n',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual(['new-name.ts']);
+    });
+    it('caps changedPaths at 25 entries', () => {
+        const lines = Array.from({ length: 40 }, (_, i) => ` M file-${i}.ts`).join('\n');
+        mockGitCommands({
+            'git status --porcelain': lines,
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toHaveLength(25);
+    });
+    it('is fail-open: a clean worktree yields no changedPaths key', () => {
+        mockGitCommands({
+            'git status --porcelain': '',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals).not.toHaveProperty('changedPaths');
+    });
+    it('is fail-open: git absence/error yields an empty signal set, never throws', () => {
+        execSyncMock.mockImplementation(() => {
+            throw new Error('git: command not found');
+        });
+        expect(() => collectCliStartupSignals('/not-a-repo')).not.toThrow();
+        expect(collectCliStartupSignals('/not-a-repo')).toEqual({});
+    });
+    // WP-486 Slice 1 — PR #369 review (Codex): two P2 findings, both reproduced live against a
+    // disposable git repo before being fixed here (see PR thread replies for the live evidence).
+    it('requests --untracked-files=all so a wholly-untracked directory expands to its files, not one directory entry', () => {
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': '?? src/routes/new-feature/page.svelte\n?? src/routes/new-feature/logic.ts\n',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual([
+            'src/routes/new-feature/page.svelte',
+            'src/routes/new-feature/logic.ts',
+        ]);
+        // Locks in the actual flag used — a regression back to bare `--porcelain` (default
+        // `--untracked-files=normal`, which collapses the directory to one `?? src/` entry) would
+        // fail this assertion even though the mock above still matches the substring.
+        const calledCommands = execSyncMock.mock.calls.map((c) => c[0]);
+        expect(calledCommands.some((c) => c.includes('--untracked-files=all'))).toBe(true);
+    });
+    it('includes committed branch paths (vs origin/main) when the worktree is clean — the normal re-orientation case', () => {
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': '',
+            'git merge-base HEAD origin/main': 'abc1234\n',
+            'git diff --name-only -z abc1234 HEAD': 'convex/agentKnowledge/orientView.ts\0packages/cli/src/commands/orient.ts\0',
+            'git branch --show-current': 'ontology-r4-r5-signal-envelope\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual([
+            'convex/agentKnowledge/orientView.ts',
+            'packages/cli/src/commands/orient.ts',
+        ]);
+    });
+    it('unions dirty-worktree and committed-branch paths, de-duped', () => {
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': ' M convex/agentKnowledge/orientView.ts\n',
+            'git merge-base HEAD origin/main': 'abc1234\n',
+            'git diff --name-only -z abc1234 HEAD': 'convex/agentKnowledge/orientView.ts\0packages/cli/src/commands/orient.ts\0',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual([
+            'convex/agentKnowledge/orientView.ts',
+            'packages/cli/src/commands/orient.ts',
+        ]);
+    });
+    it('parses -z NUL-separated branch-diff paths without quote-corruption on non-ASCII filenames (PR #369 review)', () => {
+        // `-z` disables git's default core.quotePath octal-escaping outright, so a non-ASCII path
+        // comes through as raw UTF-8 bytes, NUL-terminated — never as a quoted `"caf\303\251.txt"`
+        // string a newline-splitting parser would mis-glob against. This test locks in that the
+        // `-z` flag is actually present on the branch-diff git invocation (a regression back to
+        // bare `--name-only` would corrupt this exact case).
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': '',
+            'git merge-base HEAD origin/main': 'abc1234\n',
+            'git diff --name-only -z abc1234 HEAD': 'café.txt\0src/edge-case.ts\0',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual(['café.txt', 'src/edge-case.ts']);
+    });
+    it('is fail-open: no origin/main (e.g. no network) yields dirty-worktree paths only, never throws', () => {
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': ' M convex/agentKnowledge/orientView.ts\n',
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+            // No 'git merge-base' entry — mockGitCommands throws "unmocked git command", simulating a
+            // real git failure (e.g. unknown revision origin/main); runGit's try/catch degrades this
+            // to undefined, same as any other fail-open git error.
+        });
+        expect(() => collectCliStartupSignals('/repo')).not.toThrow();
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toEqual(['convex/agentKnowledge/orientView.ts']);
+    });
+    it('caps the UNIONED changedPaths at 25 entries', () => {
+        const dirty = Array.from({ length: 15 }, (_, i) => ` M dirty-${i}.ts`).join('\n');
+        const branch = Array.from({ length: 15 }, (_, i) => `branch-${i}.ts`).join('\0');
+        mockGitCommands({
+            'git status --porcelain --untracked-files=all': dirty,
+            'git merge-base HEAD origin/main': 'abc1234\n',
+            'git diff --name-only -z abc1234 HEAD': branch,
+            'git branch --show-current': 'main\n',
+            'git rev-parse --show-toplevel': '/repo\n',
+        });
+        const signals = collectCliStartupSignals('/repo');
+        expect(signals.changedPaths).toHaveLength(25);
+    });
+});
+//# sourceMappingURL=startup-signals.test.js.map

package/dist/lib/startup-signals.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"startup-signals.test.js","sourceRoot":"","sources":["../../src/lib/startup-signals.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAE9D,MAAM,YAAY,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAC7B,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC;IACnC,QAAQ,EAAE,CAAC,GAAG,IAAe,EAAE,EAAE,CAAC,YAAY,CAAC,GAAG,IAAI,CAAC;CACxD,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,wBAAwB,EAAE,MAAM,sBAAsB,CAAC;AAEhE,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;IACxC,UAAU,CAAC,GAAG,EAAE;QACd,YAAY,CAAC,SAAS,EAAE,CAAC;IAC3B,CAAC,CAAC,CAAC;IAEH,SAAS,eAAe,CAAC,SAAiC;QACxD,YAAY,CAAC,kBAAkB,CAAC,CAAC,OAAe,EAAE,EAAE;YAClD,KAAK,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,CAAC;gBACxD,IAAI,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAC;oBAAE,OAAO,MAAM,CAAC;YAC7C,CAAC;YACD,MAAM,IAAI,KAAK,CAAC,yBAAyB,OAAO,EAAE,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;IACL,CAAC;IAED,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,eAAe,CAAC;YACd,wBAAwB,EAAE,sFAAsF;YAChH,2BAA2B,EAAE,kCAAkC;YAC/D,+BAA+B,EAAE,wDAAwD;SAC1F,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,sDAAsD,CAAC,CAAC;QAEjG,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC;YACtB,YAAY,EAAE;gBACZ,qCAAqC;gBACrC,yCAAyC;aAC1C;YACD,UAAU,EAAE,gCAAgC;YAC5C,YAAY,EAAE,gCAAgC;SAC/C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,eAAe,CAAC;YACd,wBAAwB,EAAE,iCAAiC;YAC3D,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC;IACxD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjF,eAAe,CAAC;YACd,wBAAwB,EAAE,KAAK;YAC/B,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,YAAY,CAAC,EAAE,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,eAAe,CAAC;YACd,wBAAwB,EAAE,EAAE;YAC5B,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,cAAc,CAAC,cAAc,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0EAA0E,EAAE,GAAG,EAAE;QAClF,YAAY,CAAC,kBAAkB,CAAC,GAAG,EAAE;YACnC,MAAM,IAAI,KAAK,CAAC,wBAAwB,CAAC,CAAC;QAC5C,CAAC,CAAC,CAAC;QAEH,MAAM,CAAC,GAAG,EAAE,CAAC,wBAAwB,CAAC,aAAa,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QACpE,MAAM,CAAC,wBAAwB,CAAC,aAAa,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC9D,CAAC,CAAC,CAAC;IAEH,2FAA2F;IAC3F,6FAA6F;IAE7F,EAAE,CAAC,8GAA8G,EAAE,GAAG,EAAE;QACtH,eAAe,CAAC;YACd,8CAA8C,EAAE,6EAA6E;YAC7H,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC;YACnC,oCAAoC;YACpC,iCAAiC;SAClC,CAAC,CAAC;QACH,mFAAmF;QACnF,0FAA0F;QAC1F,8EAA8E;QAC9E,MAAM,cAAc,GAAG,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAW,CAAC,CAAC;QAC1E,MAAM,CAAC,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,uBAAuB,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACrF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8GAA8G,EAAE,GAAG,EAAE;QACtH,eAAe,CAAC;YACd,8CAA8C,EAAE,EAAE;YAClD,iCAAiC,EAAE,WAAW;YAC9C,sCAAsC,EAAE,4EAA4E;YACpH,2BAA2B,EAAE,kCAAkC;YAC/D,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC;YACnC,qCAAqC;YACrC,qCAAqC;SACtC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,eAAe,CAAC;YACd,8CAA8C,EAAE,0CAA0C;YAC1F,iCAAiC,EAAE,WAAW;YAC9C,sCAAsC,EAAE,4EAA4E;YACpH,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC;YACnC,qCAAqC;YACrC,qCAAqC;SACtC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4GAA4G,EAAE,GAAG,EAAE;QACpH,0FAA0F;QAC1F,2FAA2F;QAC3F,wFAAwF;QACxF,wFAAwF;QACxF,qDAAqD;QACrD,eAAe,CAAC;YACd,8CAA8C,EAAE,EAAE;YAClD,iCAAiC,EAAE,WAAW;YAC9C,sCAAsC,EAAE,8BAA8B;YACtE,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,EAAE,kBAAkB,CAAC,CAAC,CAAC;IACzE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+FAA+F,EAAE,GAAG,EAAE;QACvG,eAAe,CAAC;YACd,8CAA8C,EAAE,0CAA0C;YAC1F,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;YAC1C,0FAA0F;YAC1F,yFAAyF;YACzF,uDAAuD;SACxD,CAAC,CAAC;QAEH,MAAM,CAAC,GAAG,EAAE,CAAC,wBAAwB,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;QAC9D,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,CAAC,qCAAqC,CAAC,CAAC,CAAC;IAChF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAClF,MAAM,MAAM,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjF,eAAe,CAAC;YACd,8CAA8C,EAAE,KAAK;YACrD,iCAAiC,EAAE,WAAW;YAC9C,sCAAsC,EAAE,MAAM;YAC9C,2BAA2B,EAAE,QAAQ;YACrC,+BAA+B,EAAE,SAAS;SAC3C,CAAC,CAAC;QAEH,MAAM,OAAO,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,YAAY,CAAC,EAAE,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}