Repository navigation
docs(devlog): archive 205 landed units from _plan to _fin - #6884
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
WalkthroughWarning Review details and warnings were omitted to fit the comment limit. |
There was a problem hiding this comment.
💡 Codex Review
_plan
This unit still explicitly records Status: open, and its only companion document is a prospective implementation and verification plan with no terminal outcome, so moving it to _fin hides unfinished work from the open-work directory. This is not isolated: 260930_minimax_m31_flash_preview and 261001_jev_decision_routing are also archived while retaining the same open status and no outcome record. Restore these units to _plan, or add the concrete landing/outcome evidence before archiving them.
AGENTS.md reference: AGENTS.md:L108-L111
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
✅ Deterministic PR hygiene checks passed. |
Summary
Moves 205 devlog units whose work has landed from
devlog/_plan/todevlog/_fin/, so_planlists only open work again (250 → 43 units).Each unit was checked against live state instead of its own status line: merged PRs, commits reachable from
origin/dev, and closed issues. Units that still own an open issue, an unimplemented scope, or an empty verification record stay in_plan. Deferred polish alone did not keep a unit open._planunits already had a_fincopy.260908_provider_runtime_stackwas byte-identical.260913_devin_image_passthroughwas a strict subset, missing the merge-result section. The_planduplicates are deleted.devlog/_plan/<unit>todevlog/_fin/<unit>(195 occurrences in 141 files), including source comments,structure/docs, and oneisScannedPathexample in the file-size ratchet test. Three../../_plan/links and one sibling link that crossed the split are fixed by hand.Verification
bun test tests/ci-workflows/file-size-ratchet.test.ts tests/ci-workflows/repo-hygiene.test.ts: 25 pass, 0 failbun run structure:check: passedbun run privacy:scan: passedorigin/dev: no newly broken links after the move.git grepfinds no remaining reference to a moved unit under its_planpath.Checklist