@productbrain/mcp 0.0.1-beta.3409 → 0.0.1-beta.3417

package/dist/{chunk-3JRSYADK.js → chunk-GFBV6BUO.js}

@@ -8513,6 +8513,71 @@ function formatShapeAdvisorySummaryLine(sas) {
   return `Shape advisories (${sas.windowDays}d): ${sas.pendingCount} pending / ${sas.compoundTotal} compound${pendingRatePart} \u2014 precisionAgent ${agentPart}, precisionHuman ${humanPart}.${truncatedNote}${candidatePart}`;
 }
 
+// src/tools/orient-rituals.ts
+function buildRitualsSurfaceLines(view, opts) {
+  const lines = [""];
+  if (opts?.fetchFailed) {
+    lines.push("Direction: unavailable \u2014 orient fetch failed, retry `orient`");
+    lines.push("Spine check: unavailable \u2014 orient fetch failed, retry `orient`");
+    return lines;
+  }
+  lines.push(view?.directionLine ?? "Direction: not computed yet \u2014 `pb direction refresh`");
+  if (view?.verdictAskLine) lines.push(view.verdictAskLine);
+  lines.push(view?.spineLine ?? "Spine check: not computed yet \u2014 `pb quality spine-check`");
+  return lines;
+}
+async function buildAlreadyOrientedResult(kernelQuery2, agentSessionId, taskGroundingWarningLines2) {
+  let ritualsFetchFailed = false;
+  let rituals = null;
+  try {
+    rituals = await kernelQuery2("chain.getRitualsSurface", {});
+  } catch {
+    ritualsFetchFailed = true;
+  }
+  const ritualsLines = buildRitualsSurfaceLines(rituals, { fetchFailed: ritualsFetchFailed });
+  return {
+    content: [{
+      type: "text",
+      text: "Already oriented. Session active, writes unlocked.\n" + ritualsLines.join("\n") + "\n\n" + taskGroundingWarningLines2.join("\n") + "Use `orient mode='full'` for workspace context or `orient task='...'` before substantive work."
+    }],
+    structuredContent: success(
+      "Already oriented. Session active, writes unlocked.",
+      {
+        alreadyOriented: true,
+        sessionId: agentSessionId,
+        taskGroundingStatus: "workspace_only",
+        taskGroundingRequired: true,
+        nextStep: 'Run `orient task="describe the work"` before substantive work.',
+        // T1's undefined-vs-null distinction: omit the key on a fetch failure, keep an explicit
+        // null for a genuine no-row arm.
+        ...rituals?.directionLine !== void 0 ? { directionLine: rituals.directionLine } : {},
+        ...rituals?.verdictAskLine !== void 0 ? { verdictAskLine: rituals.verdictAskLine } : {},
+        ...rituals?.spineLine !== void 0 ? { spineLine: rituals.spineLine } : {}
+      }
+    )
+  };
+}
+
+// src/tools/orient-metrics.ts
+function reportOrientWrapperBytes(toolResult, truncated, tier, taskProvided) {
+  try {
+    const fullToolOutput = JSON.stringify({ content: toolResult.content ?? [] });
+    const bytes = Buffer.byteLength(fullToolOutput, "utf8");
+    const sessionId = getAgentSessionId();
+    const conversationId = getConversationId();
+    void kernelMutation("agent.reportOrientMetric", {
+      bytes,
+      truncated,
+      tier,
+      taskProvided,
+      ...sessionId ? { sessionId } : {},
+      ...conversationId ? { conversationId } : {}
+    }).catch(() => {
+    });
+  } catch {
+  }
+}
+
 // src/lib/gapToPrompt.ts
 function cleanLabel(label) {
   return label.replace(/ has entries$/i, "").replace(/ coverage$/i, "").replace(/^Strategy — /i, "");
@@ -9015,24 +9080,6 @@ function domainDetailFromRetrieval(retrieval) {
   const evidenceGap = typeof retrieval.evidenceGap === "string" ? retrieval.evidenceGap : null;
   return reason || nextAction || evidenceGap ? { reason, nextAction, evidenceGap } : null;
 }
-function reportOrientWrapperBytes(toolResult, truncated, tier, taskProvided) {
-  try {
-    const fullToolOutput = JSON.stringify({ content: toolResult.content ?? [] });
-    const bytes = Buffer.byteLength(fullToolOutput, "utf8");
-    const sessionId = getAgentSessionId();
-    const conversationId = getConversationId();
-    void kernelMutation("agent.reportOrientMetric", {
-      bytes,
-      truncated,
-      tier,
-      taskProvided,
-      ...sessionId ? { sessionId } : {},
-      ...conversationId ? { conversationId } : {}
-    }).catch(() => {
-    });
-  } catch {
-  }
-}
 async function markOrientedWithSnapshotFallback(agentSessionId, coherenceSnapshot) {
   try {
     await kernelCall("agent.markOriented", {
@@ -9194,22 +9241,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
   }
   const agentSessionId = getAgentSessionId();
   if (isSessionOriented() && mode === "brief" && !task) {
-    return {
-      content: [{
-        type: "text",
-        text: "Already oriented. Session active, writes unlocked.\n\n" + taskGroundingWarningLines().join("\n") + "Use `orient mode='full'` for workspace context or `orient task='...'` before substantive work."
-      }],
-      structuredContent: success(
-        "Already oriented. Session active, writes unlocked.",
-        {
-          alreadyOriented: true,
-          sessionId: agentSessionId,
-          taskGroundingStatus: "workspace_only",
-          taskGroundingRequired: true,
-          nextStep: 'Run `orient task="describe the work"` before substantive work.'
-        }
-      )
-    };
+    return buildAlreadyOrientedResult(kernelQuery, agentSessionId, taskGroundingWarningLines());
   }
   let wsCtx = null;
   try {
@@ -9220,6 +9252,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
   let priorSessions = [];
   let recoveryBlock = null;
   let orientView = null;
+  let orientFetchFailed = false;
   try {
     const orientArgs = {};
     if (task) orientArgs.task = task;
@@ -9237,6 +9270,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
     if (orientConversationId) orientArgs.conversationId = orientConversationId;
     orientView = await kernelQuery("chain.getOrientView", orientArgs);
   } catch {
+    orientFetchFailed = true;
   }
   const resolvedTier = orientView?.resolvedTier ?? callTier ?? "standard";
   const taskMeaningful = orientView?.taskMeaningful ?? false;
@@ -9360,6 +9394,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
     if (orientView?.taskAlignment) {
       lines.push(`Task alignment: ${orientView.taskAlignment.verdict} \u2014 ${orientView.taskAlignment.wording.replace(/\*\*/g, "")}`);
     }
+    lines.push(...buildRitualsSurfaceLines(orientView, { fetchFailed: orientFetchFailed }));
     if (orientView?.startup?.domainRetrieval) {
       const retrieval = orientView.startup.domainRetrieval;
       const scopedTo = retrieval.resolvedDomainSlug ? ` (${retrieval.resolvedDomainSlug})` : "";
@@ -9502,6 +9537,12 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
           ...orientView?.taskContext?.isGroundingStub ? { groundingStub: true } : {},
           // WP-465 slice ③ (§4.1): served verbatim — identical on CLI and MCP.
           ...orientView?.taskAlignment ? { taskAlignment: orientView.taskAlignment } : {},
+          // WP-582 B2 round 1 (Codex P2 review, PR #532, T1): omit when `undefined` (old
+          // deploy, field never served); keep an explicit `null` for the documented no-row
+          // arm — the two states must stay distinguishable to a structuredContent consumer.
+          ...orientView?.directionLine !== void 0 ? { directionLine: orientView.directionLine } : {},
+          ...orientView?.verdictAskLine !== void 0 ? { verdictAskLine: orientView.verdictAskLine } : {},
+          ...orientView?.spineLine !== void 0 ? { spineLine: orientView.spineLine } : {},
           // RCRT miss-alarm: machine-readable verdict for structuredContent
           // consumers (parity with the rendered prose alarm).
           ...orientView?.retrievalMiss ? { retrievalMiss: orientView.retrievalMiss } : {},
@@ -9684,6 +9725,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
       }
       if (orientView) {
         lines.push(...buildTaskAlignmentLines(orientView.taskAlignment));
+        lines.push(...buildRitualsSurfaceLines(orientView, { fetchFailed: orientFetchFailed }));
         lines.push(...formatInitiativeStatusLines(orientView.initiativeStatus));
         lines.push(...formatWorkstreamHealthLines(orientView.workstreamHealth));
         lines.push(...formatWhatNeedsAttentionLines(orientView.whatNeedsAttention));
@@ -9811,6 +9853,7 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
     } else {
       lines.push(`**Brain stage: ${orientStage}.**`);
       lines.push("");
+      lines.push(...buildRitualsSurfaceLines(orientView, { fetchFailed: orientFetchFailed }));
       if (orientView?.activeWorkPackages?.length) {
         lines.push("## Active work packages \u2014 current scope");
         lines.push("_Work outside these work packages requires explicit user confirmation._");
@@ -9934,6 +9977,12 @@ async function _handleOrient({ mode = "full", tier, task, scope, startupSignals,
         ...orientView?.taskContext?.isGroundingStub ? { groundingStub: true } : {},
         // WP-465 slice ③ (§4.1): served verbatim — identical on CLI and MCP.
         ...orientView?.taskAlignment ? { taskAlignment: orientView.taskAlignment } : {},
+        // WP-582 B2 round 1 (Codex P2 review, PR #532, T1): served verbatim ONLY when the
+        // server actually sent the field — see the matching brief-mode comment above for the
+        // undefined-vs-null distinction this preserves.
+        ...orientView?.directionLine !== void 0 ? { directionLine: orientView.directionLine } : {},
+        ...orientView?.verdictAskLine !== void 0 ? { verdictAskLine: orientView.verdictAskLine } : {},
+        ...orientView?.spineLine !== void 0 ? { spineLine: orientView.spineLine } : {},
         // RCRT miss-alarm: machine-readable verdict for structuredContent
         // consumers (parity with the rendered prose alarm).
         ...orientView?.retrievalMiss ? { retrievalMiss: orientView.retrievalMiss } : {},
@@ -13097,6 +13146,12 @@ async function handleClose() {
       `| Contradiction warnings | ${warnings} |`
     );
   }
+  let rituals = null;
+  try {
+    rituals = await kernelCall("chain.getRitualsSurface", {});
+  } catch {
+  }
+  if (rituals) lines.push(...buildRitualsSurfaceLines(rituals));
   lines.push("", "Write tools are now blocked. Session data saved for future orientation.");
   const fullResponse = wrapupNudge + lines.join("\n");
   const summary = `Session ${sessionId} closed. ${created} created, ${modified} modified.`;
@@ -13109,7 +13164,13 @@ async function handleClose() {
       relationsCreated: relations,
       gateFailures: gates,
       contradictionWarnings: warnings,
-      wrapupNudge: wrapupNudge.length > 0
+      wrapupNudge: wrapupNudge.length > 0,
+      // T1's undefined-vs-null distinction applies here too — omit the key entirely when the
+      // fetch failed, rather than forcing a null a machine consumer can't distinguish from a
+      // genuine no-row arm.
+      ...rituals?.directionLine !== void 0 ? { directionLine: rituals.directionLine } : {},
+      ...rituals?.verdictAskLine !== void 0 ? { verdictAskLine: rituals.verdictAskLine } : {},
+      ...rituals?.spineLine !== void 0 ? { spineLine: rituals.spineLine } : {}
     })
   };
 }
@@ -16400,4 +16461,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-3JRSYADK.js.map
+//# sourceMappingURL=chunk-GFBV6BUO.js.map