Repository navigation
Conversation
A story failure throws with NO_ERROR_REPORT, so no error report (and no Video.mp4) was written for it, and a retry reloaded the game before anyone saw the failure frame. SegmentHooks gets a `failed` hook, told of each failed attempt and verify run before any reload. FailureCapture attaches to it and saves USER_FILE_PATH()/<game>/AutoStoryFailures/<time>-<checkpoint>-<event>/ with Video.mp4 (stream history), Screenshot.png and Failure.json, whatever the error-report send mode; save_thrown() covers exceptions out of the run. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
…r symbol Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
The unit-test runner's registration function is at its length limit on the game stacks. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
…frlg/autostory-failure-capture
FailureCapture is attached to the story hooks of story rows and verify mode, so each failed attempt saves its video, screenshot and JSON before the soft reset, and a run that throws saves under the last step under way. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
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.
Depends on #308. Base
frlg/stage8-rocktunnel(#293, the stage tip) with #308 merged in, so the diff also shows #308 until it lands innav/story-stepsand is merged down.The 00.27 failures on console B kept only their final screenshot, so their frame-replay fixtures are single frames: story failures throw with
NO_ERROR_REPORTand a retry soft-resets before anything is saved. Story rows and verify mode now attach #308'sFailureCaptureto their hooks, so each failed attempt savesUSER_FILE_PATH()/PokemonFRLG/AutoStoryFailures/<time>-<checkpoint>-<event>/(Video.mp4, Screenshot.png, Failure.json) before the soft reset, and a run that throws saves under the last step under way. Compiled locally; Pokemon::StorySegments, Pokemon::AutoStoryFailureCapture and the FRLG AutoStory sim suites run in the headless runner; not yet run on console B.🤖 Generated with Claude Code
https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss