@saadnvd1/agent-os 0.3.821 → 0.3.824

package/content/whats-new.json

@@ -1921,6 +1921,22 @@
       "area": "AgentOS",
       "text": "You can set your name in Settings, and AgentOS uses it in chats and notes instead of a fixed name.",
       "url": "https://github.com/saadnvd1/agent-os/pull/550"
+    },
+    {
+      "id": 551,
+      "date": "2026-11-01",
+      "kind": "fixed",
+      "area": "Chat",
+      "text": "Questions from GitHub Copilot chats now always show up at the latest message after the chat restarts, instead of going missing.",
+      "url": "https://github.com/saadnvd1/agent-os/pull/551"
+    },
+    {
+      "id": 552,
+      "date": "2026-11-01",
+      "kind": "fixed",
+      "area": "Chat",
+      "text": "Cancelled questions from GitHub Copilot chats now clear right away instead of staying on screen.",
+      "url": "https://github.com/saadnvd1/agent-os/pull/552"
     }
   ]
 }

package/lib/chat/drivers/copilot-ask.ts

@@ -66,6 +66,8 @@ export function askUserQuestion(a: AskArgs): ChatQuestion {
   };
 }
 
+const requestKey = (id: string | number) => `${typeof id}:${id}`;
+
 const text = (t: string, isError = false): CallToolResult => ({
   content: [{ type: "text", text: t }],
   ...(isError ? { isError } : {}),
@@ -105,8 +107,10 @@ export function isAskUserCall(call: P): boolean {
 export class CopilotAskServer {
   private http?: http.Server;
   private secret = crypto.randomBytes(32).toString("hex");
-  private next = 0;
   private live = new Set<{ close(): Promise<void> }>();
+  // Waiting ask_user calls by JSON-RPC request id. A cancel arrives as its
+  // own POST, on a fresh MCP server, so only this map can match it.
+  private waiting = new Map<string, string>();
 
   constructor(private pending: PendingApprovals) {}
 
@@ -119,22 +123,49 @@ export class CopilotAskServer {
           "Ask the user a question and wait for the answer. Use it when you need a decision or missing information instead of asking in plain text. Offer options for a choice (multiSelect to allow several); with no options the user types an answer. Blocks until they answer.",
         inputSchema: askInput,
       },
-      async (args) => {
+      async (args, extra) => {
         const q = askUserQuestion(args);
-        const o = await this.pending.ask({
-          id: `approval-ask-${++this.next}`,
+        // Unique across worker restarts: a card's id is its chat row, so a
+        // reused id would rewrite an old card up the history instead.
+        const id = `approval-ask-${crypto.randomUUID()}`;
+        const asked = this.pending.ask({
+          id,
           toolName: "AskUserQuestion",
           title: "Questions",
           input: { questions: [q] },
           questions: [q],
           canAlways: false,
         });
-        return askUserResult(q, o);
+        // Copilot gave up on the call (its timeout, or the connection went):
+        // nobody is waiting for this answer any more.
+        const expire = () => this.pending.respond(id, { decision: "expired" });
+        if (extra.signal.aborted) expire();
+        else extra.signal.addEventListener("abort", expire, { once: true });
+        const key = requestKey(extra.requestId);
+        this.waiting.set(key, id);
+        try {
+          return askUserResult(q, await asked);
+        } finally {
+          if (this.waiting.get(key) === id) this.waiting.delete(key);
+        }
       }
     );
     return server;
   }
 
+  // notifications/cancelled names the request to give up on. An id with no
+  // waiting call (answered already, or never ours) is ignored.
+  private cancelWaiting(body: unknown): void {
+    for (const m of Array.isArray(body) ? body : [body]) {
+      const msg = m as { method?: unknown; params?: { requestId?: unknown } };
+      const rid = msg?.params?.requestId;
+      if (msg?.method !== "notifications/cancelled") continue;
+      if (typeof rid !== "string" && typeof rid !== "number") continue;
+      const card = this.waiting.get(requestKey(rid));
+      if (card) this.pending.respond(card, { decision: "expired" });
+    }
+  }
+
   private allowed(req: http.IncomingMessage): number {
     if (req.headers.origin) return 403;
     const host = (req.headers.host ?? "").replace(/:\d+$/, "");
@@ -176,6 +207,7 @@ export class CopilotAskServer {
       res.writeHead(400).end();
       return;
     }
+    this.cancelWaiting(body);
     const server = this.build();
     const transport = new StreamableHTTPServerTransport({
       sessionIdGenerator: undefined,
@@ -222,6 +254,7 @@ export class CopilotAskServer {
   // and stops listening.
   close(): void {
     this.pending.expireAll();
+    this.waiting.clear();
     const server = this.http;
     this.http = undefined;
     server?.close();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saadnvd1/agent-os",
-  "version": "0.3.821",
+  "version": "0.3.824",
   "description": "Self-hosted web UI for managing Claude Code sessions",
   "bin": {
     "agent-os": "scripts/agent-os"