@productbrain/cli 0.1.0-beta.47 → 0.1.0-beta.4721

package/dist/__tests__/promote-batch.test.js

@@ -0,0 +1,185 @@
+/**
+ * pb promote — batch (multi-ID) promotion (INS-1813).
+ * Accepts multiple entry IDs, promotes each independently, and a single failure
+ * does NOT abort the rest of the batch. Reports per-entry results and exits
+ * non-zero when any entry failed.
+ */
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { runPromote } from '../commands/promote.js';
+import { CLIError, ErrorCode } from '../lib/errors.js';
+const kernelCallWithSessionMock = vi.fn();
+const { MockMcpError } = vi.hoisted(() => {
+    class MockMcpError extends Error {
+        code;
+        details;
+        constructor(message, code, details) {
+            super(message);
+            this.name = 'McpError';
+            this.code = code;
+            this.details = details;
+        }
+    }
+    return { MockMcpError };
+});
+vi.mock('../lib/client.js', () => ({
+    kernelCallWithSession: (...args) => kernelCallWithSessionMock(...args),
+    McpError: MockMcpError,
+}));
+vi.mock('../lib/config.js', () => ({
+    getConfigOrGuide: vi.fn(() => Promise.resolve({ apiKey: 'pb_sk_test', siteUrl: 'https://test.convex.site' })),
+}));
+vi.mock('../lib/runner.js', () => ({
+    isJsonMode: () => false,
+    runCliCommand: vi.fn(async ({ fn, formatPretty }) => {
+        const data = await fn();
+        process.stdout.write(formatPretty(data) + '\n');
+    }),
+}));
+let mockSession = { sessionId: 'sess-test' };
+vi.mock('../lib/session.js', () => ({
+    readSession: () => mockSession,
+}));
+/** Route by fn + entryId so batches don't depend on call ordering. */
+function installRoutes(opts = {}) {
+    const failIds = new Set(opts.failIds ?? []);
+    kernelCallWithSessionMock.mockImplementation((fn, args) => {
+        const id = args?.entryId ?? '';
+        if (fn === 'chain.getEntry') {
+            return Promise.resolve({ name: `Name ${id}`, collectionSlug: 'tensions', data: { description: 'A valid description' } });
+        }
+        if (fn === 'chain.detectSemanticConflicts')
+            return Promise.resolve([]);
+        if (fn === 'chain.commitEntry') {
+            if (failIds.has(id))
+                return Promise.reject(new Error(`commit failed for ${id}`));
+            return Promise.resolve({ _id: `j-${id}`, entryId: id, name: `Name ${id}`, versionId: 'v1', version: 1 });
+        }
+        return Promise.resolve({});
+    });
+}
+function commitCallsFor(id) {
+    return kernelCallWithSessionMock.mock.calls.filter((c) => c[0] === 'chain.commitEntry' && c[1]?.entryId === id).length;
+}
+describe('runPromote — batch promotion (INS-1813)', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        mockSession = { sessionId: 'sess-test' };
+    });
+    it('promotes every ID in the batch and prints a receipt for each', async () => {
+        installRoutes();
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryIds: ['TEN-2', 'INS-3', 'DEC-4'] });
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        expect(commitCallsFor('TEN-2')).toBe(1);
+        expect(commitCallsFor('INS-3')).toBe(1);
+        expect(commitCallsFor('DEC-4')).toBe(1);
+        expect(output).toContain('Promoted TEN-2');
+        expect(output).toContain('Promoted INS-3');
+        expect(output).toContain('Promoted DEC-4');
+    });
+    it('does not abort the batch when one entry fails — the rest still promote', async () => {
+        installRoutes({ failIds: ['INS-3'] });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        let caught;
+        try {
+            await runPromote({ entryIds: ['TEN-2', 'INS-3', 'DEC-4'] });
+        }
+        catch (err) {
+            caught = err;
+        }
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        // Every entry was attempted, including the ones after the failure.
+        expect(commitCallsFor('TEN-2')).toBe(1);
+        expect(commitCallsFor('INS-3')).toBe(1);
+        expect(commitCallsFor('DEC-4')).toBe(1);
+        // The successful ones still got promoted.
+        expect(output).toContain('Promoted TEN-2');
+        expect(output).toContain('Promoted DEC-4');
+        // The failure is surfaced per-entry...
+        expect(output).toContain('INS-3');
+        // ...and the batch exits non-zero so automation knows.
+        expect(caught).toBeInstanceOf(Error);
+    });
+    it('still works for a single ID passed via entryIds', async () => {
+        installRoutes();
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryIds: ['TEN-2'] });
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        expect(commitCallsFor('TEN-2')).toBe(1);
+        expect(output).toContain('Promoted TEN-2');
+    });
+    it('fails fast and preserves the category when a global precondition (auth) fails the batch', async () => {
+        // An expired/invalid key fails every id identically. The batch must NOT retry each id
+        // and relabel it as a per-entry VALIDATION_FAILED — it rethrows the real auth error
+        // immediately, just like the single-id path (Codex P2, PR #196).
+        kernelCallWithSessionMock.mockImplementation((fn, args) => {
+            const id = args?.entryId ?? '';
+            if (fn === 'chain.getEntry') {
+                return Promise.resolve({ name: `Name ${id}`, collectionSlug: 'tensions', data: { description: 'A valid description' } });
+            }
+            if (fn === 'chain.detectSemanticConflicts')
+                return Promise.resolve([]);
+            if (fn === 'chain.commitEntry') {
+                return Promise.reject(new CLIError('API key expired.', {
+                    code: ErrorCode.AUTH_INVALID,
+                    category: 'auth',
+                    guidance: 'Run `pb login` to re-authenticate.',
+                }));
+            }
+            return Promise.resolve({});
+        });
+        let caught;
+        try {
+            await runPromote({ entryIds: ['TEN-2', 'INS-3', 'DEC-4'] });
+        }
+        catch (err) {
+            caught = err;
+        }
+        // The real auth category/code survives — not masked as validation.
+        expect(caught).toBeInstanceOf(CLIError);
+        expect(caught.category).toBe('auth');
+        expect(caught.code).toBe(ErrorCode.AUTH_INVALID);
+        // Fail fast: it aborted on the first id, never attempting the rest against a dead key.
+        expect(commitCallsFor('TEN-2')).toBe(1);
+        expect(commitCallsFor('INS-3')).toBe(0);
+        expect(commitCallsFor('DEC-4')).toBe(0);
+    });
+    it('fails fast on a session-category failure (closed/expired session dooms every id)', async () => {
+        kernelCallWithSessionMock.mockImplementation((fn, args) => {
+            const id = args?.entryId ?? '';
+            if (fn === 'chain.getEntry') {
+                return Promise.resolve({ name: `Name ${id}`, collectionSlug: 'tensions', data: { description: 'A valid description' } });
+            }
+            if (fn === 'chain.detectSemanticConflicts')
+                return Promise.resolve([]);
+            if (fn === 'chain.commitEntry') {
+                return Promise.reject(new CLIError('Session expired.', {
+                    code: ErrorCode.SESSION_REQUIRED,
+                    category: 'session',
+                    guidance: 'Run `pb session start`.',
+                }));
+            }
+            return Promise.resolve({});
+        });
+        let caught;
+        try {
+            await runPromote({ entryIds: ['TEN-2', 'INS-3'] });
+        }
+        catch (err) {
+            caught = err;
+        }
+        expect(caught.category).toBe('session');
+        expect(caught.code).toBe(ErrorCode.SESSION_REQUIRED);
+        expect(commitCallsFor('TEN-2')).toBe(1);
+        expect(commitCallsFor('INS-3')).toBe(0);
+    });
+    it('rejects when the batch is empty after trimming', async () => {
+        installRoutes();
+        await expect(runPromote({ entryIds: ['  ', ''] })).rejects.toThrow('Entry ID is required');
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalled();
+    });
+});
+//# sourceMappingURL=promote-batch.test.js.map

package/dist/__tests__/promote-batch.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"promote-batch.test.js","sourceRoot":"","sources":["../../src/__tests__/promote-batch.test.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AACpD,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAEvD,MAAM,yBAAyB,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC;AAE1C,MAAM,EAAE,YAAY,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE;IACvC,MAAM,YAAa,SAAQ,KAAK;QAC9B,IAAI,CAAU;QACd,OAAO,CAAa;QACpB,YAAY,OAAe,EAAE,IAAa,EAAE,OAAmB;YAC7D,KAAK,CAAC,OAAO,CAAC,CAAC;YACf,IAAI,CAAC,IAAI,GAAG,UAAU,CAAC;YACvB,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC;YACjB,IAAI,CAAC,OAAO,GAAG,OAAO,CAAC;QACzB,CAAC;KACF;IACD,OAAO,EAAE,YAAY,EAAE,CAAC;AAC1B,CAAC,CAAC,CAAC;AAEH,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,GAAG,EAAE,CAAC,CAAC;IACjC,qBAAqB,EAAE,CAAC,GAAG,IAAe,EAAE,EAAE,CAAC,yBAAyB,CAAC,GAAG,IAAI,CAAC;IACjF,QAAQ,EAAE,YAAY;CACvB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,GAAG,EAAE,CAAC,CAAC;IACjC,gBAAgB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,YAAY,EAAE,OAAO,EAAE,0BAA0B,EAAE,CAAC,CAAC;CAC9G,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,kBAAkB,EAAE,GAAG,EAAE,CAAC,CAAC;IACjC,UAAU,EAAE,GAAG,EAAE,CAAC,KAAK;IACvB,aAAa,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,EAAK,EAAE,EAAE,EAAE,YAAY,EAA4D,EAAE,EAAE;QAC/G,MAAM,IAAI,GAAG,MAAM,EAAE,EAAE,CAAC;QACxB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,IAAI,CAAC,CAAC;IAClD,CAAC,CAAC;CACH,CAAC,CAAC,CAAC;AAEJ,IAAI,WAAW,GAAiC,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC;AAC3E,EAAE,CAAC,IAAI,CAAC,mBAAmB,EAAE,GAAG,EAAE,CAAC,CAAC;IAClC,WAAW,EAAE,GAAG,EAAE,CAAC,WAAW;CAC/B,CAAC,CAAC,CAAC;AAEJ,sEAAsE;AACtE,SAAS,aAAa,CAAC,OAA+B,EAAE;IACtD,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC;IAC5C,yBAAyB,CAAC,kBAAkB,CAAC,CAAC,EAAU,EAAE,IAA0B,EAAE,EAAE;QACtF,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,IAAI,EAAE,CAAC;QAC/B,IAAI,EAAE,KAAK,gBAAgB,EAAE,CAAC;YAC5B,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,cAAc,EAAE,UAAU,EAAE,IAAI,EAAE,EAAE,WAAW,EAAE,qBAAqB,EAAE,EAAE,CAAC,CAAC;QAC3H,CAAC;QACD,IAAI,EAAE,KAAK,+BAA+B;YAAE,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACvE,IAAI,EAAE,KAAK,mBAAmB,EAAE,CAAC;YAC/B,IAAI,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;gBAAE,OAAO,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,qBAAqB,EAAE,EAAE,CAAC,CAAC,CAAC;YACjF,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,GAAG,EAAE,KAAK,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC,CAAC;QAC3G,CAAC;QACD,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,cAAc,CAAC,EAAU;IAChC,OAAO,yBAAyB,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,mBAAmB,IAAK,CAAC,CAAC,CAAC,CAA0B,EAAE,OAAO,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC;AACnJ,CAAC;AAED,QAAQ,CAAC,yCAAyC,EAAE,GAAG,EAAE;IACvD,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,WAAW,GAAG,EAAE,SAAS,EAAE,WAAW,EAAE,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,aAAa,EAAE,CAAC;QAChB,MAAM,QAAQ,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;QAElF,MAAM,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC,CAAC;QAC5D,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACrE,QAAQ,CAAC,WAAW,EAAE,CAAC;QAEvB,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wEAAwE,EAAE,KAAK,IAAI,EAAE;QACtF,aAAa,CAAC,EAAE,OAAO,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACtC,MAAM,QAAQ,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;QAElF,IAAI,MAAe,CAAC;QACpB,IAAI,CAAC;YACH,MAAM,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC,CAAC;QAC9D,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,GAAG,GAAG,CAAC;QACf,CAAC;QACD,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACrE,QAAQ,CAAC,WAAW,EAAE,CAAC;QAEvB,mEAAmE;QACnE,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,0CAA0C;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC3C,uCAAuC;QACvC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QAClC,uDAAuD;QACvD,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,KAAK,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,aAAa,EAAE,CAAC;QAChB,MAAM,QAAQ,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;QAElF,MAAM,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC1C,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACrE,QAAQ,CAAC,WAAW,EAAE,CAAC;QAEvB,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yFAAyF,EAAE,KAAK,IAAI,EAAE;QACvG,sFAAsF;QACtF,oFAAoF;QACpF,iEAAiE;QACjE,yBAAyB,CAAC,kBAAkB,CAAC,CAAC,EAAU,EAAE,IAA0B,EAAE,EAAE;YACtF,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,IAAI,EAAE,CAAC;YAC/B,IAAI,EAAE,KAAK,gBAAgB,EAAE,CAAC;gBAC5B,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,cAAc,EAAE,UAAU,EAAE,IAAI,EAAE,EAAE,WAAW,EAAE,qBAAqB,EAAE,EAAE,CAAC,CAAC;YAC3H,CAAC;YACD,IAAI,EAAE,KAAK,+BAA+B;gBAAE,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;YACvE,IAAI,EAAE,KAAK,mBAAmB,EAAE,CAAC;gBAC/B,OAAO,OAAO,CAAC,MAAM,CACnB,IAAI,QAAQ,CAAC,kBAAkB,EAAE;oBAC/B,IAAI,EAAE,SAAS,CAAC,YAAY;oBAC5B,QAAQ,EAAE,MAAM;oBAChB,QAAQ,EAAE,oCAAoC;iBAC/C,CAAC,CACH,CAAC;YACJ,CAAC;YACD,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC7B,CAAC,CAAC,CAAC;QAEH,IAAI,MAAe,CAAC;QACpB,IAAI,CAAC;YACH,MAAM,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC,CAAC;QAC9D,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,GAAG,GAAG,CAAC;QACf,CAAC;QAED,mEAAmE;QACnE,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC;QACxC,MAAM,CAAE,MAAmB,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACnD,MAAM,CAAE,MAA4B,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACxE,uFAAuF;QACvF,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kFAAkF,EAAE,KAAK,IAAI,EAAE;QAChG,yBAAyB,CAAC,kBAAkB,CAAC,CAAC,EAAU,EAAE,IAA0B,EAAE,EAAE;YACtF,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,IAAI,EAAE,CAAC;YAC/B,IAAI,EAAE,KAAK,gBAAgB,EAAE,CAAC;gBAC5B,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,cAAc,EAAE,UAAU,EAAE,IAAI,EAAE,EAAE,WAAW,EAAE,qBAAqB,EAAE,EAAE,CAAC,CAAC;YAC3H,CAAC;YACD,IAAI,EAAE,KAAK,+BAA+B;gBAAE,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;YACvE,IAAI,EAAE,KAAK,mBAAmB,EAAE,CAAC;gBAC/B,OAAO,OAAO,CAAC,MAAM,CACnB,IAAI,QAAQ,CAAC,kBAAkB,EAAE;oBAC/B,IAAI,EAAE,SAAS,CAAC,gBAAgB;oBAChC,QAAQ,EAAE,SAAS;oBACnB,QAAQ,EAAE,yBAAyB;iBACpC,CAAC,CACH,CAAC;YACJ,CAAC;YACD,OAAO,OAAO,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC7B,CAAC,CAAC,CAAC;QAEH,IAAI,MAAe,CAAC;QACpB,IAAI,CAAC;YACH,MAAM,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC,CAAC;QACrD,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,GAAG,GAAG,CAAC;QACf,CAAC;QAED,MAAM,CAAE,MAAmB,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACtD,MAAM,CAAE,MAA4B,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC5E,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxC,MAAM,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,aAAa,EAAE,CAAC;QAChB,MAAM,MAAM,CAAC,UAAU,CAAC,EAAE,QAAQ,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,sBAAsB,CAAC,CAAC;QAC3F,MAAM,CAAC,yBAAyB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC3D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/__tests__/promote.test.js

@@ -1,22 +1,50 @@
 /**
- * pb promote — verify mcpCallWithSession sequence and args.
- * Covers: happy path (direct commit), governance proposal, no session, empty entryId.
+ * pb promote — verify kernelCallWithSession sequence and args.
+ * Covers: happy path (direct commit), governance proposal, no session, empty entryId,
+ * and the SERVER-side auto-verify receipt (WP-B, attestation-model spec §4.3 — the
+ * old client-side verify-then-retry on QUALITY_GATE_BLOCKED is deleted dead code).
  */
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { runPromote } from '../commands/promote.js';
-const mcpCallWithSessionMock = vi.fn();
+const kernelCallWithSessionMock = vi.fn();
+// Hoist McpError so it's available both in the mock factory and in test bodies.
+// The client mock must export the same class other lib consumers import; the
+// tests use it to shape server-thrown errors (WP-B deleted promote.ts's own
+// McpError-instanceof retry, so only the mock factory needs the class now).
+const { MockMcpError } = vi.hoisted(() => {
+    class MockMcpError extends Error {
+        code;
+        details;
+        constructor(message, code, details) {
+            super(message);
+            this.name = 'McpError';
+            this.code = code;
+            this.details = details;
+        }
+    }
+    return { MockMcpError };
+});
 vi.mock('../lib/client.js', () => ({
-    mcpCallWithSession: (...args) => mcpCallWithSessionMock(...args),
+    kernelCallWithSession: (...args) => kernelCallWithSessionMock(...args),
+    McpError: MockMcpError,
 }));
 vi.mock('../lib/config.js', () => ({
     getConfigOrGuide: vi.fn(() => Promise.resolve({ apiKey: 'pb_sk_test', siteUrl: 'https://test.convex.site' })),
 }));
-// Force pretty mode for all tests so formatPromoteReceipt output is tested
+// Output mode is controllable per-test via runnerState.jsonMode (defaults to pretty,
+// so existing formatPromoteReceipt assertions keep working). The mock runCliCommand
+// mirrors the real one: JSON.stringify in json mode, formatPretty otherwise.
+const { runnerState } = vi.hoisted(() => ({ runnerState: { jsonMode: false } }));
 vi.mock('../lib/runner.js', () => ({
-    isJsonMode: () => false,
+    isJsonMode: () => runnerState.jsonMode,
     runCliCommand: vi.fn(async ({ fn, formatPretty }) => {
         const data = await fn();
-        process.stdout.write(formatPretty(data) + '\n');
+        if (runnerState.jsonMode) {
+            process.stdout.write(JSON.stringify(data) + '\n');
+        }
+        else {
+            process.stdout.write(formatPretty(data) + '\n');
+        }
     }),
 }));
 let mockSession = { sessionId: 'sess-test' };
@@ -27,9 +55,10 @@ describe('runPromote', () => {
     beforeEach(() => {
         vi.clearAllMocks();
         mockSession = { sessionId: 'sess-test' };
+        runnerState.jsonMode = false;
     });
     it('calls chain.commitEntry with correct args (happy path)', async () => {
-        mcpCallWithSessionMock
+        kernelCallWithSessionMock
             .mockResolvedValueOnce({ name: 'Test entry', collectionSlug: 'tensions', data: { description: 'Valid description' } })
             .mockResolvedValueOnce([])
             .mockResolvedValueOnce({
@@ -41,14 +70,14 @@ describe('runPromote', () => {
         const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
         await runPromote({ entryId: 'TEN-951' });
         writeSpy.mockRestore();
-        expect(mcpCallWithSessionMock).toHaveBeenCalledWith('chain.commitEntry', {
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.commitEntry', {
             entryId: 'TEN-951',
             author: 'agent:sess-test',
             sessionId: 'sess-test',
         });
     });
     it('passes commitMessage when --message is provided', async () => {
-        mcpCallWithSessionMock
+        kernelCallWithSessionMock
             .mockResolvedValueOnce({ name: 'Test entry', collectionSlug: 'tensions', data: { description: 'Valid description' } })
             .mockResolvedValueOnce([])
             .mockResolvedValueOnce({
@@ -60,7 +89,7 @@ describe('runPromote', () => {
         const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
         await runPromote({ entryId: 'TEN-951', message: 'Validated and ready' });
         writeSpy.mockRestore();
-        expect(mcpCallWithSessionMock).toHaveBeenCalledWith('chain.commitEntry', {
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.commitEntry', {
             entryId: 'TEN-951',
             author: 'agent:sess-test',
             sessionId: 'sess-test',
@@ -68,7 +97,7 @@ describe('runPromote', () => {
         });
     });
     it('displays version result for direct commit', async () => {
-        mcpCallWithSessionMock
+        kernelCallWithSessionMock
             .mockResolvedValueOnce({ name: 'Wave 2 smoke test', collectionSlug: 'tensions', data: { description: 'Valid description' } })
             .mockResolvedValueOnce([])
             .mockResolvedValueOnce({
@@ -88,7 +117,7 @@ describe('runPromote', () => {
         expect(output).toContain('v1');
     });
     it('displays proposal result for governance mode', async () => {
-        mcpCallWithSessionMock
+        kernelCallWithSessionMock
             .mockResolvedValueOnce({ name: 'Some governed decision', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
             .mockResolvedValueOnce([])
             .mockResolvedValueOnce({
@@ -107,8 +136,8 @@ describe('runPromote', () => {
         expect(output).toContain('governance mode');
     });
     it('displays quality warnings when present', async () => {
-        mcpCallWithSessionMock
-            .mockResolvedValueOnce({ name: 'Test Bet', collectionSlug: 'bets', data: { description: 'Valid description' } })
+        kernelCallWithSessionMock
+            .mockResolvedValueOnce({ name: 'Test Bet', collectionSlug: 'work-packages', data: { description: 'Valid description' } })
             .mockResolvedValueOnce([])
             .mockResolvedValueOnce({
             _id: 'j571abc',
@@ -132,20 +161,72 @@ describe('runPromote', () => {
     it('throws CLIError with no active session', async () => {
         mockSession = null;
         await expect(runPromote({ entryId: 'TEN-951' })).rejects.toThrow('No active session');
-        expect(mcpCallWithSessionMock).not.toHaveBeenCalled();
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalled();
     });
     it('throws CLIError with empty entryId', async () => {
         await expect(runPromote({ entryId: '  ' })).rejects.toThrow('Entry ID is required');
-        expect(mcpCallWithSessionMock).not.toHaveBeenCalled();
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalled();
     });
     it('handles server errors thrown during commit', async () => {
-        mcpCallWithSessionMock.mockRejectedValue(new Error('Entry not found.'));
+        kernelCallWithSessionMock.mockRejectedValue(new Error('Entry not found.'));
         await expect(runPromote({ entryId: 'TEN-999' })).rejects.toThrow('Entry not found.');
     });
-    it('blocks promote when semantic contradiction preflight finds a contradiction', async () => {
-        mcpCallWithSessionMock
+    // WP-485 Slice 2b (R2, FEAT-1370): the client no longer decides — it discovers and hands
+    // the candidates to commitEntry as evidence. The enforcement decision (Slice 2a) lives
+    // server-side inside commitEntry's evaluate()/assessContradiction gate.
+    it('passes discovered conflicts to commitEntry as evidence instead of blocking locally', async () => {
+        const conflict = {
+            chainEntryId: 'DEC-42',
+            chainEntryName: 'Existing decision',
+            type: 'contradiction',
+            confidence: 0.9,
+            explanation: 'Directly contradicts the approved baseline.',
+        };
+        kernelCallWithSessionMock
+            .mockResolvedValueOnce({ name: 'Conflicting entry', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
+            .mockResolvedValueOnce([conflict])
+            .mockResolvedValueOnce({
+            _id: 'j571abc',
+            entryId: 'DEC-100',
+            versionId: 'v123',
+            version: 1,
+        });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryId: 'DEC-100' });
+        writeSpy.mockRestore();
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.commitEntry', {
+            entryId: 'DEC-100',
+            author: 'agent:sess-test',
+            sessionId: 'sess-test',
+            conflicts: [conflict],
+        });
+    });
+    // WP-485 Slice 2b (FEAT-1370): DEAD-BUT-TYPED — `{ graduated: false }` (DEC-1321) means
+    // the current server can never throw this, but the client must not silently swallow it
+    // as success on the day a future graduation flips disposition to block/acknowledge/queue.
+    it('surfaces a thrown CONTRADICTION_BLOCKED as a legible, actionable error (dead-but-typed, forced double)', async () => {
+        kernelCallWithSessionMock
             .mockResolvedValueOnce({ name: 'Conflicting entry', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
             .mockResolvedValueOnce([
+            {
+                chainEntryId: 'DEC-42',
+                chainEntryName: 'Existing decision',
+                type: 'contradiction',
+                confidence: 0.95,
+                explanation: 'Directly contradicts the approved baseline.',
+            },
+        ])
+            .mockRejectedValueOnce(new MockMcpError('Blocked by contradiction with DEC-42.', 'CONTRADICTION_BLOCKED'));
+        await expect(runPromote({ entryId: 'DEC-100' })).rejects.toThrow('Blocked by contradiction with DEC-42.');
+        const commitCalls = kernelCallWithSessionMock.mock.calls.filter((c) => c[0] === 'chain.commitEntry');
+        expect(commitCalls).toHaveLength(1);
+    });
+    // WP-485 Slice 2b (R3, FEAT-1370): the commit succeeded — the advisory is informational,
+    // rendered from the server's single verdict, never re-derived client-side.
+    it('renders a contradictionAdvisory in the receipt when the server attaches one', async () => {
+        kernelCallWithSessionMock
+            .mockResolvedValueOnce({ name: 'Advisory entry', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
+            .mockResolvedValueOnce([
             {
                 chainEntryId: 'DEC-42',
                 chainEntryName: 'Existing decision',
@@ -153,9 +234,197 @@ describe('runPromote', () => {
                 confidence: 0.9,
                 explanation: 'Directly contradicts the approved baseline.',
             },
-        ]);
-        await expect(runPromote({ entryId: 'DEC-100' })).rejects.toThrow('Commit blocked by contradiction with DEC-42: Directly contradicts the approved baseline.');
-        expect(mcpCallWithSessionMock).not.toHaveBeenCalledWith('chain.commitEntry', expect.anything());
+        ])
+            .mockResolvedValueOnce({
+            _id: 'j571abc',
+            entryId: 'DEC-100',
+            name: 'Advisory entry',
+            versionId: 'v123',
+            version: 1,
+            contradictionAdvisory: {
+                reason: 'Advisory only — not yet graduated to blocking.',
+                blockingCount: 1,
+                conflictDetails: [
+                    {
+                        chainEntryId: 'DEC-42',
+                        chainEntryName: 'Existing decision',
+                        type: 'contradiction',
+                        confidence: 0.9,
+                        explanation: 'Directly contradicts the approved baseline.',
+                    },
+                ],
+            },
+        });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryId: 'DEC-100' });
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        expect(output).toContain('Promoted DEC-100');
+        expect(output).toContain('Contradiction advisory');
+        expect(output).toContain('Advisory only — not yet graduated to blocking.');
+        expect(output).toContain('DEC-42');
+    });
+    // WP-B (attestation-model spec §4.3): the server auto-verifies as part of the
+    // promote itself. The CLI never verifies-and-retries — it renders the receipt.
+    it('renders the server auto-verify receipt (no client-side verify, single commit call)', async () => {
+        kernelCallWithSessionMock
+            // call 1: chain.getEntry
+            .mockResolvedValueOnce({ name: 'Agent capture', collectionSlug: 'insights', data: { description: 'Some insight' } })
+            // call 2: chain.detectSemanticConflicts
+            .mockResolvedValueOnce([])
+            // call 3: chain.commitEntry → committed WITH the server-side auto-verify receipt
+            .mockResolvedValueOnce({
+            _id: 'j571abc', entryId: 'INS-100', versionId: 'v1', version: 1,
+            autoVerified: { strength: 'delegated', basis: 'attested', context: 'promote-gate', verifiedBy: 'agent for Randy Test' },
+        });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryId: 'INS-100' });
+        const output = writeSpy.mock.calls.map(c => String(c[0])).join('');
+        writeSpy.mockRestore();
+        // THE RECEIPT SAYS SO (spec §4.3) — silence here is a structural regression.
+        expect(output).toContain('Promoted INS-100');
+        expect(output).toContain('Auto-verified as part of promote — delegated (agent for Randy Test)');
+        // The client-side verify-then-retry is dead code: verifyEntry never called,
+        // commitEntry called exactly once.
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.verifyEntry', expect.anything());
+        const commitCalls = kernelCallWithSessionMock.mock.calls.filter(c => c[0] === 'chain.commitEntry');
+        expect(commitCalls).toHaveLength(1);
+    });
+    it('surfaces ANY QUALITY_GATE_BLOCKED as a legible error — no verify-retry, even for a verification-shaped block from an old server (spec §11 rollout rule)', async () => {
+        kernelCallWithSessionMock
+            // call 1: chain.getEntry
+            .mockResolvedValueOnce({ name: 'Agent capture', collectionSlug: 'insights', data: { description: 'Some insight' } })
+            // call 2: chain.detectSemanticConflicts
+            .mockResolvedValueOnce([])
+            // call 3: chain.commitEntry → an OLD server still throws the verification block
+            .mockRejectedValueOnce(new MockMcpError('Agent-origin entry requires verification before commit', 'QUALITY_GATE_BLOCKED'));
+        await expect(runPromote({ entryId: 'INS-100' })).rejects.toThrow('requires verification');
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.verifyEntry', expect.anything());
+        const commitCalls = kernelCallWithSessionMock.mock.calls.filter(c => c[0] === 'chain.commitEntry');
+        expect(commitCalls).toHaveLength(1);
+    });
+    it('surfaces other QUALITY_GATE_BLOCKED reasons unchanged', async () => {
+        kernelCallWithSessionMock
+            // call 1: chain.getEntry
+            .mockResolvedValueOnce({ name: 'Incomplete entry', collectionSlug: 'work-packages', data: { description: 'A bet' } })
+            // call 2: chain.detectSemanticConflicts
+            .mockResolvedValueOnce([])
+            // call 3: chain.commitEntry → blocked for a different reason (missing fields)
+            .mockRejectedValueOnce(new MockMcpError('Entry missing required fields', 'QUALITY_GATE_BLOCKED'));
+        await expect(runPromote({ entryId: 'BET-200' })).rejects.toThrow('Entry missing required fields');
+        // verifyEntry must never have been called
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.verifyEntry', expect.anything());
+    });
+    // ── WP-465 slice ⑤ (Finding #25): structured coherency refusal in --json mode ──
+    it('emits machine-readable refusal routes to stdout on a Standard/Strict coherency refusal (--json)', async () => {
+        runnerState.jsonMode = true;
+        const refusal = {
+            outcome: 'refused',
+            code: 'COHERENCY_REFUSED',
+            disposition: 'acknowledge',
+            effectiveDisposition: 'acknowledge-required',
+            overridable: true,
+            mode: 'standard',
+            reason: "Steering gate returned 'acknowledge': this governance write requires an explicit response.",
+            subject: { entryId: 'DEC-500', name: 'Standalone decision' },
+            routes: {
+                link: { description: 'Link this entry to an active part of the strategic spine and retry.' },
+                acknowledge: {
+                    param: 'coherencyAcknowledgement',
+                    responses: ['linked', 'accepted-fix', 'diverged'],
+                    minReasonChars: 12,
+                    description: 'Respond to the coherency challenge.',
+                },
+                override: { param: 'steeringOverrideReason', minChars: 12, description: 'Typed override.' },
+            },
+        };
+        kernelCallWithSessionMock
+            // call 1: chain.getEntry
+            .mockResolvedValueOnce({ name: 'Standalone decision', collectionSlug: 'decisions', data: { rationale: 'Stands alone.' } })
+            // call 2: chain.detectSemanticConflicts
+            .mockResolvedValueOnce([])
+            // call 3: chain.commitEntry → coherency refusal (RETURNED, not thrown)
+            .mockResolvedValueOnce({ status: 'coherency_refused', refusal });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        // The throw carries the gate's own reason (so humans see the cause on stderr).
+        await expect(runPromote({ entryId: 'DEC-500' })).rejects.toThrow(/Steering gate returned/);
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        // The structured { outcome: 'refused', refusal } envelope reached stdout (machine-readable),
+        // carrying the subject + the retry routes the agent needs to populate the controls.
+        const parsed = JSON.parse(output.trim());
+        expect(parsed.outcome).toBe('refused');
+        expect(parsed.refusal.subject.entryId).toBe('DEC-500');
+        expect(parsed.refusal.routes.acknowledge.param).toBe('coherencyAcknowledgement');
+        expect(parsed.refusal.routes.override.param).toBe('steeringOverrideReason');
+    });
+    it('does not emit a refusal JSON line in human (pretty) mode', async () => {
+        runnerState.jsonMode = false;
+        kernelCallWithSessionMock
+            .mockResolvedValueOnce({ name: 'Standalone decision', collectionSlug: 'decisions', data: { rationale: 'Stands alone.' } })
+            .mockResolvedValueOnce([])
+            .mockResolvedValueOnce({
+            status: 'coherency_refused',
+            refusal: { reason: 'Refused by the coherency gate.' },
+        });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await expect(runPromote({ entryId: 'DEC-500' })).rejects.toThrow(/coherency/i);
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        expect(output).not.toContain('"outcome":"refused"');
+    });
+    // ── WP-465 slice ⑤ (Finding #9): Light-mode steering advisory in the receipt ──
+    it('surfaces a non-blocking steeringAdvisory in the promote receipt when present', async () => {
+        kernelCallWithSessionMock
+            .mockResolvedValueOnce({ name: 'Misaligned decision', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
+            .mockResolvedValueOnce([])
+            .mockResolvedValueOnce({
+            _id: 'j571abc',
+            entryId: 'DEC-600',
+            name: 'Misaligned decision',
+            versionId: 'v123',
+            version: 1,
+            steeringAdvisory: 'This decision is not linked to the strategic spine.',
+        });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryId: 'DEC-600' });
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        expect(output).toContain('Promoted DEC-600');
+        expect(output).toContain('Coherency advisory');
+        expect(output).toContain('This decision is not linked to the strategic spine.');
+    });
+    // WP-485 Slice 2b round 2 (Codex P1, FEAT-1370): batch --json used to reduce every success
+    // to `{entryId, ok:true}`, discarding the server's entire contradictionAdvisory — a detected
+    // contradiction could commit while `pb promote ID1 ID2 --json` gave automated callers no
+    // trace it happened. The single-id JSON path already returns the full result object; batch
+    // must not throw the advisory away just because it takes a different output branch.
+    it('preserves contradictionAdvisory in batch --json output (not reduced to {entryId, ok:true})', async () => {
+        runnerState.jsonMode = true;
+        const advisory = {
+            reason: '1 possible contradiction(s) with existing Chain entries (advisory).',
+            blockingCount: 1,
+            conflictDetails: [
+                { chainEntryId: 'DEC-42', chainEntryName: 'Existing decision', type: 'contradiction', confidence: 0.9, explanation: 'Directly contradicts the approved baseline.' },
+            ],
+        };
+        kernelCallWithSessionMock
+            // TEN-951: no conflict, no advisory
+            .mockResolvedValueOnce({ name: 'Clean entry', collectionSlug: 'tensions', data: { description: 'Valid description' } })
+            .mockResolvedValueOnce([])
+            .mockResolvedValueOnce({ _id: 'j1', entryId: 'TEN-951', versionId: 'v1', version: 1 })
+            // DEC-100: contradiction found, advisory attached
+            .mockResolvedValueOnce({ name: 'Conflicting entry', collectionSlug: 'decisions', data: { rationale: 'Valid description' } })
+            .mockResolvedValueOnce([{ chainEntryId: 'DEC-42', chainEntryName: 'Existing decision', type: 'contradiction', confidence: 0.9, explanation: 'Directly contradicts the approved baseline.' }])
+            .mockResolvedValueOnce({ _id: 'j2', entryId: 'DEC-100', versionId: 'v1', version: 1, contradictionAdvisory: advisory });
+        const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        await runPromote({ entryIds: ['TEN-951', 'DEC-100'] });
+        const output = writeSpy.mock.calls.map((c) => String(c[0])).join('');
+        writeSpy.mockRestore();
+        const parsed = JSON.parse(output.trim());
+        expect(parsed).toHaveLength(2);
+        expect(parsed[0]).toEqual({ entryId: 'TEN-951', ok: true });
+        expect(parsed[1].contradictionAdvisory).toEqual(advisory);
     });
 });
 //# sourceMappingURL=promote.test.js.map