docs(plan): commit the executed evidence-wiring plan (Slice 3, PR #102) - #104
Merged
Conversation
Committed as an executed artifact, not as current truth. Execution proved four of its statements wrong and they are NOT corrected here, so the record shows what was planned against what shipped: the bound order is bundle -> record -> object (the plan says bundle -> object -> record); doctor's missing_event_store tripwire does NOT catch a store deleted after an ordinary run (the external anchor does); several test counts are stale; and four of its proposed doc assertions were non-discriminating. The shipped reference docs and tests are authoritative wherever they disagree.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Files the Slice 3 plan as an executed artifact, matching how Slice 1 (#84) and Slice 2 (#99) were recorded.
Deliberately not corrected. Execution falsified four of its statements, and leaving them visible is most of the value of keeping the file — it shows what was planned against what shipped:
bundle → record → object; the plan saysbundle → object → record. No test pinned order, so nothing caught it until Task 8's conformance vector.missing_event_storetripwire catches a store deleted out from under a real run is false — measured three times independently. An ordinary dispatch leaves no-wal/-shmresidue, the tripwire is gated on that residue, and deleting the store makes plainloop doctorquieter. The control that holds is the external anchor,--expect-chain-head.parametrize("mode", _MODES)test as one collected item where the extras leg collects two.The shipped
reference/docs and the test suite are authoritative wherever they disagree with this file.