Skip to content

FRLG AutoStory: type the player's name in the new game - #177

Closed
dayfine wants to merge 38 commits into
frlg/autostory-mvpfrom
frlg/autostory-player-name
Closed

FRLG AutoStory: type the player's name in the new game#177
dayfine wants to merge 38 commits into
frlg/autostory-mvpfrom
frlg/autostory-player-name

Conversation

@dayfine

@dayfine dayfine commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Depends on #120 and #81. Base frlg/autostory-mvp (#120), with frlg/naming-screen (#81) merged in; the diff shows #81's files plus this change.

AutoStory's 00.01 now names the player instead of confirming an empty name.

  • New Player name option, default "Kosmos". Empty keeps today's default-name behaviour.
  • The intro types it on the first keyboard with FRLG: type names on the in-game naming screen #81's type_name(); the rival's keyboard still confirms an empty name, so the rival keeps the default name.
  • A name the keyboard can't type is a setup error before the run starts.

Testing: the full app builds locally (macOS, at #179's tip). I didn't run unit tests locally because test mode can flip the lab app's panel. Ubuntu CI is dispatched: https://github.com/dayfine/Arduino-Source/actions/runs/41528875876

Console checks: nobody has seen #81's naming timings on a Switch yet (the first presses right after the keyboard fades in, and the 2 s wait before the "So your name is…?" text).

🤖 Generated with Claude Code

https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E

dayfine and others added 7 commits October 29, 2026 23:45
@dayfine @claude
FRLG: type names on the in-game naming screen
f92f9e4 
Adds the naming screen keyboard from pret/pokefirered's naming_screen.c
(three pages, the button column, wrap-around and page-change cursor
rules), a planner that turns a name into the fewest presses, a
type_name() routine that presses them, and a detector for the screen.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge origin/main into frlg/naming-screen
a848603 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge origin/main into frlg/naming-screen
c2090aa 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge origin/main into frlg/naming-screen
4506cb7 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
FRLG naming screen: type a name without pressing OK
47c3ffc 
plan_naming_keys() plans the presses that type a name and stops before
START A; plan_naming_input() is now that plan plus START A.
type_name_without_confirming() types the name and leaves the naming screen
open, which is where SID Helper starts.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine
Merge remote-tracking branch 'origin/frlg/naming-screen' into frlg/au…
7695156 
…tostory-player-name

# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/PokemonFRLG_Tests.cpp
@dayfine @claude
FRLG AutoStory: type the player's name in the new game's intro
651cad2 
A PLAYER_NAME option (default "Kosmos") is typed on the player's keyboard
in 00.01 with the naming screen's type_name(). Empty keeps the default
name, as before. The rival's keyboard still confirms an empty name.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine dayfine mentioned this pull request Nov 2, 2026
Closed
dayfine and others added 4 commits November 2, 2026 14:05
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
68bdfc7 
# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/Programs/AutoStory/PokemonFRLG_AutoStoryEvents.h
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
fd6ce8c 
# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/Programs/AutoStory/PokemonFRLG_AutoStoryActions.cpp
#	SerialPrograms/Source/VideoGames/PokemonFRLG/Programs/AutoStory/PokemonFRLG_AutoStoryActions.h
@dayfine @claude
Merge commit 'f4bbb71de' into HEAD
c98160d 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge frlg/autostory-mvp into frlg/autostory-player-name
701564a 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine added a commit that referenced this pull request Nov 2, 2026
@dayfine @claude
AutoStory PR progress: 2026-11-02 15:10 PT, FRLG #177-#179, #180/#181,…
87953ab 
… CI per tip

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine and others added 17 commits November 2, 2026 15:53
@dayfine @claude
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
4bef31d 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
1dbf3c7 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
32cc761
@dayfine @claude
FRLG AutoStory: give StoryEnv's player_name a default
75ea90f 
Windows Clang fails on -Wmissing-field-initializers where
AutoStory::program() builds StoryEnv without player_name, which it sets
right after. Behavior is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
51fa636
@dayfine
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
4efc512
@dayfine
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
f387f0b
@dayfine
Merge branch 'frlg/autostory-mvp' into frlg/autostory-player-name
b8dceff
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
af95529
@dayfine
Merge frlg/autostory-mvp into frlg/autostory-player-name
ff997e3
@dayfine
Merge frlg/autostory-mvp into frlg/autostory-player-name
34d71b7 
# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/PokemonFRLG_Tests.cpp
@dayfine @claude
Merge frlg/autostory-mvp (10a4232, #119 recovery) into frlg/autostory…
021ed57 
…-player-name

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine
Merge frlg/autostory-mvp into frlg/autostory-player-name
f1b5ad0 
# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/PokemonFRLG_Tests.cpp
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
ee98a84
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
694bb50
@dayfine
Merge remote-tracking branch 'origin/frlg/autostory-mvp' into HEAD
61b5bb0
@dayfine @claude
Merge frlg/autostory-mvp into frlg/autostory-player-name
1b6ef12 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine and others added 10 commits November 3, 2026 02:44
@dayfine
Merge frlg/autostory-mvp (battle loop: end on the overworld after RUN…
215cf3f 
… in a dark cave) into frlg/autostory-player-name
@dayfine
Merge frlg/autostory-mvp (a disabled move, and a move list that never…
fb70b32 
… showed) into frlg/autostory-player-name
@dayfine
Merge frlg/autostory-mvp (matchup attacks in every battle) into frlg/…
4c40482 
…autostory-player-name
@dayfine
Merge frlg/autostory-mvp (HP bar share, nurse for yellow HP, move pro…
c1f07a3 
…mpts out of step) into frlg/autostory-player-name
@dayfine
Merge frlg/autostory-mvp (c3edf46: verify mode from #122) into frlg/a…
fb56c02 
…utostory-player-name
@dayfine
Merge frlg/autostory-mvp (cb51ddf: battle sim from #120) into frlg/au…
29ca9d3 
…tostory-player-name
@dayfine @claude
Merge frlg/autostory-mvp into frlg/autostory-player-name
8fb3048 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine @claude
Merge frlg/autostory-mvp (TM moves in the summary reader) into frlg/a…
f092f93 
…utostory-player-name

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine
Merge frlg/autostory-mvp (bag rows below an item the reader can't nam…
0b6bf8a 
…e) into frlg/autostory-player-name

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

# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/Programs/AutoStory/PokemonFRLG_AutoStory.cpp
@dayfine @claude
FRLG AutoStory: check the options in check_setup(), keeping program()…
ca6321a 
… within code health's 50 lines

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
@dayfine dayfine mentioned this pull request Nov 3, 2026
Draft
@dayfine

dayfine commented Nov 3, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #214 (FRLG stack consolidated into stage PRs; will be resplit for upstream).

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@dayfine