docs: split CLAUDE.md's Cold Storage and Closed Backlog into non-imported files

[mixmansoundude]

Summary

• CLAUDE.md is loaded in full every session via Claude Code's context, and its "Closed
  Backlog" section alone was over 60% of the file's ~4200 lines despite being a pure
  historical record with no forward-looking action attached to any entry.
• Moved "Cold Storage" and "Closed Backlog" (plus the now-fully-closed Active Backlog item 7)
  out into two new files: docs/agent-closed-backlog.md and docs/agent-cold-storage.md,
  leaving short pointer stubs in CLAUDE.md.
• Unlike docs/agent-lessons-learned.md / agent-interconnect.md / agent-ndjson.md, these
  two new files are deliberately not referenced via CLAUDE.md's @import mechanism, so
  they're read on demand rather than loaded into every session's context — that's the whole
  point of the move.
• CLAUDE.md shrinks from ~4200 to ~1065 lines with zero content loss (verified via
  substring-containment checks of every moved section against the pre-move file).
• Item 7's "fully closed" status was independently re-confirmed by reading its own closing
  lines before moving it, not assumed from the task description alone.
• Updated the three files that previously documented item 7's literal 7. marker as a
  permanent, expected MD029 lint exception (.markdownlint-cli2.jsonc, AGENTS.md,
  tools/run_sanity_sweep.sh) to reflect that the exception no longer applies — confirmed
  locally that markdownlint-cli2 CLAUDE.md now reports 0 issues.
• Fixed two stale cross-references (docs/agent-ndjson.md, docs/agent-scratchlog.md) that
  pointed at item 7's old location inside CLAUDE.md.

Test plan

• python -m compileall -q . — clean
• python tools/check_delimiters.py run_setup.bat — clean (unchanged file)
• markdownlint-cli2 CLAUDE.md — 0 issues (previously 1 permanent/expected finding for
  item 7's marker; confirms the exception no longer applies)
• python -m yamllint .github/workflows/ — clean (no workflow changes in this PR)
• ASCII sweep across all touched files (including the two new docs) — clean
• python -m pytest tests/test_*.py -q — 398 passed, 41 skipped (no code changes)
• Verified every moved section (Cold Storage, Closed Backlog, item 7) is byte-identical
  to its pre-move content via Python substring-containment checks

---

Generated by Claude Code

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: c78de0b3-6153-42b0-872b-f30fa8c420fc

📥 Commits

Reviewing files that changed from the base of the PR and between 67726af and c6454cd.

📒 Files selected for processing (2)

• docs/agent-closed-backlog.md
• docs/agent-cold-storage.md

📜 Recent review details

🧰 Additional context used

📓 Path-based instructions (4)

**/*

📄 CodeRabbit inference engine (AGENTS.md)

Use GitHub Actions CI results, especially Windows runner results, as the source of truth; local or sandbox runs are advisory.

Files:

• docs/agent-cold-storage.md
• docs/agent-closed-backlog.md

**/*.{md,markdown}

📄 CodeRabbit inference engine (AGENTS.md)

Cite run_setup.bat locations by stable label or subroutine name rather than line number in documentation.

Files:

• docs/agent-cold-storage.md
• docs/agent-closed-backlog.md

**/*.{bat,cmd,ps1,py,yml,yaml,json,md}

📄 CodeRabbit inference engine (CLAUDE.md)

Use ASCII-only text; do not add emojis, curly quotes, em dashes, or other non-ASCII characters.

Files:

• docs/agent-cold-storage.md
• docs/agent-closed-backlog.md

**/*.{bat,cmd,ps1,py,md}

📄 CodeRabbit inference engine (CLAUDE.md)

When a change teaches or invalidates a repository lesson, update the relevant knowledge document in the same commit; move resolved backlog items to docs/agent-closed-backlog.md.

Files:

• docs/agent-cold-storage.md
• docs/agent-closed-backlog.md

🧠 Learnings (1)

📓 Common learnings

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: Follow the iteration order: plan and check the plan, execute one change set, run self-checks, fix CI or tests according to whether they error or fail assertions, then stop and open or append a PR.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: After every push, poll CI; after CI is green, verify setup logs, artifact inventory, and non-decreasing NDJSON row counts before opening a PR.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: Push every commit before it can be lost; do not leave commits only in the local repository when a session or iteration ends.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: Any new observable log line, disk file, or assertion-detectable behavior must emit an NDJSON row and have its artifact path added to the batch-check test-logs upload using both existing slash-style variants.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: External or environmental failures must be made legible in logs or NDJSON, including installer exit codes and a `reason=` token, without being mistaken for repository regressions.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:37:52.039Z
Learning: Use minimal, surgical patches; leave professional comments explaining non-obvious constraints, and annotate untraceable code with `derived requirement: <why needed>` while proposing a requirement.

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:38:04.606Z
Learning: During each iteration loop, freeze scope, fix CI before tests and product code, implement exactly one missing feature slice, and add exactly one missing test.

🔇 Additional comments (2)

docs/agent-closed-backlog.md (1)

3-5: LGTM!

Also applies to: 291-294

docs/agent-cold-storage.md (1)

3-5: LGTM!

Also applies to: 18-28, 29-53, 54-80, 82-116

---

📝 Walkthrough

Summary by CodeRabbit

• Documentation

  • Reorganized agent guidance to distinguish automatically loaded instructions from reference-only material.
  • Moved resolved backlog history and shelved technical ideas into dedicated reference documents.
  • Expanded documentation navigation and clarified backlog numbering and archival practices.
  • Updated related notes and scratch documentation to reflect the new locations.

• Chores

  • Updated Markdown quality checks to require zero outstanding findings and flag future regressions.

Walkthrough

The change moves closed backlog history and shelved ideas out of CLAUDE.md, updates related references, and changes MD029 guidance to expect zero findings.

Changes

Documentation archive and lint baseline

Layer / File(s)	Summary
Agent documentation archive CLAUDE.md, docs/agent-closed-backlog.md, docs/agent-cold-storage.md	CLAUDE.md now contains pointers and repository-map entries. Closed backlog history and cold-storage proposals now reside in dedicated documents.
Historical reference updates docs/agent-ndjson.md, docs/agent-scratchlog.md	References for Active Backlog item 7 now point to docs/agent-closed-backlog.md.
Markdownlint baseline update .markdownlint-cli2.jsonc, AGENTS.md, tools/run_sanity_sweep.sh	MD029 guidance now expects zero findings and treats any finding as a regression.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• mixmansoundude/Python_vs_Windows#392: Updates the same MD029 configuration and advisory checks.
• mixmansoundude/Python_vs_Windows#399: Moves Active Backlog item 7 history from CLAUDE.md to the closed-backlog archive.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main documentation change: moving Cold Storage and Closed Backlog into non-imported files.
Description check	✅ Passed	The description directly explains the file moves, context reduction, reference updates, and validation results.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/bootstrapper-execution-branches-ox2izi

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/agent-closed-backlog.md`:
- Around line 3-5: Keep every path literal in docs/agent-closed-backlog.md on a
single source line within its backtick-delimited code span, including the
references near the top and the condabin\conda.bat path near the later section;
wrap surrounding prose outside the inline code span when necessary.

In `@docs/agent-cold-storage.md`:
- Around line 3-4: Keep the inline-code path `docs/agent-lessons-learned.md`
contiguous on a single Markdown line in the opening paragraph of
`docs/agent-cold-storage.md`, without changing the surrounding wording.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 62ff70f2-2f9b-4958-973d-3b2c0f202433

📥 Commits

Reviewing files that changed from the base of the PR and between a6f5968 and 67726af.

📒 Files selected for processing (8)

• .markdownlint-cli2.jsonc
• AGENTS.md
• CLAUDE.md
• docs/agent-closed-backlog.md
• docs/agent-cold-storage.md
• docs/agent-ndjson.md
• docs/agent-scratchlog.md
• tools/run_sanity_sweep.sh

📜 Review details

⏰ Context from checks skipped due to timeout. (9)

• GitHub Check: auto_merge
• GitHub Check: Batch syntax/run check (justme-test)
• GitHub Check: Batch syntax/run check (contract-uv-fail)
• GitHub Check: Batch syntax/run check (contract-uv)
• GitHub Check: Batch syntax/run check (cache)
• GitHub Check: Batch syntax/run check (conda-full)
• GitHub Check: Batch syntax/run check (uv)
• GitHub Check: Batch syntax/run check (uv-dl-fallback)
• GitHub Check: Batch syntax/run check (real)

🧰 Additional context used

📓 Path-based instructions (5)

**/*

📄 CodeRabbit inference engine (AGENTS.md)

**/*: Work in the order Plan, Check the plan, Execute, Self-check/tests, and keep each loop to one change set.
Push every commit before it can be lost; do not leave commits unpushed at the end of a turn.

Files:

• AGENTS.md
• docs/agent-cold-storage.md
• docs/agent-scratchlog.md
• tools/run_sanity_sweep.sh
• docs/agent-ndjson.md
• docs/agent-closed-backlog.md
• CLAUDE.md

**/{CLAUDE.md,README.md,docs/**/*.md}

📄 CodeRabbit inference engine (AGENTS.md)

Cite run_setup.bat locations by stable label or subroutine name rather than line number, unless a line number provides immediate value for the introducing change.

Files:

• docs/agent-cold-storage.md
• docs/agent-scratchlog.md
• docs/agent-ndjson.md
• docs/agent-closed-backlog.md
• CLAUDE.md

docs/*.md

📄 CodeRabbit inference engine (CLAUDE.md)

When a change teaches or invalidates a lesson, update the relevant knowledge document in the same commit; edit existing entries rather than only appending.

Files:

• docs/agent-cold-storage.md
• docs/agent-scratchlog.md
• docs/agent-ndjson.md
• docs/agent-closed-backlog.md

**/*.{py,bat,cmd,ps1,yml,yaml,json,sh}

📄 CodeRabbit inference engine (AGENTS.md)

Keep source text ASCII plain text and do not manually change line endings; follow .gitattributes.

Files:

• tools/run_sanity_sweep.sh

CLAUDE.md

📄 CodeRabbit inference engine (AGENTS.md)

Run markdownlint-cli2 only on CLAUDE.md with the repository's MD029-only configuration; new Active Backlog entries must be bullets with the number in prose, not literal ordered-list markers.

Files:

• CLAUDE.md

🧠 Learnings (1)

📓 Common learnings

Learnt from: CR
Repo: mixmansoundude/Python_vs_Windows

Timestamp: 2026-07-31T09:26:58.985Z
Learning: Freeze scope during an iteration loop; fix CI issues before test failures, assertions, and product code; implement exactly one feature slice and add exactly one test per loop.

🪛 LanguageTool

docs/agent-cold-storage.md

[style] ~50-~50: Consider an alternative for the overused word “exactly”.
Context: ...t manufacture that evidence, which is exactly why this belongs in Cold Storage rather...

(EXACTLY_PRECISELY)

---

[style] ~70-~70: Consider an alternative for the overused word “exactly”.
Context: ...just a version bump. Condition (b) is exactly what the "Next-pin probe concept" secti...

(EXACTLY_PRECISELY)

---

[grammar] ~74-~74: Ensure spelling is correct
Context: ...nal reasoning plus a deliberately blunt writeup of why Tier B's narrow, well-justifie...

(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)

🔇 Additional comments (7)

.markdownlint-cli2.jsonc (1)

13-25: LGTM!

AGENTS.md (1)

243-243: LGTM!

tools/run_sanity_sweep.sh (1)

97-101: LGTM!

CLAUDE.md (1)

16-30: LGTM!

Also applies to: 85-88, 484-486, 737-742, 1059-1065

docs/agent-cold-storage.md (1)

1-2: LGTM!

Also applies to: 5-116

docs/agent-ndjson.md (1)

533-534: LGTM!

Also applies to: 550-552

docs/agent-scratchlog.md (1)

53-55: LGTM!