fix: remove AskUserQuestion from allowed-tools to restore interactive picker (#504)

AskUserQuestion listed in allowed-tools causes the permission system to
auto-approve it, bypassing the requiresUserInteraction() guard that
renders the interactive picker. Removing it from all command and agent
frontmatter forces AskUserQuestion through the normal permission path
where the picker actually renders.

Fixes #503

Author: costajohnt

agents/contribution-strategist.md

@@ -40,7 +40,7 @@ User wants help with goal-setting.
 
 model: inherit
 color: magenta
-tools: ["Bash", "Read", "Write", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Write", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/issue-scout.md

@@ -22,7 +22,7 @@ User wants to evaluate a specific issue before investing time.
 
 model: inherit
 color: green
-tools: ["Bash", "Read", "Write", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Write", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/pr-compliance-checker.md

@@ -31,7 +31,7 @@ User wants pre-submission guidance on PR quality.
 
 model: inherit
 color: orange
-tools: ["Bash", "Read", "Glob", "Grep", "WebFetch", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Glob", "Grep", "WebFetch", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/pr-health-checker.md

@@ -31,7 +31,7 @@ Merge conflicts are a health issue this agent handles.
 
 model: inherit
 color: yellow
-tools: ["Bash", "Read", "Write", "Grep", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Write", "Grep", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/pr-responder.md

@@ -22,7 +22,7 @@ User needs help understanding and responding to a specific code review comment.
 
 model: inherit
 color: cyan
-tools: ["Bash", "Read", "Glob", "Grep", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Glob", "Grep", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/pre-commit-reviewer.md

@@ -22,7 +22,7 @@ Post-conflict resolution is a critical moment where bugs can be introduced. Revi
 
 model: inherit
 color: red
-tools: ["Bash", "Read", "Glob", "Grep", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Glob", "Grep", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

agents/repo-evaluator.md

@@ -22,7 +22,7 @@ User wants to predict maintainer engagement before contributing.
 
 model: inherit
 color: blue
-tools: ["Bash", "Read", "Write", "Glob", "AskUserQuestion", "mcp__*"]
+tools: ["Bash", "Read", "Write", "Glob", "mcp__*"]
 ---
 
 > **Input validation:** The "AskUserQuestion Validation Protocol" from `commands/oss.md` applies to ALL `AskUserQuestion` calls in this agent. After every call, check for empty/missing answers and fall back to text-based input if the picker auto-completed.

commands/oss-search.md

@@ -1,7 +1,7 @@
 ---
 name: oss-search
 description: "Search for new open source issues to contribute to — parallel multi-strategy search with vetting"
-allowed-tools: Bash, Read, Write, Edit, Glob, Grep, AskUserQuestion, Task, mcp__*
+allowed-tools: Bash, Read, Write, Edit, Glob, Grep, Task, mcp__*
 ---
 
 # OSS Issue Search

commands/oss.md

@@ -1,7 +1,7 @@
 ---
 name: oss
 description: "Daily OSS contribution check - uses CLI with --json for structured data"
-allowed-tools: Bash, Read, Write, Edit, Glob, Grep, AskUserQuestion, Task, mcp__*
+allowed-tools: Bash, Read, Write, Edit, Glob, Grep, Task, mcp__*
 ---
 
 # OSS Autopilot Daily Check

commands/setup-oss.md

@@ -1,7 +1,7 @@
 ---
 name: setup-oss
 description: Configure OSS autopilot preferences
-allowed-tools: Bash, Write, Read, Glob, AskUserQuestion, mcp__*
+allowed-tools: Bash, Write, Read, Glob, mcp__*
 ---
 
 # OSS Autopilot Setup