Repository navigation
Conversation
…ibility ownership
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.
Native Gold/Silver/Crystal trainer battles currently award friendship only from screen entry and only against the sixteen badge leaders. Apply the source pre-turn constructor award to all 22 classes in the combined leader list, including the Elite Four, Champion and Red, over the existing native ordered party references. Fainted, egg and out-of-scope party members are excluded, without constructing another battle or changing a mod.
The older offered host also lacked
playerPartyView()and ignored the fourth trainer-constructor argument. Include the real ordered native party-selection prerequisite: lead/doubles, benches, generation-appropriate menus, party-wide effects, EXP/EXP.ALL and defeat use the selected original mons. Preserve the actual save-party table and ordering. A standalone older-host regression drives these production paths; the full trainer-preparation callback feature is not silently claimed here.Expose the source model's class tables and
isGymLeader/isKantoGymLeaderpredicates. The real native constructor calls the public seam, allowing API2 decorations to reach the existing controller. Imported numeric class identity is mapped canonically instead of comparing translatedLEADERnames. Keep badge/music/Yellow companion markers separate and other editions/hacks unchanged.Bring the older host's actual friendship producer onto
happiness.changedas well. Source event names (for exampleGYMBATTLE, not numeric4) and saturated/zero-delta applied-event callbacks match the frozen Gen1 implementation; the cartridge index remains available aseventId. Unchanged walking steps stay silent.This draft continues the offered native stack: RNG #394, protection #393, volatile #392, priority #390 and their previously documented model/host prerequisites. The offered branch includes those prerequisites; please assess this incremental feature after the preceding changes. No upstream merge is performed by this work.
Validation status:
535a5ec0comparison on the corresponding fork implementation: 18,636 comparisons, 1,393 actual constructor pairs, 252 other native happiness-event cases; all pass with reference bytes unchanged. An earlier actual numeric4/stringGYMBATTLE mismatch failed and was corrected, not normalized away in the comparison.4e37741epasses the full official ROM-free command: 923/923 engine suites, 4/4 SDK suites, including 24,270 constructor/API2 checks, 30 semantic checkpoint checks and the corrected lifecycle suite. This is a result for that fork source only, not for this older offered host.607a0e77: 24,270/24,270 constructor/API2 friendship checks and 262/262 existing model-contract checks pass. The prior offered heada92fd975fails directly in the actual constructor with missingplayerPartyView, confirming the prerequisite. The separate new scope regression initially queried AI before real screen entry and supplied Gen1-shaped species to NDS constructors; that failed run is retained. The fixture now enters the actual native screen and supplies the correct six-stat/IV species layout without dropping an assertion or edition. On offered sourceb17a8f1b, the corrected production-path scope regression passes 651/651 checks across all supported editions. A separately executed frozen-Gen1 comparison on this older host also passes 18,636 comparisons / 1,393 actual constructor pairs / 252 event cases, using class identities directly from the pinned frozen Gold/Crystal manifests; the original reference bytes remain unchanged. The independently executed full older-host command now reports 73/84 engine suites and 3/3 SDK suites, with exactly the same eleven baseline failures as the preceding RNG host (71/82); both added suites pass and no new failure is introduced. The retained failures are fingerprint/meta/string gates, quit-thread shutdown, rebind capture/swap, soft reset, tool hooks, update-check/update expectations and wide-battle layout. This is not an all-green upstream baseline or release claim.Contributions by Roxas2712 — https://github.com/Roxas2712. Offered to Cedric / UNDERdecoded under the specific PR permission in LICENSE-ROXAS2712.md, with existing upstream and third-party rights retained.