Complete the Week 4 executable capstone - #284
Merged
Merged
Conversation
AI-Assisted: GPT-5.6 Sol + Sentinel
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.
Why
Week 4 needed one executable cumulative learner path and a capstone record whose identity does not change with the absolute spelling of the active Python interpreter.
Change
Final exact-head review
The reviewed PR head is
e2b409ae82520416c7173333463c0b5298be1e48/ treec00b63288e61577c5001344ec452d02594137ab8on baseb852b2087eb8821a7fc57b5108e793ff4242675b. The direct successor from6a51d9925c9287e5a97932635e2a2f1c2cb0c583changes onlyweek4-capstone.pyandtests_refsol/test_week_4_day_9.py; the cumulative base-to-head scope is exactly 30 paths, including all eleven accepted prose paths. The other 313 records reproduce preservation manifest8344a306a53fbd27d87cc8ae05c58cb9af46ab5ae934d57b136e5c5cfb4d5fd5.Two distinct absolute interpreter spellings produce the same newline-terminated 2,995-byte capstone record, receipt IDs,
7207 -> 5203compaction accounting, selected branch, and E42 range, with no environment path leakage. The fixed SHA-256 ise906b7977917b44eacc7e59932ddf393828614dd3a737a929f077f0d84730901.The cumulative learner commands pass at 16/24/33/40/46/58/80/86/112; 171/171 focused tests and the full 503 passed / 8 intentional skips are green, as are native build/install, lint/compile/format, mdBook build, sitemap, exact scope/preservation, and hosted run
33393503902. The inheritedmdbook testfailures remain confined to unchanged setup/Weeks 1–3 snippets; every Week 4 chapter passes.Wholly fresh independent reviews are GO on this exact head:
This merge does not authorize model download, benchmark, Phase C, publication, release, tag, or deployment.
AI-Assisted: GPT-5.6 Sol + Forge + Sentinel