Skip to content

AutoStory: move AutoStoryStats to Pokemon/AutoStory - #45

Draft
dayfine wants to merge 7 commits into
autostory/r2-story-indexfrom
autostory/r3-shared-stats
Draft

dayfine wants to merge 7 commits into
autostory/r2-story-indexfrom
autostory/r3-shared-stats

Conversation

@dayfine

@dayfine dayfine commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Depends on #44. Base: autostory/r2-story-index; the diff shows only this change.

Moves AutoStoryStats unchanged to Pokemon/AutoStory/Pokemon_AutoStoryStats.h; SV keeps the name through an alias, and the stat keys are the same. Review with git diff --color-moved. Fork CI passed on Mac, Windows clang and Ubuntu (runs 41350947416, 41350954722, 41350961631).

🤖 Generated with Claude Code

https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E

Pure move, so BDSP and FRLG AutoStories can use the same stats; SV keeps the
name through an alias. The stat keys are unchanged.

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 1, 2026
…s PR

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 1, 2026
dayfine added a commit that referenced this pull request Nov 1, 2026
dayfine added a commit that referenced this pull request Nov 1, 2026
…d AutoStory stats, for story runs

StoryRunOptions (now "Story Run") takes the game's checkpoint database
and shows Start and End checkpoint dropdowns, labeled as SV labels its
segments ("00.09 Pewter Gym: Camper Liam, then Brock") and built with
#43's make_story_select_database(); a story in sections passes them joined,
so a section's entry is at #44's story_index(). A start after the end is
refused. record_story_stats() adds a segment's run to #45's
AutoStoryStats: the segment, each passed checkpoint, each reload.

Tests: the dropdowns' labels, slugs and defaults, the start/end check,
and the stats.

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 1, 2026
The program's integer START/END options give way to the shared story run
options' checkpoint dropdowns (#122), labelled from the table ("00.09
Pewter Gym: Camper Liam, then Brock"), and it counts the shared
AutoStoryStats (#45): a segment per run, the checkpoints passed and the
reloads. The reload and log hooks move into a helper, keeping program()
under the function size limit.

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

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine and others added 2 commits November 2, 2026 07:00
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E

# Conflicts:
#	SerialPrograms/cmake/SourceFiles.cmake
@dayfine
dayfine added this pull request to stack #230 November 5, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autostory AutoStory programs game:shared Shared or game-neutral code refactor No behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant