feat(defaults): every device on the Devices page ships OFF -- pure opt-in - #238
Conversation
…t-in (NathanNeurotic directive)
All start modes (BDM, APA HDD, Network, Apps, MMCE, Favourites) and all block-device toggles (USB,
iLink, MX4SIO, exFAT HDD, UDPBD) now default to OFF/disabled. HDD, network, and the non-USB toggles
already were; this flips BDM/Apps/MMCE/Favourites (were Manual) and USB (was on). A fresh install
boots to the start menu with no tabs -- deferredInit's boot select handles the nothing-registered
case explicitly ("the start menu stays") -- and the user enables exactly what their rig has in
Settings. Saved configs override every one of these on load, so existing installs are untouched.
The GUI-less autolaunch path (miniInit BDM force-enables) is deliberately untouched -- an external
argv launch has no Settings screen to opt in with.
Builds clean (make opl.elf, exit 0).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout. (5)
🔇 Additional comments (2)
📝 WalkthroughWalkthrough
ChangesDefault startup settings
Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the default configuration in src/opl.c to disable all device start modes (gBDMStartMode, gAPPStartMode, gMMCEStartMode, gFAVStartMode) and the USB block-device toggle (gEnableUSB) by default, adhering to the opt-in doctrine. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
…ying the runner's exact diff) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Directive
All start modes (BDM, APA HDD, Network, Apps, MMCE, Favourites) and all block-device toggles (USB, iLink, MX4SIO, exFAT HDD, UDPBD) now default OFF. HDD, network, and the non-USB toggles already were; this flips BDM/Apps/MMCE/Favourites (were Manual) and USB (was on).
Safety checked
deferredInit's boot select explicitly covers nothing-registered ("the start menu stays") — the user lands on the start menu and opts in via Settings. No NULL-tab crash path.miniInit's BDM force-enables stay — an external argv launch has no Settings screen to opt in with.Note this supersedes the defaults commit in #236 (which wanted Manual + Network=SMB — contradicting the shipped Network-OFF default). This PR is the maintainer's actual doctrine: everything off, pure opt-in.
Builds clean (
make opl.elf, exit 0).🤖 Generated with Claude Code
Summary by CodeRabbit