David KayaandCopilot 042cec6065 fix: resolve hook permissions to proper categories for approval ...

When the pre-tool-use hook returns 'ask', the Copilot CLI creates
PermissionRequestHook instead of categorized PermissionRequestRead/
Write/Shell. This caused 'Permission: hook' labels and broke category-
based auto-approval ('Always approve read' wouldn't cover grep/glob).

Add ResolveHookToolCategory mapping in CopilotApprovalCoordinator to
map known tool names (view/grep/glob→read, edit/create→write, etc.)
to their permission categories. Wire into GetFallbackToolName,
BuildPermissionApprovalEvent, and CreateApprovalPolicyOutput so:
- Approval banner shows 'Permission: read' instead of 'Permission: hook'
- 'Always approve' stores the category key, covering all tools in it
- Hook short-circuits when category is already auto-approved
Unknown tools (MCP, custom) keep existing 'hook' behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

2026-03-30 09:41:51 +02:00

..

src/Aryx.AgentHost

fix: resolve hook permissions to proper categories for approval

2026-03-30 09:41:51 +02:00

tests/Aryx.AgentHost.Tests

fix: resolve hook permissions to proper categories for approval

2026-03-30 09:41:51 +02:00

Aryx.AgentHost.slnx

refactor: rename sidecar host to Aryx

2026-03-26 00:14:12 +01:00