docs(devlog): archive the 48 units whose work landed on dev - #1840
Merged
Merged
Conversation
The prior sweep (260805_devlog_fin_sweep) required a unit's own record to state a past-tense outcome before archiving; landed code was explicitly not enough. The repository owner overrode that: a unit is done when its work is on dev, regardless of whether its phase docs were ever rewritten. Eight read-only adjudicators verified all 55 open units against origin/dev -- merge-base for every cited SHA, gh for every PR and issue, rg for every named symbol and test file. 48 landed, 7 did not. The 7 that stay carry positive evidence of unlanded work, not merely a stale record: no blobless guidance in any workflow (260801); no /api/codex-prompt route and no WP2-WP7 tests (260802); PR #1008 closed unmerged and no src/usage/rollup.ts (260806_disposition_sweep); #1302/#1059 still open and no websocket-buffering-bound test (260814_bug_resolution_campaign); six of eight usage phases with no segments/usage-index/projection-summary modules (260814_usage_memory_roadmap); C2/C3 unmerged with history-provider and features.ts still on the old behavior (260816_codexrs_multiagent_v2); and a master plan still marked proposed with FAB-01 unauthorised (800_agent-fabric). 331 files moved: _plan 430 -> 99, _fin 1884 -> 2215. 65 files that cited a moved unit by path -- source comments, test headers, AGENTS.md, CODEOWNERS, the superpowers plans -- are repointed to _fin, so no reference dangles. Gates: repo-hygiene 11 pass, privacy:scan passed, tsc --noEmit clean.
Contributor
|
Important Review skippedToo many files! This PR contains 356 files, which is 56 over the limit of 300. To get a review, reduce the PR to 300 files or fewer by splitting it into smaller PRs or changing its base branch. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (13)
📒 Files selected for processing (356)
You can disable this status message by setting the 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. Comment |
github-actions
Bot
added
the
documentation
Contributor
|
✅ Deterministic PR hygiene checks passed. |
Contributor
✅ READY
UI screenshot waived by the Hygiene✅ Deterministic PR hygiene checks passed. |
Owner
Author
|
This PR does not change gui behavior: the only two gui/ hunks are comment-string repointings from devlog/_plan/ to devlog/_fin/ (Models.tsx:100 and page-loading-contract.test.tsx:11). No rendered output changes, so there is no UI screenshot to take. Applying the maintainer waiver label. |
lidge-jun
added
the
gui-screenshot-waived
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Keep unfinished Compatibility Lab phases in _plan
Do not archive the entire Compatibility Lab unit yet: this line explicitly states that CL-10.5 remains blocked, and the delivery sequence also reserves CL-10.6 for final programme acceptance. Moving the unit to _fin removes this unfinished work from the repository's open-unit inventory and incorrectly represents the programme as closed. Leave the unit in _plan, or split CL-10.5/CL-10.6 into a new open unit, until the transport contract, implementation, and acceptance have landed.
AGENTS.md reference: AGENTS.md:L75-L78
ℹ️ 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".
Both build their path from separate string segments, so the text-level repointing pass could not see them and CI caught it: - tests/lab-automation-ingwannu-regressions.test.ts reads the CL-08 plan via join(..., "devlog", "_plan", "260807_compatibility_lab", ...) to assert it carries no trailing whitespace. That unit moved, so the read threw. - src/lab/live/manifest.ts resolves the normative 024 live-case authority the same way. Its mirror check is a no-op when the file is missing, so this silently disabled the drift guard rather than failing loudly. With the path corrected the runtime copy verifies byte-identical again. A grep for '"devlog"' as a string literal finds these two and nothing else.
agentHits
pushed a commit
to agentHits/opencodex
that referenced
this pull request
Sep 17, 2026
docs(devlog): archive the 48 units whose work landed on dev
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Archives the 48
devlog/_plan/units whose work has landed ondev, leaving 7 open.The prior sweep (
260805_devlog_fin_sweep) required a unit's own record to state a past-tense outcome before archiving — landed code was explicitly not enough. That standard is what kept nine units open whose implementation was visibly ondevwhile their phase docs still read as plans. The repository owner overrode it: a unit is done when its work is ondev, regardless of whether the record was ever rewritten.Eight read-only adjudicators verified all 55 open units against
origin/dev:git merge-base --is-ancestorfor every cited SHA,gh pr view/gh issue viewfor every cited PR and issue, andrgfor every named symbol and test file. 48 landed; 7 did not.The 7 that stay carry positive evidence of unlanded work, not merely a stale record:
260801_monorepo_git_blobless_strategyCONTRIBUTING.md,README.md, or any workflow260802_codex_set_prompt_composer/api/codex-promptroute, no WP2-WP7 tests260806_disposition_sweepsrc/usage/rollup.tsdoes not exist260814_bug_resolution_campaigntests/websocket-buffering-bound.test.tsabsent260814_usage_memory_roadmap260816_codexrs_multiagent_v2_and_history_perfhistory-provider.tsandfeatures.tsstill on old behavior800_agent-fabricstatus: proposed; FAB-01 unauthorised331 files moved:
_plan430 → 99 tracked files,_fin1884 → 2215. 65 files that cited a moved unit by path — source comments, test headers,AGENTS.md,.github/CODEOWNERS, thedocs/superpowers/plans — are repointed to_fin/, so no reference dangles. Relative../links were checked separately; every one resolves inside its own unit.The sweep's own record is at
devlog/_fin/260816_devlog_fin_sweep/000_record.md.Verification
bun test tests/repo-hygiene.test.ts— 11 pass, 0 fail. The security tripwire reads_plan/only, and its shrunken scope still asserts non-empty.bun run privacy:scan— passed.bun x tsc --noEmit— clean.git ls-files -s devlogshows one100755(the pre-existing_fin/260731_structure_sot_refresh/004_measure.sh); every moved file is100644.rgfinds nodevlog/_plan/<moved-unit>path anywhere in the tree.Checklist
dev