fix: address PR review findings

Cover the new question, approval, and prompt lifecycle events in the SDK
event exhaustiveness test, align the session listing test with the
intentional skip-invalid contract, and drop an unused path import from
the desktop staging script.

Author: elkaix

apps/desktop/scripts/stage-runtime.ts

@@ -4,7 +4,7 @@ import { spawn } from 'node:child_process'
 import { existsSync } from 'node:fs'
 import { cp, lstat, mkdir, mkdtemp, readFile, readdir, realpath, rm, writeFile } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
-import { dirname, join, resolve, sep } from 'node:path'
+import { join, resolve, sep } from 'node:path'
 
 const desktopRoot = resolve(import.meta.dirname, '..')
 const repositoryRoot = resolve(desktopRoot, '../..')

packages/node-sdk/test/list-sessions.test.ts

@@ -311,15 +311,15 @@ describe('SessionStore.list', () => {
     expect(sessions.find((session) => session.id === fallback.id)?.title).toBe('Fallback Title');
   });
 
-  it('rejects malformed indexed state instead of treating it as an incomplete summary', async () => {
+  it('skips malformed indexed state instead of treating it as an incomplete summary', async () => {
     const homeDir = await makeTempDir();
     const workDir = await makeTempDir();
     const store = new SessionStore(homeDir);
 
     const malformed = await createStoredSession(store, { id: 'ses_bad_state', workDir });
     await writeFile(join(malformed.sessionDir, 'state.json'), '{bad json', 'utf-8');
 
-    await expect(store.list({ workDir })).rejects.toMatchObject({ code: 'session.state_invalid' });
+    await expect(store.list({ workDir })).resolves.toEqual([]);
   });
 
   it('sorts by filesystem activity descending', async () => {

packages/node-sdk/test/session-event-types.test.ts

@@ -76,6 +76,13 @@ describe('Event public types', () => {
         case 'event.workspace.updated':
         case 'event.workspace.deleted':
         case 'event.config.changed':
+        case 'event.question.requested':
+        case 'event.question.answered':
+        case 'event.question.dismissed':
+        case 'event.question.expired':
+        case 'event.approval.requested':
+        case 'event.approval.resolved':
+        case 'event.approval.expired':
         case 'goal.updated':
         case 'skill.activated':
         case 'error':
@@ -111,6 +118,9 @@ describe('Event public types', () => {
         case 'background.task.terminated':
         case 'cron.fired':
         case 'prompt.submitted':
+        case 'prompt.completed':
+        case 'prompt.aborted':
+        case 'prompt.steered':
           return;
         default:
           assertNever(event);