docs(readme): v5 — model-driven focus, unit consistency, move cache section - #21
Merged
Merged
Conversation
Per user feedback (5 rounds of README refinement): - Why ACP: "ACP hands all context-management authority to the model itself" — best implementation to date. Two effects: saves ~2/3 tokens (a 1M-window model runs in the 200,000-300,000 range); ultra-long sessions without losing key content (500M-level context, 100K messages). Unify units; drop the 37-bug-fixes footnote (explained later). - Proven at scale: lead with a one-line summary (500M-level context, p95 ~30%, avg cache hit >85% — average not per-session, explained in the cache section). Drop Span / Model turns / Tokens-reclaimed rows. - How It Works: model is 100% responsible; tools = compress/decompress/delete. Lifecycle state-machine (mermaid) simplified to 3 objects. Compression strategy with the source priority list (agent results, verbose command output, dead-end exploration, ...). Decompression + Deletion strategies. - Impact on Prompt Caching: ~87% hit rate; beats traditional compression; context ~30% vs traditional 50-80%; conclusion = higher hit rate AND no key info lost. Moved to right after How It Works. EN + ZH consistent. Docs-only. Includes anonymized real-world session stats. Reviewed via dual-agent code review (AGENTS.md §5.3) before this PR.
Reviewer B (dual-agent review, AGENTS.md §5.3) flagged that the devlog still described the earlier v3 revision (wrong numbers like 2.6M/1.8M reclaimed, H3 subsections, 'demoted to footnote', stale 'targets Gitea fork' note) which do not match the v5 diff. Rewrote REQ.md + WORKLOG.md to accurately reflect PR #21's shipped content. mark_block/'delete' framing retained per maintainer decision (Option B): it is conceptually deletion (mark -> eventually truly deleted); the merge-vs-true- delete mechanism is intentionally not elaborated in the README.
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
README refinement, 5 rounds of user feedback consolidated. Docs-only (no
lib/changes). EN + ZH kept consistent.lib/prompts/system.ts: agent results, verbose command output, dead-end exploration, ...). Decompression + Deletion strategies (deletion irrecoverable, replaces forced GC).Verification
lib/touched, no build/typecheck/test impact.[English](./README.md) | [中文](./README.zh-CN.md)).devlog/2026-06-27_readme-v3/.Non-goals