Sync takeover docs after provider selection

[100yenadmin]

Summary

• update takeover truth blocks from the post-Fix OpenWorlds first-minute click chrome #470 baseline to the current post-Respect native selected provider in OpenWorlds starts #472 main baseline (5dd1391)
• document that OpenWorlds now honors the native selected provider, but that this is not yet Claude-free release proof
• clarify the Codex wrapper is currently a player actor/move-facade, while support VM details remain operator-only

Tests

• git diff --check
• rg -n "36d8ac3|178.|104.123|Permission denied|publickey|TCP/22|SSH|auth posture|endpoint" WorldOS-OPERATING-GOAL.md WorldOS-GUI-RUNBOOK.md WorldOS-RUNBOOK.md || true

Notes

• Docs-only. No code path or release gate behavior changed.
• This keeps release: rerun clean non-partial 5-persona RRI after gate hardening #466 gated on a DM-capable first-turn provider path rather than treating the existing Codex actor wrapper as a Dungeon Master replacement.

Summary by CodeRabbit

• Documentation
  • Updated operational guides for system setup, provider configuration, and app routing procedures.
  • Clarified how native provider selection works and configuration requirements needed for full functionality.
  • Refreshed system baseline references and next operational action steps.
  • Enhanced documentation of app play routing, session management, and provider integration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e6784296-0d0c-44bc-80cd-e909f0a6247d

📥 Commits

Reviewing files that changed from the base of the PR and between 5dd1391 and f3f3de8.

📒 Files selected for processing (3)

• WorldOS-GUI-RUNBOOK.md
• WorldOS-OPERATING-GOAL.md
• WorldOS-RUNBOOK.md

---

📝 Walkthrough

Walkthrough

Three documentation files are updated to establish a new baseline commit reference (after PR #472) and clarify operational guidance around native provider selection. The baseline is synchronized across operating-goal and runbook docs, then operational guidance is expanded to explain how local routing, provider session startup, and first-turn validation work under the new provider-selection behavior.

Changes

Baseline Sync and Provider Routing Guidance

Layer / File(s)	Summary
Baseline commit synchronization WorldOS-OPERATING-GOAL.md, WorldOS-GUI-RUNBOOK.md, WorldOS-RUNBOOK.md	The canonical MAIN BASELINE commit is advanced to 5dd1391 (after merged PRs #470/#471/#472) across all three docs, establishing the current verified state for local worktree and app checkout references.
Native provider routing and validation requirements WorldOS-RUNBOOK.md, WorldOS-GUI-RUNBOOK.md, WorldOS-OPERATING-GOAL.md	Local/VM routing guidance is clarified to specify support-vm-1 use after Codex config verification; provider session startup is documented to show startProviderSession's optional provider parameter with Swift fallback to selectedProviderRaw; a new "Native provider reality check" section warns that missing provider status is not release proof and specifies first-turn validation requiring /session-surface to end with can_act:true; and NEXT ACTION is updated to require DM-capable first-turn play before running #466.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• electricsheephq/WorldOS#472: The runbook and docs changes directly reflect PR #472's code change where native start surfaces now conditionally pass a provider/preferredProvider parameter into startProviderSession instead of hardcoding "claude".
• electricsheephq/WorldOS#471: Both PRs update local checkout and takeover routing guidance in WorldOS-RUNBOOK.md, so this PR's revisions build on #471's routing changes.
• electricsheephq/WorldOS#469: Both PRs advance origin/main baseline commit references and stabilize runbook documentation authority across the same docs.

Poem

🐰 A baseline bounce from forty-seventy-two, / Provider routing now says "pick a path, it's up to you!" / Native checks whisper "not released just yet," / While first-turn sessions validate—no regrets.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description provides a clear summary of changes, includes validation details, and adds relevant notes. However, it lacks the required CLA and licensing checklist items specified in the repository template.	Add the Licensing / CLA section with the required checkboxes to confirm CLA agreement and verify no confidential/third-party restricted material is included.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Sync takeover docs after provider selection' accurately reflects the main change—updating documentation to reflect recent provider selection implementation post-PR #472.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[100yenadmin]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.