@gotcos/glasses-server 6.18.6 → 6.18.7

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 6.18.7
+
+- **Phone Restart no longer leaves the server Stopped.** Control LaunchAgent
+  KeepAlive is `SuccessfulExit: false`, so the old SIGTERM→exit(0) path never
+  came back — phone showed "Restart requested; reconnect is taking longer than
+  expected" while Control sat at Stopped. Restart now `launchctl kickstart -k`
+  (fallback `exit(1)` so KeepAlive still fires).
+
 ## 6.18.6
 
 - **Phone Restart works on COS Control managed installs.** Recovery status /

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotcos/glasses-server",
-  "version": "6.18.6",
+  "version": "6.18.7",
   "description": "COS Glasses \u2014 self-hosted AI heads-up-display server for Even G2 smart glasses, powered by Claude Code, Codex, or Cursor Agent CLI",
   "type": "module",
   "bin": {

package/server/lib/managed-restart.ts

@@ -0,0 +1,47 @@
+import { spawn } from 'node:child_process'
+import { userInfo } from 'node:os'
+
+/** LaunchAgent label Control installs for the managed glasses server. */
+export const DEFAULT_LAUNCHD_LABEL = 'com.cos.glasses-server'
+
+/**
+ * COS Control's LaunchAgent uses KeepAlive SuccessfulExit:false — a clean
+ * SIGTERM/exit(0) leaves the service STOPPED. Phone Restart must either
+ * kickstart -k (kill + start) or exit non-zero so KeepAlive brings it back.
+ */
+export function scheduleManagedServerRestart(options?: {
+  delayMs?: number
+  label?: string
+  kickstart?: (args: string[]) => void
+  exitProcess?: (code: number) => void
+  onError?: (error: unknown) => void
+}): void {
+  const delayMs = options?.delayMs ?? 350
+  const label = (options?.label ?? process.env.COS_LAUNCHD_LABEL?.trim()) || DEFAULT_LAUNCHD_LABEL
+  const kickstart = options?.kickstart ?? ((args: string[]) => {
+    const child = spawn('launchctl', args, { detached: true, stdio: 'ignore' })
+    child.unref()
+  })
+  const exitProcess = options?.exitProcess ?? ((code: number) => { process.exit(code) })
+  const onError = options?.onError ?? ((error: unknown) => {
+    console.error('[recovery] Failed to schedule managed server restart:', error)
+  })
+
+  const timer = setTimeout(() => {
+    if (process.env.COS_DISABLE_SELF_RESTART === '1') return
+    try {
+      const uid = userInfo().uid
+      kickstart(['kickstart', '-k', `gui/${uid}/${label}`])
+      // If kickstart did not kill us quickly (misconfigured label), force a
+      // non-zero exit so KeepAlive SuccessfulExit:false still restarts.
+      setTimeout(() => {
+        if (process.env.COS_DISABLE_SELF_RESTART === '1') return
+        try { exitProcess(1) } catch (error) { onError(error) }
+      }, 2_000).unref?.()
+    } catch (error) {
+      onError(error)
+      try { exitProcess(1) } catch (exitError) { onError(exitError) }
+    }
+  }, delayMs)
+  timer.unref?.()
+}

package/server/routes/recovery.ts

@@ -2,6 +2,7 @@ import { Router } from 'express'
 import { readFileSync } from 'node:fs'
 import { resolve } from 'node:path'
 import { atomicWriteFileSync } from '../lib/atomic-fs.js'
+import { scheduleManagedServerRestart } from '../lib/managed-restart.js'
 import { isManagedRuntime } from '../lib/managed-runtime.js'
 import { acquireMaintenance, getRecoveryActivityStatus } from '../lib/recovery-activity.js'
 import { getWhisperHealth, restartWhisperServer } from '../lib/whisper-local.js'
@@ -65,15 +66,15 @@ recoveryRouter.post('/recovery/server/restart', (_req, res) => {
   res.status(202).json({ accepted: true, oldBootId: serverMetrics.bootId })
   // Schedule independently of the response socket. The phone may change
   // network/close the sheet immediately after receiving 202; that must not
-  // cancel an accepted restart or strand maintenance forever.
-  const timer = setTimeout(() => {
-    if (process.env.COS_DISABLE_SELF_RESTART === '1') { gate.release(); return }
-    try {
-      process.kill(process.pid, 'SIGTERM')
-    } catch (error) {
+  // cancel an accepted restart. Maintenance is in-process only — it dies with us.
+  //
+  // Do NOT SIGTERM→exit(0): Control's LaunchAgent KeepAlive is
+  // SuccessfulExit:false, so a clean exit leaves the service STOPPED and the
+  // phone shows "Restart requested; reconnect is taking longer than expected."
+  scheduleManagedServerRestart({
+    onError: (error) => {
       gate.release()
-      console.error('[recovery] Failed to signal managed server restart:', error)
-    }
-  }, 350)
-  timer.unref?.()
+      console.error('[recovery] Failed to schedule managed server restart:', error)
+    },
+  })
 })