docs(devlog): close the 260819 merge loop with its outcome record

[lidge-jun]

Summary

Docs-only. Closes out the batched merge loop that landed 14 PRs today, including the full mega-file split stack.

• 090_merge_loop_outcome.md — what merged, what was held and why, and the three lessons worth keeping.
• 070_...md gets an executed-status banner recording two things: its own dogfooding argument was disproved during execution, and the preview soak gate it specifies has not run.

The substance worth reading is in 090. Short version:

The small-batch rule caught three defects a single sweep would have shipped behind green CI. The sharpest was #2105, where reconcileShellHook(false) unconditionally removes the shell hook — but false also means "claude is not on this process's PATH", and a service-started proxy does not inherit the interactive login shell's PATH. A user with Claude Code installed would have had their working .zshrc hook deleted.

"No test is possible" turned out to be a claim, not a fact. This devlog asserted across three campaigns that a barrel extraction's oracle is tsc plus its importers. A review lane disproved it: forking MODEL_ADAPTER_OVERRIDE_ALLOWED into a second Set inside the barrel leaves tsc --noEmit at exit 0, and no test imported the leaves directly, so barrel and leaf were never compared. The test now exists (landed in #2119).

A hold is worth exactly what the re-check before merge is worth. wp6 held three PRs; two got follow-up fixes and one did not, and #2112 landed with its .passthrough() gap intact. Nothing failed, because that is what passthrough does. Caught at closeout and fixed in #2119.

Verification

Docs-only change; no code gates apply. The claims it records were verified at closeout:

14 PRs MERGED, 8 HOLD-list PRs still OPEN   (gh pr view, each)
tsc --noEmit                                 -> exit 0
config + types-barrel-identity + app-server  -> 215 pass, 0 fail

Checklist

• Targets dev
• Docs-only — no src/ change, so no regression test applies
• Every claim in the record is backed by a command or a PR/commit id
• No UI change (no screenshot needed)

Summary by CodeRabbit

• Documentation
  • Added roadmap progress updates covering execution status, outcomes, and evidence for stages A–C.
  • Documented outstanding preview soak gates and unresolved attribution decisions.
  • Added a merge-loop closeout summarizing merged and held changes, identified blockers, review lessons, and remaining work.
  • Recorded a schema-validation process issue and updated testing rationale.

[github-actions]

✅ Deterministic PR hygiene checks passed.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 549df09b-c094-43c0-8948-0f0f638465e4

📥 Commits

Reviewing files that changed from the base of the PR and between 0fc8d13 and 54afa93.

📒 Files selected for processing (2)

• devlog/_plan/260819_next_roadmap/070_next_roadmap_split_and_dogfood.md
• devlog/_plan/260819_next_roadmap/090_merge_loop_outcome.md

---

📝 Walkthrough

Walkthrough

The PR updates two roadmap planning documents. It records stages A–C execution, merge-loop results, detected defects, validation lessons, the codexToolMode schema omission, and remaining preview, soak, CI, and author-response work.

Changes

Roadmap and merge-loop closeout

Layer / File(s)	Summary
Roadmap execution status devlog/_plan/260819_next_roadmap/070_next_roadmap_split_and_dogfood.md:7-23	Records 14 merged PRs, 8 blocked PRs, barrel identity-test coverage, and pending C3/C5 preview soak and attribution decisions.
Merge-loop outcome and detected defects devlog/_plan/260819_next_roadmap/090_merge_loop_outcome.md:1-44	Documents the merge-loop outcome and three defects involving shell-hook removal, catalog guidance, and paid-tier enablement.
Validation lessons and follow-up work devlog/_plan/260819_next_roadmap/090_merge_loop_outcome.md:45-86	Records review-head, rebase, and compiler-testing lessons; documents the missing codexToolMode schema fix; lists remaining author responses, preview/soak work, and CI status.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: ingwannu

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/merge-loop-outcome

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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