Repository navigation
docs: restore the green and privacy sections word for word from v4.20.0 - #549
Conversation
Follow-up to #548, cut from main at 5e94667. README.md only. A fresh-context review of the merged head found that the blocks the PR had described as restored from v4.20.0 were not. The Green software engineering section had lost 23 lines, including two hedges that are the point of the section: that the operator supplies the region, observation period, lifecycle assessment and reservation assumptions, and the closing paragraph stating that no token-savings or CO2 figure is published for end-to-end sessions because none has been measured. Security had been folded into Privacy Policy and both paraphrased; Support and the License opener carried small wording changes. Every one of those blocks is now the v4.20.0 text, plus the Verification section that follows the green section there (the per-mechanism ablation evidence and both paper PDFs), which had been dropped in an earlier cut. Each block was compared to `git show v4.20.0:README.md` at word level after whitespace normalisation inside the splicing script; all comparisons print True. The cross-platform positioning paragraph no longer lists hosts on its own, since that list did not match the table it pointed to; it names Claude Code, the Claude Desktop bundle and Claude Cowork and defers every other host to the table. Deliberately not reintroduced: the independent-project disclaimer the owner removed on main in 5e94667. Gates on this head: link and cited-path resolution, no duplicated heading, check_doc_claims.py, check_version_surfaces.py (17 sites), and the codex-contract and version-surface tests (33 pass). Co-Authored-By: Claude Fable 5.1 <[email protected]>
|
ZETETIC-REVIEW: APPROVE Verdict on head 97d7dd9. An independent fresh-context reviewer, the same one whose REQUEST-CHANGES on #548's head f20cb17 this PR closes, re-verified this head with its own comparisons rather than the PR body's: the Green software engineering section together with the Verification section that follows it diff empty against v4.20.0's README; Security, Privacy Policy and Support diff empty as well; the License opener matches v4.20.0. "not an Anthropic product" and "Independent project" have zero matches, so the disclaimer the owner removed on main in 5e94667 is not reintroduced. The cross-platform paragraph names only Claude Code, the Claude Desktop bundle and Claude Cowork, all three documented in Install, and defers every other host to the table. Regression against 5e94667: the diff touches only the cross-platform paragraph, the Green section, the new Verification section, Security, Privacy Policy, Support and the License opener. No other line of main's README changed or dropped. Gates from a detached scratch worktree at this head: check_doc_claims.py passed, check_version_surfaces.py passed across 17 sites, test_codex_plugin_contract.py and test_version_surfaces.py passed all 33 cases. The migration block is untouched. CI on this head: 14 pass, 11 skipping, zero failures; the skips are the docs-only path filters. |
Summary
Narrow follow-up to #548, cut from current
main(5e94667) after the merge. It changes README.md only, and it deliberately does not reintroduce the independent-project disclaimer that 5e94667 removed.A fresh-context review of #548's merged head found that two blocks described as restored from v4.20.0 were not. This PR makes them so and proves it mechanically.
Restored word for word from v4.20.0
The whole Green software engineering section. The merged version had lost 23 lines, among them two hedges that are the point of that section: the sentence that the operator supplies the region, observation period, lifecycle assessment and reservation assumptions, and the closing paragraph stating that Cortex publishes no token-savings or CO₂ figure for end-to-end agent sessions because it has not measured one. The ~0.05 s / ~0.6 s hook-boot figures and their citation were already intact.
The Verification section that follows it in v4.20.0: the per-mechanism ablation evidence (LongMemEval-S 17 rows, LoCoMo 14 rows, with their result files) and the two paper PDFs. It backs the benchmark table above it and had been dropped in an earlier cut.
Security and Privacy Policy as two separate sections, Support, and the License opener, all with v4.20.0's punctuation. The merged version had folded Security into Privacy Policy and paraphrased both.
Each block was compared to
git show v4.20.0:README.mdat word level after whitespace normalisation inside the script that spliced it; all four comparisons print True.One wording change
The cross-platform positioning paragraph no longer carries its own host list, which did not match the "Every other MCP host" table it pointed to. It now names Claude Code, the Claude Desktop bundle and Claude Cowork and defers every other host to that table, the only place that enumerates them.
Left out on purpose
The independent-project disclaimer, removed on
mainby the owner in 5e94667 after the merge. Nothing here re-adds it.Type of change
Test plan
Word-level identity of the four restored blocks against v4.20.0: True.
No duplicated H2 heading; every relative link resolves; the three
docs/benchmarks/e1-v3-*.mdfiles and both PDFs cited by the Verification section exist.scripts/check_doc_claims.pyandscripts/check_version_surfaces.py(17 sites): green.tests_py/scripts/test_codex_plugin_contract.pyandtest_version_surfaces.py: 33 pass; the migration block is untouched.All existing tests pass.
Manual verification: identity comparison, heading uniqueness, link and path resolution.
Audit notes
The finding this closes came from a fresh-context reviewer on #548's head f20cb17 (REQUEST-CHANGES, two blockers, one should-fix, one nit); the same reviewer approved the equivalent content on dcb591d, which never reached
mainbecause #548 had already been merged. This PR is that content rebased onto currentmainminus the disclaimer.Reviewer checklist
manifest.jsonalready declares).🤖 Generated with Claude Code