Repository navigation
Skill playbooks, renderer-free simulation code, and examples as integration tests - #56
Merged
SandroMaglione merged 5 commits intoOct 19, 2026
Conversation
…ion code, and turn examples into integration tests - The ritmo skill gets one playbook per kind of change (mechanic, content, UI and rendering, bug fix, refactor, state and upgrades); SKILL.md routes to them and keeps the principles. - ritmo/keep-simulation-headless reports type-only renderer imports and ritmo-render / ritmo-pixi subpath imports. - examples/ is replaced by integration/top-down and integration/algorithms: headless simulations with their tests, renderer-free, covered by one lint glob. The browser-only examples and vite setup are removed. - The root README links the games built with Ritmo. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Release steps move to .changeset/README.md. AGENTS.md keeps only rules that change a decision and are not enforced by tooling (112 to 50 lines). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Skill: playbooks per kind of change
skills/ritmo/SKILL.mdnow routes each change to a playbook and keeps the six principles plus the self-check. There are six playbooks:mechaniccontentui-and-renderbug-fixrefactorstate-and-upgradesEach is a short numbered list of steps that names the Ritmo API to use and ends with a Done when check. The total is 112 lines. The structure follows pstack: a router plus playbooks loaded on demand. The new lessons come from inkline-town#30:
ritmo/keep-simulation-headless(breaking)import type { Container } from "pixi.js"), so neither the schema nor the simulation can name a renderer type. Renderer services live in render code, and services are not registered in the schema.@typeonce/ritmo-render/*and@typeonce/ritmo-pixi/*, which used to slip past it.examples/→integration/integration/top-downis the headless simulation,debug.tsand the test. It names no renderer or page type, andritmo-devtools linton it reports no lints and no uncovered modules.integration/algorithmskeeps its test.concepts.tsmoved next toConcepts.test.ts.pixi,platformerandgeneticsexamples,vite.config.ts, theexample:*scripts, andmatter-js.integration/*/**, covers all integration simulations.Repo cleanup
RELEASING.mdandARCHITECTURE.md..changeset/README.md.assets/spr_player.png,vitest.shared.ts(folded intovitest.config.ts), and staledocs/**andoutput/**lint ignores.Verification
pnpm run checkpasses (77 files, 568 tests), and so doespnpm pack:check, which now also asserts the playbooks ship.🤖 Generated with Claude Code