@productbrain/mcp 0.0.1-beta.2206 → 0.0.1-beta.2226

package/dist/{chunk-WM3GBWEF.js → chunk-OD5YVZBQ.js}

@@ -1680,7 +1680,7 @@ No DB writes \u2014 call without \`preview:true\` to capture for real.${result.w
 
 ${msg}
 
-Use \`entries action=get\` to inspect the existing entry, or \`update-entry\` to modify it.`
+Use \`entries action=get\` to inspect the existing entry, or \`entries action=update\` to modify it.`
         }],
         structuredContent: failure(
           "DUPLICATE",
@@ -2472,7 +2472,9 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
             const { type: relationType } = inferRelationType(resolvedSlug, c.collSlug, profile);
             batchAutoLinks.push({ fromEntryId: finalEntryId, toEntryId: c.entryId, type: relationType, proposedBy: "auto-link", confidence: c.confidence });
           }
-          if (batchAutoLinks.length > 0) {
+          if (batchAutoLinks.length > 0 && preview) {
+            batchEntryWarnings.push(`${batchAutoLinks.length} auto-link(s) would be created (preview \u2014 no DB writes).`);
+          } else if (batchAutoLinks.length > 0) {
             const batchRes = await kernelMutation("chain.createEntryRelations", {
               relations: batchAutoLinks,
               sessionId: agentId ?? void 0
@@ -2487,18 +2489,30 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
         }
       }
       let relationsCreatedCount = 0;
+      let relationsPreviewedCount = 0;
+      let relationsProposedCount = 0;
       const relationsFailedList = [];
       if (entry.relations && entry.relations.length > 0) {
         for (const rel of entry.relations) {
           try {
-            await kernelMutation("chain.createEntryRelation", {
-              fromEntryId: finalEntryId,
-              toEntryId: rel.to,
-              type: rel.type,
-              proposedBy: "user",
-              sessionId: agentId ?? void 0
-            });
-            relationsCreatedCount++;
+            const relResult = await kernelMutation(
+              "chain.createEntryRelation",
+              {
+                fromEntryId: finalEntryId,
+                toEntryId: rel.to,
+                type: rel.type,
+                proposedBy: "user",
+                sessionId: agentId ?? void 0,
+                ...preview ? { preview: true } : {}
+              }
+            );
+            if (relResult?.preview) {
+              relationsPreviewedCount++;
+            } else if (relResult?.status === "agent_proposal_created") {
+              relationsProposedCount++;
+            } else {
+              relationsCreatedCount++;
+            }
           } catch (relErr) {
             const relMsg = relErr instanceof Error ? relErr.message : String(relErr);
             relationsFailedList.push(`${rel.to} (${rel.type}): ${relMsg}`);
@@ -2507,6 +2521,12 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
         if (relationsFailedList.length > 0) {
           batchEntryWarnings.push(`${relationsFailedList.length} inline relation(s) failed: ${relationsFailedList.join("; ")}`);
         }
+        if (relationsPreviewedCount > 0) {
+          batchEntryWarnings.push(`${relationsPreviewedCount} inline relation(s) would be created (preview \u2014 no DB writes).`);
+        }
+        if (relationsProposedCount > 0) {
+          batchEntryWarnings.push(`${relationsProposedCount} inline relation(s) converted to an agent proposal (misuse pattern) \u2014 review in Cortex UI.`);
+        }
       }
       if (autoCommitApplied && !batchWasAutoCommittedServerSide) {
         try {
@@ -2570,7 +2590,9 @@ async function handleBatchCapture(server, { entries, autoCommit, preview }) {
         // TEN-2365: capture-time authority-domain proposal echo.
         ...result.authorityDomain ? { authorityDomain: result.authorityDomain } : {},
         // TEN-957 (WP-484 S2): inline relations created for this entry.
-        ...relationsCreatedCount > 0 ? { relationsCreated: relationsCreatedCount } : {}
+        ...relationsCreatedCount > 0 ? { relationsCreated: relationsCreatedCount } : {},
+        // Finding #14: distinct from relationsCreated — never counted as a write.
+        ...relationsProposedCount > 0 ? { relationsProposed: relationsProposedCount } : {}
       });
     } catch (error) {
       const msg = error instanceof Error ? error.message : String(error);
@@ -2763,7 +2785,9 @@ _Use \`entries action=move\` to correct any misclassified entries._`);
             ...r.commitRefusal ? { coherencyRefusal: r.commitRefusal } : {},
             ...r.authorityDomain ? { domain: r.authorityDomain.slug } : {},
             // TEN-957 (WP-484 S2): inline relations created for this entry.
-            ...r.relationsCreated ? { relationsCreated: r.relationsCreated } : {}
+            ...r.relationsCreated ? { relationsCreated: r.relationsCreated } : {},
+            // Finding #14: proposal outcomes are distinct from writes — never folded in.
+            ...r.relationsProposed ? { relationsProposed: r.relationsProposed } : {}
           })),
           total: created.length,
           failed: failed.length,
@@ -3978,7 +4002,7 @@ var coherencyAcknowledgementFlatSchema = z5.object({
 });
 var entriesSchema = z5.object({
   action: z5.enum(ENTRIES_ACTIONS).describe(
-    "'list': browse entries with filters. 'get': fetch one entry by ID. 'batch': fetch multiple entries. 'search': full-text search. 'update': change fields on an existing entry (draft by default). 'commit': accept a draft entry onto the Chain. 'history': audit trail for an entry. 'move': reclassify an entry to a different collection. 'verify': mark an entry as verified (lightweight \u2014 no codebase scan; see the separate `verify` tool for the codebase-scanning check)."
+    "'list': browse entries with filters. 'get': fetch one entry by ID. 'batch': fetch multiple entries. 'search': full-text search. 'update': change fields on an existing entry (draft by default). 'commit': accept a draft entry onto the Chain. 'history': audit trail for an entry. 'move': reclassify an entry to a different collection. 'verify': mark an entry as verified (lightweight \u2014 no codebase scan; see `quality action=verify-chain` for the codebase-scanning check)."
   ),
   entryId: z5.string().max(200).optional().describe(
     "Entry ID, e.g. '<PREFIX>-<n>'. Required for: get, update, commit, history, move, verify."
@@ -4174,7 +4198,7 @@ function registerEntriesTools(server) {
     "entries",
     {
       title: "Entries",
-      description: 'Read and manage entries on the Chain. One tool for the entry lifecycle.\n\n- **list**: Browse entries with optional filters (collection, status, tag, label). Use collections action=list first to discover slugs.\n- **get**: Fetch a single entry by ID \u2014 full record with data, labels, relations, history.\n- **batch**: Fetch multiple entries (max 20) in one call. Same shape as get per entry.\n- **search**: Full-text search across entries. Scope by collection or filter by status.\n- **update**: Change fields on an existing entry (draft by default \u2014 never autoPublish without explicit user confirmation).\n- **commit**: Accept a draft entry onto the Chain (SSOT). Runs the contradiction preflight first.\n- **history**: Audit trail for an entry.\n- **move**: Reclassify an entry to a different collection.\n- **verify**: Mark an entry as verified (lightweight, no codebase scan \u2014 see the separate `verify` tool for that).\n\nUse `entries action=get entryId="..."` to fetch one. Use `entries action=search query="..."` to discover.',
+      description: 'Read and manage entries on the Chain. One tool for the entry lifecycle.\n\n- **list**: Browse entries with optional filters (collection, status, tag, label). Use collections action=list first to discover slugs.\n- **get**: Fetch a single entry by ID \u2014 full record with data, labels, relations, history.\n- **batch**: Fetch multiple entries (max 20) in one call. Same shape as get per entry.\n- **search**: Full-text search across entries. Scope by collection or filter by status.\n- **update**: Change fields on an existing entry (draft by default \u2014 never autoPublish without explicit user confirmation).\n- **commit**: Accept a draft entry onto the Chain (SSOT). Runs the contradiction preflight first.\n- **history**: Audit trail for an entry.\n- **move**: Reclassify an entry to a different collection.\n- **verify**: Mark an entry as verified (lightweight, no codebase scan \u2014 see `quality action=verify-chain` for that).\n\nUse `entries action=get entryId="..."` to fetch one. Use `entries action=search query="..."` to discover.',
       inputSchema: entriesSchema,
       // Mixed read/write noun (§3 R3): can't be marked read-only once update/commit/move
       // live here too — accepted for the 10-tool default (§9 R3).
@@ -4457,7 +4481,15 @@ ${formatted}` }],
 }
 async function handleMove(entryId, toCollection) {
   try {
-    const result = await kernelMutation("chain.moveToCollection", { entryId, toCollectionSlug: toCollection });
+    const result = await kernelMutation("chain.moveToCollection", {
+      entryId,
+      toCollectionSlug: toCollection,
+      // Finding #11: without an identity, moveToCollection's requirePermission call
+      // resolves to { agentSessionId: undefined } and permissions.ts's write pre-check
+      // denies EVERY call — this action always failed live. Sibling write actions
+      // (knowledge.ts:137) all thread this same changedBy shape.
+      changedBy: getAgentSessionId() ? `agent:${getAgentSessionId()}` : void 0
+    });
     const lines = [
       `## Move Result`,
       "",
@@ -12292,7 +12324,11 @@ async function handleStart2() {
       workspaceName: result.workspaceName,
       toolsScope: result.toolsScope,
       superseded: !!result.superseded
-    }, [{ tool: "orient", description: "Orient session", parameters: {} }])
+      // Finding #16: bare `parameters: {}` lets orient's schema default kick in
+      // (action defaults to "task", which requires a `task` string) — the suggested
+      // next-action call failed validation instead of running. `action: "start"` is
+      // the guided-setup entrypoint this session just opened write access for.
+    }, [{ tool: "orient", description: "Orient session", parameters: { action: "start" } }])
   };
 }
 async function handleClose() {
@@ -12319,7 +12355,7 @@ async function handleClose() {
           "",
           text,
           "",
-          `> **${data.drafts.length} uncommitted draft(s) remain.** Run \`session-wrapup\` with action \`commit-all\` before closing next time.`,
+          `> **${data.drafts.length} uncommitted draft(s) remain.** Run \`session action=wrapup-commit\` before closing next time.`,
           "",
           "---",
           ""
@@ -12470,7 +12506,13 @@ var chainBranchSchema = z22.object({
 });
 var chainReviewSchema = z22.object({
   action: z22.enum(["gate", "comment", "resolve-comment", "list-comments"]).describe("Action: run coherence gate, add a comment, resolve a comment, or list comments"),
-  chainEntryId: z22.string().max(200).describe("The chain's entry ID"),
+  // Finding #12: optional at the base (mirrors chainSchema's chainEntryId pattern at
+  // line ~690) — resolve-comment resolves purely by commentId (handleChainReview never
+  // reads chainEntryId in that branch) and the compound-tool's advertised schema
+  // (chainReviewCompoundSchema below) already documents it as optional for that action.
+  // Per-action variants that DO need it (gate/comment/list-comments) re-require it below,
+  // same pattern as chainGetVariant/chainEditVariant re-requiring over chainSchema's base.
+  chainEntryId: z22.string().max(200).optional().describe("The chain's entry ID. Required for every action except 'resolve-comment'."),
   commitMessage: z22.string().max(2e3).optional().describe("Commit message to lint (for gate action)"),
   versionNumber: z22.number().optional().describe("Version to comment on or list comments for"),
   linkId: z22.string().max(200).optional().describe("Link this comment targets (optional for comment)"),
@@ -13040,10 +13082,10 @@ var chainActionUnion = z22.discriminatedUnion("action", [
   chainVersionHistoryVariant
 ]);
 var reviewBase = chainReviewSchema.omit({ action: true });
-var chainReviewGateVariant = reviewBase.extend({ action: z22.literal("gate") });
-var chainReviewCommentVariant = reviewBase.extend({ action: z22.literal("comment"), versionNumber: z22.number(), body: z22.string().max(2e4) });
+var chainReviewGateVariant = reviewBase.extend({ action: z22.literal("gate"), chainEntryId: z22.string().max(200) });
+var chainReviewCommentVariant = reviewBase.extend({ action: z22.literal("comment"), chainEntryId: z22.string().max(200), versionNumber: z22.number(), body: z22.string().max(2e4) });
 var chainReviewResolveCommentVariant = reviewBase.extend({ action: z22.literal("resolve-comment"), commentId: z22.string().max(200) });
-var chainReviewListCommentsVariant = reviewBase.extend({ action: z22.literal("list-comments") });
+var chainReviewListCommentsVariant = reviewBase.extend({ action: z22.literal("list-comments"), chainEntryId: z22.string().max(200) });
 var branchBase = chainBranchSchema.omit({ action: true });
 var chainBranchCreateVariant = branchBase.extend({ action: z22.literal("branch.create") });
 var chainBranchListVariant = branchBase.extend({ action: z22.literal("branch.list") });
@@ -15277,4 +15319,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-WM3GBWEF.js.map
+//# sourceMappingURL=chunk-OD5YVZBQ.js.map