Repository navigation
Conversation
Unity now opens a bump PR from native-sdk-release, and it was missing from the dispatch list. Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
I reviewed the one-commit change that adds Unity to the list of wrappers notified on an Android release.
- Unity added to
WRAPPER_REPOS:notify-wrappers.ymlnow sendsnative-sdk-releasetosuperwall/Superwall-Unityalong with Flutter, Expo and KMP, through the same per-repo loop. One repo failing does not stop the others. - Comment updates: the header in
notify-wrappers.ymland thenotify_wrappersjob comment inbuild+test+deploy.ymlnow list Unity.
ℹ️ Until Superwall-Unity#12 lands, the Unity dispatch is silently ignored rather than failing
The description says a dispatch to Unity fails until native-sdk-bump.yml is on Unity's main. That is not quite right. POST /repos/{owner}/{repo}/dispatches returns 204 whether or not any workflow listens for the event, so the job stays green and nothing happens on the Unity side. This means the two PRs can merge in either order. The only thing that would turn this job red is WRAPPER_DISPATCH_TOKEN lacking access to Superwall-Unity (a 404 from gh api), which the test plan already covers. Note that a version released before #12 lands will not start a Unity bump on its own. You would re-send it with notify_wrappers_only.
claude-opus-5.5 | 𝕏

Summary
superwall/Superwall-Unityto the wrapper dispatch list, so an Android release tells Unity to open a bump PR the way Flutter, Expo, and KMP already are.native-sdk-bump.ymlonmain. Until then a dispatch to Unity fails, and the other wrappers are still notified.Test plan
WRAPPER_DISPATCH_TOKENhas Contents read & write onsuperwall/Superwall-Unity. The dispatch API checks that permission, and a token that cannot see the repo fails this job.main, run this workflow by hand for a version Unity does not already pin and confirm the Unity bump workflow starts.