fix: address code review findings

- Fix invalid HTTP header name: X-Pilot Memory-Client → X-Pilot-Memory-Client
- Complete truncated comment in ResponseProcessor.ts
- Add error handling for file read in PlanRoutes.ts

Author: maxritter

console/src/services/worker/agents/ResponseProcessor.ts

@@ -128,7 +128,7 @@ export async function processAgentResponse(
     agentName
   );
 
-  // Note: Don't mark session as completed here - summaries are generated during
+  // Note: Session completion is handled by stale cleanup, not here (summaries are generated during handoffs)
 
   cleanupProcessedMessages(session, worker);
 }

console/src/services/worker/http/routes/PlanRoutes.ts

@@ -102,13 +102,17 @@ export class PlanRoutes extends BaseRouteHandler {
         return;
       }
       const firstSpec = specs[0];
-      const content = readFileSync(firstSpec.filePath, 'utf-8');
-      res.json({
-        content,
-        name: firstSpec.name,
-        status: firstSpec.status,
-        filePath: firstSpec.filePath,
-      });
+      try {
+        const content = readFileSync(firstSpec.filePath, 'utf-8');
+        res.json({
+          content,
+          name: firstSpec.name,
+          status: firstSpec.status,
+          filePath: firstSpec.filePath,
+        });
+      } catch {
+        res.status(404).json({ error: 'Plan file not found' });
+      }
       return;
     }
 

console/src/shared/remote-endpoint.ts

@@ -48,7 +48,7 @@ export function getWorkerEndpointConfig(): WorkerEndpointConfig {
     if (remoteConfig.token) {
       authHeaders['Authorization'] = `Bearer ${remoteConfig.token}`;
     }
-    authHeaders['X-Pilot Memory-Client'] = 'local-hooks';
+    authHeaders['X-Pilot-Memory-Client'] = 'local-hooks';
 
     cachedEndpoint = {
       mode: 'remote',