Conversation
- Confirms starting Bleached Earth quests - Handle Out of Pods messages in non-JP servers
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds handling for the pod confirmation dialog when starting Bleached Earth quests and ensures the out-of-pods message is handled on non-JP servers.
- Introduces a new region and detection for the “Start Quest” confirmation dialog.
- Updates
AutoBattleto click the confirm button and include the new branch in its decision logic. - Registers the
StartQuestimage inImagesandImageLoader, and fixes a scoping issue inDroidCvPattern.
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/src/main/java/io/github/fate_grand_automata/scripts/locations/Locations.kt | Added ordealCallConfirmPodUseRegion for the confirm dialog. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/entrypoints/AutoBattle.kt | Created isInOrdealCallConfirmPodUseScreen(), added confirmPodUsage branch, and updated exception catch. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/Images.kt | Added new enum value StartQuest. |
| app/src/main/java/io/github/fate_grand_automata/util/ImageLoader.kt | Mapped Images.StartQuest to its PNG. |
| app/src/main/java/io/github/fate_grand_automata/imaging/DroidCvPattern.kt | Moved Mat instantiation inside branch but removed outer scope declaration. |
Comments suppressed due to low confidence (3)
scripts/src/main/java/io/github/fate_grand_automata/scripts/entrypoints/AutoBattle.kt:524
- The new
confirmPodUsagebranch isn't covered by existing tests. Add a test that simulates the StartQuest dialog on both JP and non-JP servers and verifies the click behavior.
var confirmPodUsage = false
app/src/main/java/io/github/fate_grand_automata/imaging/DroidCvPattern.kt:81
- The
matchfunction removes the outerresultdeclaration and only declaresresultinside theifblock, but there's no return ofresultat the end. This will cause a compilation error and break template matching. Consider declaringval result = Mat()at the method start and returning it aftermatchTemplate.
val result = Mat()
scripts/src/main/java/io/github/fate_grand_automata/scripts/entrypoints/AutoBattle.kt:535
- After clicking the pod usage confirmation, the code continues to
refill.refill()without exiting or pausing. This may trigger a refill too early. Consider throwing anExitExceptionor returning immediately, similar to other dialog handlers.
confirmPodUsage -> locations.ordealCallConfirmPodUseRegion.center.click()
Collaborator
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.

Fixes #1598