docs(agents): correct input capability list in AGENTS.md - #588
Merged
Conversation
src/input/ has keyboard, mouse, and gamepad input sources, not touch. Also audited the rest of the Project Overview list against /src; the other bullets (ECS, rendering, physics, audio, animations, particles, asset loading, FSM) already match the implementation, and no other file duplicates the touch claim. Fixes #582
stormmuller
enabled auto-merge (squash)
August 6, 2026 21:59
The Aug 7 2026 GitHub Actions incident (ARC runner pods stuck idle) dropped this branch's push/pull_request-triggering events, so the required CI workflow never ran on this PR. Empty commit to retrigger it.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
stormmuller
pushed a commit
that referenced
this pull request
Aug 7, 2026
…ng on dev Merges dev, which brings in #587 (the parent-transform fix filed as #581) and #588 (the AGENTS.md input capability correction filed as #582). Both were surfaced by the review of this document, and both are now fixed, so the document's claims about them were stale. The transform gap row and its explanatory note move to past tense. #587 landed exactly the composition the note proposed - scale the child's local offset by the parent's world scale, rotate it by the parent's world rotation, then add - and deleted the three superseded parent-*-system.ts files along with it. The note is retained rather than removed, since the design's assumptions were formed against the broken behavior and it is worth knowing they no longer are. DL-03's "does this survive the fix" subsection moves to past tense too, and now records that option (b) is no longer broken while still losing on the load-bearing arguments. Open question 2 loses its blocker: world-space canvas mode depended on the transform fix, since a health bar parented to a rotating ship was exactly the broken case. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WmVKWzRGU5FPVEwAa2oqVw
9 tasks
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
2 participants
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
AGENTS.md's Project Overview listed the engine's input capabilities as "Keyboard, mouse, and touch input handling".src/input/has keyboard, mouse, and gamepad input sources — there is noTouchInputSource. Corrected the line to say "Keyboard, mouse, and gamepad input handling".While in there, audited the rest of the Project Overview list against
/src: ECS, rendering (WebGL2), physics (rigid bodies, collision, gravity), audio (Howler.js), animations, particles, asset loading, and FSM all match their implementations — no further drift found. Also checkedREADME.mdanddocumentation-site/docsfor the same "touch" claim; neither duplicates it, so no other file needed a change.This is documentation-only; no
TouchInputSourcewas added (that's explicitly out of scope per the issue).Related issue(s)
Closes #582
Verification checklist
npm run check-typespasses with 0 errors (no/srcchanges made)npm testpasses (no/srcchanges made)npm run lintpasses with 0 errorsnpm run cspellpasses with 0 errorsnpm run check-exportspasses (no/srcchanges made)index.ts— N/A, no API change/documentation-site/docs/docsis updated if this change affects documented behavior — N/A, no documented behavior changed,AGENTS.mditself is the doc being corrected/srcmodule touchedChangelog
docsGenerated by Claude Code