pi-condense 2.4.3 → 2.6.0

package/src/spill.test.ts

@@ -1,9 +1,12 @@
 import { describe, it, expect } from "bun:test";
-import { mkdtemp, readFile, rm, writeFile } from "node:fs/promises";
+import { mkdir, mkdtemp, readFile, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { sanitizeId, blobDirFor, blobPathFor, headPreview, spillOversizedBatch } from "./spill.js";
 import { ToolCallIndexer } from "./indexer.js";
+import { registerQueryTool } from "./query-tool.js";
+import { occKey } from "./occurrence-key.js";
+import { CUSTOM_TYPE_INDEX } from "./types.js";
 import type { CapturedBatch } from "./types.js";
 
 describe("sanitizeId", () => {
@@ -36,6 +39,94 @@ describe("headPreview", () => {
   });
 });
 
+describe("occurrence-keyed spill", () => {
+  it("blobPathFor distinguishes two occurrences of one id", () => {
+    const a = blobPathFor("/tmp/s", "sess", occKey("bash_23", 1150));
+    const b = blobPathFor("/tmp/s", "sess", occKey("bash_23", 3150));
+    expect(a).not.toBe(b);
+    expect(a.endsWith("bash_23_1150.txt")).toBe(true);
+  });
+
+  it("legacy bare-id sidecar path is unchanged", () => {
+    expect(blobPathFor("/tmp/s", "sess", "bash_23").endsWith("bash_23.txt")).toBe(true);
+  });
+
+  it("registerDuplicate is called with occurrence keys on both sides", async () => {
+    const calls: string[][] = [];
+    const indexer = {
+      lookupByContent: () => "bash_1@1000",
+      registerDuplicate: (a: string, b: string) => calls.push([a, b]),
+    } as any;
+    const batch = {
+      turnIndex: 0,
+      timestamp: 2000,
+      assistantText: "",
+      toolCalls: [
+        { toolCallId: "bash_2", toolName: "bash", args: {}, resultText: "x".repeat(100), isError: false, resultTimestamp: 2150 },
+      ],
+    };
+    await spillOversizedBatch({
+      batch: batch as any,
+      indexer,
+      config: { spillThreshold: 10, spillPreviewBytes: 10, dedupByContentHash: true },
+      sessionDir: "/tmp/s",
+      sessionId: "sess",
+      appendEntry: () => {},
+    });
+    expect(calls).toEqual([["bash_2@2150", "bash_1@1000"]]);
+  });
+});
+
+describe("G4/C4: legacy bare-id sidecar recovery", () => {
+  it("a pre-upgrade legacy record whose spillPath points at a bare-id-named sidecar still resolves through context_tree_query", async () => {
+    const dir = await mkdtemp(join(tmpdir(), "spill-legacy-"));
+    try {
+      // A pre-occurrence-key sidecar, written and named exactly the way a
+      // pre-upgrade session would have (bare id, no resultTimestamp suffix).
+      const sidecarPath = blobPathFor(dir, "sid", "bash_7");
+      await mkdir(blobDirFor(dir, "sid"), { recursive: true });
+      await writeFile(sidecarPath, "OLD SPILLED BODY".repeat(20));
+
+      const indexEntry = {
+        type: "custom",
+        customType: CUSTOM_TYPE_INDEX,
+        data: {
+          toolCalls: [
+            {
+              toolCallId: "bash_7",
+              toolName: "fetch",
+              args: { url: "https://x" },
+              resultText: "",
+              resultPreview: "OLD SPILLED",
+              spillPath: sidecarPath,
+              spillBytes: 340,
+              isError: false,
+              turnIndex: 0,
+              timestamp: 500,
+            },
+          ],
+        },
+      };
+      // No matching ToolResultMessage in the branch (a genuinely pre-upgrade,
+      // truncated session) - an index entry persisted without resultTimestamp
+      // stays bare-keyed (no migration), so its sidecar keeps its bare-id
+      // filename and still resolves via the persisted spillPath.
+      const indexer = new ToolCallIndexer();
+      indexer.reconstructFromSession({ sessionManager: { getBranch: () => [indexEntry] } } as any);
+      expect(indexer.hasLegacyBareRecord("bash_7")).toBe(true);
+
+      let registered: any;
+      registerQueryTool({ registerTool: (def: any) => (registered = def) } as any, indexer);
+      const result = await registered.execute("call-1", { toolCallIds: ["bash_7"] }, undefined, undefined, undefined);
+      const text = result.content[0].text as string;
+
+      expect(text).toContain("OLD SPILLED BODY");
+    } finally {
+      await rm(dir, { recursive: true, force: true });
+    }
+  });
+});
+
 describe("spillOversizedBatch", () => {
   const cfg = { spillThreshold: 10, spillPreviewBytes: 8, dedupByContentHash: true };
   const mkBatch = (toolCalls: any[]): CapturedBatch => ({ turnIndex: 0, timestamp: 1, assistantText: "", toolCalls });
@@ -85,6 +176,22 @@ describe("spillOversizedBatch", () => {
     }
   });
 
+  it("two occurrences of one toolCallId spill to distinct sidecar files", async () => {
+    const dir = await mkdtemp(join(tmpdir(), "spill-"));
+    try {
+      const indexer = new ToolCallIndexer();
+      const batch1 = mkBatch([{ toolCallId: "bash_23", toolName: "bash", args: {}, resultText: "FIRST".repeat(20), isError: false, resultTimestamp: 1150 }]);
+      const batch2 = mkBatch([{ toolCallId: "bash_23", toolName: "bash", args: {}, resultText: "SECOND".repeat(20), isError: false, resultTimestamp: 3150 }]);
+      await spillOversizedBatch({ batch: batch1, indexer, config: { ...cfg, dedupByContentHash: false }, sessionDir: dir, sessionId: "sid", appendEntry: () => {} });
+      await spillOversizedBatch({ batch: batch2, indexer, config: { ...cfg, dedupByContentHash: false }, sessionDir: dir, sessionId: "sid", appendEntry: () => {} });
+      const rec1 = indexer.getRecord("bash_23@1150")!;
+      const rec2 = indexer.getRecord("bash_23@3150")!;
+      expect(rec1.spillPath).not.toBe(rec2.spillPath);
+      expect(await readFile(rec1.spillPath!, "utf-8")).toBe("FIRST".repeat(20));
+      expect(await readFile(rec2.spillPath!, "utf-8")).toBe("SECOND".repeat(20));
+    } finally { await rm(dir, { recursive: true, force: true }); }
+  });
+
   it("dedups an oversized duplicate to the original without a second file", async () => {
     const dir = await mkdtemp(join(tmpdir(), "spill-"));
     try {

package/src/spill.ts

@@ -3,6 +3,7 @@ import { join } from "node:path";
 import type { CapturedBatch, CapturedToolCall } from "./types.js";
 import type { ToolCallIndexer } from "./indexer.js";
 import { hashToolResult } from "./content-hash.js";
+import { occKey } from "./occurrence-key.js";
 
 /** Replace anything outside [A-Za-z0-9_-] so the id can't escape the blob dir. */
 export function sanitizeId(toolCallId: string): string {
@@ -50,18 +51,19 @@ export async function spillOversizedBatch(args: {
   for (const tc of batch.toolCalls) {
     if (tc.resultText.length < config.spillThreshold) continue;
 
+    const key = occKey(tc.toolCallId, tc.resultTimestamp);
     const hash = hashToolResult(tc.toolName, tc.resultText);
 
     if (config.dedupByContentHash) {
       const original = indexer.lookupByContent(tc.toolName, tc.resultText);
-      if (original && original !== tc.toolCallId) {
-        indexer.registerDuplicate(tc.toolCallId, original, appendEntry);
+      if (original && original !== key) {
+        indexer.registerDuplicate(key, original, appendEntry);
         handled.add(tc.toolCallId);
         continue;
       }
     }
 
-    const path = blobPathFor(sessionDir, sessionId, tc.toolCallId);
+    const path = blobPathFor(sessionDir, sessionId, key);
     try {
       await mkdir(blobDirFor(sessionDir, sessionId), { recursive: true });
       await writeFile(path, tc.resultText, "utf-8");

package/src/summary-refs.test.ts

@@ -1,5 +1,11 @@
 import { describe, expect, test } from "bun:test";
-import { substituteInlineRefs, formatSummaryToolCallRefs, type SummaryToolCallRef } from "./summary-refs.js";
+import {
+  substituteInlineRefs,
+  formatSummaryToolCallRefs,
+  buildShortToolCallRefs,
+  normalizeSummaryToolCallRefs,
+  type SummaryToolCallRef,
+} from "./summary-refs.js";
 
 describe("substituteInlineRefs", () => {
   const refs: SummaryToolCallRef[] = [
@@ -120,3 +126,47 @@ describe("substituteInlineRefs", () => {
     }
   });
 });
+
+describe("occurrence-aware summary refs", () => {
+  test("buildShortToolCallRefs carries resultTimestamp through", () => {
+    const { refs, nextIndex } = buildShortToolCallRefs(
+      [
+        { toolCallId: "bash_23", resultTimestamp: 2150 },
+        { toolCallId: "bash_23", resultTimestamp: 3150 },
+      ],
+      5,
+    );
+    expect(refs).toEqual([
+      { shortId: "t5", toolCallId: "bash_23", resultTimestamp: 2150 },
+      { shortId: "t6", toolCallId: "bash_23", resultTimestamp: 3150 },
+    ]);
+    expect(nextIndex).toBe(7);
+  });
+
+  test("buildShortToolCallRefs omits resultTimestamp when absent", () => {
+    const { refs } = buildShortToolCallRefs([{ toolCallId: "bash_1" }], 1);
+    expect(refs).toEqual([{ shortId: "t1", toolCallId: "bash_1" }]);
+    expect("resultTimestamp" in refs[0]).toBe(false);
+  });
+
+  test("normalizeSummaryToolCallRefs preserves a numeric resultTimestamp", () => {
+    const refs = normalizeSummaryToolCallRefs({
+      toolCallRefs: [{ shortId: "t1", toolCallId: "bash_1", resultTimestamp: 1150 }],
+    });
+    expect(refs).toEqual([{ shortId: "t1", toolCallId: "bash_1", resultTimestamp: 1150 }]);
+  });
+
+  test("normalizeSummaryToolCallRefs drops a non-numeric resultTimestamp", () => {
+    const refs = normalizeSummaryToolCallRefs({
+      toolCallRefs: [{ shortId: "t1", toolCallId: "bash_1", resultTimestamp: "nope" }],
+    });
+    expect(refs).toEqual([{ shortId: "t1", toolCallId: "bash_1" }]);
+    expect("resultTimestamp" in refs[0]).toBe(false);
+  });
+
+  test("legacy toolCallIds-only details still normalize", () => {
+    expect(normalizeSummaryToolCallRefs({ toolCallIds: ["bash_1"] })).toEqual([
+      { shortId: "bash_1", toolCallId: "bash_1" },
+    ]);
+  });
+});

package/src/summary-refs.ts

@@ -1,8 +1,11 @@
 import type { CapturedBatch } from "./types.js";
+import { resultTimestampOf } from "./occurrence-key.js";
 
 export interface SummaryToolCallRef {
   shortId: string;
   toolCallId: string;
+  /** ToolResultMessage timestamp; combines with toolCallId into the occurrence key. */
+  resultTimestamp?: number;
 }
 
 export interface SummaryMessageDetailsLike {
@@ -13,12 +16,13 @@ export interface SummaryMessageDetailsLike {
 const SHORT_ID_PREFIX = "t";
 
 export function buildShortToolCallRefs(
-  toolCallIds: string[],
+  calls: { toolCallId: string; resultTimestamp?: number }[],
   startIndex: number,
 ): { refs: SummaryToolCallRef[]; nextIndex: number } {
-  const refs = toolCallIds.map((toolCallId, offset) => ({
+  const refs = calls.map((call, offset) => ({
     shortId: `${SHORT_ID_PREFIX}${startIndex + offset}`,
-    toolCallId,
+    toolCallId: call.toolCallId,
+    ...(call.resultTimestamp !== undefined ? { resultTimestamp: call.resultTimestamp } : {}),
   }));
   return { refs, nextIndex: startIndex + refs.length };
 }
@@ -33,7 +37,14 @@ export function normalizeSummaryToolCallRefs(details: unknown): SummaryToolCallR
         (ref): ref is SummaryToolCallRef =>
           !!ref && typeof ref.shortId === "string" && typeof ref.toolCallId === "string",
       )
-      .map((ref) => ({ shortId: ref.shortId, toolCallId: ref.toolCallId }));
+      .map((ref) => {
+        const resultTimestamp = resultTimestampOf((ref as any).resultTimestamp);
+        return {
+          shortId: ref.shortId,
+          toolCallId: ref.toolCallId,
+          ...(resultTimestamp !== undefined ? { resultTimestamp } : {}),
+        };
+      });
   }
 
   if (Array.isArray(raw.toolCallIds)) {

package/src/test-support.ts

@@ -0,0 +1,54 @@
+import { expect } from "bun:test";
+import { sweepOrphanToolResults } from "./orphan-sweep.js";
+import { pruneMessages } from "./pruner.js";
+import { DiagnosticSink } from "./diagnostics.js";
+
+/** Shared by chain-range-prune.test.ts and id-collision.integration.test.ts. */
+export function expectNoOrphanToolResults(messages: any[]): void {
+  let open = new Set<string>();
+  for (const m of messages) {
+    if (m.role === "assistant") {
+      open = new Set((m.content ?? []).filter((c: any) => c.type === "toolCall").map((c: any) => c.id));
+    } else if (m.role === "toolResult") {
+      expect(open.has(m.toolCallId)).toBe(true);
+      open.delete(m.toolCallId);
+    }
+  }
+}
+
+/**
+ * G4/C3: proof that the orphan sweep (src/pruner.ts Phase 4) is a net, not a
+ * crutch. Runs the real `sweepOrphanToolResults` (the exact function backing
+ * Phase 4's diagnostic) over an already-pruned message array and fails if it
+ * finds anything to sweep. Equivalent to asserting the Phase 4 diagnostic
+ * never fires for this output: `sweepOrphanToolResults` returns the input
+ * array reference and an empty `sweptIds` when nothing is orphaned, which is
+ * precisely the condition under which `pruneMessages` skips the
+ * `diagnostics?.report("orphan-sweep", ...)` call.
+ */
+export function expectZeroOrphanSweep(messages: any[]): void {
+  const { messages: swept, sweptIds } = sweepOrphanToolResults(messages);
+  expect(sweptIds).toEqual([]);
+  expect(swept).toBe(messages);
+}
+
+/**
+ * G4/C3: wraps a `pruneMessages` call with a counting `DiagnosticSink` and
+ * fails if the `orphan-sweep` diagnostic fires. For fixtures that already go
+ * through the full pruner pipeline (pruner.test.ts), this is the more direct
+ * proof than `expectZeroOrphanSweep` since it exercises the actual Phase 4
+ * call site, not just the underlying pure function.
+ */
+export function pruneWithZeroSweepAssertion(
+  messages: any[],
+  indexer: any,
+  chainCompression?: any,
+  errorPurge?: any,
+  protection?: any,
+  recoveryGraceTurns: number = 0,
+): ReturnType<typeof pruneMessages> {
+  const sink = new DiagnosticSink(() => {});
+  const result = pruneMessages(messages, indexer, chainCompression, errorPurge, protection, recoveryGraceTurns, sink);
+  expect(sink.counts()["orphan-sweep"]).toBe(0);
+  return result;
+}

package/src/tree-browser.ts

@@ -7,6 +7,7 @@ import type { ToolCallRecord } from "./types.js";
 import { CUSTOM_TYPE_SUMMARY } from "./types.js";
 import { normalizeSummaryToolCallRefs } from "./summary-refs.js";
 import type { ToolCallIndexer } from "./indexer.js";
+import { occKey } from "./occurrence-key.js";
 
 // ── Tree node types ─────────────────────────────────────────────────────────
 
@@ -119,7 +120,7 @@ export function buildPruneTree(
 
     const children: TreeNode[] = [];
     for (const ref of toolCallRefs) {
-      const record = indexer.getRecord(ref.toolCallId);
+      const record = indexer.getRecord(occKey(ref.toolCallId, ref.resultTimestamp));
       if (!record) continue;
       children.push(toolCallNode(record, 1));
     }

package/src/types.ts

@@ -10,7 +10,7 @@
  *   event.toolResults = ToolResultMessage[] (one per tool call in this turn)
  *
  * STATE MODEL (Ph1 step 3):
- *   - Runtime state: Map<toolCallId, ToolCallRecord> rebuilt on session_start
+ *   - Runtime state: Map<occurrenceKey, ToolCallRecord> rebuilt on session_start
  *   - Session metadata: pi.appendEntry("context-prune-index", IndexEntryData)
  *     stored once per summarized batch; NOT in LLM context
  *   - User config: .pi/settings.json → "contextPrune" key (JSON merge safe,
@@ -78,6 +78,22 @@ export const CUSTOM_TYPE_DEDUP_ALIAS = "context-prune-dedup-alias";
  */
 export const CUSTOM_TYPE_CHAIN = "context-prune-chain";
 
+/**
+ * Written via pi.appendEntry(CUSTOM_TYPE_DIAGNOSTIC, data) when a prune-time
+ * invariant degrades. NOT in LLM context: zero tokens, zero cache-prefix
+ * change. Deduplication is a runtime concern of the diagnostic sink
+ * (src/diagnostics.ts), which takes a caller-supplied dedup key and never
+ * persists it - the persisted entry carries only `kind` plus a freeform `detail`.
+ */
+export const CUSTOM_TYPE_DIAGNOSTIC = "context-prune-diagnostic";
+
+export type DiagnosticKind = "unresolved-range" | "range-id-mismatch" | "orphan-sweep";
+
+export interface DiagnosticEntryData {
+  kind: DiagnosticKind;
+  detail: string;
+}
+
 /** The registered name of the recovery tool (src/query-tool.ts). Shared so the
  * grace checks in pruner.ts / chain-compressor.ts cannot drift from registration. */
 export const QUERY_TOOL_NAME = "context_tree_query";
@@ -163,19 +179,6 @@ export const ROLLING_WINDOW_PRESETS: { value: string; label: string }[] = [
   { value: "10", label: "10" },
 ];
 
-/**
- * Cycling preset values for the `thinkingStrip.keepLastTurns` setting.
- * Stored as strings because SettingsList cycles string values; converted to
- * number when applied. Counts ASSISTANT turns (messages), not closed chains.
- */
-export const KEEP_LAST_TURNS_PRESETS: { value: string; label: string }[] = [
-  { value: "4", label: "4" },
-  { value: "8", label: "8" },
-  { value: "16", label: "16 (default)" },
-  { value: "32", label: "32" },
-  { value: "64", label: "64" },
-];
-
 /**
  * Cycling preset values for the `minBatchChars` setting in the SettingsList.
  * Stored as strings because SettingsList cycles string values; converted to
@@ -349,8 +352,6 @@ export interface ContextPruneConfig {
   chainCompression: ChainCompressionConfig;
   /** Replace failed toolCall argument bodies with compact stubs after a cooldown window. */
   purgeErrors: ErrorPurgeConfig;
-  /** Rolling main-loop thinking-block strip: keep thinking only on the last K assistant turns. */
-  thinkingStrip: ThinkingStripConfig;
   /**
    * Pre-flush content-hash dedup pass. When `true`, each captured tool call
    * is hashed by `(toolName, normalize(resultText))` and compared against
@@ -412,13 +413,22 @@ export interface ChainRange {
   /** Timestamp of the user message that opens the chain. */
   startUserTimestamp: number;
   /**
-   * All toolCallIds in the chain's middle (deduplicated).
-   * Collected from both AssistantMessage ToolCall blocks AND matching
-   * ToolResultMessages. Used to: (1) drop ToolResultMessages, (2) identify
-   * and drop middle AssistantMessages, (3) suppress per-batch summary
-   * CustomMessages whose toolCallRefs overlap.
+   * All toolCallIds in the chain's middle (deduplicated). Collected from both
+   * AssistantMessage ToolCall blocks AND matching ToolResultMessages.
+   * Identifies the chain's middle tool calls for detection, recovery-grace
+   * filtering and diagnostics. NOT used for the load-bearing indexer lookups
+   * (summary bodies, tool refs) - those maps are occurrence-keyed, so use
+   * the sibling `middleOccurrenceKeys` instead. Drops themselves are decided
+   * positionally by `resolveRange` in chain-range-prune.ts, not by these ids.
    */
   middleToolCallIds: string[];
+  /**
+   * Occurrence keys (`id@resultTimestamp`) for the chain's middle tool
+   * results, collected from the ToolResultMessages themselves. Used for
+   * indexer summary-body / toolRef lookups, which are occurrence-keyed.
+   * Optional so hand-built ChainRange fixtures need not set it.
+   */
+  middleOccurrenceKeys?: string[];
   /**
    * Subset of middleToolCallIds whose tool name ∈ protectedTools (detection-time
    * fact). The detector always emits it ([] when no protected tool ran); optional
@@ -440,12 +450,19 @@ export interface ChainCompressionEntry {
   /** Timestamp of the user message that opens the chain. Keep raw; synthetic inserted after. */
   startUserTimestamp: number;
   /**
-   * ToolCallIds of all dropped middle messages.
-   * Used at context-transform time to: drop matching ToolResultMessages,
-   * drop AssistantMessages that contain any of these as ToolCall blocks,
-   * and suppress per-batch summary messages whose toolCallRefs overlap.
+   * All toolCallIds in the chain's middle. **Diagnostic only** since the
+   * positional-range change: drops are decided by index range (see
+   * resolveRange in chain-range-prune.ts). Retained as a cross-check - a
+   * mismatch against the ids actually dropped emits `range-id-mismatch`.
    */
   droppedToolCallIds: string[];
+  /**
+   * Occurrence keys for the same calls as droppedToolCallIds. Load-bearing at
+   * render time: summaryBodies are occurrence-keyed, so the synthetic chain
+   * body is looked up by these. Absent on pre-upgrade entries, which fall back
+   * to droppedToolCallIds against their own bare-keyed summaryBodies.
+   */
+  droppedOccurrenceKeys?: string[];
   /**
    * Subset of droppedToolCallIds whose tool was user-protected. Membership is decided
    * per call by tool name (every call whose name ∈ protectedTools), not a per-id allowlist.
@@ -497,18 +514,6 @@ export interface ErrorPurgeConfig {
   minArgChars: number;
 }
 
-export interface ThinkingStripConfig {
-  enabled: boolean;
-  /**
-   * Keep `thinking` blocks on the last K assistant turns; strip them from
-   * older assistant messages (preserving text + toolCall blocks). Counts
-   * assistant messages, not closed chains. Clamped to >= 1 so the most-recent
-   * assistant turn always keeps its thinking (Anthropic requires the last
-   * assistant turn's thinking during tool use). Default 16.
-   */
-  keepLastTurns: number;
-}
-
 export const DEFAULT_CONFIG: ContextPruneConfig = {
   enabled: false,
   showPruneStatusLine: true,
@@ -534,10 +539,6 @@ export const DEFAULT_CONFIG: ContextPruneConfig = {
     cooldownTurns: 2,
     minArgChars: 500,
   },
-  thinkingStrip: {
-    enabled: true,
-    keepLastTurns: 16,
-  },
   dedupByContentHash: true,
   autoBudgetThreshold: null,
   spillThreshold: 65536,
@@ -554,6 +555,13 @@ export interface CapturedToolCall {
   args: Record<string, unknown>;
   resultText: string;
   isError: boolean;
+  /**
+   * Timestamp of the ToolResultMessage this call was paired with. The
+   * occurrence discriminant (see src/occurrence-key.ts): provider ids repeat
+   * within a session, this does not. Optional so pre-upgrade persisted
+   * entries stay readable; absent => the record is legacy bare-id keyed.
+   */
+  resultTimestamp?: number;
   spillPath?: string;
   spillBytes?: number;
   resultPreview?: string;
@@ -596,6 +604,8 @@ export interface ToolCallRecord {
   isError: boolean;
   turnIndex: number;
   timestamp: number;
+  /** See CapturedToolCall.resultTimestamp. */
+  resultTimestamp?: number;
   /** Absolute path to the sidecar blob holding the full body (set only when the result was spilled). */
   spillPath?: string;
   /** Full byte length of the spilled body. */
@@ -634,6 +644,9 @@ export interface IndexEntryData {
 export interface DedupAliasEntryData {
   newToolCallId: string;
   originalToolCallId: string;
+  /** Occurrence timestamps for each side; absent on pre-upgrade entries. */
+  newResultTimestamp?: number;
+  originalResultTimestamp?: number;
   hash?: string;
 }
 
@@ -644,6 +657,8 @@ export interface DedupAliasEntryData {
 export interface SummaryToolCallRef {
   shortId: string;
   toolCallId: string;
+  /** ToolResultMessage timestamp; with toolCallId this forms the occurrence key. */
+  resultTimestamp?: number;
 }
 
 /**
@@ -730,14 +745,6 @@ export interface PruneFrontier {
   summaryCharCount: number;
   /** Whether the attempt actually pruned or was skipped for being oversized */
   outcome: PruneFrontierOutcome;
-  /**
-   * Assistant-message timestamp marking the flush-gated thinking-strip boundary:
-   * thinking is stripped from every assistant message older than this. Advances
-   * only at flushes (stays fixed between them so renders are prefix-stable and the
-   * prompt cache survives a tool loop). Absent on pre-feature entries - the render
-   * path then falls back to the live-count window. See src/thinking-strip.ts.
-   */
-  thinkingStripBoundaryTimestamp?: number;
 }
 
 /**