@productbrain/mcp 0.0.1-beta.5646 → 0.0.1-beta.5663

package/dist/{chunk-C5BIZH5N.js → chunk-A4WEYWDI.js}

@@ -202,6 +202,9 @@ function classifyError(err) {
   }
   return { code: "INTERNAL_ERROR", message: message || "An unexpected error occurred." };
 }
+function retryIsFutile(code) {
+  return code === "VALIDATION_ERROR";
+}
 
 // src/envelope.ts
 function success(summary, data, next) {
@@ -15561,7 +15564,7 @@ ${lines.join("\n")}${truncationNote}`;
 function questionErrorResult(err, action, params = {}) {
   const classified = classifyError(err);
   const definedParams = Object.fromEntries(Object.entries(params).filter(([, v]) => v !== void 0));
-  const fallbackActions = classified.code === "VALIDATION_ERROR" ? void 0 : [{ tool: "question", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
+  const fallbackActions = retryIsFutile(classified.code) ? void 0 : [{ tool: "question", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
   return failureResult(
     classified.message,
     classified.code,
@@ -15625,12 +15628,13 @@ var ROLE_ACTION_SPECS = {
 };
 function formatRow(row) {
   const id = row.entryId ?? row.id;
-  return `    ${id}  ${sanitizeSingleLineField(row.name)}  [${row.canonicalKey ?? "?"}/${row.status}]`;
+  const statusDisplay = row.workflowStatus ? `${row.status} / ${sanitizeSingleLineField(row.workflowStatus)}` : row.status;
+  return `    ${id}  ${sanitizeSingleLineField(row.name)}  [${row.canonicalKey ?? "?"}/${statusDisplay}]`;
 }
 function formatSection(label, rows, truncated, cursorHint) {
-  if (rows.length === 0) return [`  ${label}: not recorded yet`];
-  const lines = [`  ${label}:`, ...rows.map(formatRow)];
-  if (truncated) lines.push(`    (more exist \u2014 ${cursorHint ?? "raise limit (capped at 200, no cursor for this list)"})`);
+  if (rows.length === 0 && !truncated) return [`  ${label}: not recorded yet`];
+  const lines = rows.length === 0 ? [`  ${label}: not recorded yet, but the scan is incomplete`] : [`  ${label}:`, ...rows.map(formatRow)];
+  if (truncated) lines.push(`    (scan incomplete, more may exist \u2014 ${cursorHint ?? "raise limit (capped at 200, no cursor for this list)"})`);
   return lines;
 }
 function formatRoleAccountable(result2) {
@@ -15656,12 +15660,13 @@ async function handleRoleAccountable(roleId, canonicalKey, limit, cursor) {
 function roleErrorResult(err, action, params = {}) {
   const classified = classifyError(err);
   const definedParams = Object.fromEntries(Object.entries(params).filter(([, v]) => v !== void 0));
+  const fallbackActions = retryIsFutile(classified.code) ? void 0 : [{ tool: "role", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
   return failureResult(
     classified.message,
     classified.code,
     classified.message,
     classified.recovery ?? "Use action='accountable' with roleId set to a real role's chain id or entry id.",
-    classified.availableActions ?? [{ tool: "role", description: `Retry ${action}`, parameters: { action, ...definedParams } }],
+    classified.availableActions ?? fallbackActions,
     classified.diagnostics
   );
 }
@@ -15698,7 +15703,13 @@ var outcomeSchema = z35.object({
     "'record': record a reading (evidence, not a verdict). 'list': list readings, newest first. 'show': the derived view \u2014 current, progress, pace, staleness, recorded completion."
   ),
   entryId: z35.string().max(200).optional().describe("The key result (or other outcome-bearing) entry, by its chain id. Required for all three actions."),
-  value: z35.number().optional().describe("'record': the measured value. Required for 'record'."),
+  // PR #858 review round 3 (Codex P2, PRRT_kwDORS4ils6gXHj9): a JSON number that overflows to
+  // `Infinity` (e.g. `1e400`) parses fine in JS and passes a plain `z.number()`, but
+  // `callGateway()` serializes it with `JSON.stringify`, which turns `Infinity` into `null` —
+  // the backend's `v.number()` validator then rejects a call this tool's own schema accepted.
+  // `.finite()` rejects it here, matching the explicit finite-value checks the CLI/other outcome
+  // entry surfaces already use.
+  value: z35.number().finite().optional().describe("'record': the measured value. Required for 'record'."),
   valueText: z35.string().max(500).optional().describe("'record': verbatim as-reported form when the source was not clean, e.g. '~62%'. Display only."),
   asOf: z35.string().max(64).optional().describe("'record': when the world was in this state, as a parseable date (default: now). Back-dating a backfill is legitimate \u2014 it shows up as asOf earlier than recordedAt."),
   kind: z35.enum(READING_KINDS).optional().describe("'record': measurement | estimate | correction (default: measurement)."),
@@ -15711,7 +15722,7 @@ var outcomeSchema = z35.object({
 var recordVariant = z35.object({
   action: z35.literal("record"),
   entryId: z35.string().max(200),
-  value: z35.number(),
+  value: z35.number().finite(),
   valueText: z35.string().max(500).optional(),
   asOf: z35.string().max(64).optional(),
   kind: z35.enum(READING_KINDS).optional(),
@@ -15735,6 +15746,10 @@ var OUTCOME_ACTION_SPECS = {
   list: { params: ["entryId", "limit"], description: "entryId is required; limit is optional." },
   show: { params: ["entryId"], description: "entryId is required." }
 };
+function safeDateSlice(ms) {
+  const d = new Date(ms);
+  return Number.isNaN(d.getTime()) ? "invalid-date" : d.toISOString().slice(0, 10);
+}
 function formatRecordReceipt(numericValue, entryId, valueText, receipt) {
   const tier = receipt.independenceTier ?? "unknown";
   const lines = [
@@ -15751,7 +15766,7 @@ function formatReadingsList(entryId, result2) {
   const rows = result2?.readings ?? [];
   if (rows.length === 0) return `No readings recorded on ${safeEntryId} yet.`;
   const lines = rows.map((row) => {
-    const when = new Date(row.asOf).toISOString().slice(0, 10);
+    const when = safeDateSlice(row.asOf);
     const superseded = row.supersededByReadingId ? `  <- superseded by ${row.supersededByReadingId}` : "";
     const text = row.valueText ? ` ("${sanitizeSingleLineField(row.valueText)}")` : "";
     const note = row.note ? `  - ${sanitizeSingleLineField(row.note)}` : "";
@@ -15774,7 +15789,7 @@ function formatOutcomeShow(result2) {
     `  progress    ${pct(o.progressPct)}   direction ${o.direction}`,
     `  pace        ${pct(o.pacePct)}   ${o.pacePct === void 0 ? "(no cycle end set)" : o.pacePct >= 0 ? "(ahead of pace)" : "(behind pace)"}`,
     `  state       ${o.state}${o.hasNoReadings ? " \u2014 never read" : ""}${o.overdueByMs ? ` (by ${Math.floor(o.overdueByMs / 864e5)}d)` : ""}`,
-    `  next due    ${o.nextDueAt ? new Date(o.nextDueAt).toISOString().slice(0, 10) : "no cadence set"}`,
+    `  next due    ${o.nextDueAt ? safeDateSlice(o.nextDueAt) : "no cadence set"}`,
     `  owner       ${roleLabel(o.ownerRole)}`,
     `  reporter    ${roleLabel(o.reporterRole)}`,
     `  completion  recorded as ${o.completion} (source: ${o.completionProvenance})`,
@@ -15792,16 +15807,17 @@ function outcomeErrorResult(err, action, entryId, params = {}) {
   const classified = classifyError(err);
   const definedParams = Object.fromEntries(Object.entries(params).filter(([, v]) => v !== void 0));
   const isUnverifiedRecordTimeout = action === "record" && classified.code === "GATEWAY_TIMEOUT";
+  const isMissingCorrectionTarget = action === "record" && classified.code === "NOT_FOUND" && classified.message.startsWith("Reading not found");
   const defaultActions = isUnverifiedRecordTimeout ? [
     { tool: "outcome", description: "Check whether the reading already landed", parameters: { action: "list", entryId } },
     { tool: "outcome", description: "Check the derived view", parameters: { action: "show", entryId } }
-  ] : [{ tool: "outcome", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
+  ] : isMissingCorrectionTarget ? [{ tool: "outcome", description: "List readings for this entry to find a valid correctsReadingId", parameters: { action: "list", entryId } }] : retryIsFutile(classified.code) ? void 0 : [{ tool: "outcome", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
   return failureResult(
     classified.message,
     classified.code,
     classified.message,
-    classified.recovery ?? "Use action='show' or action='list' with a valid entryId to check the key result exists.",
-    classified.availableActions ?? defaultActions,
+    isMissingCorrectionTarget ? `Reading correctsReadingId not found for entryId='${entryId}'. Use action='list' to see valid reading ids.` : classified.recovery ?? "Use action='show' or action='list' with a valid entryId to check the key result exists.",
+    isMissingCorrectionTarget ? defaultActions : classified.availableActions ?? defaultActions,
     classified.diagnostics
   );
 }
@@ -16841,4 +16857,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-C5BIZH5N.js.map
+//# sourceMappingURL=chunk-A4WEYWDI.js.map