@productbrain/mcp 0.0.1-beta.6566 → 0.0.1-beta.6576

package/dist/{chunk-JALJ3MPU.js → chunk-3HALD6EU.js}

@@ -2752,6 +2752,46 @@ async function runConflictPreflight(name, description, collectionHint) {
   }
 }
 
+// src/tools/knowledge/updateEntryAppliedFields.ts
+function computeUpdateEntryAppliedFields(args, normalization) {
+  const rejectedDataFields = normalization?.rejected ?? [];
+  const acceptedDataFields = normalization?.accepted ?? [];
+  const appliedFields = [];
+  if (args.name !== void 0) appliedFields.push("name");
+  if (args.status !== void 0) appliedFields.push("status");
+  if (args.workflowStatus !== void 0) appliedFields.push("workflowStatus");
+  if (args.order !== void 0) appliedFields.push("order");
+  if (args.canonicalKey !== void 0) appliedFields.push("canonicalKey");
+  appliedFields.push(...acceptedDataFields);
+  return { appliedFields, rejectedDataFields };
+}
+function buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields) {
+  if (!(fieldsProvided.length > 0 && appliedFields.length === 0)) return null;
+  const rejectedNote = rejectedDataFields.length > 0 ? ` Rejected: ${rejectedDataFields.join(", ")}.` : "";
+  return failureResult(
+    `No fields were applied to ${entryId}.${rejectedNote}`,
+    "FIELDS_REJECTED",
+    `No fields were applied to ${entryId}.${rejectedNote}`,
+    "Run `collections action=describe` for this entry's collection to see valid fields."
+  );
+}
+function filterRedundantRejectionWarnings(warnings, rejectedDataFields) {
+  if (rejectedDataFields.length === 0) return warnings;
+  return warnings.filter((warning) => !rejectedDataFields.some((field) => warning.includes(field)));
+}
+function pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields) {
+  if (fieldsProvided.length > 0) {
+    responseLines.push(`**Fields applied:** ${appliedFields.join(", ")}`);
+    if (rejectedDataFields.length > 0) {
+      responseLines.push("");
+      responseLines.push(`\u26A0\uFE0F **Fields rejected:** ${rejectedDataFields.join(", ")}`);
+    }
+  } else {
+    responseLines.push("");
+    responseLines.push("\u26A0\uFE0F No fields to update were provided \u2014 only `entryId` was set.");
+  }
+}
+
 // src/tools/knowledge/getHistory.ts
 import { z as z5 } from "zod/v3";
 var getHistorySchema = z5.object({
@@ -2852,6 +2892,10 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     return coherencyRefusedResult(`${toolName} action=update`, entryId, maybeRefused.refusal);
   }
   const id = updateResult.id;
+  const fieldArgs = { name, status: rawStatus, workflowStatus: rawWorkflowStatus, order, canonicalKey };
+  const { appliedFields, rejectedDataFields } = computeUpdateEntryAppliedFields(fieldArgs, updateResult.normalization);
+  const totalFailure = buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields);
+  if (totalFailure) return totalFailure;
   await recordSessionActivity({ entryModified: id });
   const wsCtx = await getWorkspaceContext();
   const updated = await kernelQuery("chain.getEntry", { entryId });
@@ -2865,18 +2909,12 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     `Internal ID: ${id}`,
     `**Workspace:** ${wsCtx.workspaceSlug} (${wsCtx.workspaceId})`
   ];
-  if (fieldsProvided.length > 0) {
-    responseLines.push(`**Fields provided:** ${fieldsProvided.join(", ")}`);
-  } else {
-    responseLines.push("");
-    responseLines.push("\u26A0\uFE0F No fields to update were provided \u2014 only `entryId` was set.");
-  }
-  if ((updateResult.normalizationWarnings ?? []).length > 0) {
+  pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields);
+  const shownWarnings = filterRedundantRejectionWarnings(updateResult.normalizationWarnings ?? [], rejectedDataFields);
+  if (shownWarnings.length > 0) {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F **Normalization warnings:**");
-    for (const warning of updateResult.normalizationWarnings ?? []) {
-      responseLines.push(`  - ${warning}`);
-    }
+    for (const warning of shownWarnings) responseLines.push(`  - ${warning}`);
   }
   if ((updateResult.validationWarnings ?? []).length > 0) {
     responseLines.push("");
@@ -2885,7 +2923,7 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
       responseLines.push(`  - ${warning}`);
     }
   }
-  const summary = `Updated ${entryId} \u2014 ${versionMode}. Fields: ${fieldsProvided.join(", ") || "none"}.`;
+  const summary = `Updated ${entryId} \u2014 ${versionMode}. Fields: ${appliedFields.join(", ") || "none"}.`;
   const next = [
     { tool: toolName, description: "View updated entry", parameters: { action: "get", entryId } },
     { tool: toolName, description: "Accept onto Chain", parameters: { action: "commit", entryId } }
@@ -2895,7 +2933,8 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     structuredContent: success(summary, {
       entryId: id,
       versionMode,
-      fieldsProvided,
+      fieldsApplied: appliedFields,
+      // PR B: renamed from fieldsProvided (caller intent) — this is the server's verdict, matching the text above
       // BET-192: normalization breakdown in structuredContent — symmetric with capture tool
       ...updateResult.normalization && (Object.keys(updateResult.normalization.remapped).length > 0 || updateResult.normalization.rejected.length > 0) && {
         normalization: {
@@ -14916,4 +14955,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-JALJ3MPU.js.map
+//# sourceMappingURL=chunk-3HALD6EU.js.map