fix(vscode): show tool name in permission prompt title#8813
fix(vscode): show tool name in permission prompt title#8813KeWang0622 wants to merge 1 commit intoKilo-Org:mainfrom
Conversation
The auto-approve permission prompt title now includes the tool name (e.g. "Permission required: Read" or "Permission required: playwright_click") so users can quickly identify which tool is requesting approval without expanding the checkboxes. Uses the existing resolveLabel utility which returns human-readable names for built-in tools and falls back to the raw name for MCP tools. Fixes Kilo-Org#8738
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files)
Reviewed by gpt-5.4-20260305 · 175,783 tokens |
|
@KeWang0622 can you show some screenshots? Before after? Thanks |
|
@marius-kilocode Sure! I don't have screenshots of the VS Code extension running, but here's what the change does: Before: The permission prompt title shows only "Permission required" with no indication of which tool is requesting access. After: The title shows "Permission required: Read" or "Permission required: playwright_click" — the tool name is appended using the existing The change is 7 net lines in Happy to add a screenshot if you can point me to the quickest way to build and run the extension locally — I'll spin it up and capture the before/after. |
Context
Fixes #8738
The auto-approve permission prompt currently shows only "Permission required" in the title, with no indication of which tool is requesting approval. Users must expand the checkbox section to discover the tool name, which hurts discoverability — especially for MCP tools where the name carries important context (e.g.
playwright_click,github_create_issue).Implementation
Appends the resolved tool label to the permission prompt title via the existing
resolveLabelutility:"Permission required: Read","Permission required: Bash""Permission required: playwright_click""Permission required (subagent): Edit"One-file change, 9 lines added (net +7). No new dependencies, no i18n key changes, no CSS changes.
How to Test
bun run extensionScreenshots
N/A — the change appends text to an existing header string. Before:
"Permission required". After:"Permission required: Read".