fix(cursor): clean up PID file on startDaemon failure and improve daemon robustness

- Add removePidFile() in safeResolve on failure to prevent stale PIDs
- Only send SIGKILL in stopDaemon if SIGTERM wait loop exhausted
- Check isDaemonRunning before model fetch to avoid 5s timeout
- Add port validation unit tests for startDaemon

src/cursor/cursor-daemon.ts

@@ -150,6 +150,7 @@ export async function startDaemon(
       if (resolved) return;
       resolved = true;
       if (checkTimeout) clearTimeout(checkTimeout);
+      if (!result.success) removePidFile();
       resolve(result);
     };
 
@@ -280,11 +281,13 @@ export async function stopDaemon(): Promise<{ success: boolean; error?: string }
       }
     }
 
-    // Escalate to SIGKILL if process still alive after SIGTERM attempts
-    try {
-      process.kill(pid, 'SIGKILL');
-    } catch {
-      // Already dead — good
+    // Escalate to SIGKILL only if SIGTERM wait loop exhausted
+    if (attempts >= 10) {
+      try {
+        process.kill(pid, 'SIGKILL');
+      } catch {
+        // Already dead — good
+      }
     }
 
     removePidFile();

src/cursor/cursor-models.ts

@@ -7,6 +7,7 @@
 
 import * as http from 'http';
 import type { CursorModel } from './types';
+import { isDaemonRunning } from './cursor-daemon';
 
 /** Default daemon port */
 export const DEFAULT_CURSOR_PORT = 4242;
@@ -142,8 +143,12 @@ export async function fetchModelsFromDaemon(port: number): Promise<CursorModel[]
 
 /**
  * Get available models (from daemon or defaults).
+ * Checks daemon health first to avoid 5s timeout when daemon is not running.
  */
 export async function getAvailableModels(port: number): Promise<CursorModel[]> {
+  if (!(await isDaemonRunning(port))) {
+    return DEFAULT_CURSOR_MODELS;
+  }
   return fetchModelsFromDaemon(port);
 }
 

tests/unit/cursor/cursor-daemon.test.ts

@@ -13,6 +13,7 @@ import {
   isDaemonRunning,
   getDaemonStatus,
   stopDaemon,
+  startDaemon,
 } from '../../../src/cursor/cursor-daemon';
 
 // Test isolation
@@ -116,6 +117,26 @@ describe('removePidFile', () => {
   });
 });
 
+describe('startDaemon', () => {
+  it('rejects invalid port (0)', async () => {
+    const result = await startDaemon({ port: 0, model: 'test' });
+    expect(result.success).toBe(false);
+    expect(result.error).toContain('Invalid port');
+  });
+
+  it('rejects invalid port (65536)', async () => {
+    const result = await startDaemon({ port: 65536, model: 'test' });
+    expect(result.success).toBe(false);
+    expect(result.error).toContain('Invalid port');
+  });
+
+  it('rejects non-integer port', async () => {
+    const result = await startDaemon({ port: 3.14, model: 'test' });
+    expect(result.success).toBe(false);
+    expect(result.error).toContain('Invalid port');
+  });
+});
+
 describe('isDaemonRunning', () => {
   it('returns false when no daemon is running on port', async () => {
     // Use a port that should not have anything running