mixdog 0.9.153 → 0.9.154

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixdog",
-  "version": "0.9.153",
+  "version": "0.9.154",
   "private": false,
   "type": "module",
   "description": "Standalone mixdog coding-agent CLI/TUI workspace.",
@@ -31,6 +31,7 @@
     "!scripts/bench/",
     "!scripts/recall-bench-*.txt",
     "!scripts/*-test.mjs",
+    "!scripts/*.test.mjs",
     "!scripts/*-smoke.mjs",
     "!scripts/smoke-loop*.mjs",
     "!scripts/*-bench.mjs",
@@ -40,6 +41,8 @@
     "!scripts/*.jsx",
     "src/",
     "!src/tui/dev",
+    "!src/**/*.test.mjs",
+    "!src/**/*.test.jsx",
     "vendor/"
   ],
   "scripts": {
@@ -68,6 +71,7 @@
     "test:spec-advisory": "node scripts/advisory-spec-test.mjs && npm run test:spec-advisory --prefix apps/desktop",
     "test:session-transport": "node --test scripts/session-transport-test.mjs scripts/daemon-bootstrap-test.mjs src/runtime/shared/child-spawn-remote.test.mjs src/standalone/session-runtime-agent-control-client.test.mjs src/standalone/session-runtime-dispatch-cancel.test.mjs src/standalone/session-runtime-host-factory.test.mjs src/standalone/session-runtime-inline-host.test.mjs src/standalone/session-runtime-provider-cooldown.test.mjs src/standalone/session-runtime-shard-host.test.mjs src/standalone/session-runtime-shard-router.test.mjs",
     "test:session": "node --test scripts/runtime-turn-contract-test.mjs scripts/session-save-fault-store-test.mjs scripts/turn-review-revert-test.mjs src/runtime/shared/tool-surface.test.mjs src/runtime/bridge-clients.test.mjs",
+    "test:quick": "npm run test:session && npm run test:session-transport && npm run test:desktop-main --prefix apps/desktop",
     "test:office": "node --test src/runtime/office/office-runtime.test.mjs src/runtime/office/office-assurance.test.mjs && npm run test:office-approval --prefix apps/desktop",
     "test:office:live": "node scripts/run-office-live-tests.mjs",
     "test:media": "node --test src/runtime/media/store.test.mjs src/runtime/media/renditions.test.mjs src/runtime/media/adapters/codex-image.test.mjs",

package/src/rules/shared/60-verification.md

@@ -9,6 +9,8 @@
   buildability invariants. Treat mutable behavior, UX, exact text, snapshots,
   and implementation shape as advisory specifications; update them when the
   requested behavior changes instead of preserving obsolete behavior.
+- A new test asserts observable behavior, never source text or implementation
+  shape, and never re-asserts a contract another test already owns.
 - A check runs at the strictness the task requires; never raise a tool's own
   severity beyond it.
 - If verification fails, collect all failures, leave Verification, complete all

package/src/runtime/agent/orchestrator/session/agent-loop.mjs

@@ -27,7 +27,6 @@ import {
     _intraTurnSig,
 } from './loop/tool-classify.mjs';
 import { preDispatchDenyForSession } from './loop/pre-dispatch-deny.mjs';
-import { runRecallFastTrackCompact } from './loop/recall-fasttrack.mjs';
 import { executeTool, _scopedCacheOutcomeForCall, resolveLiveToolCwd } from './loop/tool-exec.mjs';
 
 // classifyResultKind is imported from result-classification.mjs at the top of

package/src/runtime/agent/orchestrator/session/compact/handoff.mjs

@@ -239,28 +239,6 @@ export function collectWorkingFileGroups(messages, cap = Number.POSITIVE_INFINIT
     }, prior, limit, cwd);
 }
 
-export function collectWorkingFiles(messages, cap = Number.POSITIVE_INFINITY, options = {}) {
-    const groups = collectWorkingFileGroups(messages, cap, options);
-    return [...groups.modified, ...groups.referenced].map((entry) => entry.path);
-}
-
-export function stripWorkingFileSections(text) {
-    const out = [];
-    let skipping = false;
-    for (const raw of String(text || '').split('\n')) {
-        const line = raw.trim();
-        if (line === '## Working files') {
-            skipping = true;
-            continue;
-        }
-        if (skipping && (/^##\s+/.test(line) || /^<\/?prior-compacted-context>$/.test(line))) {
-            skipping = false;
-        }
-        if (!skipping) out.push(raw);
-    }
-    return out.join('\n').replace(/\n{3,}/g, '\n\n');
-}
-
 function indexToolResults(messages) {
     const map = new Map();
     for (const m of messages || []) {

package/src/runtime/agent/orchestrator/session/compact/messages.mjs

@@ -43,8 +43,8 @@ export function isProtectedContextUserMessage(m) {
 // content prefix (the `<skill>` envelope from buildSkillResultEnvelope) so the
 // check survives even if the synthetic `meta` field is dropped during a tail
 // rebuild. It is NOT the human's latest prompt and must be excluded from
-// "latest human request" selection (deriveCurrentRequest /
-// buildRecallFastTrackQuery). The `meta:'skill'` marker is also honoured.
+// "latest human request" selection (deriveCurrentRequest). The `meta:'skill'`
+// marker is also honoured.
 export function isInjectedSkillBodyMessage(m) {
     if (m?.role !== 'user') return false;
     if (m.meta === 'skill') return true;

package/src/runtime/agent/orchestrator/session/compact/runner.mjs

@@ -717,13 +717,6 @@ function truncateTailToCap(messages, cap) {
     return estimateMessagesTokens(sanitized) <= cap ? sanitized : [];
 }
 
-function splitRecallFitInputs(recallText) {
-    return {
-        recall: String(recallText || '').trim(),
-        prior: '',
-    };
-}
-
 function recallTailStartIndex(live, tail) {
     if (!tail.length) return live.length;
     const first = tail[0];
@@ -837,9 +830,9 @@ function _recallFastTrackCompactMessages(messages, budgetTokens, opts = {}) {
         head: recallHead,
         lastTurnStartIdx = recallHead.length,
     } = selectRecallPreservedTail(live, recallTailOpts);
-    const recallFit = splitRecallFitInputs(opts.recallText);
+    const recallText = String(opts.recallText || '').trim();
     if (recallHead.length === 0
-        && !(recallFit.recall || recallFit.prior || opts.allowEmptyRecall === true)) {
+        && !(recallText || opts.allowEmptyRecall === true)) {
         throw new Error('recallFastTrackCompactMessages: no compactable prior history before preserved tail');
     }
 
@@ -851,7 +844,7 @@ function _recallFastTrackCompactMessages(messages, budgetTokens, opts = {}) {
     }
     const budgetRaisedBy = Math.max(0, budget - originalBudget);
 
-    if (!recallFit.recall && !recallFit.prior && opts.allowEmptyRecall !== true) {
+    if (!recallText && opts.allowEmptyRecall !== true) {
         throw new Error('recallFastTrackCompactMessages: recall text is empty');
     }
     const oldHistory = recallHead;
@@ -871,14 +864,14 @@ function _recallFastTrackCompactMessages(messages, budgetTokens, opts = {}) {
         ? Math.min(recallRoomUncapped, Math.max(512, recallTokenCap - tailTokens))
         : recallRoomUncapped;
     const toolLines = collectToolOutcomeLines(recallHead)
-        .filter((line) => !recallFit.recall.includes(String(line || '').trim()));
+        .filter((line) => !recallText.includes(String(line || '').trim()));
     const workingFiles = collectWorkingFileGroups(recallHead, undefined, {
         cwd: opts.cwd,
         previousSummary: '',
         now: Date.now(),
     });
     const conversationLines = excludeTailFromConversation(
-        conversationLinesFromMemoryText(recallFit.recall),
+        conversationLinesFromMemoryText(recallText),
         recallTail,
     );
     const composedRecall = composeRecallHandoff({
@@ -894,7 +887,6 @@ function _recallFastTrackCompactMessages(messages, budgetTokens, opts = {}) {
         fittedRecall,
         recallRoom,
         recallMeta,
-        '',
     );
     if (!summaryMessage) {
         throw new Error(`recallFastTrackCompactMessages: summary cannot fit remaining budget=${recallRoom}`);
@@ -936,20 +928,15 @@ function _recallFastTrackCompactMessages(messages, budgetTokens, opts = {}) {
         remainingTokens: budget - mandatoryCost,
         recallTokenCap: (Number.isFinite(recallTokenCap) && recallTokenCap > 0) ? recallTokenCap : null,
         recallRoom,
-        recallChars: recallFit.recall.length,
-        recallBytes: textByteLength(recallFit.recall),
-        priorChars: 0,
-        priorBytes: 0,
+        recallChars: recallText.length,
+        recallBytes: textByteLength(recallText),
         summaryMessageChars: summaryContent.length,
         summaryMessageBytes: textByteLength(summaryContent),
-        recallEmpty: !recallFit.recall,
-        priorEmpty: !recallFit.prior,
-        recallTruncatedInSummary: !!recallFit.recall && !summaryContent.includes(recallFit.recall),
-        priorTruncatedInSummary: false,
+        recallEmpty: !recallText,
+        recallTruncatedInSummary: !!recallText && !summaryContent.includes(recallText),
         tailTruncated: recallTail.some((m) => messageContentHasMarker(m, RECALL_TAIL_TRUNCATION_MARKER) || messageContentHasMarker(m, RECALL_TAIL_SHORT_TRUNCATION_MARKER)),
         fileReattached: reattach.reattached,
         tailOptions: recallTailOpts,
-        previousSummary: false,
         durationMs: Date.now() - startedAt,
     };
     compactDebugLog('recall-fasttrack result', diagnostics);

package/src/runtime/agent/orchestrator/session/compact/summary.mjs

@@ -346,28 +346,6 @@ function makeSemanticSummaryMessage(oldHistory, summary, semanticMeta = {}, pres
     return makeSummaryMessage(parts.join('\n\n'));
 }
 
-export function buildRecallFastTrackQuery(messages, opts = {}) {
-    const maxChars = Math.max(200, Number(opts.maxChars) || 2_000);
-    const hints = String(opts.hints || 'current task decisions constraints file paths changed files verification failures next steps').trim();
-    let latestUser = '';
-    const recent = [];
-    const input = Array.isArray(messages) ? messages : [];
-    for (let i = input.length - 1; i >= 0; i -= 1) {
-        const m = input[i];
-        const text = extractText(m).trim();
-        if (!text) continue;
-        if (recent.length < 6) recent.unshift(text);
-        if (!latestUser && m?.role === 'user' && !isProtectedContextUserMessage(m) && !isInjectedSkillBodyMessage(m)) {
-            latestUser = text;
-        }
-        if (latestUser && recent.length >= 6) break;
-    }
-    const parts = [latestUser, hints, recent.join('\n')]
-        .map((s) => String(s || '').trim())
-        .filter(Boolean);
-    return truncateMiddle([...new Set(parts)].join('\n'), maxChars);
-}
-
 // Fit the structured semantic summary into the remaining token budget WITHOUT
 // dropping any required section. The incoming `summary` is already schema-valid
 // (enforceSemanticSummarySchema ran upstream); here we shrink section bodies via
@@ -410,106 +388,15 @@ export function fitSemanticSummaryMessage(oldHistory, summary, remainingTokens,
 export const RECALL_TAIL_TRUNCATION_MARKER = '[... truncated during recall tail preservation ...]';
 export const RECALL_TAIL_SHORT_TRUNCATION_MARKER = '[truncated]';
 
-const PRIOR_COMPACTED_CONTEXT_OPEN = '<prior-compacted-context>';
-const PRIOR_COMPACTED_CONTEXT_CLOSE = '</prior-compacted-context>';
-// Matches ONLY a structural wrapper BOUNDARY line — a line whose sole content is
-// the open or close tag (optionally surrounded by whitespace). It deliberately
-// does NOT match an inline occurrence embedded in real content (e.g. a user note
-// like "keep <prior-compacted-context> literal here"), so flattening the wrapper
-// can never splice words together or corrupt marker-like text. The production
-// wrapper is always emitted on its own line, so boundary-line stripping removes
-// every real wrapper while leaving inline literals verbatim.
-const PRIOR_COMPACTED_CONTEXT_BOUNDARY_RE = /^[ \t]*<\/?prior-compacted-context>[ \t]*$/;
-
-// Strip the STRUCTURAL <prior-compacted-context> / </prior-compacted-context>
-// boundary lines from prior text so re-wrapping never nests, while preserving
-// any inline marker-like text inside real content exactly as written. Returns
-// the bare inner content with the blank lines the removed boundary lines leave
-// behind collapsed.
-export function stripPriorCompactedContextWrappers(text) {
-    const raw = String(text ?? '');
-    if (!raw) return '';
-    // Keep every non-structural byte, including leading/trailing and repeated
-    // newlines.  Removing only the structural line itself (not trim/collapse)
-    // preserves the layout of all remaining content.
-    return raw
-        .split('\n')
-        .filter((line) => !PRIOR_COMPACTED_CONTEXT_BOUNDARY_RE.test(line))
-        .join('\n');
-}
-
-// Remove only STRUCTURALLY IDENTICAL blank-line-separated blocks (byte-for-byte
-// repeats) so a prior context re-fed across many compaction cycles keeps every
-// distinct requirement/fact and cannot grow without bound. The dedupe is
-// content-preserving: the key is the block's EXACT text — internal whitespace is
-// never collapsed and meaningful block content is never trimmed — so distinct
-// strings such as `printf 'a  b'` and `printf 'a b'` are BOTH kept verbatim.
-// Only an exact repeat of a previously emitted block is dropped; blank
-// separators (whitespace-only splits) are skipped, never real content.
-function dedupePriorCompactedBlocks(text) {
-    const raw = String(text ?? '');
-    if (!raw.trim()) return '';
-    const seen = new Set();
-    const parts = raw.split(/(\n{2,})/);
-    const out = [];
-    let separator = '';
-    for (let i = 0; i < parts.length; i += 1) {
-        const part = parts[i];
-        if (i % 2 === 1) {
-            separator += part;
-            continue;
-        }
-        // Whitespace-only parts are layout, not a duplicate candidate.
-        if (!part.trim()) {
-            out.push(separator, part);
-            separator = '';
-            continue;
-        }
-        if (seen.has(part)) {
-            // Drop only the exact repeated structural block and its preceding
-            // separator; all retained text and whitespace stay byte-for-byte.
-            separator = '';
-            continue;
-        }
-        out.push(separator, part);
-        separator = '';
-        seen.add(part);
-    }
-    out.push(separator);
-    return out.join('');
-}
-
-// Canonicalize prior compacted context to AT MOST ONE wrapper: flatten any
-// nested/duplicated wrappers accumulated by earlier cycles, dedupe repeated
-// blocks, then wrap the surviving content exactly once. Repeated compaction can
-// therefore never nest or duplicate the prior context, each distinct
-// requirement/fact is preserved exactly once, and repeated-cycle token size
-// stays bounded.
-//
-// Optimization-safe empty-prior interpretation: when there is NO prior content
-// (empty / blank / boundary-tag-only input) this returns '' so the generated
-// summary carries ZERO wrappers instead of an empty
-// <prior-compacted-context></prior-compacted-context> pair. The production
-// summary body joins only non-empty parts (makeRecall*SummaryMessageParts), so
-// an empty wrapper cannot be carried and would only waste tokens. "Exactly one
-// wrapper" is thus realized as: exactly one when prior content exists, none when
-// it does not — and never more than one, never nested.
-export function formatPriorCompactedContextBlock(priorText) {
-    const flattened = dedupePriorCompactedBlocks(stripPriorCompactedContextWrappers(priorText));
-    if (!flattened) return '';
-    return `${PRIOR_COMPACTED_CONTEXT_OPEN}\n${flattened}\n${PRIOR_COMPACTED_CONTEXT_CLOSE}`;
-}
-
+// Peel the structural summary header off a prior summary body so semantic
+// compaction can feed bare text into its <previous-summary> block. Sessions
+// stored before recall-fasttrack stopped chaining summaries may still carry a
+// <prior-compacted-context> wrapper, so those boundary lines are dropped too.
 export function stripNestedSummaryHeaderLines(text) {
     const raw = String(text ?? '');
-    // A generated recall summary has a canonical, provenance-bearing shape:
-    //   header + "\n\n" + OPEN + "\n" + prior + "\n" + CLOSE + "\n\n" + recall
-    // Extracting it as lines loses ownership of a run of newlines: in
-    // `X\n` + join + live `X`, generic block dedupe cannot know that one
-    // newline belongs to X and must survive dropping the duplicate live X.
-    // Peel only the emitted wrapper/header/join bytes and retain the inner
-    // prior slice verbatim. The live recall is dropped only when it is the
-    // same payload the wrapper already carries.
+    // Peel only the emitted wrapper/header/join bytes and keep the inner slice
+    // verbatim: splitting into lines would lose ownership of a run of newlines,
+    // where one newline belongs to the content and must survive.
     const openMatch = /^[ \t]*<prior-compacted-context>[ \t]*\n/m.exec(raw);
     if (openMatch) {
         const closeRe = /\n[ \t]*<\/prior-compacted-context>[ \t]*(?=\n|$)/g;
@@ -544,10 +431,9 @@ export function stripNestedSummaryHeaderLines(text) {
             followsStructuralHeader = true;
             continue;
         }
-        // Summary parts are joined with "\n\n".  The first empty line after
+        // Summary parts are joined with "\n\n". The first empty line after
         // stripped summary metadata is that join's structural separator, not
-        // prior content; retaining it makes every refeed gain a newline when
-        // formatPriorCompactedContextBlock wraps the result again.
+        // content.
         if (followsStructuralHeader && line === '') {
             followsStructuralHeader = false;
             continue;
@@ -570,46 +456,19 @@ export function stripNestedSummaryHeaderLines(text) {
     return out.join('\n');
 }
 
-function makeRecallFastTrackSummaryMessageParts(oldHistory, recallPart, priorPart, recallMeta = {}) {
+function makeRecallFastTrackSummaryMessageParts(oldHistory, recallPart, recallMeta = {}) {
     const header = compactHeader(oldHistory);
     header.push(`compact_type=${COMPACT_TYPE_RECALL_FASTTRACK} source=recall-fasttrack query_sha=${recallMeta.querySha || 'none'}`);
     const parts = [header.join('\n')];
-    const priorBlock = formatPriorCompactedContextBlock(priorPart);
-    if (priorBlock) parts.push(priorBlock);
     const recall = String(recallPart || '').trim();
     if (recall) parts.push(recall);
     return makeSummaryMessage(parts.join('\n\n'));
 }
 
-export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remainingTokens, recallMeta = {}, priorPart = '') {
+export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remainingTokens, recallMeta = {}) {
     const recall = String(recallText || '').trim();
-    const prior = String(priorPart || '');
-
-    let fittedPrior = prior;
-    if (prior) {
-        let lo = 0;
-        let hi = prior.length;
-        let bestPriorLen = 0;
-        while (lo <= hi) {
-            const mid = Math.floor((lo + hi) / 2);
-            const candidate = makeRecallFastTrackSummaryMessageParts(oldHistory, '', prior.slice(0, mid), recallMeta);
-            if (estimateMessagesTokens([candidate]) <= remainingTokens) {
-                bestPriorLen = mid;
-                lo = mid + 1;
-            } else {
-                hi = mid - 1;
-            }
-        }
-        fittedPrior = prior.slice(0, bestPriorLen);
-        if (!fittedPrior && prior) {
-            const markerOnly = makeRecallFastTrackSummaryMessageParts(oldHistory, '', RECALL_TAIL_TRUNCATION_MARKER, recallMeta);
-            if (estimateMessagesTokens([markerOnly]) <= remainingTokens) {
-                fittedPrior = RECALL_TAIL_TRUNCATION_MARKER;
-            }
-        }
-    }
 
-    const minimal = makeRecallFastTrackSummaryMessageParts(oldHistory, '', fittedPrior, recallMeta);
+    const minimal = makeRecallFastTrackSummaryMessageParts(oldHistory, '', recallMeta);
     if (estimateMessagesTokens([minimal]) > remainingTokens) return null;
     if (!recall) return minimal;
 
@@ -618,14 +477,14 @@ export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remaini
         // Root-block granularity fit: drop the OLDEST blocks WHOLE (never cut
         // a `# chunk` / `# raw_pending` / `# raw_terminal` block mid-entry);
         // dropping more leading blocks only shrinks the body, so binary-search
-        // the minimal drop count (mirrors fitRecallRootsMessage).
+        // the minimal drop count.
         let loB = 0;
         let hiB = blocks.length;
         let bestLo = -1;
         while (loB <= hiB) {
             const midB = Math.floor((loB + hiB) / 2);
             const body = [preamble, ...blocks.slice(midB)].filter(Boolean).join('\n\n');
-            const candidate = makeRecallFastTrackSummaryMessageParts(oldHistory, body, fittedPrior, recallMeta);
+            const candidate = makeRecallFastTrackSummaryMessageParts(oldHistory, body, recallMeta);
             if (estimateMessagesTokens([candidate]) <= remainingTokens) {
                 bestLo = midB;
                 hiB = midB - 1;
@@ -635,12 +494,11 @@ export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remaini
         }
         if (bestLo >= 0) {
             const body = [preamble, ...blocks.slice(bestLo)].filter(Boolean).join('\n\n');
-            return makeRecallFastTrackSummaryMessageParts(oldHistory, body, fittedPrior, recallMeta);
+            return makeRecallFastTrackSummaryMessageParts(oldHistory, body, recallMeta);
         }
-        // Even zero blocks (preamble alone) overflows alongside the fitted
-        // prior - try preamble alone, else the no-recall minimal message.
+        // Even the preamble alone overflows: try it, else the minimal message.
         if (preamble) {
-            const preambleOnly = makeRecallFastTrackSummaryMessageParts(oldHistory, preamble, fittedPrior, recallMeta);
+            const preambleOnly = makeRecallFastTrackSummaryMessageParts(oldHistory, preamble, recallMeta);
             if (estimateMessagesTokens([preambleOnly]) <= remainingTokens) return preambleOnly;
         }
         return minimal;
@@ -654,7 +512,7 @@ export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remaini
     let best = minimal;
     while (lo <= hi) {
         const mid = Math.floor((lo + hi) / 2);
-        const candidate = makeRecallFastTrackSummaryMessageParts(oldHistory, recall.slice(0, mid), fittedPrior, recallMeta);
+        const candidate = makeRecallFastTrackSummaryMessageParts(oldHistory, recall.slice(0, mid), recallMeta);
         if (estimateMessagesTokens([candidate]) <= remainingTokens) {
             best = candidate;
             lo = mid + 1;
@@ -665,28 +523,23 @@ export function fitRecallFastTrackSummaryMessage(oldHistory, recallText, remaini
     return best;
 }
 
-// --- Smart-compact root-based fitting (arrival-time replacement) -----------
+// --- Root-block splitting for the recall digest ----------------------------
 //
-// dump_session_roots (memory/index.mjs dumpSessionRootChunks) renders chunks
-// TIME-ORDERED (oldest first; chunks.sort by sourceTurn/ts/id ascending),
-// each root/raw block starting with one of:
+// A memory digest renders chunks TIME-ORDERED (oldest first), each root/raw
+// block starting with one of:
 //   # chunk N root=ID[ category=X]
 //   # raw_pending N id=ID
 //   # raw_terminal N id=ID
-// and blocks joined by "\n\n". runRecallFastTrackForSession additionally
-// prepends a "session_id=..." / cycle1-drain-status preamble before the dump
-// text (also "\n\n"-joined) — preserved verbatim as a non-block segment.
+// and blocks joined by "\n\n", after an optional preamble that is kept verbatim
+// as a non-block segment.
 //
-// Unlike fitRecallFastTrackSummaryMessage (character-slice binary search,
-// used by the LLM-summary-free but still-mid-turn recall-fasttrack compact
-// path), the smart-compact arrival path must never cut a root block
-// mid-entry — losing half a root's content silently corrupts that entry.
-// This splitter finds block boundaries by the label pattern (robust to
-// blank lines inside member/raw content, since it anchors on the distinctive
-// "# chunk /raw_pending/raw_terminal" line rather than a blank-line split).
+// fitRecallFastTrackSummaryMessage drops whole blocks instead of cutting one
+// mid-entry, since losing half a root's content silently corrupts that entry.
+// Boundaries come from the label pattern, which stays robust to blank lines
+// inside member/raw content.
 const RECALL_ROOT_BLOCK_HEADER_RE = /^# (?:chunk \d+ root=\d+(?: category=\S+)?|raw_pending \d+ id=\d+|raw_terminal \d+ id=\d+)[ \t]*$/;
 
-export function splitRecallRootBlocks(text) {
+function splitRecallRootBlocks(text) {
     const value = String(text || '');
     if (!value.trim()) return { preamble: '', blocks: [] };
     const re = new RegExp(RECALL_ROOT_BLOCK_HEADER_RE.source, 'gm');
@@ -708,97 +561,3 @@ export function splitRecallRootBlocks(text) {
     return { preamble, blocks };
 }
 
-// Minimal-header summary-message wrapper for the smart-compact roots path.
-// Keeps the SUMMARY_PREFIX anchor (isSummaryMessage / selectCompactionWindow
-// / clear-preserve / TUI all key off startsWith(SUMMARY_PREFIX)) but skips the
-// full sha256/roleCounts header line that fitRecallFastTrackSummaryMessage
-// computes — smart-arrival replacement is a lightweight prefix swap, not the
-// anchored LLM-summary compact, so a heavy per-call header is unneeded cost.
-function makeRecallRootsSummaryMessageParts(oldHistory, rootsPart, priorPart, recallMeta = {}) {
-    const header = `${SUMMARY_PREFIX}\nmessages=${(oldHistory || []).length} compact_type=${COMPACT_TYPE_RECALL_FASTTRACK} source=smart-arrival query_sha=${recallMeta.querySha || 'none'}`;
-    const parts = [header];
-    const priorBlock = formatPriorCompactedContextBlock(priorPart);
-    if (priorBlock) parts.push(priorBlock);
-    const roots = String(rootsPart || '').trim();
-    if (roots) parts.push(roots);
-    return makeSummaryMessage(parts.join('\n\n'));
-}
-
-// Root-block-aware fit for the smart-compact arrival path (Step1). Mirrors
-// fitRecallFastTrackSummaryMessage's prior-block binary-search fit, but the
-// recall body is fit at ROOT-BLOCK granularity: when the full set of root
-// blocks (kept in original time order) exceeds remainingTokens, the OLDEST
-// blocks are dropped WHOLE (never character-truncated mid-block) until the
-// remaining (newest-biased) suffix fits. Because dropping more leading
-// blocks can only shrink (never grow) the serialized size, the minimal-drop
-// threshold is found via binary search on the drop count.
-export function fitRecallRootsMessage(oldHistory, recallText, remainingTokens, recallMeta = {}, priorPart = '') {
-    const prior = String(priorPart || '');
-
-    let fittedPrior = prior;
-    if (prior) {
-        let lo = 0;
-        let hi = prior.length;
-        let bestPriorLen = 0;
-        while (lo <= hi) {
-            const mid = Math.floor((lo + hi) / 2);
-            const candidate = makeRecallRootsSummaryMessageParts(oldHistory, '', prior.slice(0, mid), recallMeta);
-            if (estimateMessagesTokens([candidate]) <= remainingTokens) {
-                bestPriorLen = mid;
-                lo = mid + 1;
-            } else {
-                hi = mid - 1;
-            }
-        }
-        fittedPrior = prior.slice(0, bestPriorLen);
-        if (!fittedPrior && prior) {
-            const markerOnly = makeRecallRootsSummaryMessageParts(oldHistory, '', RECALL_TAIL_TRUNCATION_MARKER, recallMeta);
-            if (estimateMessagesTokens([markerOnly]) <= remainingTokens) {
-                fittedPrior = RECALL_TAIL_TRUNCATION_MARKER;
-            }
-        }
-    }
-
-    const minimal = makeRecallRootsSummaryMessageParts(oldHistory, '', fittedPrior, recallMeta);
-    if (estimateMessagesTokens([minimal]) > remainingTokens) return null;
-
-    const { preamble, blocks } = splitRecallRootBlocks(recallText);
-    if (blocks.length === 0) {
-        // No parseable root-block boundaries (empty / non-dump recallText) —
-        // degrade to keep-whole-if-it-fits, else drop entirely. Never
-        // mid-truncates: a non-block body is treated as a single atomic unit.
-        const whole = String(recallText || '').trim();
-        if (!whole) return minimal;
-        const full = makeRecallRootsSummaryMessageParts(oldHistory, whole, fittedPrior, recallMeta);
-        if (estimateMessagesTokens([full]) <= remainingTokens) return full;
-        return minimal;
-    }
-
-    let loB = 0;
-    let hiB = blocks.length;
-    let bestLo = -1;
-    while (loB <= hiB) {
-        const mid = Math.floor((loB + hiB) / 2);
-        const kept = blocks.slice(mid);
-        const body = [preamble, ...kept].filter(Boolean).join('\n\n');
-        const candidate = makeRecallRootsSummaryMessageParts(oldHistory, body, fittedPrior, recallMeta);
-        if (estimateMessagesTokens([candidate]) <= remainingTokens) {
-            bestLo = mid;
-            hiB = mid - 1;
-        } else {
-            loB = mid + 1;
-        }
-    }
-    if (bestLo >= 0) {
-        const kept = blocks.slice(bestLo);
-        const body = [preamble, ...kept].filter(Boolean).join('\n\n');
-        return makeRecallRootsSummaryMessageParts(oldHistory, body, fittedPrior, recallMeta);
-    }
-    // Even zero root blocks (preamble alone) overflows alongside the fitted
-    // prior — try preamble alone, else the no-recall minimal header.
-    if (preamble) {
-        const preambleOnly = makeRecallRootsSummaryMessageParts(oldHistory, preamble, fittedPrior, recallMeta);
-        if (estimateMessagesTokens([preambleOnly]) <= remainingTokens) return preambleOnly;
-    }
-    return minimal;
-}

package/src/runtime/agent/orchestrator/session/compact.mjs

@@ -44,12 +44,6 @@ export {
     pruneToolOutputsUnanchored,
 } from './compact/budget.mjs';
 
-export {
-    buildRecallFastTrackQuery,
-    splitRecallRootBlocks,
-    fitRecallRootsMessage,
-} from './compact/summary.mjs';
-
 export {
     semanticCompactMessages,
     recallFastTrackCompactMessages,