Skip to content

Latest commit

 

History

History
132 lines (106 loc) · 6.97 KB

CACHE.md

File metadata and controls

132 lines (106 loc) · 6.97 KB

Prompt-cache stability (the pinned prefix)

阅读简体中文版:zh_hans/CACHE.md。

Provider prompt caches (DeepSeek KV cache, Anthropic cache_control) only pay off when the byte prefix of a request matches the previous one: the system prompt, then the tool catalog, then messages[0..n-1]. Any change to those bytes invalidates the cache for every token after the first difference.

The invariant

After session start, the system prompt and tool catalog are frozen bytes. History only grows. A cache miss is allowed only when we can name why.

Concretely:

  • The header (system prompt + tools) is composed once at session start and re-composed only on an explicit, logged header-change op. The tool loop performs no mid-loop system-prompt refresh, so an agent writing a file (which changes the project-context pack, a directory listing, a skills scan) cannot move the pinned prefix under the model's feet mid-turn.
  • History only grows. Volatile facts the model must see (LSP diagnostics, steer input, subagent completions) are appended to the message list, never spliced into the frozen prefix. Workspace drift is delivered the same way: at the start of each new user turn (never mid-tool-loop) the engine recomposes the volatile contributors and, if anything differs from what the model last saw, appends one <context_update> user-role message with a bounded +/- line delta (new files in the project pack, edited AGENTS.md lines, added skills, memory entries, goal text) before the user's message. The header bytes stay pinned; the update is a normal append, so the prefix still extends. The pinned system prompt tells the model once that updates arrive this way. Each delta is delivered exactly once (/cache stats shows Context updates: N).
  • Every miss is attributable. PrefixStabilityManager (prefix_cache.rs) records each change with a reason and reports it through /cache stats.

What counts as a declared header change

These re-pin the prefix under a logged change:<what> reason. A changed tool header can invalidate the remaining conversation prefix, requiring a full refill and, for some providers, cache-write charges. A named change does not make that cost small or prove a provider cache miss:

Op Reason
/model (SetModel) change:model
Mode change (agent/plan/operate/yolo) change:mode
Goal set / pause / resume / clear / status change:goal
Mid-turn tool-surface change (deferred-tool admission/eviction, tool-search activation, runtime MCP tool arrival) change:tool_surface
Session sync / restore (SyncSession) resume
Session construction initial

History resets that legitimately invalidate the tail (not the header) are logged as reset:<what> — reset:compaction, reset:clear, and reset:image_window when a provider's per-request image limit moves the request image window (the oldest images go as fixed text notes).

Anything else that changes the header bytes with no declared reason is drift: it is logged as drift:<component>, the original pin is kept (so the same undeclared prefix keeps counting as a miss instead of quietly becoming the new baseline), and /cache stats shows a WARNING. After the mid-loop-refresh removal, drift should stay at zero in normal operation; a non-zero drift count is a real bug to investigate.

Attribution vs. the old behavior

Two earlier behaviors are rejected, matching the DeepSeek Harness design:

  • Detect-and-report + re-pin on drift. The manager used to re-pin to the new prefix on every change, so /cache stats looked "stable" again after one bad step while the provider cache was already dead. It now keeps the original pin on undeclared drift.
  • Recompose the system prompt from disk on every tool step. The turn loop used to call refresh_system_prompt() before every model request, including mid-tool-loop. That is removed. Header refreshes happen only at the declared edges above.

Tool-result redaction (prepare_model_bound_request) is content-preserving when no secret is configured (the common case), so it does not move the prefix. When a configured secret appears in a tool result, redacting it is a security requirement that correctly overrides cache stability for that one message.

Per-request receipts

provider_request_receipt in exec stream JSON and turn.provider_request in Runtime events join the current preparation's logical prefix outcome with provider-wire tool bytes and digest, route identity and endpoint fingerprint, and that request's reported usage. A prepared wire observation does not prove provider delivery. Missing usage or adapter measurements remain unknown. Transparent retries settle their own receipt before cumulative usage is folded; their preparation is explicitly marked reused. Transport retries are counted, but missing per-attempt usage is never presented as free work.

These are observations, not additional billing records. They do not establish that eviction caused every cache miss. A cache refill can depend on provider retention, routing, message changes and other conditions outside the local pin.

Verifying the fix

/cache stats reports prefix stability, the pin reason, the last miss reason, the undeclared-drift count, and the aggregate provider cache hit rate. In a coding session, a stable eligible prefix can be reused within the provider's retention window. Read reported usage to establish a hit; local stability alone does not prove one.

Live end-to-end check (manual, key-gated)

With a real DEEPSEEK_API_KEY, run a session that makes the agent take at least three tool steps in one turn, then open /cache inspect. Keep the route and tool surface fixed and compare prompt_cache_hit_tokens, reported input usage and the per-request receipt. Stable prefix and tool-catalog hashes show local continuity; provider cache counters establish reported reuse. A local re-pin can explain a change, but its absence does not establish the cause of a provider miss.

KV-cache effect note (for contributors)

Any new contributor to the session context must state its KV-cache effect: does it belong in the frozen prefix (system + tools) or in append-only history? Never splice a volatile fact (time, an instruction edit, a skill-catalog change, a project-file change) into the prefix — append it as a user-role message instead. A later request must be previous ⊕ suffix unless a logged header change or a history reset explains the difference.

Deferred: full reconstructability (Layer 3)

DeepSeek Harness derives every request from an append-only session log via a pure deriveMessages() projection, so prefix-extension is emergent rather than managed. Codewhale now pins the header and delivers drift as <context_update> appends; the remaining step is to make the session log the single source of truth with a pure projection (and to persist the context-update baseline with it). That is a follow-up lane, not part of this change.