Revisions batch: on-ramp + model currency + D13 scope row (spec), six guide rules, first full live-AC run (6/6 after the thread-identity fix) - #25
Open
Carr1005 wants to merge 10 commits into
Conversation
…ency updates, importable fixture filename Owner-approved texts from the #24 batch: blockquote gains 'what you're building' + fastest-path/completion sentences (R16/R18 density feedback; keyless finish blessed); D11/D12 course-default entries carry inline setup weight; D4 Options name providers not current model names, with gate-time recommendation deferred to the build agent; CTX-D gains the perishable request-parameter-behavior category (temperature rejection); §2 key wording acknowledges compatible endpoints; fixtures/llm-stubs.py -> llm_stubs.py. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…o canon) First contradicted-category row: thread-scoped default (the app path) vs the Lesson-5 notebook's unscoped global pool, both course-demonstrated; stakes test passes because the choice changes what a summary read returns. Side edits: contradicted-group paragraph updated; R18 defers to D13 (no deciding twice); §5 coverage note and CTX-C11 cross-reference D13; provenance note records the batch. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…nline heavy-option weights, model-currency timing, id alphabets, importable fixture filenames, scoping-contradiction example G1 blockquote anatomy: orientation lead (goal-row default as a user-visible interaction, never a component inventory) + fastest-path/completion sentence. G2 §5.5 Options: heavy options state setup weight inline at choice time. G3 §6.0 + CTX-D anatomy: model currency is the canonical generation-vs-gate timing split; request-parameter-compatibility perishables. G4 Pass A: identifier formats are working parameters, alphabet included. G5 §5 anatomy: fixture module filenames must be importable identifiers. G6 §5.5 contradicted route: worked storage-scoping example that passes the stakes test (stabilizes the D13-class subject). §14: reference-style audit lines for G1/G2/G3/G5; G4 rides the existing Pass-A coverage line. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…1-V6 properties Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…1 residual class recurrence Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… (augmentation fold) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…marization (R11) AC21's live run caught the gap deterministically: the model, never told its thread id, called summarize_and_store with a guessed 'current_thread' and the tool honestly consolidated nothing. The constraint is now stated; AC21 is its end-to-end test. Living-document rule applied; guide-level generalization (harness-known identities are injected, never model-guessed) queued as a pass finding. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ct PASS; AC21 finding + live-run record Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 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
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.
The revisions batch from the tester-feedback triage (#24) plus the promoted #18 findings — spec edits and their guide generalizations, validated Option-B style (surgical edits + clean-room Opus regens; no adoption, no rebuild). Also includes the lab's first full keyed live-AC run, which caught and fixed a real design gap.
Spec (3 commits)
2835655— on-ramp + currency: the opening blockquote gains the orientation lead ("what you're building" as a user-visible scene) and the fastest-path/completion sentence (offline suite green = the recommended baseline is complete; keys unlock only live tests/live use); inline heavy-setup notes on D11 Oracle and D12 Tavily; D4 Options name providers, never "current" model names ("expect the build agent to recommend current ones at the gate"); CTX-D gains the request-parameter-compatibility perishable (the temperature-rejection class);fixtures/llm-stubs.py→llm_stubs.py.809dea8— D13 summary-memory-scope row (owner decision, first contradicted-category row): thread-scoped default (the app path) vs the Lesson-5 notebook's unscoped global pool — both course-demonstrated; stakes test passes because the choice changes what a read returns. Side edits: contradicted-group paragraph, R18 defers to D13, §5 coverage note, CTX-C11 cross-ref.Guide (1 commit,
0cd5ff2)Six course-agnostic rules: G1 orientation-block template + reader test; G2 heavy options state setup weight inline at choice time; G3 model currency is the generation-vs-gate timing split, CTX-D parameter-constraint category; G4 identifier formats are Pass-A parameters, alphabet included; G5 fixture module filenames must be importable identifiers; G6 the contradicted route's worked storage-scoping example. Reference-style §14 lines.
Validation
docs/research/guide-revisions-batch-matrix.md(decision log incl. F8 parked and Recording: Claude Code transcript compaction can erase the /prepare-build announcement bookend #19 excluded by owner).Live-AC run (run-09, owner-supplied key) — the headline
First keyed run: AC4, AC9, AC19, AC22, AC24 passed live. AC21 — the course's cross-session continuity demo — failed deterministically: the assembled context never tells the model its thread id, so it called
summarize_and_storewith a guessed"current_thread"and the tool honestly consolidated nothing. The oracle worked exactly as designed: the spec gained the missing constraint (R11: harness-known identities are injected, never model-supplied), the run-09 build was conformed (~20 lines), offline suite re-green 21/21, and AC21 passed on rerun — live suite 6/6. The course-agnostic guide rule ("harness-known identities are injected, never model-guessed") is recorded in the matrix as the next pass's first candidate, per fix-and-test-travel-together.Issue routing
🤖 Generated with Claude Code