#827: Correct GOALS.md, PLAN.md, PROJECT_STATE.md and IRREDUCIBLE_SURFACE.md — audit found them materially stale - #838
Merged
JDonaghy merged 1 commit intoSep 5, 2026
Conversation
…E.md A four-agent audit of develop @ ee26268 found these planning docs materially stale, in ways that actively misdirect work: - The "#47 closed with blocker filed nowhere" claim was already false by the time the revision carrying it was written: quadraui#699 was filed and closed within the same 33-minute window (2026-09-03 16:38Z-17:11Z), and vimcode#47 was reopened into milestone #5 at 16:38Z. quadraui milestone #9 was never closed either, so the "re-open it" instruction was acting on a wrong premise. - The "44 call sites" figure conflated every use of the `backend` field with the two Rc-handle methods actually in question. Real count, verified at ee26268: 19 (modal_stack_handle x12, drag_state_handle x7). - GOALS.md §2's "duplication moved down a level into quadraui" claims (1,671 byte-identical lines, EventOutcome declared twice, 7 private UTF-8 copies, no desktop/, etc.) were checked against quadraui's pinned rev and mostly don't hold up; quadraui#481's own correction comment had already withdrawn the headline number. - The post-#735 size table's src/gtk/ column (9,650) predated #785's move of struct App into src/app.rs. Regenerated all columns at ee26268 with scripts/prod_lines.py and added the src/app.rs row. - docs/IRREDUCIBLE_SURFACE.md's folder-picker verdict was wrong, not just stale: quadraui::compose::FolderPickerController has existed since 2026-05-25 and its module doc already instructs vimcode to adopt it. Struck the row and added a caveat on how thin the unit_w/unit_h seam actually is (7 signatures, two explicit char_width > 1.0 sniffs). Every regenerated number is cited with the script and revision it was measured at, per GOALS.md's own "regenerate, don't trust a number typed into prose" rule. Pure documentation change, no code touched. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.
Closes #827
Automated PR opened by coordinator for review of issue #827.