@productbrain/mcp 0.0.1-beta.6660 → 0.0.1-beta.6668

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

@@ -2753,39 +2753,47 @@ async function runConflictPreflight(name, description, collectionHint) {
 }
 
 // src/tools/knowledge/updateEntryAppliedFields.ts
-function computeUpdateEntryAppliedFields(args, normalization) {
+function computeUpdateEntryAppliedFields(args, normalization, provenanceApplied = {}, revertedFields = []) {
   const rejectedDataFields = normalization?.rejected ?? [];
-  const acceptedDataFields = normalization?.accepted ?? [];
+  const revertedSet = new Set(revertedFields);
+  const acceptedDataFields = (normalization?.accepted ?? []).filter((field) => !revertedSet.has(field));
   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");
+  if (args.sourceRef && provenanceApplied.sourceRef) appliedFields.push("sourceRef");
+  if (args.sourceExcerpt && provenanceApplied.sourceExcerpt) appliedFields.push("sourceExcerpt");
   appliedFields.push(...acceptedDataFields);
-  return { appliedFields, rejectedDataFields };
+  const effectiveReverted = [...revertedFields];
+  if (args.sourceRef && provenanceApplied.sourceRef === false) effectiveReverted.push("sourceRef");
+  if (args.sourceExcerpt && provenanceApplied.sourceExcerpt === false) effectiveReverted.push("sourceExcerpt");
+  return { appliedFields, rejectedDataFields, revertedFields: effectiveReverted };
 }
-function buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields) {
+function buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields, revertedFields = []) {
   if (!(fieldsProvided.length > 0 && appliedFields.length === 0)) return null;
   const rejectedNote = rejectedDataFields.length > 0 ? ` Rejected: ${rejectedDataFields.join(", ")}.` : "";
+  const revertedNote = revertedFields.length > 0 ? ` Reverted (platform-owned or write-once, cannot be set by this call): ${revertedFields.join(", ")}.` : "";
+  const message = `No fields were applied to ${entryId}.${rejectedNote}${revertedNote}`;
   return failureResult(
-    `No fields were applied to ${entryId}.${rejectedNote}`,
+    message,
     "FIELDS_REJECTED",
-    `No fields were applied to ${entryId}.${rejectedNote}`,
+    message,
     "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) {
+function pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields, revertedFields = []) {
   if (fieldsProvided.length > 0) {
     responseLines.push(`**Fields applied:** ${appliedFields.join(", ")}`);
     if (rejectedDataFields.length > 0) {
       responseLines.push("");
       responseLines.push(`\u26A0\uFE0F **Fields rejected:** ${rejectedDataFields.join(", ")}`);
     }
+    if (revertedFields.length > 0) {
+      responseLines.push("");
+      responseLines.push(`\u26A0\uFE0F **Fields reverted (platform-owned or write-once):** ${revertedFields.join(", ")}`);
+    }
   } else {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F No fields to update were provided \u2014 only `entryId` was set.");
@@ -2868,7 +2876,7 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
   if (rawWorkflowStatus !== void 0) fieldsProvided.push("workflowStatus");
   if (data !== void 0) fieldsProvided.push("data");
   if (order !== void 0) fieldsProvided.push("order");
-  if (canonicalKey !== void 0) fieldsProvided.push("canonicalKey");
+  for (const [k, v] of [["canonicalKey", canonicalKey], ["sourceRef", sourceRef], ["sourceExcerpt", sourceExcerpt]]) if (v !== void 0) fieldsProvided.push(k);
   const status2 = rawStatus;
   const workflowStatus = rawWorkflowStatus;
   const updateResult = await kernelMutation("chain.updateEntry", {
@@ -2882,8 +2890,8 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     autoPublish,
     changeNote,
     changedBy: getAgentSessionId() ? `agent:${getAgentSessionId()}` : void 0,
-    ...sourceRef ? { sourceRef } : {},
-    ...sourceExcerpt ? { sourceExcerpt } : {},
+    ...sourceRef !== void 0 ? { sourceRef } : {},
+    ...sourceExcerpt !== void 0 ? { sourceExcerpt } : {},
     ...steeringOverrideReason ? { steeringOverrideReason } : {},
     ...coherencyAcknowledgement ? { coherencyAcknowledgement } : {}
   });
@@ -2892,9 +2900,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);
+  const fieldArgs = { name, status: rawStatus, workflowStatus: rawWorkflowStatus, order, canonicalKey, sourceRef, sourceExcerpt };
+  const revertedByServer = updateResult.revertedFields ?? [];
+  const { appliedFields, rejectedDataFields, revertedFields } = computeUpdateEntryAppliedFields(fieldArgs, updateResult.normalization, updateResult.provenanceApplied, revertedByServer);
+  const totalFailure = buildFieldsRejectedFailure(entryId, fieldsProvided, appliedFields, rejectedDataFields, revertedFields);
   if (totalFailure) return totalFailure;
   await recordSessionActivity({ entryModified: id });
   const wsCtx = await getWorkspaceContext();
@@ -2909,12 +2918,12 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
     `Internal ID: ${id}`,
     `**Workspace:** ${wsCtx.workspaceSlug} (${wsCtx.workspaceId})`
   ];
-  pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields);
-  const shownWarnings = filterRedundantRejectionWarnings(updateResult.normalizationWarnings ?? [], rejectedDataFields);
-  if (shownWarnings.length > 0) {
+  pushAppliedFieldsLines(responseLines, fieldsProvided, appliedFields, rejectedDataFields, revertedFields);
+  const warnings = updateResult.normalizationWarnings ?? [];
+  if (warnings.length > 0) {
     responseLines.push("");
     responseLines.push("\u26A0\uFE0F **Normalization warnings:**");
-    for (const warning of shownWarnings) responseLines.push(`  - ${warning}`);
+    for (const warning of warnings) responseLines.push(`  - ${warning}`);
   }
   if ((updateResult.validationWarnings ?? []).length > 0) {
     responseLines.push("");
@@ -2947,7 +2956,9 @@ async function handleUpdateEntry({ entryId, name, status: rawStatus, workflowSta
       },
       ...(updateResult.validationWarnings ?? []).length > 0 && {
         validationWarnings: updateResult.validationWarnings
-      }
+      },
+      ...revertedFields.length > 0 && { revertedFields }
+      // #1026 R2: symmetric with the text above
     }, next)
   };
 }
@@ -14955,4 +14966,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-3HALD6EU.js.map
+//# sourceMappingURL=chunk-MEJLUGP4.js.map