fix(desktop): a signed-out executor goes idle once and never speaks for its successor

dispose() now reports idle once. A recovered delivery that settles after
sign-out no longer reports through the shared busy callback, where it
could release the sleep blocker the next session's executor was holding.

Author: waleedlatif1

apps/desktop/src/main/desktop-executor/executor.test.ts

@@ -488,6 +488,34 @@ describe('keeping the machine awake', () => {
     expect(sim.completions).toHaveLength(1)
   })
 
+  it('goes idle at sign-out and stays silent when a recovered delivery settles afterwards', async () => {
+    const { sim, journal, executor, busy } = setup()
+    const answer = deferred<void>()
+    const complete = sim.client.complete
+    let sending = false
+    sim.client.complete = async (request) => {
+      sending = true
+      await answer.promise
+      return complete(request)
+    }
+    await journal.put({
+      toolCallId: 'r-1',
+      state: 'result',
+      executionToken: 't-1',
+      completion: DONE,
+    })
+    await executor.recover()
+    await vi.waitFor(() => expect(sending).toBe(true))
+    expect(busy).toEqual([true])
+
+    await executor.dispose()
+    expect(busy).toEqual([true, false])
+    answer.resolve()
+    await sleep(40)
+
+    expect(busy).toEqual([true, false])
+  })
+
   it('stays busy while a result a previous run left is still on its way to Sim', async () => {
     const { sim, journal, executor, busy } = setup()
     sim.completeErrors = [new DeviceRequestError(503, 'deploying')]

apps/desktop/src/main/desktop-executor/executor.ts

@@ -188,6 +188,7 @@ export class DesktopExecutor {
     // cleared only after it, so its `claiming` record does not outlive the session.
     await this.reconciling?.catch(() => {})
     await Promise.all([stopping, this.dropHeld()])
+    this.updateBusy()
     await this.options.journal.clear()
   }
 
@@ -441,7 +442,9 @@ export class DesktopExecutor {
   }
 
   private updateBusy(): void {
-    const busy = this.held.size > 0 || this.recoveredInFlight > 0
+    // A disposed executor reports idle once, at dispose; a delivery that settles later must not
+    // speak for the executor that replaced it.
+    const busy = !this.disposed && (this.held.size > 0 || this.recoveredInFlight > 0)
     if (busy === this.busy) return
     this.busy = busy
     this.options.onBusyChange?.(busy)