Skip to content

Experiment: explicit iOS controller player assignment - #3

Closed
oliverames wants to merge 1 commit into
codex/experiment-basefrom
codex/experiment-controller-assignment
Closed

oliverames wants to merge 1 commit into
codex/experiment-basefrom
codex/experiment-controller-assignment

Conversation

@oliverames

Copy link
Copy Markdown
Owner

This is a fork-only experiment targeting codex/experiment-base. Keep it isolated from main and the existing upstream PR heads. The switch is off by default.

What and why

Automatic controller slots already support multiple players, but identical model names are reconnect hints rather than persistent identity. DM_PAD_ASSIGN=1 adds a session-only Players sheet with live connection identification and explicit move/swap choices. Merge mode keeps its existing behavior and disables this option.

Reassignment releases old buttons, motion and haptic routes before publishing the new assignment. Opted-in callbacks serialize on the main queue. Touch remains Player 1 and is available when only later players have controllers. Player indicators cover the framework's supported first four slots; later slots use sheet labels.

Checked

  • 128 host input checks; production iOS build; source/docs/lint checks pass.
  • Newt, iOS 27.2: 39 programmatic production-adapter checks with writable/synthetic controller state, recorded host endpoints and synthetic gyro. Release order, slot masks, reconnect hints, stale motion callbacks and sheet data are covered.
  • Physical USB/Bluetooth devices, LEDs, rumble, real gyro, interactive sheet presentation and accessibility remain pending. No physical-controller gameplay route was run.

Docs

  • Assignment semantics, opt-in and identity/hardware limits documented.
  • No game files, keys or extracted game data.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-29T01:41:01.215592Z 01a2d95 PR opened
🔒 Security Review ✅ Completed 2026-10-29T01:39:27.510511Z 01a2d95 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@oliverames

Copy link
Copy Markdown
Owner Author

Closing: experiment evidence and testing are tracked in the upstream nxnative PRs, so the fork-only experiment copy comes down. Keeping the branch here for reference.

@oliverames oliverames closed this Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant