feat: add detection for strengthened skills#2180
feat: add detection for strengthened skills#2180sheng-28 wants to merge 7 commits intoFate-Grand-Automata:masterfrom
Conversation
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Show resolved
Hide resolved
|
Thanks for the hard work! |
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/io/github/fate_grand_automata/ui/pref_support/PreferredSupportScreen.kt
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
Last night I asked someone from Discord to test it on Grand Duels, the first version works fine with Emiya according to them |
|
Could you also add checking if there's the non-strengthened skill? so that if neither the strengthened nor not strengthened template image were found it means automatically 0 so we can either throw an error with the configuration set not found or just let it pass as true |
That's a good idea, I'll add a detection for non-strengthened template when the strengthened template is not found, and throw a notification to let the user know if there's no strengthening at all for that skill. |
|
btw, if you are finished with this PR. Ping reconman on discord so he can take a look |
…script when required strengthened skills are empty
There was a problem hiding this comment.
Pull request overview
Adds a new support-selection filter to detect whether a support servant’s skills have been strengthened (rank-up cleared), alongside UI controls to configure the requirement and an explicit script exit reason when templates can’t be detected.
Changes:
- Add strengthened-skill detection in preferred support selection and expose three per-skill strengthened requirements in support preferences.
- Add Preferred Support UI controls (and a note dialog) to configure strengthened skill requirements.
- Add new image templates and a new battle exit reason + UI string mapping for “strengthened skill empty”.
Reviewed changes
Copilot reviewed 10 out of 13 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt | Adds strengthened-skill filtering + detection logic during preferred support matching. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/prefs/ISupportPreferences.kt | Adds strengthened-skill preference fields to scripts-facing support prefs interface. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/locations/SupportScreenLocations.kt | Introduces a new region constant intended for strengthened-skill icon detection. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/entrypoints/AutoBattle.kt | Adds a new ExitReason for strengthened-skill template/detection failure. |
| scripts/src/main/java/io/github/fate_grand_automata/scripts/Images.kt | Registers new image identifiers for strengthened/unstrengthened templates. |
| prefs/src/main/java/io/github/fate_grand_automata/prefs/core/SupportPrefsCore.kt | Persists strengthened-skill preferences (ints) in prefs core. |
| prefs/src/main/java/io/github/fate_grand_automata/prefs/SupportPreferences.kt | Wires prefs core strengthened-skill fields into the scripts preferences implementation. |
| app/src/main/res/values/localized.xml | Adds strings for the new UI section, note, and exit message. |
| app/src/main/java/io/github/fate_grand_automata/ui/pref_support/PreferredSupportScreen.kt | Adds UI controls for configuring strengthened-skill requirements and an info dialog. |
| app/src/main/java/io/github/fate_grand_automata/ui/exit/BattleExit.kt | Maps the new ExitReason to a user-facing message. |
| app/src/main/assets/Jp/skill_strengthened.png | Adds strengthened template image asset (JP folder). |
| app/src/main/assets/En/skill_unstrengthened.png | Adds unstrengthened template image asset (EN folder). |
| app/src/main/assets/En/skill_strengthened.png | Adds strengthened template image asset (EN folder). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/io/github/fate_grand_automata/ui/pref_support/PreferredSupportScreen.kt
Show resolved
Hide resolved
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
...pts/src/main/java/io/github/fate_grand_automata/scripts/supportSelection/ServantSelection.kt
Outdated
Show resolved
Hide resolved
|
I think you could try to request another test on discord and ping Lord FGA so it can be brought to his attention |
Pull Request Type
Related Issue
Description
Screenshots
Buttons added under Max Skills setting




Screenshot of a match in debug mode
Screenshot of a mis-match in debug mode
Screenshot of exit prompt when required strengthening couldn't be found e.g. Gilgamesh with strengthening requirement of [1, 0, 2]