openhermes 4.12.1 → 4.13.0

package/harness/lib/routing/route-guidance.ts

@@ -1,147 +1,147 @@
-import {
-  ROUTE_ACTIONS,
-  ROUTE_OUTCOMES,
-  ROUTE_VERIFICATIONS,
-  ROUTE_WORK_TYPES,
-} from "./types.ts";
-import type {
-  RouteAction,
-  RouteOutcome,
-  RouteResolution,
-  RouteVerification,
-  RouteWork,
-  RuntimeRouteDecision,
-} from "./types.ts";
-
-export const ROUTE_GUIDANCE_PREFIX = "ROUTE_GUIDANCE:";
-export const NEXT_ROUTE_PREFIX = "NEXT_ROUTE:";
-
-interface ConsumedRouteGuidance {
-  output: string;
-  selected: string | null;
-}
-
-const runtimeRouteState = new Map<string, RuntimeRouteDecision>();
-
-function isRouteOutcome(value: unknown): value is RouteOutcome {
-  return typeof value === "string" && ROUTE_OUTCOMES.includes(value as RouteOutcome);
-}
-
-function isRouteVerification(value: unknown): value is RouteVerification {
-  return typeof value === "string" && ROUTE_VERIFICATIONS.includes(value as RouteVerification);
-}
-
-function isRouteAction(value: unknown): value is RouteAction {
-  return typeof value === "string" && ROUTE_ACTIONS.includes(value as RouteAction);
-}
-
-function isRouteWork(value: unknown): value is RouteWork {
-  return typeof value === "string" && ROUTE_WORK_TYPES.includes(value as RouteWork);
-}
-
-export function extractRouteGuidance(output: string): RouteResolution | null {
-  const guidanceLine = output
-    .split(/\r?\n/)
-    .map((line) => line.trim())
-    .find((line) => line.startsWith(ROUTE_GUIDANCE_PREFIX));
-
-  if (!guidanceLine) return null;
-
-  const raw = guidanceLine.slice(ROUTE_GUIDANCE_PREFIX.length).trim();
-  if (!raw) return null;
-
-  try {
-    const parsed = JSON.parse(raw) as Partial<RouteResolution>;
-    if (!isRouteOutcome(parsed.outcome)) return null;
-    if (parsed.verification !== undefined && !isRouteVerification(parsed.verification)) return null;
-    if (parsed.action !== undefined && !isRouteAction(parsed.action)) return null;
-    if (parsed.work !== undefined && !isRouteWork(parsed.work)) return null;
-    if (!Array.isArray(parsed.candidates) || !parsed.candidates.every((candidate) => typeof candidate === "string")) {
-      return null;
-    }
-    if (parsed.selected !== null && parsed.selected !== undefined && typeof parsed.selected !== "string") {
-      return null;
-    }
-    if (typeof parsed.reason !== "string") return null;
-
-    return {
-      outcome: parsed.outcome,
-      ...(parsed.verification ? { verification: parsed.verification } : {}),
-      ...(parsed.action ? { action: parsed.action } : {}),
-      ...(parsed.work ? { work: parsed.work } : {}),
-      candidates: parsed.candidates,
-      selected: parsed.selected ?? null,
-      reason: parsed.reason,
-    };
-  } catch {
-    return null;
-  }
-}
-
-export function consumeRouteGuidance(output: string): ConsumedRouteGuidance {
-  const guidance = extractRouteGuidance(output);
-  if (!guidance?.selected || output.includes(NEXT_ROUTE_PREFIX)) {
-    return { output, selected: guidance?.selected ?? null };
-  }
-
-  return {
-    output: `${output.trimEnd()}\n${NEXT_ROUTE_PREFIX} ${guidance.selected}`,
-    selected: guidance.selected,
-  };
-}
-
-function extractExplicitNextRoute(output: string): string | null {
-  const routeLine = output
-    .split(/\r?\n/)
-    .map((line) => line.trim())
-    .find((line) => line.startsWith(NEXT_ROUTE_PREFIX));
-
-  if (!routeLine) return null;
-
-  const raw = routeLine.slice(NEXT_ROUTE_PREFIX.length).trim();
-  if (!raw || /\s/.test(raw)) return null;
-  return raw;
-}
-
-export function extractRuntimeRouteDecision(output: string): RuntimeRouteDecision | null {
-  const explicitNextRoute = extractExplicitNextRoute(output);
-  if (explicitNextRoute) {
-    return {
-      selected: explicitNextRoute,
-      source: "next_route",
-    };
-  }
-
-  const guidance = extractRouteGuidance(output);
-  if (!guidance?.selected) return null;
-
-  return {
-    selected: guidance.selected,
-    source: "route_guidance",
-    outcome: guidance.outcome,
-    verification: guidance.verification,
-    action: guidance.action,
-    work: guidance.work,
-    candidates: guidance.candidates,
-    reason: guidance.reason,
-  };
-}
-
-export function rememberRuntimeRouteDecision(sessionId: string, output: string): RuntimeRouteDecision | null {
-  const decision = extractRuntimeRouteDecision(output);
-  if (!decision) {
-    runtimeRouteState.delete(sessionId);
-    return null;
-  }
-
-  runtimeRouteState.set(sessionId, decision);
-  return decision;
-}
-
-export function getRuntimeRouteDecision(sessionId: string): RuntimeRouteDecision | null {
-  return runtimeRouteState.get(sessionId) ?? null;
-}
-
-export function clearRuntimeRouteDecision(sessionId: string): void {
-  runtimeRouteState.delete(sessionId);
-}
+import {
+  ROUTE_ACTIONS,
+  ROUTE_OUTCOMES,
+  ROUTE_VERIFICATIONS,
+  ROUTE_WORK_TYPES,
+} from "./types.ts";
+import type {
+  RouteAction,
+  RouteOutcome,
+  RouteResolution,
+  RouteVerification,
+  RouteWork,
+  RuntimeRouteDecision,
+} from "./types.ts";
+
+export const ROUTE_GUIDANCE_PREFIX = "ROUTE_GUIDANCE:";
+export const NEXT_ROUTE_PREFIX = "NEXT_ROUTE:";
+
+interface ConsumedRouteGuidance {
+  output: string;
+  selected: string | null;
+}
+
+const runtimeRouteState = new Map<string, RuntimeRouteDecision>();
+
+function isRouteOutcome(value: unknown): value is RouteOutcome {
+  return typeof value === "string" && ROUTE_OUTCOMES.includes(value as RouteOutcome);
+}
+
+function isRouteVerification(value: unknown): value is RouteVerification {
+  return typeof value === "string" && ROUTE_VERIFICATIONS.includes(value as RouteVerification);
+}
+
+function isRouteAction(value: unknown): value is RouteAction {
+  return typeof value === "string" && ROUTE_ACTIONS.includes(value as RouteAction);
+}
+
+function isRouteWork(value: unknown): value is RouteWork {
+  return typeof value === "string" && ROUTE_WORK_TYPES.includes(value as RouteWork);
+}
+
+export function extractRouteGuidance(output: string): RouteResolution | null {
+  const guidanceLine = output
+    .split(/\r?\n/)
+    .map((line) => line.trim())
+    .find((line) => line.startsWith(ROUTE_GUIDANCE_PREFIX));
+
+  if (!guidanceLine) return null;
+
+  const raw = guidanceLine.slice(ROUTE_GUIDANCE_PREFIX.length).trim();
+  if (!raw) return null;
+
+  try {
+    const parsed = JSON.parse(raw) as Partial<RouteResolution>;
+    if (!isRouteOutcome(parsed.outcome)) return null;
+    if (parsed.verification !== undefined && !isRouteVerification(parsed.verification)) return null;
+    if (parsed.action !== undefined && !isRouteAction(parsed.action)) return null;
+    if (parsed.work !== undefined && !isRouteWork(parsed.work)) return null;
+    if (!Array.isArray(parsed.candidates) || !parsed.candidates.every((candidate) => typeof candidate === "string")) {
+      return null;
+    }
+    if (parsed.selected !== null && parsed.selected !== undefined && typeof parsed.selected !== "string") {
+      return null;
+    }
+    if (typeof parsed.reason !== "string") return null;
+
+    return {
+      outcome: parsed.outcome,
+      ...(parsed.verification ? { verification: parsed.verification } : {}),
+      ...(parsed.action ? { action: parsed.action } : {}),
+      ...(parsed.work ? { work: parsed.work } : {}),
+      candidates: parsed.candidates,
+      selected: parsed.selected ?? null,
+      reason: parsed.reason,
+    };
+  } catch {
+    return null;
+  }
+}
+
+export function consumeRouteGuidance(output: string): ConsumedRouteGuidance {
+  const guidance = extractRouteGuidance(output);
+  if (!guidance?.selected || output.includes(NEXT_ROUTE_PREFIX)) {
+    return { output, selected: guidance?.selected ?? null };
+  }
+
+  return {
+    output: `${output.trimEnd()}\n${NEXT_ROUTE_PREFIX} ${guidance.selected}`,
+    selected: guidance.selected,
+  };
+}
+
+function extractExplicitNextRoute(output: string): string | null {
+  const routeLine = output
+    .split(/\r?\n/)
+    .map((line) => line.trim())
+    .find((line) => line.startsWith(NEXT_ROUTE_PREFIX));
+
+  if (!routeLine) return null;
+
+  const raw = routeLine.slice(NEXT_ROUTE_PREFIX.length).trim();
+  if (!raw || /\s/.test(raw)) return null;
+  return raw;
+}
+
+export function extractRuntimeRouteDecision(output: string): RuntimeRouteDecision | null {
+  const explicitNextRoute = extractExplicitNextRoute(output);
+  if (explicitNextRoute) {
+    return {
+      selected: explicitNextRoute,
+      source: "next_route",
+    };
+  }
+
+  const guidance = extractRouteGuidance(output);
+  if (!guidance?.selected) return null;
+
+  return {
+    selected: guidance.selected,
+    source: "route_guidance",
+    outcome: guidance.outcome,
+    verification: guidance.verification,
+    action: guidance.action,
+    work: guidance.work,
+    candidates: guidance.candidates,
+    reason: guidance.reason,
+  };
+}
+
+export function rememberRuntimeRouteDecision(sessionId: string, output: string): RuntimeRouteDecision | null {
+  const decision = extractRuntimeRouteDecision(output);
+  if (!decision) {
+    runtimeRouteState.delete(sessionId);
+    return null;
+  }
+
+  runtimeRouteState.set(sessionId, decision);
+  return decision;
+}
+
+export function getRuntimeRouteDecision(sessionId: string): RuntimeRouteDecision | null {
+  return runtimeRouteState.get(sessionId) ?? null;
+}
+
+export function clearRuntimeRouteDecision(sessionId: string): void {
+  runtimeRouteState.delete(sessionId);
+}

package/harness/lib/routing/route-resolver.ts

@@ -1,58 +1,58 @@
-import type { RouteEvidence, RouteResolution, SkillRouteMap } from "./types.ts";
-
-function findCandidate(candidates: string[], fragment: string): string | null {
-  return candidates.find((candidate) => candidate.includes(fragment)) ?? null;
-}
-
-function buildResolution(evidence: RouteEvidence, candidates: string[], selected: string | null, reason: string): RouteResolution {
-  return {
-    outcome: evidence.outcome,
-    ...(evidence.verification ? { verification: evidence.verification } : {}),
-    ...(evidence.action ? { action: evidence.action } : {}),
-    ...(evidence.work ? { work: evidence.work } : {}),
-    candidates,
-    selected,
-    reason,
-  };
-}
-
-export function resolveRoute(routeMap: SkillRouteMap, evidence: RouteEvidence): RouteResolution {
-  const candidates = [...routeMap[evidence.outcome]];
-  if (candidates.length === 0) {
-    return buildResolution(evidence, candidates, null, `No route candidates declared for outcome \"${evidence.outcome}\".`);
-  }
-
-  if (evidence.target && candidates.includes(evidence.target)) {
-    return buildResolution(evidence, candidates, evidence.target, `Selected \"${evidence.target}\" from output evidence.`);
-  }
-
-  if (evidence.action === "fixable" || evidence.work === "implement") {
-    const builderCandidate = findCandidate(candidates, "builder");
-    if (builderCandidate) {
-      return buildResolution(evidence, candidates, builderCandidate, `Selected \"${builderCandidate}\" for fixable implementation work.`);
-    }
-  }
-
-  if (evidence.verification === "unverified") {
-    const gauntletCandidate = findCandidate(candidates, "gauntlet");
-    if (gauntletCandidate) {
-      return buildResolution(evidence, candidates, gauntletCandidate, `Selected \"${gauntletCandidate}\" because work is still unverified.`);
-    }
-  }
-
-  if (evidence.work === "verify") {
-    const gauntletCandidate = findCandidate(candidates, "gauntlet");
-    if (gauntletCandidate) {
-      return buildResolution(evidence, candidates, gauntletCandidate, `Selected \"${gauntletCandidate}\" for verification work.`);
-    }
-  }
-
-  if (evidence.work === "ship" && evidence.verification === "verified" && evidence.action === "done") {
-    const shipCandidate = findCandidate(candidates, "ship");
-    if (shipCandidate) {
-      return buildResolution(evidence, candidates, shipCandidate, `Selected \"${shipCandidate}\" for verified ship-ready work.`);
-    }
-  }
-
-  return buildResolution(evidence, candidates, candidates[0], `Selected first declared route for outcome \"${evidence.outcome}\".`);
-}
+import type { RouteEvidence, RouteResolution, SkillRouteMap } from "./types.ts";
+
+function findCandidate(candidates: string[], fragment: string): string | null {
+  return candidates.find((candidate) => candidate.includes(fragment)) ?? null;
+}
+
+function buildResolution(evidence: RouteEvidence, candidates: string[], selected: string | null, reason: string): RouteResolution {
+  return {
+    outcome: evidence.outcome,
+    ...(evidence.verification ? { verification: evidence.verification } : {}),
+    ...(evidence.action ? { action: evidence.action } : {}),
+    ...(evidence.work ? { work: evidence.work } : {}),
+    candidates,
+    selected,
+    reason,
+  };
+}
+
+export function resolveRoute(routeMap: SkillRouteMap, evidence: RouteEvidence): RouteResolution {
+  const candidates = [...routeMap[evidence.outcome]];
+  if (candidates.length === 0) {
+    return buildResolution(evidence, candidates, null, `No route candidates declared for outcome \"${evidence.outcome}\".`);
+  }
+
+  if (evidence.target && candidates.includes(evidence.target)) {
+    return buildResolution(evidence, candidates, evidence.target, `Selected \"${evidence.target}\" from output evidence.`);
+  }
+
+  if (evidence.action === "fixable" || evidence.work === "implement") {
+    const builderCandidate = findCandidate(candidates, "builder");
+    if (builderCandidate) {
+      return buildResolution(evidence, candidates, builderCandidate, `Selected \"${builderCandidate}\" for fixable implementation work.`);
+    }
+  }
+
+  if (evidence.verification === "unverified") {
+    const gauntletCandidate = findCandidate(candidates, "gauntlet");
+    if (gauntletCandidate) {
+      return buildResolution(evidence, candidates, gauntletCandidate, `Selected \"${gauntletCandidate}\" because work is still unverified.`);
+    }
+  }
+
+  if (evidence.work === "verify") {
+    const gauntletCandidate = findCandidate(candidates, "gauntlet");
+    if (gauntletCandidate) {
+      return buildResolution(evidence, candidates, gauntletCandidate, `Selected \"${gauntletCandidate}\" for verification work.`);
+    }
+  }
+
+  if (evidence.work === "ship" && evidence.verification === "verified" && evidence.action === "done") {
+    const shipCandidate = findCandidate(candidates, "ship");
+    if (shipCandidate) {
+      return buildResolution(evidence, candidates, shipCandidate, `Selected \"${shipCandidate}\" for verified ship-ready work.`);
+    }
+  }
+
+  return buildResolution(evidence, candidates, candidates[0], `Selected first declared route for outcome \"${evidence.outcome}\".`);
+}