Repository navigation
refactor: rename the step model to the beat model - #200
Merged
Merged
Conversation
Record the decision that Playground traces share the story snapshot and the 2D/3D views, are recorded by AST instrumentation, and that there is exactly one (dark) Playground. Extend CONTEXT.md with the Playground vocabulary (trace, step, snapshot, lens, console, example, exercise) and the story terms that were in use but undefined. Add the October 2026 playground audit and the design research the decision rests on. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
CONTEXT.md and ADR 0004 reserve "step" for a moment of a Playground trace. A story is made of beats and its data is the beat model. - Code: StoryStep -> StoryBeat, STORY_STEPS -> STORY_BEATS, view prop step -> beat in the JavaScript and Git stories. - UI: the story counter reads "Beat n / N" and the buttons are "Previous beat" / "Next beat", as the scenario already did. - Skill step-model -> beat-model; agent definitions, AGENTS.md, the 3D standard, ADRs 0001 and 0003 and the verify-viz script follow. Generated copies are regenerated by agents:sync. No behaviour change. ADR 0001 keeps its file name. Co-Authored-By: Claude Opus 5.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.
Summary
Stacked on #199 (base branch
docs/playground-adr-0004). Merge #199 first, then retarget this tomain.A rename with no behaviour change.
CONTEXT.mdand ADR 0004 reserve "step" for a moment of a Playground trace; a story is made of beats and its data is the beat model.StoryStep→StoryBeat,STORY_STEPS→STORY_BEATS, and the 2D/3D view propstep→beatin the JavaScript and Git stories.StoryBeatrather thanBeatbecause the scenario already hasScenarioBeat.step-model→beat-model; agent definitions,AGENTS.md, the 3D standard, ADRs 0001 and 0003 and theverify-vizoverlap script follow. Generated copies come fromnpm run agents:sync."Step" stays where it is a verb ("step forward"), a numeric step (
ZOOM_STEP), an ARIA token (aria-current="step"), or the Playground's own term. ADR 0001 keeps its file name so links keep working.Checks
npx tsc -b,npm run lint,npm run test:run(33 files, 340 tests) andnpm run buildpass.npm run agents:checkpasses.Not in this PR
docs/3D-Visualization-Standard.mdstill says its prototype's hand-authored beats should later be replaced with Playground traces, which ADR 0004 rules out. That is a change of meaning, left for the owner.docs/RESEARCH/**keeps the old term as a record of what was true when written.🤖 Generated with Claude Code