opencandle 0.4.0 → 0.5.0

package/src/routing/router.ts

@@ -1,12 +1,25 @@
-import { extractEntities } from "./entity-extractor.js";
+import { extractEntities, isAmbiguousConceptUsage } from "./entity-extractor.js";
+import { classifyWithLegacyRules } from "./legacy-rule-router.js";
 import { buildRouterPrompt } from "./router-prompt.js";
+import {
+  computeMissingRequiredSlots,
+  isDispatchableWorkflow,
+  isRouteKind,
+  isToolBundleName,
+  legacyRouteForRouteKind,
+  routeKindFromLegacyRoute,
+  selectToolBundles,
+} from "./route-manifest.js";
 import type {
+  RouterDiagnostic,
   RouterInputContext,
   RouterLlmClient,
   RouterOutput,
   RouterPreferenceUpdate,
   RouterRoute,
+  RouterRouteKind,
   RouterSlot,
+  ToolBundleName,
 } from "./router-types.js";
 import type { ExtractedEntities, WorkflowType } from "./types.js";
 
@@ -19,7 +32,7 @@ const VALID_WORKFLOWS: ReadonlyArray<Exclude<WorkflowType, "unclassified">> = [
   "watchlist_or_tracking",
   "general_finance_qa",
 ];
-const VALID_SOURCES = new Set(["user", "preference", "default"]);
+const VALID_SOURCES = new Set(["user", "preference", "default", "prior_context", "memory"]);
 const VALID_CONFIDENCE = new Set(["high", "medium", "low"]);
 
 /**
@@ -38,7 +51,7 @@ export async function route(
   let firstError: string | undefined;
   try {
     const raw = await client.complete(prompt);
-    return validateRouterOutput(raw);
+    return postProcessRouterOutput(input.text, validateRouterOutput(raw));
   } catch (err) {
     firstError = err instanceof Error ? err.message : String(err);
   }
@@ -47,10 +60,10 @@ export async function route(
   try {
     const retryPrompt = `${prompt}\n\n(Your previous response failed validation: ${firstError}. Return a valid JSON object conforming to RouterOutput. Nothing else.)`;
     const raw = await client.complete(retryPrompt);
-    return validateRouterOutput(raw);
+    return postProcessRouterOutput(input.text, validateRouterOutput(raw));
   } catch {
     // Persistent failure — return a minimal fallback with regex-extracted symbols.
-    return minimalFallback(input.text);
+    return postProcessRouterOutput(input.text, minimalFallback(input.text));
   }
 }
 
@@ -62,33 +75,63 @@ export function validateRouterOutput(raw: string): RouterOutput {
   }
   const obj = parsed as Record<string, unknown>;
 
-  const route = obj.route;
-  if (typeof route !== "string" || !VALID_ROUTES.includes(route as RouterRoute)) {
-    throw new Error(`invalid route: ${JSON.stringify(route)}`);
+  const rawMissingRequired = validateStringArray(obj.missing_required, "missing_required");
+
+  const explicitRouteKind = obj.routeKind;
+  if (
+    explicitRouteKind !== undefined &&
+    (typeof explicitRouteKind !== "string" || !isRouteKind(explicitRouteKind))
+  ) {
+    throw new Error(`invalid routeKind: ${JSON.stringify(explicitRouteKind)}`);
+  }
+
+  const rawRoute = obj.route;
+  let route: RouterRoute;
+  if (typeof rawRoute === "string") {
+    if (!VALID_ROUTES.includes(rawRoute as RouterRoute)) {
+      throw new Error(`invalid route: ${JSON.stringify(rawRoute)}`);
+    }
+    route = rawRoute as RouterRoute;
+  } else if (typeof explicitRouteKind === "string" && isRouteKind(explicitRouteKind)) {
+    route = legacyRouteForRouteKind(explicitRouteKind);
+  } else {
+    throw new Error(`invalid route: ${JSON.stringify(rawRoute)}`);
   }
 
   let workflow: RouterOutput["workflow"];
-  if (route === "workflow") {
+  const routeKind: RouterRouteKind =
+    typeof explicitRouteKind === "string" && isRouteKind(explicitRouteKind)
+      ? explicitRouteKind
+      : routeKindFromLegacyRoute(route, rawMissingRequired);
+
+  if (route === "workflow" || routeKind === "workflow_dispatch") {
     if (typeof obj.workflow !== "string" || !VALID_WORKFLOWS.includes(obj.workflow as Exclude<WorkflowType, "unclassified">)) {
       throw new Error(`workflow route requires a valid workflow; got ${JSON.stringify(obj.workflow)}`);
     }
     workflow = obj.workflow as Exclude<WorkflowType, "unclassified">;
+  } else if (typeof obj.workflow === "string" && VALID_WORKFLOWS.includes(obj.workflow as Exclude<WorkflowType, "unclassified">)) {
+    workflow = obj.workflow as Exclude<WorkflowType, "unclassified">;
   }
 
   const entities = validateEntities(obj.entities);
   const slots = validateSlots(obj.slots);
   const preference_updates = validatePreferenceUpdates(obj.preference_updates);
-  const missing_required = validateStringArray(obj.missing_required, "missing_required");
+  const missing_required = rawMissingRequired;
+  const tool_bundles = validateToolBundles(obj.tool_bundles);
+  const diagnostics = validateDiagnostics(obj.diagnostics);
   const reasoning =
     typeof obj.reasoning === "string" ? obj.reasoning : "";
 
   return {
-    route: route as RouterRoute,
+    routeKind,
+    route: legacyRouteForRouteKind(routeKind),
     workflow,
     entities,
     slots,
     preference_updates,
     missing_required,
+    tool_bundles,
+    diagnostics,
     reasoning,
   };
 }
@@ -120,13 +163,433 @@ function validateEntities(raw: unknown): ExtractedEntities {
   const out: ExtractedEntities = { symbols };
   if (typeof e.budget === "number") out.budget = e.budget;
   if (typeof e.maxPremium === "number") out.maxPremium = e.maxPremium;
+  if (typeof e.costBasis === "number") out.costBasis = e.costBasis;
+  if (typeof e.shareQuantity === "number") out.shareQuantity = e.shareQuantity;
   if (typeof e.timeHorizon === "string") out.timeHorizon = e.timeHorizon;
   if (typeof e.riskProfile === "string") out.riskProfile = e.riskProfile;
   if (e.direction === "bullish" || e.direction === "bearish") out.direction = e.direction;
   if (typeof e.dteHint === "string") out.dteHint = e.dteHint;
+  if (e.optionStrategy === "covered_call" || e.optionStrategy === "protective_put") out.optionStrategy = e.optionStrategy;
+  if (typeof e.heldSymbol === "string") out.heldSymbol = e.heldSymbol.toUpperCase();
+  const catalystSymbols = validateStringArray(e.catalystSymbols, "entities.catalystSymbols").map((s) =>
+    s.toUpperCase(),
+  );
+  if (catalystSymbols.length > 0) out.catalystSymbols = catalystSymbols;
+  const compareMetrics = validateStringArray(e.compareMetrics, "entities.compareMetrics");
+  if (compareMetrics.length > 0) out.compareMetrics = compareMetrics;
   return out;
 }
 
+export function postProcessRouterOutput(text: string, output: RouterOutput): RouterOutput {
+  const extracted = extractEntities(text);
+  const deterministic = classifyWithLegacyRules(text);
+  let diagnostics: RouterDiagnostic[] = [...output.diagnostics];
+  let next: RouterOutput = {
+    ...output,
+    entities: {
+      ...output.entities,
+      symbols: output.entities.symbols.filter((symbol) =>
+        !isAmbiguousConceptUsage(text, symbol),
+      ),
+      budget: output.entities.budget ?? extracted.budget,
+      maxPremium: output.entities.maxPremium ?? extracted.maxPremium,
+      timeHorizon: output.entities.timeHorizon ?? extracted.timeHorizon,
+      riskProfile: output.entities.riskProfile ?? extracted.riskProfile,
+      assetScope: output.entities.assetScope ?? extracted.assetScope,
+      compareMetrics: mergeStringArrays(output.entities.compareMetrics, extracted.compareMetrics),
+      direction: output.entities.direction ?? extracted.direction,
+      optionStrategy: output.entities.optionStrategy ?? extracted.optionStrategy,
+      costBasis: output.entities.costBasis ?? extracted.costBasis,
+      shareQuantity: output.entities.shareQuantity ?? extracted.shareQuantity,
+      heldSymbol: output.entities.heldSymbol ?? extracted.heldSymbol,
+      catalystSymbols: output.entities.catalystSymbols ?? extracted.catalystSymbols,
+      dteHint: output.entities.dteHint ?? (output.workflow === "options_screener" ? extracted.dteHint : undefined),
+    },
+    diagnostics,
+  };
+
+  if (next.workflow === "options_screener" && isExistingPositionOptionRequest(text, extracted) && extracted.heldSymbol) {
+    const reorderedSymbols = [
+      extracted.heldSymbol,
+      ...mergeSymbols(next.entities.symbols, extracted.symbols).filter((symbol) => symbol !== extracted.heldSymbol),
+    ];
+    if (next.entities.symbols[0] !== extracted.heldSymbol) {
+      diagnostics.push({
+        code: extracted.optionStrategy === "protective_put"
+          ? "existing_position_underlying_corrected"
+          : "covered_call_underlying_corrected",
+        message: `using owned position ${extracted.heldSymbol} as the option-chain underlying`,
+      });
+    }
+    next = {
+      ...next,
+      entities: {
+        ...next.entities,
+        symbols: reorderedSymbols,
+        optionStrategy: extracted.optionStrategy ?? next.entities.optionStrategy,
+        direction: extracted.direction ?? next.entities.direction,
+        heldSymbol: extracted.heldSymbol,
+        catalystSymbols: reorderedSymbols.filter((symbol) => symbol !== extracted.heldSymbol),
+        costBasis: extracted.costBasis ?? next.entities.costBasis,
+        shareQuantity: extracted.shareQuantity ?? next.entities.shareQuantity,
+        dteHint: extracted.dteHint ?? next.entities.dteHint,
+      },
+      diagnostics,
+    };
+  }
+
+  if (
+    next.workflow === "options_screener" &&
+    isOptionsEducationOrSuitabilityRequest(text) &&
+    !isSpecificOptionContractSelectionRequest(text)
+  ) {
+    diagnostics.push({
+      code: "options_workflow_corrected_to_policy_task",
+      message: "options education or suitability prompt should use policy-card synthesis, not contract-screen workflow dispatch",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      workflow: "general_finance_qa",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  // Legacy rules may recover a primary route only when the LLM router path has
+  // already failed validation. Otherwise they are limited to enrichment and
+  // narrow corrections below.
+  if (
+    next.diagnostics.some((d) => d.code === "router_validation_failed") &&
+    deterministic.workflow !== "unclassified"
+  ) {
+    next = {
+      ...next,
+      routeKind: isDispatchableWorkflow(deterministic.workflow)
+        ? "workflow_dispatch"
+        : "agent_task",
+      route: isDispatchableWorkflow(deterministic.workflow) ? "workflow" : "fallback",
+      workflow: deterministic.workflow,
+      entities: {
+        ...deterministic.entities,
+        budget: deterministic.entities.budget ?? extracted.budget,
+        maxPremium: deterministic.entities.maxPremium ?? extracted.maxPremium,
+        timeHorizon: deterministic.entities.timeHorizon ?? extracted.timeHorizon,
+        riskProfile: deterministic.entities.riskProfile ?? extracted.riskProfile,
+        assetScope: deterministic.entities.assetScope ?? extracted.assetScope,
+        compareMetrics: mergeStringArrays(deterministic.entities.compareMetrics, extracted.compareMetrics),
+        direction: deterministic.entities.direction ?? extracted.direction,
+        costBasis: deterministic.entities.costBasis ?? extracted.costBasis,
+        shareQuantity: deterministic.entities.shareQuantity ?? extracted.shareQuantity,
+        heldSymbol: deterministic.entities.heldSymbol ?? extracted.heldSymbol,
+        catalystSymbols: deterministic.entities.catalystSymbols ?? extracted.catalystSymbols,
+      },
+      diagnostics: [
+        ...diagnostics,
+        {
+          code: "deterministic_failure_recovery",
+          message: `deterministic classifier selected ${deterministic.workflow} after router validation failure`,
+        },
+      ],
+      reasoning: next.reasoning
+        ? `${next.reasoning}; deterministic classifier selected ${deterministic.workflow}`
+        : `deterministic classifier selected ${deterministic.workflow}`,
+    };
+    diagnostics = next.diagnostics;
+  }
+
+  if (next.routeKind === "workflow_dispatch" && !isDispatchableWorkflow(next.workflow)) {
+    diagnostics.push({
+      code: "route_kind_corrected_to_agent_task",
+      message: next.workflow
+        ? `${next.workflow} is not a dispatchable workflow`
+        : "workflow_dispatch requires a dispatchable workflow",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      diagnostics,
+    };
+  }
+
+  if (next.routeKind === "agent_task" && isDispatchableWorkflow(next.workflow)) {
+    diagnostics.push({
+      code: "dispatchable_workflow_corrected_to_workflow_dispatch",
+      message: `${next.workflow} is a dispatchable workflow`,
+    });
+    next = {
+      ...next,
+      routeKind: "workflow_dispatch",
+      route: "workflow",
+      diagnostics,
+    };
+  }
+
+  if (
+    next.workflow === "compare_assets" &&
+    next.entities.symbols.length === 0 &&
+    isExplicitMacroDataRequest(text)
+  ) {
+    diagnostics.push({
+      code: "compare_route_corrected_to_macro_task",
+      message: "macro/source acronyms were not explicit tickers",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      workflow: "general_finance_qa",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  if (next.workflow === "compare_assets" && isPortfolioEvaluationRequest(text)) {
+    diagnostics.push({
+      code: "portfolio_evaluation_corrected_to_agent_task",
+      message: "existing portfolio/allocation risk review should not be reduced to asset comparison",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      workflow: "general_finance_qa",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  if (
+    next.routeKind === "agent_task" &&
+    !next.workflow &&
+    next.entities.symbols.length === 0 &&
+    isExplicitMacroDataRequest(text)
+  ) {
+    diagnostics.push({
+      code: "macro_task_inferred_from_prompt",
+      message: "macro data terms were present without explicit tickers",
+    });
+    next = {
+      ...next,
+      workflow: "general_finance_qa",
+      diagnostics,
+    };
+  }
+
+  if (next.workflow === "portfolio_builder" && isCryptoSizingRequest(text)) {
+    diagnostics.push({
+      code: "crypto_sizing_corrected_to_agent_task",
+      message: "crypto allocation-range and drawdown questions are advisory tradeoffs, not portfolio construction",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      workflow: "general_finance_qa",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  if (next.workflow === "portfolio_builder" && isPortfolioEvaluationRequest(text)) {
+    diagnostics.push({
+      code: "portfolio_evaluation_corrected_to_agent_task",
+      message: "existing portfolio/allocation evaluation does not require portfolio-construction budget",
+    });
+    next = {
+      ...next,
+      routeKind: "agent_task",
+      route: "fallback",
+      workflow: "general_finance_qa",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  if (
+    next.workflow === "portfolio_builder" &&
+    next.entities.symbols.length >= 2 &&
+    isPortfolioTradeoffComparisonRequest(text)
+  ) {
+    diagnostics.push({
+      code: "portfolio_tradeoff_corrected_to_compare_assets",
+      message: "explicit multi-asset tradeoff question should compare the requested assets before constructing a portfolio",
+    });
+    next = {
+      ...next,
+      routeKind: "workflow_dispatch",
+      route: "workflow",
+      workflow: "compare_assets",
+      missing_required: [],
+      diagnostics,
+    };
+  }
+
+  if (
+    next.workflow === "single_asset_analysis" &&
+    isSpecializedSingleAssetPolicyRequest(text)
+  ) {
+    diagnostics.push({
+      code: "single_asset_workflow_corrected_to_general_policy_task",
+      message: "prompt asks for policy-card planning outside a single-asset buy/sell analysis",
+    });
+    next = {
+      ...next,
+      workflow: "general_finance_qa",
+      diagnostics,
+    };
+  }
+
+  const missingRequired = computeMissingRequiredSlots(
+    next.workflow,
+    next.entities,
+    next.slots,
+    next.missing_required,
+  );
+  if (missingRequired.length > 0 && next.routeKind !== "pass_through") {
+    if (next.routeKind !== "clarification") {
+      diagnostics.push({
+        code: "route_kind_corrected_to_clarification",
+        message: `missing required slots: ${missingRequired.join(", ")}`,
+      });
+    }
+    next = {
+      ...next,
+      routeKind: "clarification",
+      route: "fallback",
+      missing_required: missingRequired,
+      diagnostics,
+    };
+  }
+
+  const selectedToolBundles = isConceptualEducationRequest(text, next)
+    ? []
+    : selectToolBundles(next);
+  if (selectedToolBundles.length === 0 && isConceptualEducationRequest(text, next)) {
+    diagnostics.push({
+      code: "conceptual_education_no_tools",
+      message: "conceptual education prompt does not need live finance tools",
+    });
+  }
+  const emittedUnsupported = next.tool_bundles.filter((bundle) => !selectedToolBundles.includes(bundle));
+  if (emittedUnsupported.length > 0) {
+    diagnostics.push({
+      code: "tool_bundles_corrected",
+      message: `unsupported emitted bundles dropped: ${emittedUnsupported.join(", ")}`,
+    });
+  }
+
+  return omitUndefined({
+    ...next,
+    route: legacyRouteForRouteKind(next.routeKind),
+    tool_bundles: selectedToolBundles,
+    diagnostics,
+  });
+}
+
+function isExplicitMacroDataRequest(text: string): boolean {
+  return /\b(?:get_economic_data|fred|cpi|inflation|fed\s+funds?|unemployment|gdp|macro)\b/i.test(text);
+}
+
+function isConceptualEducationRequest(text: string, output: RouterOutput): boolean {
+  if (output.routeKind !== "agent_task") return false;
+  if (output.entities.symbols.length > 0) return false;
+  if (isForwardLookingMacroContextRequest(text)) return false;
+  if (/\b(?:current|recent|today|right now|latest|news|sentiment|build|portfolio|buy|sell|allocate|compare)\b/i.test(text)) {
+    return false;
+  }
+  return /\b(?:explain|what is|define|how (?:do|should|to)|teach me|help me understand)\b/i.test(text);
+}
+
+function isForwardLookingMacroContextRequest(text: string): boolean {
+  return /\b(?:rates?|rate\s*cuts?|fed|inflation|macro)\b/i.test(text) &&
+    /\b(?:next\s+(?:year|12\s*months?)|over\s+the\s+next|outlook|affect|impact|falling|rising)\b/i.test(text);
+}
+
+function isCoveredCallRequest(text: string): boolean {
+  return /\bcovered\s+calls?\b/i.test(text);
+}
+
+function isPortfolioEvaluationRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  const hasEvaluationIntent =
+    /\b(?:evaluat(?:e|ion)|review|assess|analy[sz]e|prospects?|risks?|risky|opportunities?|mitigat(?:e|ion)|adjustment|rebalance|diversify|concentration|overweight|underweight|target\s+bands?|drift|worried|crash|protect|protection|missing\s+out\s+on\s+growth)\b/.test(lower);
+  const hasPortfolioObject =
+    /\b(?:portfolio|allocation|asset\s+allocation|60\/40|equity|fixed\s+income|bonds?)\b/.test(lower);
+  const hasConstructionIntent =
+    /\b(?:build|create|construct|put\s+together|invest|allocate)\b/.test(lower) &&
+    (/\$\s*\d|\b\d+(?:\.\d+)?\s*k\b|\bbudget\b|\bcapital\b/.test(lower));
+  return hasEvaluationIntent && hasPortfolioObject && !hasConstructionIntent;
+}
+
+function isPortfolioTradeoffComparisonRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  return /\b(?:prioritize|tradeoffs?|growth[-\s]?oriented|dividend|income|which\s+(?:one|is)\s+better|should\s+i)\b/.test(lower) &&
+    /\b(?:or|vs\.?|versus|compare)\b/.test(lower);
+}
+
+function isCryptoSizingRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  const hasPortfolioConstructionIntent =
+    /\b(?:build|create|construct|put\s+together)\b/.test(lower) &&
+    /\b(?:portfolio|allocation)\b/.test(lower);
+  if (hasPortfolioConstructionIntent) return false;
+  return /\b(?:btc|bitcoin|crypto)\b/.test(lower) &&
+    /\b(?:allocation|range|position\s+size|sizing|exposure|drawdown)\b/.test(lower);
+}
+
+function isSpecializedSingleAssetPolicyRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  return /\b(?:ticker|symbol|formerly|old ticker|earnings are|earnings tonight)\b/.test(lower) ||
+    /\b(?:today|right now|this morning|after close|moved|catalyst)\b/.test(lower) ||
+    /\b(?:sentiment|mood|reddit|twitter|x\/twitter)\b/.test(lower) ||
+    /\b(?:filing|10-k|10-q|8-k|sec)\b/.test(lower);
+}
+
+function isExistingPositionOptionRequest(text: string, extracted: ExtractedEntities): boolean {
+  return isCoveredCallRequest(text) || extracted.optionStrategy === "protective_put";
+}
+
+function isOptionsEducationOrSuitabilityRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  return /\b(?:how\s+does|how\s+do|explain|what\s+is|good\s+idea|make\s+sense|suitable|suitability|is\s+it\s+(?:good|worth|smart))\b/.test(lower) &&
+    /\b(?:covered\s+calls?|protective\s+puts?|options?|selling\s+calls?|option\s+income)\b/.test(lower);
+}
+
+function isSpecificOptionContractSelectionRequest(text: string): boolean {
+  const lower = text.toLowerCase();
+  return /\b(?:best|which|what\s+(?:strike|contract|option)|rank|screen|specific|right\s+now|today|around\s+earnings|expiration|dte|premium\s+under)\b/.test(lower) &&
+    /\b(?:sell|buy|trade|contract|strike|expiration|premium|call|put)\b/.test(lower);
+}
+
+function mergeSymbols(primary: string[], secondary: string[]): string[] {
+  const merged: string[] = [];
+  for (const symbol of [...primary, ...secondary]) {
+    if (!merged.includes(symbol)) merged.push(symbol);
+  }
+  return merged;
+}
+
+function mergeStringArrays(primary?: string[], secondary?: string[]): string[] | undefined {
+  const merged: string[] = [];
+  for (const value of [...(primary ?? []), ...(secondary ?? [])]) {
+    if (!merged.includes(value)) merged.push(value);
+  }
+  return merged.length > 0 ? merged : undefined;
+}
+
+function omitUndefined<T>(value: T): T {
+  if (Array.isArray(value)) return value.map(omitUndefined) as T;
+  if (!value || typeof value !== "object") return value;
+  const out: Record<string, unknown> = {};
+  for (const [key, entry] of Object.entries(value)) {
+    if (entry !== undefined) out[key] = omitUndefined(entry);
+  }
+  return out as T;
+}
+
 function validateSlots(raw: unknown): Record<string, RouterSlot> {
   if (raw === undefined || raw === null) return {};
   if (typeof raw !== "object") {
@@ -187,6 +650,34 @@ function validatePreferenceUpdates(raw: unknown): RouterPreferenceUpdate[] {
   });
 }
 
+function validateToolBundles(raw: unknown): ToolBundleName[] {
+  const bundles = validateStringArray(raw, "tool_bundles");
+  return bundles.filter(isToolBundleName);
+}
+
+function validateDiagnostics(raw: unknown): RouterDiagnostic[] {
+  if (raw === undefined || raw === null) return [];
+  if (!Array.isArray(raw)) {
+    throw new Error("diagnostics must be an array");
+  }
+  return raw.map((item, idx) => {
+    if (!item || typeof item !== "object") {
+      throw new Error(`diagnostics[${idx}] must be an object`);
+    }
+    const diagnostic = item as Record<string, unknown>;
+    if (typeof diagnostic.code !== "string" || diagnostic.code.length === 0) {
+      throw new Error(`diagnostics[${idx}].code must be a non-empty string`);
+    }
+    if (typeof diagnostic.message !== "string") {
+      throw new Error(`diagnostics[${idx}].message must be a string`);
+    }
+    return {
+      code: diagnostic.code,
+      message: diagnostic.message,
+    };
+  });
+}
+
 function validateStringArray(raw: unknown, field: string): string[] {
   if (raw === undefined || raw === null) return [];
   if (!Array.isArray(raw)) {
@@ -203,11 +694,19 @@ function validateStringArray(raw: unknown, field: string): string[] {
 function minimalFallback(text: string): RouterOutput {
   const entities = extractEntities(text);
   return {
+    routeKind: "agent_task",
     route: "fallback",
-    entities: { symbols: entities.symbols },
+    entities,
     slots: {},
     preference_updates: [],
     missing_required: [],
+    tool_bundles: [],
+    diagnostics: [
+      {
+        code: "router_validation_failed",
+        message: "router validation failed persistently; emitted minimal fallback",
+      },
+    ],
     reasoning: "router validation failed; emitted minimal fallback",
   };
 }