fix(copilot): add dangerously_skip_permissions alias for --allow-all

[pocky]

Summary

• Add dangerously_skip_permissions as a cross-provider alias for allow_all in the Copilot provider, mapping it to the --allow-all CLI flag
• Ensures workflows using the Claude-style dangerously_skip_permissions option work without modification when targeting Copilot, improving provider portability
• When both allow_all and dangerously_skip_permissions are set, allow_all takes precedence and --allow-all appears exactly once in the CLI args

Changes

Infrastructure

• internal/infrastructure/agents/copilot_provider.go: Add else if branch in appendCopilotOptions to handle dangerously_skip_permissions as an alias for --allow-all

Tests

• internal/infrastructure/agents/copilot_provider_unit_test.go: Add table-driven test case for dangerously_skip_permissions alias, plus two dedicated tests — one verifying allow_all takes precedence (deduplication), one verifying false value omits the flag

Test plan

• make test-unit passes with no failures
• Run a Copilot workflow with dangerously_skip_permissions: true and confirm --allow-all is passed to the CLI
• Run with both allow_all: true and dangerously_skip_permissions: true and confirm --allow-all appears exactly once
• Run with dangerously_skip_permissions: false and confirm --allow-all is absent from CLI args

---

Generated with awf commit workflow