[mvschwarz]

A Codex in-place restart can report native_unknown when its resume screen is ready before the native process is visible. The restart now immediately checks the existing strict identity and secondary observations, then conditionally resamples within the existing finite timing budget.

Defaults allow 31 evaluations and 30 yields of 100 ms. Each successful strict attempt still needs two fresh matching process observations and the existing secondary condition. Only observation repeats; launch actions remain once, exceptions retain finalization, and the direct-native pane hold restores only after final proof. Shared identity APIs and the secondary's cached-array behavior are unchanged. Exhaustion remains unknown; success remains account-unverified.

Validation: tests-first red had 18 intended failures, all in the new describe; the existing cases and immediate-success case passed. Fixed CI passed all eight jobs, all 70 restart cases and 16,983 daemon tests. Added coverage includes both restart branches, delayed/invalid evidence, sample drift, pane joins, secondary failure, verdict writes, one launch/final event and held-pane restoration.

The budget counts scheduled yields, not observation I/O; it is not a hard three-second deadline. These are synthetic lifecycle tests, not provider/native or account acceptance. Main's effort, configuration and tmux behavior stay intact.

Refs #1196.

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: d5d2e3d3-8321-4712-b1a3-cbc8a0c1b617

📥 Commits

Reviewing files that changed from the base of the PR and between 57f2d71 and d98e0a2.

📒 Files selected for processing: 2

• packages/daemon/src/domain/seat-restart-in-place.ts
• packages/daemon/test/seat-restart-in-place.test.ts

+-----------------------------------------------------------------------------------------------------------------------------------------------------------------+
| Don't use wizard code you don't understand. Wizards can generate reams of code. Make sure you understand all of it before you incorporate it into your project. |
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------+
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[openrig-review]

Approved at d98e0a2. An in-place Codex restart now gives the replacement's identity a bounded window to appear: verifyReplacement evaluates the existing strict rebind and secondary observation at once, then resamples within the class's existing timing (31 evaluations, 30 yields of 100 ms by default). Every successful attempt still needs its own two fresh native observations; nothing joins across attempts, only observation repeats, exhaustion stays native_unknown and success stays restarted_account_unverified. Required CI passed at this head; no main commit since its base touches its files.

— dev60-planner@v-openrig-build