fix: add request expiration to extension queue to prevent stale reque…

packages/chrome-extension/src/protocol.ts

@@ -7,6 +7,7 @@
   id: number;
   method: string;
   params: Record<string, unknown>;
+  _deadline?: number;
 }
 
 export interface BridgeResponse {
@@ -51,7 +52,7 @@
     }
 
     this.ws.onopen = () => {
       console.log('[Markus] Connected to bridge');
       this._connected = true;
       this.startKeepalive();
       chrome.action.setIcon({ path: {
@@ -62,7 +63,7 @@
     };
 
     this.ws.onclose = () => {
       console.log('[Markus] Disconnected from bridge');
       this._connected = false;
       this.stopKeepalive();
       chrome.action.setTitle({ title: 'Markus Browser Automation (Disconnected)' });
@@ -89,12 +90,21 @@
    * when multiple agents issue concurrent tool calls.
    */
   private enqueueRequest(req: BridgeRequest): void {
+    if (!req._deadline) {
+      req._deadline = Date.now() + 110_000;
+    }
     this.requestQueue = this.requestQueue
       .then(() => this.handleRequest(req))
       .catch((err) => console.error('[Markus] Request queue error:', err));
   }
 
   private async handleRequest(req: BridgeRequest): Promise<void> {
+    if (req._deadline && Date.now() > req._deadline) {
+      console.warn(`[Markus] Skipping expired request ${req.method} (id=${req.id})`);
+      this.send({ id: req.id, error: `Request expired while queued (bridge timeout likely already fired)` });
+      return;
+    }
+
     const handler = this.handlers.get(req.method);
     if (!handler) {
       this.send({ id: req.id, error: `Unknown method: ${req.method}` });

packages/core/src/tools/file.ts

@@ -52,7 +52,7 @@ export function createFileReadTool(security?: SecurityGuard, workspacePath?: str
     },
 
     async execute(args: Record<string, unknown>): Promise<string> {
-      const rawPath = (args['path'] ?? args['file_path'] ?? args['filePath']) as string | undefined;
+      const rawPath = (args['path'] ?? args['file'] ?? args['file_path'] ?? args['filePath']) as string | undefined;
       if (!rawPath || typeof rawPath !== 'string') {
         return JSON.stringify({ status: 'error', error: 'path is required and must be a non-empty string' });
       }
@@ -145,7 +145,7 @@ export function createFileWriteTool(security?: SecurityGuard, workspacePath?: st
     },
 
     async execute(args: Record<string, unknown>): Promise<string> {
-      const rawPath = (args['path'] ?? args['file_path'] ?? args['filePath']) as string;
+      const rawPath = (args['path'] ?? args['file'] ?? args['file_path'] ?? args['filePath']) as string;
       const { resolved: path, access } = resolveAndCheckAccess(rawPath, workspacePath, policy);
 
       if (access === 'denied') {
@@ -187,7 +187,7 @@ export function createFileEditTool(security?: SecurityGuard, workspacePath?: str
     },
 
     async execute(args: Record<string, unknown>): Promise<string> {
-      const rawPath = (args['path'] ?? args['file_path'] ?? args['filePath']) as string;
+      const rawPath = (args['path'] ?? args['file'] ?? args['file_path'] ?? args['filePath']) as string;
       const { resolved: path, access } = resolveAndCheckAccess(rawPath, workspacePath, policy);
 
       if (access === 'denied') {

packages/core/src/tools/markus-browser-bridge.ts

@@ -124,7 +124,8 @@ export class MarkusBrowserBridge {
     }
 
     const id = ++this.requestId;
-    const message = JSON.stringify({ id, method: name, params: args });
+    const _deadline = Date.now() + TOOL_CALL_TIMEOUT_MS;
+    const message = JSON.stringify({ id, method: name, params: args, _deadline });
 
     return new Promise<BridgeToolResult>((resolve, reject) => {
       const timer = setTimeout(() => {