From e70e46cbdad2a9de9d3e6324973f3091bf5be31e Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 03:47:15 +0000
Subject: [PATCH 1/3] Initial plan


From 7ab7839397b6e2033d844cc44a8a8eaee0328997 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 03:54:03 +0000
Subject: [PATCH 2/3] fix: support multiple Claude ask question answers

Co-authored-by: OpenSource03 <29690431+OpenSource03@users.noreply.github.com>
---
 src/components/PermissionPrompt.tsx           | 47 ++++++------
 .../tool-renderers/AskUserQuestion.tsx        | 17 +----
 src/lib/ask-user-question.test.ts             | 73 +++++++++++++++++++
 src/lib/ask-user-question.ts                  | 62 ++++++++++++++++
 src/types/protocol.ts                         |  2 +
 vitest.config.electron.ts                     |  3 +-
 6 files changed, 168 insertions(+), 36 deletions(-)
 create mode 100644 src/lib/ask-user-question.test.ts
 create mode 100644 src/lib/ask-user-question.ts

diff --git a/src/components/PermissionPrompt.tsx b/src/components/PermissionPrompt.tsx
index 94e5798..846f360 100644
--- a/src/components/PermissionPrompt.tsx
+++ b/src/components/PermissionPrompt.tsx
@@ -1,6 +1,7 @@
 import { useState } from "react";
 import { ShieldAlert, Check, X, Send, Play } from "lucide-react";
 import { Button } from "@/components/ui/button";
+import { getAskUserQuestionKey } from "@/lib/ask-user-question";
 import type { PermissionRequest, RespondPermissionFn } from "@/types";
 
 const TOOL_LABELS: Record<string, string> = {
@@ -24,7 +25,7 @@ interface QuestionOption {
 }
 
 interface Question {
-  id: string;
+  id?: string;
   question: string;
   header: string;
   isOther?: boolean;
@@ -128,16 +129,16 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
   const questions = (request.toolInput.questions ?? []) as Question[];
   const [selections, setSelections] = useState<Record<string, Set<string>>>(() => {
     const init: Record<string, Set<string>> = {};
-    for (const q of questions) {
-      init[q.id] = new Set();
+    for (const [index, q] of questions.entries()) {
+      init[getAskUserQuestionKey(q, index)] = new Set();
     }
     return init;
   });
   const [freeText, setFreeText] = useState<Record<string, string>>({});
 
-  const toggleOption = (questionId: string, label: string, multiSelect: boolean) => {
+  const toggleOption = (questionKey: string, label: string, multiSelect: boolean) => {
     setSelections((prev) => {
-      const current = prev[questionId] ?? new Set();
+      const current = prev[questionKey] ?? new Set();
       const next = new Set(current);
       if (multiSelect) {
         if (next.has(label)) next.delete(label);
@@ -149,23 +150,24 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
           next.add(label);
         }
       }
-      return { ...prev, [questionId]: next };
+      return { ...prev, [questionKey]: next };
     });
-    setFreeText((prev) => ({ ...prev, [questionId]: "" }));
+    setFreeText((prev) => ({ ...prev, [questionKey]: "" }));
   };
 
   const handleSubmit = () => {
     const answers: Record<string, string> = {};
     const answersByQuestionId: Record<string, string[]> = {};
-    for (const q of questions) {
-      const custom = freeText[q.id]?.trim();
+    for (const [index, q] of questions.entries()) {
+      const questionKey = getAskUserQuestionKey(q, index);
+      const custom = freeText[questionKey]?.trim();
       if (custom) {
         answers[q.question] = custom;
-        answersByQuestionId[q.id] = [custom];
+        answersByQuestionId[questionKey] = [custom];
       } else {
-        const selected = [...(selections[q.id] ?? [])];
+        const selected = [...(selections[questionKey] ?? [])];
         answers[q.question] = selected.join(", ");
-        answersByQuestionId[q.id] = selected;
+        answersByQuestionId[questionKey] = selected;
       }
     }
     onRespond("allow", {
@@ -175,9 +177,10 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
     });
   };
 
-  const hasAllAnswers = questions.every((q) => {
-    const custom = freeText[q.id]?.trim();
-    const selected = selections[q.id];
+  const hasAllAnswers = questions.every((q, index) => {
+    const questionKey = getAskUserQuestionKey(q, index);
+    const custom = freeText[questionKey]?.trim();
+    const selected = selections[questionKey];
     return custom || (selected && selected.size > 0);
   });
 
@@ -186,19 +189,20 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
       <div className="pointer-events-auto rounded-2xl border border-border/60 bg-background/55 shadow-lg backdrop-blur-lg">
         {questions.map((q, qi) => (
           <div
-            key={q.id}
+            key={q.id ?? `${qi}-${q.question}`}
             className={`flex flex-col gap-3 px-4 py-3.5 ${qi > 0 ? "border-t border-border/40" : ""}`}
           >
             <p className="text-[13px] text-foreground">{q.question}</p>
 
             <div className="grid grid-cols-2 gap-1.5">
               {(q.options ?? []).map((opt) => {
-                const isSelected = selections[q.id]?.has(opt.label);
+                const questionKey = getAskUserQuestionKey(q, qi);
+                const isSelected = selections[questionKey]?.has(opt.label);
                 return (
                   <button
                     key={opt.label}
                     type="button"
-                    onClick={() => toggleOption(q.id, opt.label, q.multiSelect)}
+                    onClick={() => toggleOption(questionKey, opt.label, q.multiSelect)}
                     className={`flex flex-col items-start rounded-lg border px-3 py-2 text-start transition-colors ${
                       isSelected
                         ? "border-border bg-accent text-foreground"
@@ -215,12 +219,13 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
             <input
               type={q.isSecret ? "password" : "text"}
               placeholder="Or type your own answer..."
-              value={freeText[q.id] ?? ""}
+              value={freeText[getAskUserQuestionKey(q, qi)] ?? ""}
               onChange={(e) => {
                 const value = e.target.value;
-                setFreeText((prev) => ({ ...prev, [q.id]: value }));
+                const questionKey = getAskUserQuestionKey(q, qi);
+                setFreeText((prev) => ({ ...prev, [questionKey]: value }));
                 if (value.trim()) {
-                  setSelections((prev) => ({ ...prev, [q.id]: new Set() }));
+                  setSelections((prev) => ({ ...prev, [questionKey]: new Set() }));
                 }
               }}
               onKeyDown={(e) => {
diff --git a/src/components/tool-renderers/AskUserQuestion.tsx b/src/components/tool-renderers/AskUserQuestion.tsx
index a8c901d..36796b0 100644
--- a/src/components/tool-renderers/AskUserQuestion.tsx
+++ b/src/components/tool-renderers/AskUserQuestion.tsx
@@ -1,4 +1,5 @@
 import { Loader2 } from "lucide-react";
+import { getAskUserQuestionAnswer } from "@/lib/ask-user-question";
 import type { UIMessage } from "@/types";
 
 interface AskQuestionOption {
@@ -7,6 +8,7 @@ interface AskQuestionOption {
 }
 
 interface AskQuestionItem {
+  id?: string;
   question: string;
   header: string;
   options: AskQuestionOption[];
@@ -16,12 +18,6 @@ interface AskQuestionItem {
 export function AskUserQuestionContent({ message }: { message: UIMessage }) {
   const questions = (message.toolInput?.questions ?? []) as AskQuestionItem[];
   const hasResult = !!message.toolResult;
-  const answers = (() => {
-    const raw = message.toolResult?.answers;
-    if (!raw || typeof raw !== "object") return null;
-    return raw as Record<string, unknown>;
-  })();
-  const orderedAnswers = answers ? Object.values(answers) : [];
 
   return (
     <div className="space-y-2 text-xs">
@@ -47,14 +43,7 @@ export function AskUserQuestionContent({ message }: { message: UIMessage }) {
             <div className="mt-1.5">
               <span className="text-[11px] text-foreground/40">Answer: </span>
               <span className="text-[12px] text-foreground/80">
-                {(() => {
-                  const direct = answers?.[q.question];
-                  if (typeof direct === "string" && direct.trim()) return direct;
-                  // Fallback for edge cases where question text keys differ
-                  const indexed = orderedAnswers[qi];
-                  if (typeof indexed === "string" && indexed.trim()) return indexed;
-                  return "No answer captured";
-                })()}
+                {getAskUserQuestionAnswer(q, qi, message.toolResult)}
               </span>
             </div>
           )}
diff --git a/src/lib/ask-user-question.test.ts b/src/lib/ask-user-question.test.ts
new file mode 100644
index 0000000..2327446
--- /dev/null
+++ b/src/lib/ask-user-question.test.ts
@@ -0,0 +1,73 @@
+import { describe, expect, it } from "vitest";
+import { getAskUserQuestionAnswer, getAskUserQuestionKey } from "./ask-user-question";
+
+describe("getAskUserQuestionKey", () => {
+  it("uses a fallback key when Claude questions do not provide ids", () => {
+    expect(getAskUserQuestionKey({ question: "First question?" }, 0)).toBe("question-0");
+    expect(getAskUserQuestionKey({ question: "Second question?" }, 1)).toBe("question-1");
+  });
+
+  it("prefers the provided question id when available", () => {
+    expect(getAskUserQuestionKey({ id: "q-1", question: "First question?" }, 0)).toBe("q-1");
+  });
+});
+
+describe("getAskUserQuestionAnswer", () => {
+  it("reads answers keyed by question id", () => {
+    const answer = getAskUserQuestionAnswer(
+      { id: "q-1", question: "Pick one" },
+      0,
+      {
+        answersByQuestionId: {
+          "q-1": ["Option A"],
+        },
+      },
+    );
+
+    expect(answer).toBe("Option A");
+  });
+
+  it("falls back to answers keyed by question text for Claude", () => {
+    const answer = getAskUserQuestionAnswer(
+      { question: "Second question?" },
+      1,
+      {
+        answers: {
+          "First question?": "Alpha",
+          "Second question?": "Beta",
+        },
+      },
+    );
+
+    expect(answer).toBe("Beta");
+  });
+
+  it("falls back to indexed answers when keys do not match", () => {
+    const answer = getAskUserQuestionAnswer(
+      { question: "Second question?" },
+      1,
+      {
+        answers: {
+          first: "Alpha",
+          second: "Beta",
+        },
+      },
+    );
+
+    expect(answer).toBe("Beta");
+  });
+
+  it("supports Codex-style structured answers", () => {
+    const answer = getAskUserQuestionAnswer(
+      { id: "q-2", question: "Choose many" },
+      0,
+      {
+        answersByQuestionId: {
+          "q-2": { answers: ["One", "Two"] },
+        },
+      },
+    );
+
+    expect(answer).toBe("One, Two");
+  });
+});
diff --git a/src/lib/ask-user-question.ts b/src/lib/ask-user-question.ts
new file mode 100644
index 0000000..9d3e782
--- /dev/null
+++ b/src/lib/ask-user-question.ts
@@ -0,0 +1,62 @@
+interface AskUserQuestion {
+  id?: string;
+  question: string;
+}
+
+function asRecord(value: unknown): Record<string, unknown> | null {
+  if (!value || typeof value !== "object" || Array.isArray(value)) return null;
+  return value as Record<string, unknown>;
+}
+
+function formatAnswerValue(value: unknown): string | null {
+  if (typeof value === "string") {
+    const trimmed = value.trim();
+    return trimmed ? trimmed : null;
+  }
+
+  if (Array.isArray(value)) {
+    const parts = value
+      .map((item) => (typeof item === "string" ? item.trim() : ""))
+      .filter((item) => item.length > 0);
+    return parts.length > 0 ? parts.join(", ") : null;
+  }
+
+  const record = asRecord(value);
+  if (!record) return null;
+
+  return (
+    formatAnswerValue(record.answers) ??
+    formatAnswerValue(record.answer) ??
+    formatAnswerValue(record.value)
+  );
+}
+
+export function getAskUserQuestionKey(question: AskUserQuestion, index: number): string {
+  const id = typeof question.id === "string" ? question.id.trim() : "";
+  return id || `question-${index}`;
+}
+
+export function getAskUserQuestionAnswer(
+  question: AskUserQuestion,
+  index: number,
+  toolResult?: Record<string, unknown>,
+): string {
+  const answers = asRecord(toolResult?.answers);
+  const answersByQuestionId = asRecord(toolResult?.answersByQuestionId);
+  const questionKey = getAskUserQuestionKey(question, index);
+
+  const direct =
+    formatAnswerValue(answersByQuestionId?.[questionKey]) ??
+    formatAnswerValue(answers?.[questionKey]) ??
+    formatAnswerValue(answers?.[question.question]);
+  if (direct) return direct;
+
+  const orderedAnswersById = answersByQuestionId ? Object.values(answersByQuestionId) : [];
+  const orderedAnswers = answers ? Object.values(answers) : [];
+
+  return (
+    formatAnswerValue(orderedAnswersById[index]) ??
+    formatAnswerValue(orderedAnswers[index]) ??
+    "No answer captured"
+  );
+}
diff --git a/src/types/protocol.ts b/src/types/protocol.ts
index 65b073c..9da3be9 100644
--- a/src/types/protocol.ts
+++ b/src/types/protocol.ts
@@ -129,6 +129,8 @@ export interface ToolUseResult {
   totalToolUseCount?: number;
   /** AskUserQuestion answers keyed by question text */
   answers?: Record<string, unknown>;
+  /** AskUserQuestion answers keyed by stable question id when available */
+  answersByQuestionId?: Record<string, unknown>;
   /** ACP agents include a detailed version of the result (e.g., unified diff for edits/reads) */
   detailedContent?: string;
 }
diff --git a/vitest.config.electron.ts b/vitest.config.electron.ts
index 45ad5eb..eaabbb0 100644
--- a/vitest.config.electron.ts
+++ b/vitest.config.electron.ts
@@ -3,9 +3,10 @@ import path from "path";
 
 export default defineConfig({
   test: {
-    include: ["electron/src/**/*.test.ts"],
+    include: ["electron/src/**/*.test.ts", "src/**/*.test.{ts,tsx}"],
     environment: "node",
     alias: {
+      "@/": path.resolve(__dirname, "src/"),
       "@shared/": path.resolve(__dirname, "shared/"),
     },
     restoreMocks: true,

From baeb51637b4533557b9950dcd290338d170693c9 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 7 Mar 2026 05:04:08 +0000
Subject: [PATCH 3/3] test: harden Claude ask question regression coverage

Co-authored-by: OpenSource03 <29690431+OpenSource03@users.noreply.github.com>
---
 src/components/PermissionPrompt.tsx           | 17 +----
 .../tool-renderers/AskUserQuestion.tsx        |  4 +-
 src/lib/ask-user-question.test.ts             | 71 ++++++++++++++++++-
 src/lib/ask-user-question.ts                  | 29 +++++++-
 4 files changed, 102 insertions(+), 19 deletions(-)

diff --git a/src/components/PermissionPrompt.tsx b/src/components/PermissionPrompt.tsx
index 846f360..a41b510 100644
--- a/src/components/PermissionPrompt.tsx
+++ b/src/components/PermissionPrompt.tsx
@@ -1,7 +1,7 @@
 import { useState } from "react";
 import { ShieldAlert, Check, X, Send, Play } from "lucide-react";
 import { Button } from "@/components/ui/button";
-import { getAskUserQuestionKey } from "@/lib/ask-user-question";
+import { buildAskUserQuestionResult, getAskUserQuestionKey } from "@/lib/ask-user-question";
 import type { PermissionRequest, RespondPermissionFn } from "@/types";
 
 const TOOL_LABELS: Record<string, string> = {
@@ -156,20 +156,7 @@ function AskUserQuestionPrompt({ request, onRespond }: PermissionPromptProps) {
   };
 
   const handleSubmit = () => {
-    const answers: Record<string, string> = {};
-    const answersByQuestionId: Record<string, string[]> = {};
-    for (const [index, q] of questions.entries()) {
-      const questionKey = getAskUserQuestionKey(q, index);
-      const custom = freeText[questionKey]?.trim();
-      if (custom) {
-        answers[q.question] = custom;
-        answersByQuestionId[questionKey] = [custom];
-      } else {
-        const selected = [...(selections[questionKey] ?? [])];
-        answers[q.question] = selected.join(", ");
-        answersByQuestionId[questionKey] = selected;
-      }
-    }
+    const { answers, answersByQuestionId } = buildAskUserQuestionResult(questions, selections, freeText);
     onRespond("allow", {
       questions: request.toolInput.questions,
       answers,
diff --git a/src/components/tool-renderers/AskUserQuestion.tsx b/src/components/tool-renderers/AskUserQuestion.tsx
index 36796b0..b3c9d55 100644
--- a/src/components/tool-renderers/AskUserQuestion.tsx
+++ b/src/components/tool-renderers/AskUserQuestion.tsx
@@ -1,5 +1,5 @@
 import { Loader2 } from "lucide-react";
-import { getAskUserQuestionAnswer } from "@/lib/ask-user-question";
+import { getAskUserQuestionAnswer, getAskUserQuestionKey } from "@/lib/ask-user-question";
 import type { UIMessage } from "@/types";
 
 interface AskQuestionOption {
@@ -23,7 +23,7 @@ export function AskUserQuestionContent({ message }: { message: UIMessage }) {
     <div className="space-y-2 text-xs">
       {questions.map((q, qi) => (
         <div
-          key={q.question}
+          key={getAskUserQuestionKey(q, qi)}
           className={qi > 0 ? "border-t border-border/40 pt-2" : ""}
         >
           <span className="text-[13px] text-foreground/80 leading-snug">
diff --git a/src/lib/ask-user-question.test.ts b/src/lib/ask-user-question.test.ts
index 2327446..7df272d 100644
--- a/src/lib/ask-user-question.test.ts
+++ b/src/lib/ask-user-question.test.ts
@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { getAskUserQuestionAnswer, getAskUserQuestionKey } from "./ask-user-question";
+import { buildAskUserQuestionResult, getAskUserQuestionAnswer, getAskUserQuestionKey } from "./ask-user-question";
 
 describe("getAskUserQuestionKey", () => {
   it("uses a fallback key when Claude questions do not provide ids", () => {
@@ -13,6 +13,21 @@ describe("getAskUserQuestionKey", () => {
 });
 
 describe("getAskUserQuestionAnswer", () => {
+  it("reads answers keyed by fallback question ids when Claude questions have no ids", () => {
+    const answer = getAskUserQuestionAnswer(
+      { question: "Second question?" },
+      1,
+      {
+        answersByQuestionId: {
+          "question-0": ["Alpha"],
+          "question-1": ["Beta"],
+        },
+      },
+    );
+
+    expect(answer).toBe("Beta");
+  });
+
   it("reads answers keyed by question id", () => {
     const answer = getAskUserQuestionAnswer(
       { id: "q-1", question: "Pick one" },
@@ -71,3 +86,57 @@ describe("getAskUserQuestionAnswer", () => {
     expect(answer).toBe("One, Two");
   });
 });
+
+describe("buildAskUserQuestionResult", () => {
+  it("keeps answers separate for multiple Claude questions without ids", () => {
+    const result = buildAskUserQuestionResult(
+      [
+        { question: "First question?" },
+        { question: "Second question?" },
+      ],
+      {
+        "question-0": new Set(["Alpha"]),
+        "question-1": new Set(["Beta"]),
+      },
+      {},
+    );
+
+    expect(result).toEqual({
+      answers: {
+        "First question?": "Alpha",
+        "Second question?": "Beta",
+      },
+      answersByQuestionId: {
+        "question-0": ["Alpha"],
+        "question-1": ["Beta"],
+      },
+    });
+  });
+
+  it("prefers free text over selected options for the matching question only", () => {
+    const result = buildAskUserQuestionResult(
+      [
+        { question: "First question?" },
+        { question: "Second question?" },
+      ],
+      {
+        "question-0": new Set(["Alpha"]),
+        "question-1": new Set(["Beta"]),
+      },
+      {
+        "question-1": "Custom answer",
+      },
+    );
+
+    expect(result).toEqual({
+      answers: {
+        "First question?": "Alpha",
+        "Second question?": "Custom answer",
+      },
+      answersByQuestionId: {
+        "question-0": ["Alpha"],
+        "question-1": ["Custom answer"],
+      },
+    });
+  });
+});
diff --git a/src/lib/ask-user-question.ts b/src/lib/ask-user-question.ts
index 9d3e782..27d1324 100644
--- a/src/lib/ask-user-question.ts
+++ b/src/lib/ask-user-question.ts
@@ -1,3 +1,5 @@
+import type { ToolUseResult } from "@/types";
+
 interface AskUserQuestion {
   id?: string;
   question: string;
@@ -36,10 +38,35 @@ export function getAskUserQuestionKey(question: AskUserQuestion, index: number):
   return id || `question-${index}`;
 }
 
+export function buildAskUserQuestionResult(
+  questions: AskUserQuestion[],
+  selections: Record<string, Set<string>>,
+  freeText: Record<string, string>,
+): Pick<ToolUseResult, "answers" | "answersByQuestionId"> {
+  const answers: Record<string, string> = {};
+  const answersByQuestionId: Record<string, string[]> = {};
+
+  for (const [index, question] of questions.entries()) {
+    const questionKey = getAskUserQuestionKey(question, index);
+    const custom = freeText[questionKey]?.trim();
+    if (custom) {
+      answers[question.question] = custom;
+      answersByQuestionId[questionKey] = [custom];
+      continue;
+    }
+
+    const selected = [...(selections[questionKey] ?? [])];
+    answers[question.question] = selected.join(", ");
+    answersByQuestionId[questionKey] = selected;
+  }
+
+  return { answers, answersByQuestionId };
+}
+
 export function getAskUserQuestionAnswer(
   question: AskUserQuestion,
   index: number,
-  toolResult?: Record<string, unknown>,
+  toolResult?: Pick<ToolUseResult, "answers" | "answersByQuestionId">,
 ): string {
   const answers = asRecord(toolResult?.answers);
   const answersByQuestionId = asRecord(toolResult?.answersByQuestionId);