Skip to content

ScreenMachine: a closed loop over known screens - #108

Closed
dayfine wants to merge 1 commit into
mainfrom
common/screen-machine
Closed

dayfine wants to merge 1 commit into
mainfrom
common/screen-machine

Conversation

@dayfine

@dayfine dayfine commented Nov 1, 2026

Copy link
Copy Markdown
Owner

BDSP's new-game intro runs as a closed loop over the screens it knows: read the screen, take the one step that screen calls for, read again, and treat an unexpected screen or a state that keeps coming back as the sign of a dropped or wrong input. This adds that loop as its own module: ScreenMachine runs it and StepTrace logs every recognized screen change with a numbered screenshot and every input, so a failure reports the screens that led up to it. Both files come unchanged out of #66, which now holds only the navigation pieces both games use; nothing here depends on navigation. BDSP's intro (PokemonBDSP_AutoStory_NewGame.cpp, on #68's stack) uses it today and FRLG's intro is next. There are no tests yet; CI checks that it builds on its own.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E

Moved unchanged out of nav/core (#66), which now holds only the
navigation pieces both games use. ScreenMachine and StepTrace have no
navigation dependency; BDSP's new-game intro drives them today.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine

dayfine commented Nov 1, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #114 (commit S3 c246d3d of the shared-core collector). Branch kept.

@dayfine dayfine closed this Nov 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant