Repository navigation
Port Super Mario Bros. Wonder Nintendo Switch 2 Edition Program 1 - #1563
Open
ishtiaq156 wants to merge 14 commits into
Open
ishtiaq156 wants to merge 14 commits into
ishtiaq156 wants to merge 14 commits into
Conversation
2 tasks done
This was referenced Nov 6, 2026
Replace older 395-check source summary with the tested 399-check baseline and the later fork-only 11cfc06 integrated review. State the original signed-app output/cleanup fail-closed guard and its 110 Mac/NSP +11 docs validations without claiming new gameplay or chain-compatible signed packaging. Retain BrentRosen attribution, original-versus-selected ID semantics and no title-specific heap patch.
Record credited b93706e safe external image symlink-in-output semantics and fail-closed staging/cache symlink and hardlink protections, plus exact 003626c 114 Mac/NSP and 11 docs validations. Keep original/selected title identities, signing precedence, real NVN capture limits and shader-chain app rebuild gate unchanged.
…tests Use stable code-test ancestry instead of self-referential integration HEAD. Credit BrentRosen generic indexed Program0 original-image code 2d7b6a4 and its safe external leaf-source link, rejected direct output .app alias and preserved symlinked parent volumes. Record newly passing 116 synthetic Mac/NSP checks; retain original/selected title ID and shader-chain mismatch limits, without runtime edits.
ishtiaq156
marked this pull request as ready for review
November 8, 2026 04:39
2 tasks done
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.
Scope — Wonder Nintendo Switch 2 Edition Program 1
This ready-for-review PR adds Super Mario Bros. Wonder: Nintendo Switch 2 Edition + Meetup in Bellabel Park as its own Ounce / NVN2 Program 1 profile: selected title ID
040015100B514001, updatev327680, version 1.2.1. The Switch 1 application remains010015100B514000, with its original save/profile; the Bellabel DLC remains010015100B515001. Selected Program 1, original application, and DLC identities are not interchangeable.The profile registers the verified Program 1
main/subsdk0/sdkbuild IDs, memory layout, 30,630 verified shader keys, stage/PSO tables and BAHS splitter correction. It keepsrender_cond on, required to prevent an unwanted shadow-mask clear that darkens the fountain/ground. It includes no proprietary files, diagnostic RESTBL overlay or title-specific heap patch.November 10 correction: Upstream main now contains the default-on general NVN recorder fix
f02d0474. The previous four-byte-ticket #1609 was closed as superseded. I removed the now-obsoletenvn_draw_ctl onprofile key. The Edition no longer needs a profile-specific NVN scratch workaround; the original RESTBL is enough. Optional #1564, #1565 and #1606 were also closed as no longer required for this port, with the independent unmerged functionality noted on each PR.Dependencies and ownership
Recommended merge order — these capabilities need to land before this profile can be merged:
capibawa's authorship.Acd_1.Then merge #1563 (this profile). The steps above are a recommended integration sequence, not necessarily strict dependencies between all the listed PRs.
Already in main: the generic NVN allocation fix
f02d0474and base SDK21 OutputStream support67f1d005(formerly #1254). Neither is a pending dependency. CI hygiene #1675 / #1682 can proceed independently and should not block this port solely because of unrelated baseline failures.Contributor and data safeguards: keep
capibawa's work in #1253 and creditBrentRosenfor #1639's general indexed RomFS work. The safe adaptation checks source/output aliases. The update-only dump contains Program 0 NPDM but no Program 0 RomFS; an explicitly supplied original-title-checkedDMROMFS1image is needed, rather than silently substituting Program 1. That check does not prove native hardware content/version parity. iOS indexed-sidecar packaging is outside this PR.Original-RESTBL validation
On November 10, manually tested New Game and Peach Continue in a signed macOS build with upstream NVN fix
f02d0474, the unmodified RESTBL, and nonvn_draw_ctlor heap workaround. Both sessions exited normally without the previous BFRES crash or reported NVN command-memory overflows. The build still included local, unmerged Program 1/audio/packaging integration.Both formerly failing BFRES model scratch allocations now fit their original budgets. Full measurements, logs and attribution caveats: #1580 retest results.
Checks
game.tsvandREADME.mdchanged; shader tables and runtime code are untouched.git diff --checkand 11/11 documentation tests passed.