@productbrain/cli 0.1.0-beta.6410 → 0.1.0-beta.6419

package/dist/__tests__/capture.test.js

@@ -3,7 +3,7 @@
  * TEN-341 test contract: capture --link calls createEntryRelation, capture --json outputs JSON.
  * WP-319 S3: grounding fires in parallel, fail-open, --suggest-only, --strict, PB_GROUNDING_MODE.
  */
-import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import { runCapture } from '../commands/capture.js';
 import { CLIError, ErrorCode } from '../lib/errors.js';
 import { setOutputMode } from '../lib/runner.js';
@@ -30,8 +30,8 @@ describe('runCapture --link', () => {
     // list the source entry's legal verbs in the refusal message.
     it('--link without --link-type refuses (no default), preserving the created entry', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1', entryId: 'TEN-710' });
             if (fn === 'chain.getEntry') {
@@ -66,8 +66,8 @@ describe('runCapture --link', () => {
     });
     it('--link without --link-type: refusal message lists the source entry\'s declared relation types', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1b', entryId: 'TEN-711' });
             if (fn === 'chain.getEntry') {
@@ -104,8 +104,8 @@ describe('runCapture --link', () => {
     // the entry still gets created, but the link itself must be explicitly refused.
     it('--link "   " (whitespace-only) refuses instead of silently dropping the link', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-ws', entryId: 'TEN-720' });
             return Promise.resolve({});
@@ -128,8 +128,8 @@ describe('runCapture --link', () => {
     // must check `!== undefined` instead, or this whole block (and the bcRUML refusal) never runs.
     it('--link "" (empty string) refuses instead of silently dropping the link', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-empty', entryId: 'TEN-722' });
             return Promise.resolve({});
@@ -151,8 +151,8 @@ describe('runCapture --link', () => {
     // guidance must say so rather than recommending `pb relate <verb> <missing-id>`.
     it('--link <missing-id> without --link-type reports the target was not found, not a verb-recovery command', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-miss', entryId: 'TEN-721' });
             if (fn === 'chain.getEntry') {
@@ -185,8 +185,8 @@ describe('runCapture --link', () => {
     // `pb relate` would name a guaranteed-to-fail re-run.
     it('excludes replaces and do_not_confuse_with from the refusal\'s legal-verb list (both are door-blocked)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1c', entryId: 'DEC-1' });
             if (fn === 'chain.getEntry') {
@@ -219,8 +219,8 @@ describe('runCapture --link', () => {
     // so it must never be listed as a `pb relate`-usable recovery verb.
     it('excludes a declared verb outside the kernel allowlist from the refusal\'s legal-verb list', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1d', entryId: 'GOAL-1' });
             if (fn === 'chain.getEntry') {
@@ -251,8 +251,8 @@ describe('runCapture --link', () => {
     // supply (guards.ts:349-363) — never recommend them as recovery verbs for such a target.
     it('excludes domain-tag verbs from the refusal\'s legal-verb list when the target is a domain', async () => {
         kernelCallWithSessionMock.mockImplementation((fn, args) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1e', entryId: 'INS-1' });
             if (fn === 'chain.getEntry') {
@@ -284,8 +284,8 @@ describe('runCapture --link', () => {
     it('uses the supplied relation type when --link-type provided (no probe, real write)', async () => {
         // Prefix "INS:" is extracted → skips classification, routes directly to insights
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-2', entryId: 'INS-50' });
             if (fn === 'chain.createEntryRelation')
@@ -311,8 +311,8 @@ describe('runCapture --link', () => {
     // --link-without---link-type branch above).
     it('a relate-door refusal during --link is deferred, naming the persisted entry — never an unhandled rejection', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-wp', entryId: 'WP-6' });
             if (fn === 'chain.createEntryRelation') {
@@ -338,8 +338,8 @@ describe('runCapture --link', () => {
     // real error's identity.
     it('a non-validation relate-door failure during --link preserves its own code/category, not a relabeled VALIDATION_FAILED', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-wp7', entryId: 'WP-7' });
             if (fn === 'chain.createEntryRelation') {
@@ -367,8 +367,8 @@ describe('runCapture --link', () => {
     // the "was refused, re-run" template, which would prompt a blind duplicate-risking retry.
     it('a NETWORK_TIMEOUT relate-door failure during --link says outcome-unknown, never "was refused"', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-wp9', entryId: 'WP-9' });
             if (fn === 'chain.createEntryRelation') {
@@ -397,8 +397,8 @@ describe('runCapture --link', () => {
     // scheduled.
     it('the quality re-eval still runs after a relate-door refusal — the entry is not left permanently deferred', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-wp8', entryId: 'WP-8' });
             if (fn === 'chain.createEntryRelation') {
@@ -424,8 +424,8 @@ describe('runCapture --link', () => {
     // fallback (formatters/capture.ts:105).
     it('the human-readable receipt prints the real relation type, never the dead surfaces_tension_in fallback', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-2b', entryId: 'INS-51' });
             if (fn === 'chain.createEntryRelation')
@@ -453,8 +453,8 @@ describe('runCapture --link', () => {
     // persisted write may claim `linkedTo`/`relationType`.
     it('a governs-consent proposal diversion (no relationId) is never reported as a persisted link', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-3', entryId: 'WPKG-1' });
             if (fn === 'chain.createEntryRelation') {
@@ -493,8 +493,8 @@ describe('runCapture --link', () => {
     // deferred-then-never-run schedule would leave the entry permanently stuck.
     it('--link without --link-type does NOT defer the LLM schedule (nothing will ever un-defer it)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-4', entryId: 'TEN-20' });
             if (fn === 'chain.createEntryRelation')
@@ -512,8 +512,8 @@ describe('runCapture --link', () => {
     // instructions must reach the operator, not just whichever check runs first.
     it('surfaces BOTH the fields-rejected error and the link-type refusal when they co-occur', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry') {
                 return Promise.resolve({
                     docId: 'doc-5', entryId: 'TEN-21',
@@ -545,8 +545,8 @@ describe('runCapture --link', () => {
     // error code must still see the timeout, not a generic validation failure.
     it('the combined fields-rejected + link-refused error preserves the link failure\'s own code (NETWORK_TIMEOUT), not VALIDATION_FAILED', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry') {
                 return Promise.resolve({
                     docId: 'doc-10', entryId: 'TEN-22',
@@ -583,8 +583,8 @@ describe('runCapture --entry-type', () => {
     });
     it('forwards --entry-type as canonicalKey on the chain.createEntry payload', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-3', entryId: 'OBJ-1' });
             return Promise.resolve({});
@@ -599,8 +599,8 @@ describe('runCapture --entry-type', () => {
     });
     it('omits canonicalKey entirely when --entry-type is absent (byte-identical default path)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-4', entryId: 'OBJ-1' });
             return Promise.resolve({});
@@ -618,8 +618,8 @@ describe('runCapture --entry-type', () => {
     // straight to the shared Objectives drawer with no classification call at all.
     it('--entry-type objective alone (no --collection) skips classification and routes to the Objectives drawer (bcI42)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-5', entryId: 'OBJ-2' });
             return Promise.resolve({});
@@ -640,8 +640,8 @@ describe('runCapture --entry-type', () => {
     // OWN 1:1 drawer (key-results, idPrefix KR) — it no longer shares the Objectives drawer.
     it('--entry-type key_result alone (no --collection) routes to the Key Results drawer (DEC-1622)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-6', entryId: 'KR-2' });
             return Promise.resolve({});
@@ -661,8 +661,8 @@ describe('runCapture --entry-type', () => {
     // classification (often landing in 'strategy') and the placement guard then rejects it.
     it('--entry-type with mixed case/whitespace still routes to the Objectives drawer, no classification call (bclZ9)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-7', entryId: 'OBJ-3' });
             return Promise.resolve({});
@@ -681,8 +681,8 @@ describe('runCapture --entry-type', () => {
     // branches (strategy) still skips classification and routes to its own drawer.
     it('--entry-type strategy alone (no --collection) skips classification and routes to the Strategy drawer — proves the derivation is general, not a hardcoded pair', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-8', entryId: 'STR-1' });
             return Promise.resolve({});
@@ -701,8 +701,8 @@ describe('runCapture --entry-type', () => {
     // --entry-type derivation silently can't resolve them during a chain.listCollections outage.
     it('--entry-type vision alone (no --collection) skips classification and routes to the Vision drawer, even offline', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-9', entryId: 'VIS-1' });
             return Promise.resolve({});
@@ -725,8 +725,8 @@ describe('runCapture --json', () => {
     it('outputs valid JSON with id field and no other stdout', async () => {
         // Prefix "TEN:" is extracted → skips classification
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1', entryId: 'TEN-711' });
             return Promise.resolve({});
@@ -777,8 +777,8 @@ describe('runCapture coaching passthrough (WP-475 E1)', () => {
     };
     it('--json output includes the raw coaching object from the write response', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-c1', entryId: 'DEC-77', coaching });
             return Promise.resolve({});
@@ -805,8 +805,8 @@ describe('runCapture coaching passthrough (WP-475 E1)', () => {
     });
     it('--json output omits coaching when the write response has none (backward compat)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-c2', entryId: 'DEC-78' });
             return Promise.resolve({});
@@ -846,8 +846,8 @@ describe('runCapture coaching passthrough (WP-475 E1)', () => {
     };
     it('--link: re-evaluates after the relation lands and renders the post-relation verdict', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-c3', entryId: 'DEC-79', coaching });
             if (fn === 'chain.createEntryRelation')
@@ -890,8 +890,8 @@ describe('runCapture coaching passthrough (WP-475 E1)', () => {
     });
     it('--link: keeps the create-response verdict when the re-eval fails (fail-open, BR-144)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-c4', entryId: 'DEC-80', coaching });
             if (fn === 'chain.createEntryRelation')
@@ -921,8 +921,8 @@ describe('runCapture coaching passthrough (WP-475 E1)', () => {
     });
     it('no --link: does NOT call the re-evaluation and does NOT defer the LLM schedule (plain capture unchanged)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-c5', entryId: 'DEC-81', coaching });
             return Promise.resolve({});
@@ -950,8 +950,8 @@ describe('runCapture --field passthrough', () => {
     });
     it('merges --field pairs into data', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-6', entryId: 'STD-1' });
             return Promise.resolve({});
@@ -972,8 +972,8 @@ describe('runCapture --field passthrough', () => {
     });
     it('--field description=X overrides the derived description', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-7', entryId: 'TEN-1' });
             return Promise.resolve({});
@@ -997,8 +997,8 @@ describe('runCapture WHY pre-flight', () => {
     });
     it('blocks with guidance when governed and whyField is missing', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.resolve({ governed: true, whyField: 'rationale', minChars: 12, fields: [] });
             }
@@ -1011,8 +1011,8 @@ describe('runCapture WHY pre-flight', () => {
     });
     it('does not block when whyField is description', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.resolve({ governed: true, whyField: 'description', minChars: 12, fields: [] });
             }
@@ -1030,8 +1030,8 @@ describe('runCapture WHY pre-flight', () => {
         // remaps data.description → rationale before the WHY gate. A bare `pb capture "DEC: …"`
         // (description set, no --field rationale) must NOT be false-blocked client-side.
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.resolve({
                     governed: true,
@@ -1054,8 +1054,8 @@ describe('runCapture WHY pre-flight', () => {
         // proof_governed has whyField='rationale' but no descriptionFieldKey remap, so a bare
         // capture (no --field rationale) is correctly blocked — the description will not satisfy it.
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.resolve({
                     governed: true,
@@ -1074,8 +1074,8 @@ describe('runCapture WHY pre-flight', () => {
     });
     it('fails open and proceeds when the contract fetch fails', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields')
                 return Promise.reject(new Error('network'));
             if (fn === 'chain.createEntry')
@@ -1093,8 +1093,8 @@ describe('runCapture WHY pre-flight', () => {
     });
     it('fails open when the contract fetch rejects with a CLIError (server 4xx/5xx surfaces as McpError, an instanceof CLIError)', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.reject(new CLIError('Could not reach test.convex.site.', {
                     code: ErrorCode.NETWORK_UNREACHABLE,
@@ -1117,8 +1117,8 @@ describe('runCapture WHY pre-flight', () => {
     it('surfaces the server WHY-gate reason on rejection', async () => {
         const serverReason = 'Governance entries (standard) require an explicit WHY before they can be created. Set "rationale" to at least 12 characters.';
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.getCollectionFields') {
                 return Promise.resolve({ governed: true, whyField: 'rationale', minChars: 12, fields: [] });
             }
@@ -1150,8 +1150,8 @@ describe('runCapture date inference', () => {
     });
     it('does not default date to today when no source date is present', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.resolveCollection')
                 return Promise.resolve({ collection: 'decisions', confidence: 0.9, tier: 'high' });
             if (fn === 'chain.createEntry')
@@ -1170,8 +1170,8 @@ describe('runCapture date inference', () => {
     });
     it('uses inferred source date when the capture text includes one', async () => {
         kernelCallWithSessionMock.mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(null);
             if (fn === 'chain.resolveCollection')
                 return Promise.resolve({ collection: 'decisions', confidence: 0.9, tier: 'high' });
             if (fn === 'chain.createEntry')
@@ -1186,28 +1186,32 @@ describe('runCapture date inference', () => {
     });
 });
 // ---------------------------------------------------------------------------
-// WP-319 S3: Pre-write grounding tests
+// WP-319 S3 / WP-755 C1a: Pre-write grounding tests (the one capture door)
 // ---------------------------------------------------------------------------
-describe('runCapture WP-319 grounding', () => {
-    const emptyGrounding = { index: 0, related: [], duplicates: [], governance: [], scanTruncated: false, v: 1 };
-    const groundingWithDuplicate = {
-        index: 0,
-        related: [],
-        duplicates: [{ entryId: 'TEN-5', name: 'Similar tension', collectionSlug: 'tensions', matchType: 'possible_duplicate', overlapRatio: 0.9 }],
-        governance: [],
-        v: 1,
+describe('runCapture WP-319/WP-755 grounding', () => {
+    const emptyVerdict = { kind: 'create_and_link', candidates: [], scanTruncated: false };
+    const askVerdict = {
+        kind: 'ask',
+        candidates: [{ entryId: 'TEN-5', name: 'Similar tension', collectionSlug: 'tensions', score: 0.6 }],
+        scanTruncated: false,
     };
     beforeEach(() => {
         kernelCallWithSessionMock.mockReset();
         mockSession = { sessionId: 'sess-test' };
         delete process.env.PB_GROUNDING_MODE;
+        // Pre-existing gap this fix round's new --strict tests surfaced: runner.ts's outputMode is
+        // module-level `let` state, set permanently by any earlier test in this file that passed
+        // `json: true` (setOutputMode is never test-scoped). Reset it here so a later test's
+        // `!json` branches (the --strict prompt gate) are never silently skipped by an earlier
+        // test's leaked JSON mode.
+        setOutputMode('auto');
     });
-    it('fires chain.suggestLinksForCapture in parallel and still calls createEntry', async () => {
+    it('fires chain.capture in parallel and still calls createEntry', async () => {
         // Prefix-based route skips classification; grounding fires for TEN prefix
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([emptyGrounding]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(emptyVerdict);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-1', entryId: 'TEN-720' });
             return Promise.resolve({});
@@ -1215,8 +1219,8 @@ describe('runCapture WP-319 grounding', () => {
         const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
         const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
         await runCapture({ text: 'TEN: grounding test tension' });
-        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.suggestLinksForCapture', expect.objectContaining({
-            entries: expect.arrayContaining([expect.objectContaining({ name: 'grounding test tension' })]),
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.capture', expect.objectContaining({
+            name: 'grounding test tension',
         }));
         expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.createEntry', expect.anything());
         writeSpy.mockRestore();
@@ -1225,7 +1229,7 @@ describe('runCapture WP-319 grounding', () => {
     it('does NOT block capture when grounding call fails', async () => {
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
+            if (fn === 'chain.capture')
                 return Promise.reject(new Error('network error'));
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-2', entryId: 'TEN-721' });
@@ -1239,12 +1243,14 @@ describe('runCapture WP-319 grounding', () => {
         writeSpy.mockRestore();
         stderrSpy.mockRestore();
     });
-    it('--suggest-only does NOT call createEntry and outputs grounding data', async () => {
-        // In test env (non-TTY), isJsonMode() returns true → JSON path is taken.
+    it('--suggest-only does NOT call createEntry and outputs the verdict', async () => {
+        // Explicit json:true — isJsonMode() is module-level state (runner.ts) with no TTY inference
+        // since WP-744/1e-B; this test wants the JSON preview shape, so it asks for it directly
+        // rather than relying on another test's leaked global mode.
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([groundingWithDuplicate]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(askVerdict);
             return Promise.resolve({});
         });
         let stdoutOutput = '';
@@ -1253,9 +1259,9 @@ describe('runCapture WP-319 grounding', () => {
             return true;
         });
         const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
-        await runCapture({ text: 'TEN: duplicate candidate', suggestOnly: true });
+        await runCapture({ text: 'TEN: possible match candidate', suggestOnly: true, json: true });
         expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.createEntry', expect.anything());
-        // JSON output contains suggestOnly flag and grounding report
+        // JSON output contains suggestOnly flag and the verdict report
         const jsonLines = stdoutOutput.split('\n').filter((l) => { try {
             JSON.parse(l);
             return true;
@@ -1273,8 +1279,8 @@ describe('runCapture WP-319 grounding', () => {
     it('--suggest-only with --json outputs machine-readable object without calling createEntry', async () => {
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([emptyGrounding]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(emptyVerdict);
             return Promise.resolve({});
         });
         let stdoutOutput = '';
@@ -1298,11 +1304,11 @@ describe('runCapture WP-319 grounding', () => {
         writeSpy.mockRestore();
         stderrSpy.mockRestore();
     });
-    it('banner is suppressed when grounding has no matches (zero-noise floor)', async () => {
+    it('banner is suppressed when the verdict has no candidates (zero-noise floor)', async () => {
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([emptyGrounding]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(emptyVerdict);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-3', entryId: 'TEN-722' });
             return Promise.resolve({});
@@ -1314,13 +1320,13 @@ describe('runCapture WP-319 grounding', () => {
             return true;
         });
         await runCapture({ text: 'TEN: no matches tension' });
-        // No "Grounding" banner in stderr when zero matches
-        const groundingInStderr = stderrLines.some((l) => l.includes('Grounding'));
+        // No "Related"/"Possible match" banner in stderr when the verdict has zero candidates.
+        const groundingInStderr = stderrLines.some((l) => l.includes('Related') || l.includes('Possible match'));
         expect(groundingInStderr).toBe(false);
         writeSpy.mockRestore();
         stderrSpy.mockRestore();
     });
-    it('PB_GROUNDING_MODE=off skips grounding call entirely', async () => {
+    it('PB_GROUNDING_MODE=off skips the door entirely', async () => {
         process.env.PB_GROUNDING_MODE = 'off';
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
@@ -1331,7 +1337,7 @@ describe('runCapture WP-319 grounding', () => {
         const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
         const stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
         await runCapture({ text: 'TEN: no grounding env off' });
-        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.suggestLinksForCapture', expect.anything());
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.capture', expect.anything());
         expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.createEntry', expect.anything());
         delete process.env.PB_GROUNDING_MODE;
         writeSpy.mockRestore();
@@ -1340,8 +1346,8 @@ describe('runCapture WP-319 grounding', () => {
     it('--json output stdout is clean (no grounding on stdout)', async () => {
         kernelCallWithSessionMock
             .mockImplementation((fn) => {
-            if (fn === 'chain.suggestLinksForCapture')
-                return Promise.resolve([groundingWithDuplicate]);
+            if (fn === 'chain.capture')
+                return Promise.resolve(askVerdict);
             if (fn === 'chain.createEntry')
                 return Promise.resolve({ docId: 'doc-5', entryId: 'TEN-724' });
             return Promise.resolve({});
@@ -1366,9 +1372,90 @@ describe('runCapture WP-319 grounding', () => {
         expect(parsed.id).toBe('TEN-724');
         // No grounding content in stdout
         expect(stdoutOutput).not.toContain('Grounding');
-        expect(stdoutOutput).not.toContain('duplicate');
+        expect(stdoutOutput).not.toContain('Possible match');
         writeSpy.mockRestore();
         stderrSpy.mockRestore();
     });
+    // WP-755 C1a fix round 1 (item 2): --strict must prompt on "version" too, not just "ask" — a
+    // 0.9-overlap duplicate returns "version" (silent-confidence band), and until C2 lands, that
+    // verdict's write is still a plain create; --strict must not silently create it unconfirmed.
+    const versionVerdict = {
+        kind: 'version',
+        entryId: 'TEN-9',
+        score: 0.95,
+        name: 'Existing near-duplicate tension',
+        collectionSlug: 'tensions',
+        docId: 'doc-9',
+        scanTruncated: false,
+    };
+    const originalStdinIsTTY = process.stdin.isTTY;
+    function mockInteractiveStdin(answer) {
+        Object.defineProperty(process.stdin, 'isTTY', { value: true, configurable: true });
+        vi.spyOn(process.stdin, 'once').mockImplementation((_event, cb) => {
+            cb(Buffer.from(answer));
+            return process.stdin;
+        });
+    }
+    afterEach(() => {
+        // vitest's own hook, not a manual end-of-test call — runs even when an assertion above threw,
+        // so one failing case in this block can never leak its stdin mock into the next test.
+        Object.defineProperty(process.stdin, 'isTTY', { value: originalStdinIsTTY, configurable: true });
+        vi.restoreAllMocks();
+    });
+    it('--strict prompts on a "version" verdict and proceeds on "y"', async () => {
+        kernelCallWithSessionMock
+            .mockImplementation((fn) => {
+            if (fn === 'chain.capture')
+                return Promise.resolve(versionVerdict);
+            if (fn === 'chain.createEntry')
+                return Promise.resolve({ docId: 'doc-10', entryId: 'TEN-726' });
+            return Promise.resolve({});
+        });
+        mockInteractiveStdin('y');
+        vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        const stderrLines = [];
+        vi.spyOn(process.stderr, 'write').mockImplementation((chunk) => {
+            stderrLines.push(String(chunk));
+            return true;
+        });
+        await runCapture({ text: 'TEN: near duplicate strict version', strict: true });
+        expect(stderrLines.some((l) => l.includes('Proceed with capture?'))).toBe(true);
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.createEntry', expect.anything());
+    });
+    it('--strict prompts on a "version" verdict and cancels on "n" (no createEntry call)', async () => {
+        kernelCallWithSessionMock
+            .mockImplementation((fn) => {
+            if (fn === 'chain.capture')
+                return Promise.resolve(versionVerdict);
+            if (fn === 'chain.createEntry')
+                return Promise.resolve({ docId: 'doc-11', entryId: 'TEN-727' });
+            return Promise.resolve({});
+        });
+        mockInteractiveStdin('n');
+        vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
+        await runCapture({ text: 'TEN: near duplicate strict version cancel', strict: true });
+        expect(kernelCallWithSessionMock).not.toHaveBeenCalledWith('chain.createEntry', expect.anything());
+    });
+    it('--strict prompts on an "ask" verdict too (unchanged from before this fix round)', async () => {
+        kernelCallWithSessionMock
+            .mockImplementation((fn) => {
+            if (fn === 'chain.capture')
+                return Promise.resolve(askVerdict);
+            if (fn === 'chain.createEntry')
+                return Promise.resolve({ docId: 'doc-12', entryId: 'TEN-728' });
+            return Promise.resolve({});
+        });
+        mockInteractiveStdin('y');
+        vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+        const stderrLines = [];
+        vi.spyOn(process.stderr, 'write').mockImplementation((chunk) => {
+            stderrLines.push(String(chunk));
+            return true;
+        });
+        await runCapture({ text: 'TEN: ask strict prompt', strict: true });
+        expect(stderrLines.some((l) => l.includes('Proceed with capture?'))).toBe(true);
+        expect(kernelCallWithSessionMock).toHaveBeenCalledWith('chain.createEntry', expect.anything());
+    });
 });
 //# sourceMappingURL=capture.test.js.map