@remnic/core 9.3.736 → 9.3.738

package/src/orchestrator.ts

@@ -101,6 +101,8 @@ import { CompressionGuidelineCoordinator } from "./orchestration/compression-gui
 import { SemanticConsolidationCoordinator } from "./orchestration/semantic-consolidation-coordinator.js";
 import { LifecyclePolicyCoordinator } from "./orchestration/lifecycle-policy-coordinator.js";
 import { EntitySynthesisCoordinator } from "./orchestration/entity-synthesis-coordinator.js";
+import { RecallResultFormatter } from "./orchestration/recall-result-formatter.js";
+export { hasIdentityRecoveryIntent, resolveEffectiveIdentityInjectionMode } from "./orchestration/recall-result-formatter.js";
 import {
   runLiveConnectorsOnce,
   type LiveConnectorsRunSummary,
@@ -265,8 +267,7 @@ import {
   type TrustZoneSearchResult,
 } from "./trust-zones.js";
 import { tryDirectAnswer, type DirectAnswerSources } from "./direct-answer-wiring.js";
-import {
-  resolveNamespaceCapabilities,
+import { resolveNamespaceCapabilities,
   resolveCapabilities,
   resolveGraphConstructionCapabilities,
   resolveMemoryLifecycleCapabilities,
@@ -280,7 +281,7 @@ import {
   resolveQmdCapabilities,
   resolveIdentityContinuityCapabilities,
   resolveLocalLlmCapabilities,
-  resolveSecurityCapabilities, resolveEvalCapabilities, resolveUtilityLearningCapabilities, resolveObjectiveStateCapabilities, resolveCompressionCapabilities} from "./capabilities.js";
+  resolveSecurityCapabilities, resolveEvalCapabilities, resolveUtilityLearningCapabilities, resolveObjectiveStateCapabilities, resolveCompressionCapabilities, resolvePresentationCapabilities, resolveConsolidationCapabilities, resolveRecallAuxiliaryCapabilities } from "./capabilities.js";
 import { DEFAULT_TAXONOMY } from "./taxonomy/index.js";
 import {
   searchHarmonicRetrieval,
@@ -1368,31 +1369,6 @@ export async function resolveRecallModeDecisionAsync(
   };
 }
 
-export function hasIdentityRecoveryIntent(prompt: string): boolean {
-  const text = typeof prompt === "string" ? prompt.toLowerCase() : "";
-  if (!text) return false;
-  return /\b(identity|continuity|recover(?:y|ing|ed)?|incident|drift|restore|regress(?:ion|ed|ing)?)\b/i.test(
-    text,
-  );
-}
-
-export function resolveEffectiveIdentityInjectionMode(options: {
-  configuredMode: IdentityInjectionMode;
-  recallMode: RecallPlanMode;
-  prompt: string;
-}): { mode: IdentityInjectionMode; shouldInject: boolean } {
-  if (
-    options.configuredMode === "recovery_only" &&
-    !hasIdentityRecoveryIntent(options.prompt)
-  ) {
-    return { mode: "recovery_only", shouldInject: false };
-  }
-  if (options.recallMode === "minimal" && options.configuredMode === "full") {
-    return { mode: "minimal", shouldInject: true };
-  }
-  return { mode: options.configuredMode, shouldInject: true };
-}
-
 export function computeArtifactCandidateFetchLimit(
   targetCount: number,
 ): number {
@@ -1928,6 +1904,11 @@ export class Orchestrator {
    * hands it to `enqueue`.
    */
   readonly extractionQueueCoordinator: ExtractionQueueCoordinator;
+  /**
+   * Issue #1526: recall result formatting + identity continuity section moved
+   * to RecallResultFormatter.
+   */
+  readonly recallResultFormatter: RecallResultFormatter;
   private heartbeatObserverChains = new Map<string, Promise<void>>();
   private recentExtractionFingerprints = new Map<string, number>();
   private readonly consolidationObservers = new Set<
@@ -2583,7 +2564,7 @@ export class Orchestrator {
   private async contentHashIndexForStorage(
     targetStorage: StorageManager,
   ): Promise<ContentHashIndex | null> {
-    if (!this.config.factDeduplicationEnabled) return null;
+    if (!resolveRecallAuxiliaryCapabilities(this.config).factDeduplication) return null;
 
     if (targetStorage.dir === this.storage.dir) {
       if (!this.contentHashIndex) {
@@ -2846,7 +2827,7 @@ export class Orchestrator {
     if (config.defaultNamespace) this.storageRouter.bindCatalogWriteHook(this.storage, config.defaultNamespace);
     // Wire page-level versioning (issue #371)
     this.storage.setVersioningConfig({
-      enabled: config.versioningEnabled,
+      enabled: resolveRecallAuxiliaryCapabilities(config).versioning,
       maxVersionsPerPage: config.versioningMaxPerPage,
       sidecarDir: config.versioningSidecarDir,
     });
@@ -2869,7 +2850,7 @@ export class Orchestrator {
     // Wire at-rest encryption (issue #690 PR 3/4).
     // If secureStoreEnabled, check whether the keyring already holds a key
     // for this memory dir (e.g. operator unlocked before daemon restart).
-    if (config.secureStoreEnabled) {
+    if (resolveRecallAuxiliaryCapabilities(config).secureStore) {
       // Mark the store as required so writes throw SecureStoreLockedError
       // instead of silently falling back to plaintext when locked (P1 finding
       // from Cursor review of PR #767).
@@ -2896,6 +2877,7 @@ export class Orchestrator {
     });
     // Issue #1526: background extraction queue lives on its own coordinator.
     this.extractionQueueCoordinator = new ExtractionQueueCoordinator();
+    this.recallResultFormatter = new RecallResultFormatter(this.config);
     const conversationIndexRuntime = createConversationIndexRuntime(config, {
       getQmd: () => this.conversationQmd,
       getFaiss: () => this.conversationFaiss,
@@ -2906,7 +2888,7 @@ export class Orchestrator {
     this.sharedContext = config.sharedContextEnabled
       ? new SharedContextManager(config)
       : undefined;
-    this.compounding = config.compoundingEnabled
+    this.compounding = resolveConsolidationCapabilities(config).compounding
       ? new CompoundingEngine(config, this.storage)
       : undefined;
     this.buffer = new SmartBuffer(config, this.storage);
@@ -3098,7 +3080,7 @@ export class Orchestrator {
       this.boxBuilders.set(
         dir,
         new BoxBuilder(dir, {
-          memoryBoxesEnabled: this.config.memoryBoxesEnabled,
+          memoryBoxesEnabled: resolvePresentationCapabilities(this.config).memoryBoxes,
           traceWeaverEnabled: this.config.traceWeaverEnabled,
           boxTopicShiftThreshold: this.config.boxTopicShiftThreshold,
           boxTimeGapMs: this.config.boxTimeGapMs,
@@ -3444,7 +3426,7 @@ export class Orchestrator {
       });
 
       // Initialize content-hash dedup index
-      if (this.config.factDeduplicationEnabled) {
+      if (resolveRecallAuxiliaryCapabilities(this.config).factDeduplication) {
         this.contentHashIndex = this.storage.createContentHashIndex();
         await this.contentHashIndex.load();
         log.info(
@@ -3470,7 +3452,7 @@ export class Orchestrator {
         );
         this.buffer.resetToEmpty();
       }
-      if (this.config.compactionResetEnabled) {
+      if (resolveRecallAuxiliaryCapabilities(this.config).compactionReset) {
         try {
           const wsDir = this.config.workspaceDir || defaultWorkspaceDir();
           const files = await readdir(wsDir).catch(() => [] as string[]);
@@ -3694,7 +3676,7 @@ export class Orchestrator {
     // Awaited so callers of `deferredReady` can rely on warmups being complete
     // and shutdown sequencing does not race with in-flight cache builds.
     const cacheWarmups: Promise<void>[] = [];
-    if (this.config.knowledgeIndexEnabled) {
+    if (resolveRecallAuxiliaryCapabilities(this.config).knowledgeIndex) {
       cacheWarmups.push(
         (async () => {
           try {
@@ -4013,7 +3995,7 @@ export class Orchestrator {
     const cadenceKey = options.namespace ?? "";
     // Master switch: a disabled feature is never bypassed, even with
     // force=true.  `force` only relaxes the cadence floor below.
-    if (!this.config.patternReinforcementEnabled) {
+    if (!resolveConsolidationCapabilities(this.config).patternReinforcement) {
       return { ran: false, skippedReason: "disabled", namespace: cadenceKey };
     }
     const cadence = this.config.patternReinforcementCadenceMs;
@@ -4072,7 +4054,7 @@ export class Orchestrator {
           ? { itemCount: result.result.clustersFound }
           : { itemCount: 0 };
       },
-      { enabled: this.config.patternReinforcementEnabled },
+      { enabled: resolveConsolidationCapabilities(this.config).patternReinforcement },
     );
   }
 
@@ -4116,7 +4098,7 @@ export class Orchestrator {
         });
         return { itemCount: result.clustersFound };
       },
-      { enabled: this.config.semanticConsolidationEnabled },
+      { enabled: resolveConsolidationCapabilities(this.config).semanticConsolidation },
     );
   }
 
@@ -5447,7 +5429,7 @@ export class Orchestrator {
     // memory directory, reject recall with a clear human-readable error
     // rather than surfacing a cryptic SecureStoreLockedError from deep
     // inside the storage layer.
-    if (this.config.secureStoreEnabled && !this.storage.isSecureStoreUnlocked()) {
+    if (resolveRecallAuxiliaryCapabilities(this.config).secureStore && !this.storage.isSecureStoreUnlocked()) {
       const lockedMsg =
         "[secure-store locked] Memory store is encrypted and locked. " +
         "Unlock the secure-store inside this daemon process, or restart the daemon through a secure-store aware launcher that installs the key.";
@@ -5585,7 +5567,7 @@ export class Orchestrator {
     });
     const observationNamespaces = observationScopePlan.readNamespaces;
     const observationQueryPolicy = buildRecallQueryPolicy(prompt, sessionKey, {
-      cronRecallPolicyEnabled: this.config.cronRecallPolicyEnabled,
+      cronRecallPolicyEnabled: resolveRecallAuxiliaryCapabilities(this.config).cronRecallPolicy,
       cronRecallNormalizedQueryMaxChars:
         this.config.cronRecallNormalizedQueryMaxChars,
       cronRecallInstructionHeavyTokenCap:
@@ -7011,7 +6993,7 @@ export class Orchestrator {
     // recall searches, incl. coding `readFallbacks`). See the
     // `lcmReadSessionIds` derivation after `recallNamespaces` is built.
     const queryPolicy = buildRecallQueryPolicy(prompt, sessionKey, {
-      cronRecallPolicyEnabled: this.config.cronRecallPolicyEnabled,
+      cronRecallPolicyEnabled: resolveRecallAuxiliaryCapabilities(this.config).cronRecallPolicy,
       cronRecallNormalizedQueryMaxChars:
         this.config.cronRecallNormalizedQueryMaxChars,
       cronRecallInstructionHeavyTokenCap:
@@ -7174,7 +7156,7 @@ export class Orchestrator {
           return limit;
         })()
       : 0;
-    const recallHeadroom = this.config.verbatimArtifactsEnabled
+    const recallHeadroom = resolvePresentationCapabilities(this.config).verbatimArtifacts
       ? Math.max(12, this.config.verbatimArtifactsMaxRecall * 4)
       : 12;
     const computedFetchLimit =
@@ -7187,7 +7169,7 @@ export class Orchestrator {
     const qmdFetchLimit = computedFetchLimit;
     const qmdHybridFetchLimit = computeQmdHybridFetchLimit(
       qmdFetchLimit,
-      this.config.verbatimArtifactsEnabled,
+      resolvePresentationCapabilities(this.config).verbatimArtifacts,
       this.config.verbatimArtifactsMaxRecall,
     );
     const embeddingFetchLimit = computedFetchLimit;
@@ -7879,11 +7861,11 @@ export class Orchestrator {
       if (
         !this.isRecallSectionEnabled(
           "entity-retrieval",
-          this.config.entityRetrievalEnabled,
+          resolveRecallAuxiliaryCapabilities(this.config).entityRetrieval,
         )
       )
         return null;
-      if (!this.config.entityRetrievalEnabled) return null;
+      if (!resolveRecallAuxiliaryCapabilities(this.config).entityRetrieval) return null;
       const maxChars =
         this.getRecallSectionMaxChars("entity-retrieval") ??
         this.config.entityRetrievalMaxChars;
@@ -7956,11 +7938,11 @@ export class Orchestrator {
       if (
         !this.isRecallSectionEnabled(
           "knowledge-index",
-          this.config.knowledgeIndexEnabled,
+          resolveRecallAuxiliaryCapabilities(this.config).knowledgeIndex,
         )
       )
         return null;
-      if (!this.config.knowledgeIndexEnabled) return null;
+      if (!resolveRecallAuxiliaryCapabilities(this.config).knowledgeIndex) return null;
       const t0 = Date.now();
       try {
         const knowledgeIndexMaxChars =
@@ -8051,11 +8033,11 @@ export class Orchestrator {
       if (
         !this.isRecallSectionEnabled(
           "verbatim-artifacts",
-          this.config.verbatimArtifactsEnabled === true,
+          resolvePresentationCapabilities(this.config).verbatimArtifacts === true,
         )
       )
         return [];
-      if (!this.config.verbatimArtifactsEnabled) return [];
+      if (!resolvePresentationCapabilities(this.config).verbatimArtifacts) return [];
       const t0 = Date.now();
       const targetCount = computeArtifactRecallLimit(
         recallMode,
@@ -8286,10 +8268,10 @@ export class Orchestrator {
     const calibrationPromise = (async (): Promise<string | null> => {
       const t0 = Date.now();
       if (
-        !this.config.calibrationEnabled ||
+        !resolveConsolidationCapabilities(this.config).calibration ||
         !this.isRecallSectionEnabled(
           "calibration-rules",
-          this.config.calibrationEnabled === true,
+          resolveConsolidationCapabilities(this.config).calibration === true,
         )
       ) {
         recordRecallSectionMetric({
@@ -8451,7 +8433,7 @@ export class Orchestrator {
               query: retrievalQuery,
               maxResults,
               sessionKey,
-              anchorsEnabled: this.config.abstractionAnchorsEnabled,
+              anchorsEnabled: resolveConsolidationCapabilities(this.config).abstractionAnchors,
               abortSignal: harmonicRetrievalAbort.signal,
             }),
           ),
@@ -8498,10 +8480,10 @@ export class Orchestrator {
     const verifiedRecallPromise = (async (): Promise<string | null> => {
       const t0 = Date.now();
       if (
-        !this.config.verifiedRecallEnabled ||
+        !resolveRecallAuxiliaryCapabilities(this.config).verifiedRecall ||
         !this.isRecallSectionEnabled(
           "verified-episodes",
-          this.config.verifiedRecallEnabled === true,
+          resolveRecallAuxiliaryCapabilities(this.config).verifiedRecall === true,
         )
       ) {
         recordRecallSectionMetric({
@@ -8587,10 +8569,10 @@ export class Orchestrator {
     const verifiedRulesPromise = (async (): Promise<string | null> => {
       const t0 = Date.now();
       if (
-        !this.config.semanticRuleVerificationEnabled ||
+        !resolveRecallAuxiliaryCapabilities(this.config).semanticRuleVerification ||
         !this.isRecallSectionEnabled(
           "verified-rules",
-          this.config.semanticRuleVerificationEnabled === true,
+          resolveRecallAuxiliaryCapabilities(this.config).semanticRuleVerification === true,
         )
       ) {
         recordRecallSectionMetric({
@@ -8676,10 +8658,10 @@ export class Orchestrator {
       const t0 = Date.now();
       if (
         !resolveCreationMemoryCapabilities(this.config).creationMemory ||
-        !this.config.workProductRecallEnabled ||
+        !resolveRecallAuxiliaryCapabilities(this.config).workProductRecall ||
         !this.isRecallSectionEnabled(
           "work-products",
-          this.config.workProductRecallEnabled === true,
+          resolveRecallAuxiliaryCapabilities(this.config).workProductRecall === true,
         )
       ) {
         recordRecallSectionMetric({
@@ -9187,7 +9169,7 @@ export class Orchestrator {
     const transcriptPromise = (async (): Promise<string | null> => {
       const t0 = Date.now();
       if (
-        !this.config.transcriptEnabled ||
+        !resolvePresentationCapabilities(this.config).transcript ||
         !this.isRecallSectionEnabled("transcript", true)
       ) {
         recordRecallSectionMetric({
@@ -9293,7 +9275,7 @@ export class Orchestrator {
         this._recallWorkspaceOverrides.get(effectiveSessionKey);
       this._recallWorkspaceOverrides.delete(effectiveSessionKey);
 
-      if (!this.config.compactionResetEnabled) return null;
+      if (!resolveRecallAuxiliaryCapabilities(this.config).compactionReset) return null;
 
       const workspaceDir =
         compactionWorkspaceDir ||
@@ -9646,9 +9628,9 @@ export class Orchestrator {
     const recentBoxesPromise = observeEnrichmentPromise(
       this.isRecallSectionEnabled(
         "memory-boxes",
-        this.config.memoryBoxesEnabled === true,
+        resolvePresentationCapabilities(this.config).memoryBoxes === true,
       ) &&
-        this.config.memoryBoxesEnabled &&
+        resolvePresentationCapabilities(this.config).memoryBoxes &&
         this.config.boxRecallDays > 0
         ? Promise.all(
             profileStorages.map((storage) =>
@@ -12751,7 +12733,7 @@ export class Orchestrator {
   ): Promise<void> {
     const mode = this.config.correctionCaptureMode;
     if (mode === "off") return;
-    if (!this.config.correctionEnabled) return;
+    if (!resolveRecallAuxiliaryCapabilities(this.config).correction) return;
 
     try {
       const corrections = detectPassiveCorrections(
@@ -12771,7 +12753,7 @@ export class Orchestrator {
       const result = await capturePassiveCorrections(
         corrections,
         {
-          correctionEnabled: this.config.correctionEnabled,
+          correctionEnabled: resolveRecallAuxiliaryCapabilities(this.config).correction,
           isLiveSession: opts.isLiveSession,
           bufferKey: opts.bufferKey,
           sessionKey: opts.sessionKey,
@@ -13205,7 +13187,7 @@ export class Orchestrator {
     }
 
     let threadIdForExtraction: string | null = null;
-    if (this.config.threadingEnabled && turns.length > 0) {
+    if (resolvePresentationCapabilities(this.config).threading && turns.length > 0) {
       const lastTurn = turns[turns.length - 1];
       try {
         threadIdForExtraction = await this.threading.processTurn(lastTurn, []);
@@ -13335,7 +13317,7 @@ export class Orchestrator {
     // Topics are derived from the current extraction's facts and entities only —
     // not from readAllMemories() — so box topics accurately reflect the current
     // session window and the call is free of expensive full-corpus I/O.
-    if (this.config.memoryBoxesEnabled && persistedIds.length > 0) {
+    if (resolvePresentationCapabilities(this.config).memoryBoxes && persistedIds.length > 0) {
       const extractionTopics = deriveTopicsFromExtraction(result);
       // Derive episodic metadata from buffer turns (REMem-inspired)
       const firstUserTurn = turns.find((t) => t.role === "user");
@@ -13356,7 +13338,7 @@ export class Orchestrator {
     // Batch-append persisted IDs so non-fact memories (entities/questions) are
     // always attached to the thread. The helper excludes pending_review ids (#1635).
     if (
-      this.config.threadingEnabled &&
+      resolvePresentationCapabilities(this.config).threading &&
       threadIdForExtraction &&
       persistedIds.length > 0
     ) {
@@ -13367,7 +13349,7 @@ export class Orchestrator {
     }
 
     // Thread title update for the already-established thread context.
-    if (this.config.threadingEnabled && threadIdForExtraction) {
+    if (resolvePresentationCapabilities(this.config).threading && threadIdForExtraction) {
       const conversationContent = turns.map((t) => t.content).join(" ");
       try {
         await this.threading.updateThreadTitle(
@@ -15268,7 +15250,7 @@ export class Orchestrator {
 
         if (chunkResult.chunked && chunkResult.chunks.length > 1) {
           // Classify memory kind (v8.0 Phase 2B: HiMem episode/note dual store)
-          const memoryKind = this.config.episodeNoteModeEnabled
+          const memoryKind = resolvePresentationCapabilities(this.config).episodeNoteMode
             ? classifyMemoryKind(fact.content, fact.tags ?? [], writeCategory)
             : undefined;
 
@@ -15525,7 +15507,7 @@ export class Orchestrator {
           }
           try {
             if (
-              this.config.verbatimArtifactsEnabled &&
+              resolvePresentationCapabilities(this.config).verbatimArtifacts &&
               this.config.verbatimArtifactCategories.includes(writeCategory) &&
               fact.confidence >= this.config.verbatimArtifactsMinConfidence &&
               !postWriteGuard
@@ -15622,7 +15604,7 @@ export class Orchestrator {
       const memoryKind =
         writeCategory === "procedure"
           ? undefined
-          : this.config.episodeNoteModeEnabled
+          : resolvePresentationCapabilities(this.config).episodeNoteMode
             ? classifyMemoryKind(fact.content, fact.tags ?? [], writeCategory)
             : undefined;
 
@@ -15822,7 +15804,7 @@ export class Orchestrator {
           }
         }
         if (
-          this.config.verbatimArtifactsEnabled &&
+          resolvePresentationCapabilities(this.config).verbatimArtifacts &&
           this.config.verbatimArtifactCategories.includes(writeCategory) &&
           fact.confidence >= this.config.verbatimArtifactsMinConfidence &&
           !postWriteGuard
@@ -16438,7 +16420,7 @@ export class Orchestrator {
       log.info(`merged ${entitiesMerged} fragmented entity files`);
     }
 
-    if (this.config.entitySummaryEnabled) {
+    if (resolvePresentationCapabilities(this.config).entitySummary) {
       try {
         const synthesized = await this.processEntitySynthesisQueue(
           this.config.defaultNamespace,
@@ -16582,7 +16564,7 @@ export class Orchestrator {
     }
 
     // Semantic consolidation pass — find similar memories, synthesize canonical versions
-    if (this.config.semanticConsolidationEnabled) {
+    if (resolveConsolidationCapabilities(this.config).semanticConsolidation) {
       try {
         const stateFilePath = path.join(
           this.config.memoryDir,
@@ -16946,226 +16928,55 @@ export class Orchestrator {
     }
   }
 
+  // Issue #1526: recall result formatting moved to RecallResultFormatter. Thin
+  // delegation keeps the private API stable for callers + tests.
   private formatQmdResults(
     title: string,
     results: QmdSearchResult[],
     sessionKey?: string,
     trustByPath?: Map<string, TrustStageResultItem> | null,
   ): string {
-    // Issue #1582 — handles are only rendered when a session key is available:
-    // resolution requires the handle history to have been recorded for this
-    // session, which only happens when sessionKey is present. Rendering handles
-    // without recording would show tokens a user can never resolve (cursor
-    // review). The rendering logic itself lives in the pure handles module.
-    const handleByIndex = buildHandleIndexForResults(
-      results,
-      this.config.recallMemoryHandles === true && sessionKey != null,
-    );
-    // Issue #1577 — epistemic hedge. Append a deterministic, component-derived
-    // suffix so the downstream model knows each memory's trust status (the
-    // cheap lever against confident-stale-answer failures). Gated separately
-    // from scoring so rendering can ship after the stage is stable. High-band
-    // and neutral items get no suffix (don't waste tokens on the common case).
-    const renderHedge = this.config.trustScoreEpistemicRendering && trustByPath !== null && trustByPath !== undefined;
-    const hedgeMap = renderHedge ? trustByPath : null;
-    const lines = results.map((r, i) => {
-      const snippet = r.snippet
-        ? r.snippet.slice(0, 500).replace(/\n/g, " ")
-        : "(no preview)";
-      const source = typeof r.line === "number" ? `${r.path}:${r.line}` : r.path;
-      const head = `[${i + 1}] ${source} (score: ${r.score.toFixed(3)})\n${snippet}`;
-      const handle = handleByIndex.get(i);
-      const hedged = head.replace(/\s+$/, "");
-      const withHandle = handle ? `${hedged} ${handle}` : hedged;
-      if (hedgeMap) {
-        const item = hedgeMap.get(r.path);
-        if (item) {
-          const hedge = renderEpistemicHedge(item.trust);
-          if (hedge.length > 0) return `${withHandle} ${hedge}`;
-        }
-      }
-      return withHandle;
-    });
-    return `## ${title}\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatQmdResults(title, results, sessionKey, trustByPath);
   }
 
   private formatObjectiveStateResults(
     results: ObjectiveStateSearchResult[],
   ): string {
-    const lines = results.map(({ snapshot }, index) => {
-      const parts = [
-        snapshot.recordedAt.replace("T", " ").slice(0, 16),
-        `${snapshot.kind}/${snapshot.changeKind}`,
-      ];
-      if (snapshot.outcome) parts.push(snapshot.outcome);
-      const header = `[${index + 1}] ${parts.join(" | ")} | ${snapshot.scope}`;
-      const detailParts = [snapshot.summary];
-      if (snapshot.command) detailParts.push(`command: ${snapshot.command}`);
-      else if (snapshot.toolName)
-        detailParts.push(`tool: ${snapshot.toolName}`);
-      return `${header}\n${detailParts.join(" | ")}`;
-    });
-    return `## Objective State\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatObjectiveStateResults(results);
   }
 
   private formatCausalTrajectoryResults(
     results: CausalTrajectorySearchResult[],
   ): string {
-    const lines = results.map(({ record, matchedFields }, index) => {
-      const header = [
-        `[${index + 1}] ${record.recordedAt.replace("T", " ").slice(0, 16)}`,
-        record.outcomeKind,
-      ].join(" | ");
-      const details = [
-        `goal: ${record.goal}`,
-        `action: ${record.actionSummary}`,
-        `observation: ${record.observationSummary}`,
-        `outcome: ${record.outcomeSummary}`,
-      ];
-      if (record.followUpSummary)
-        details.push(`follow-up: ${record.followUpSummary}`);
-      if (matchedFields.length > 0)
-        details.push(`matched: ${matchedFields.join(", ")}`);
-      return `${header}\n${details.join("\n")}`;
-    });
-
-    return `## Causal Trajectories\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatCausalTrajectoryResults(results);
   }
 
   private formatTrustZoneResults(results: TrustZoneSearchResult[]): string {
-    const lines = results.map(({ record, matchedFields }, index) => {
-      const header = [
-        `[${index + 1}] ${record.recordedAt.replace("T", " ").slice(0, 16)}`,
-        record.zone,
-        record.kind,
-      ].join(" | ");
-      const details = [
-        record.summary,
-        `provenance: ${record.provenance.sourceClass}`,
-      ];
-      if (record.entityRefs && record.entityRefs.length > 0) {
-        details.push(`entities: ${record.entityRefs.join(", ")}`);
-      }
-      if (record.tags && record.tags.length > 0) {
-        details.push(`tags: ${record.tags.join(", ")}`);
-      }
-      if (matchedFields.length > 0) {
-        details.push(`matched: ${matchedFields.join(", ")}`);
-      }
-      return `${header}\n${details.join("\n")}`;
-    });
-
-    return `## Trust Zones\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatTrustZoneResults(results);
   }
 
   private formatHarmonicRetrievalResults(
     results: HarmonicRetrievalResult[],
   ): string {
-    const lines = results.map(
-      (
-        { node, matchedAnchors, matchedFields, nodeScore, anchorScore },
-        index,
-      ) => {
-        const header = [
-          `[${index + 1}] ${node.recordedAt.replace("T", " ").slice(0, 16)}`,
-          `${node.kind}/${node.abstractionLevel}`,
-          node.sessionKey,
-        ].join(" | ");
-        const details = [
-          node.title,
-          node.summary,
-          `scores: node=${nodeScore.toFixed(1)} anchor=${anchorScore.toFixed(1)}`,
-        ];
-        if (matchedAnchors.length > 0) {
-          details.push(
-            `anchors: ${matchedAnchors.map((anchor) => `${anchor.anchorType}:${anchor.anchorValue}`).join("; ")}`,
-          );
-        }
-        if (matchedFields.length > 0) {
-          details.push(`matched: ${matchedFields.join(", ")}`);
-        }
-        return `${header}\n${details.join("\n")}`;
-      },
-    );
-
-    return `## Harmonic Retrieval\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatHarmonicRetrievalResults(results);
   }
 
   private formatWorkProductResults(
     results: WorkProductLedgerSearchResult[],
   ): string {
-    const lines = results.map(({ entry, matchedFields }, index) => {
-      const header = [
-        `[${index + 1}] ${entry.recordedAt.replace("T", " ").slice(0, 16)}`,
-        `${entry.kind}/${entry.action}`,
-        entry.sessionKey,
-      ].join(" | ");
-      const details = [entry.summary, `scope: ${entry.scope}`];
-      if (entry.artifactPath) details.push(`artifact: ${entry.artifactPath}`);
-      if (entry.tags && entry.tags.length > 0)
-        details.push(`tags: ${entry.tags.join(", ")}`);
-      if (matchedFields.length > 0)
-        details.push(`matched: ${matchedFields.join(", ")}`);
-      return `${header}\n${details.join("\n")}`;
-    });
-
-    return `## Work Products\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatWorkProductResults(results);
   }
 
   private formatVerifiedEpisodeResults(
     results: VerifiedEpisodeResult[],
   ): string {
-    const lines = results.map(
-      ({ box, verifiedEpisodeCount, matchedFields }, index) => {
-        const header = [
-          `[${index + 1}] ${box.sealedAt.replace("T", " ").slice(0, 16)}`,
-          box.traceId ? `trace:${box.traceId.slice(0, 12)}` : "trace:none",
-        ].join(" | ");
-        const details = [
-          box.goal ?? `topics: ${box.topics.join(", ")}`,
-          `verified episodes: ${verifiedEpisodeCount}`,
-        ];
-        if (box.toolsUsed && box.toolsUsed.length > 0) {
-          details.push(`tools: ${box.toolsUsed.join(", ")}`);
-        }
-        if (matchedFields.length > 0) {
-          details.push(`matched: ${matchedFields.join(", ")}`);
-        }
-        return `${header}\n${details.join("\n")}`;
-      },
-    );
-
-    return `## Verified Episodes\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatVerifiedEpisodeResults(results);
   }
 
   private formatVerifiedSemanticRuleResults(
     results: VerifiedSemanticRuleResult[],
   ): string {
-    const lines = results.map(
-      (
-        {
-          rule,
-          sourceMemoryId,
-          verificationStatus,
-          effectiveConfidence,
-          matchedFields,
-        },
-        index,
-      ) => {
-        const header = [
-          `[${index + 1}] ${rule.frontmatter.updated.replace("T", " ").slice(0, 16)}`,
-          verificationStatus,
-          `confidence:${effectiveConfidence.toFixed(2)}`,
-        ].join(" | ");
-        const details = [rule.content, `source memory: ${sourceMemoryId}`];
-        if (matchedFields.length > 0) {
-          details.push(`matched: ${matchedFields.join(", ")}`);
-        }
-        return `${header}\n${details.join("\n")}`;
-      },
-    );
-
-    return `## Verified Rules\n\n${lines.join("\n\n")}`;
+    return this.recallResultFormatter.formatVerifiedSemanticRuleResults(results);
   }
 
   private summarizeIdentityText(
@@ -17173,42 +16984,21 @@ export class Orchestrator {
     maxLines: number,
     maxChars: number,
   ): string {
-    const lines = raw
-      .replace(/\r/g, "")
-      .split("\n")
-      .map((line) => line.trim())
-      .filter((line) => line.length > 0 && !line.startsWith("#"));
-    const compact = lines.slice(0, Math.max(1, maxLines)).join(" ");
-    if (compact.length <= maxChars) return compact;
-    return `${compact.slice(0, Math.max(0, maxChars - 1))}…`;
+    return this.recallResultFormatter.summarizeIdentityText(raw, maxLines, maxChars);
   }
 
   private formatOpenIncidentLine(
     incident: ContinuityIncidentRecord,
     includeDetails: boolean,
   ): string {
-    const base = `[${incident.id}] ${incident.symptom.trim()}`;
-    if (!includeDetails) return `- ${base}`;
-    const parts = [base];
-    if (incident.suspectedCause)
-      parts.push(`cause: ${incident.suspectedCause.trim()}`);
-    if (incident.triggerWindow)
-      parts.push(`window: ${incident.triggerWindow.trim()}`);
-    return `- ${parts.join(" | ")}`;
+    return this.recallResultFormatter.formatOpenIncidentLine(incident, includeDetails);
   }
 
   private trimIdentitySection(
     content: string,
     maxChars: number,
   ): { text: string; truncated: boolean } {
-    if (maxChars <= 0) return { text: "", truncated: false };
-    if (content.length <= maxChars) return { text: content, truncated: false };
-    const suffix = "\n\n...(identity continuity trimmed)";
-    if (maxChars <= suffix.length) {
-      return { text: content.slice(0, maxChars), truncated: true };
-    }
-    const headroom = Math.max(0, maxChars - suffix.length);
-    return { text: `${content.slice(0, headroom)}${suffix}`, truncated: true };
+    return this.recallResultFormatter.trimIdentitySection(content, maxChars);
   }
 
   private async buildIdentityContinuitySection(options: {
@@ -17221,81 +17011,7 @@ export class Orchestrator {
     injectedChars: number;
     truncated: boolean;
   } | null> {
-    if (!resolveIdentityContinuityCapabilities(this.config).identityContinuity) return null;
-    if (this.config.identityMaxInjectChars <= 0) return null;
-
-    const resolved = resolveEffectiveIdentityInjectionMode({
-      configuredMode: this.config.identityInjectionMode,
-      recallMode: options.recallMode,
-      prompt: options.prompt,
-    });
-    if (!resolved.shouldInject) return null;
-
-    const [anchorRaw, loopsRaw, incidents] = await Promise.all([
-      options.storage.readIdentityAnchor(),
-      options.storage.readIdentityImprovementLoops(),
-      options.storage.readContinuityIncidents(200),
-    ]);
-    const openIncidents = incidents.filter(
-      (incident) => incident.state === "open",
-    );
-
-    const lines: string[] = [];
-    if (resolved.mode === "full") {
-      lines.push("## Identity Continuity");
-      if (anchorRaw && anchorRaw.trim().length > 0) {
-        lines.push("", "### Anchor", "", anchorRaw.trim());
-      }
-      if (loopsRaw && loopsRaw.trim().length > 0) {
-        lines.push("", "### Improvement Loops", "", loopsRaw.trim());
-      }
-      lines.push("", "### Open Incidents", "");
-      if (openIncidents.length === 0) {
-        lines.push("- none");
-      } else {
-        lines.push(
-          ...openIncidents
-            .slice(0, 5)
-            .map((incident) => this.formatOpenIncidentLine(incident, true)),
-        );
-      }
-    } else {
-      const anchorSummary = anchorRaw
-        ? this.summarizeIdentityText(anchorRaw, 3, 320)
-        : "";
-      const loopsSummary = loopsRaw
-        ? this.summarizeIdentityText(loopsRaw, 2, 240)
-        : "";
-      lines.push("## Identity Continuity Signals", "");
-      if (anchorSummary) lines.push(`- anchor: ${anchorSummary}`);
-      if (loopsSummary) lines.push(`- loops: ${loopsSummary}`);
-      if (openIncidents.length === 0) {
-        lines.push("- incidents: 0 open");
-      } else {
-        lines.push(`- incidents: ${openIncidents.length} open`);
-        lines.push(
-          ...openIncidents
-            .slice(0, 2)
-            .map((incident) => this.formatOpenIncidentLine(incident, false)),
-        );
-      }
-    }
-
-    const body = lines.join("\n").trim();
-    if (!body) return null;
-
-    const { text, truncated } = this.trimIdentitySection(
-      body,
-      this.config.identityMaxInjectChars,
-    );
-    if (!text) return null;
-
-    return {
-      section: text,
-      mode: resolved.mode,
-      injectedChars: text.length,
-      truncated,
-    };
+    return this.recallResultFormatter.buildIdentityContinuitySection(options);
   }
 
   private emitTrace(event: EngramTraceEvent): void {