Start-Copilot: map --assisted-approval, --allow-all-tools, --usage-output-file, native --enable-mcp-server - #121
Merged
Merged
Conversation
Add first-class parameters for Copilot CLI flags that previously had no mapping
(reachable only via -RemainingArgs):
- -AssistedApproval -> --assisted-approval (assisted-approval safety judge)
- -AllowAllTools -> --allow-all-tools (auto-approve tools while keeping
path/URL verification; implies not passing --allow-all)
- -UsageOutputFile -> --usage-output-file <path> (write usage JSON)
Also extend -EnableMcpServer to pass the CLI's native --enable-mcp-server, so a
server disabled in the Copilot settings is enabled for the run, in addition to
the existing path-based autoConnect override.
Parameters are added to Get-CopilotLaunchPlan (the shared core) and surfaced on
Start-Copilot, which auto-forwards them. Tests, README, and CHANGELOG updated.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: b98f01a0-a039-492f-8576-b1917e54742d
- Warn when -AssistedApproval is combined with -NoExperimental (the CLI needs experimental mode for the assisted-approval judge to engage). - Document that -AllowAllTools keeps file-path/URL verification, so fully non-interactive (-Prompt) use may also need -AllowAllPaths / -AllowAllUrls. - Document that -EnableMcpServer's native --enable-mcp-server is a no-op for an already-enabled server; refresh the README feature list. - Add interaction tests: NoExperimental+AssistedApproval warning, NoAllowAll+AllowAllTools suppression, native enable with an mcp-config present (server enabled, not disabled), usage-output-file value adjacency, and -EnableMcpServer passthrough via Start-Copilot -PassThru. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: b98f01a0-a039-492f-8576-b1917e54742d
Owner
Author
|
Peer review (Correctness, Skeptic, Craft panel). No blocking bugs — the suppression logic, no-double-emit, parameter parity, and
Copilot suite now 69 passed, 0 failed; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds first-class
Start-Copilot/Get-CopilotLaunchPlanparameters for Copilot CLI launch flags that previously had no mapping (only reachable via-RemainingArgs), and extends the MCP enable behavior.-AssistedApproval--assisted-approval-AllowAllTools--allow-all-tools--allow-all-UsageOutputFile--usage-output-file <path>-Prompt-EnableMcpServer(existing)--enable-mcp-serverautoConnectpolicyThe parameters live on
Get-CopilotLaunchPlan(the shared core that builds the argument vector);Start-Copilotsurfaces them and auto-forwards every bound parameter the core accepts.-AllowAllToolssuppresses the default--allow-allso the two are not emitted together.Tests (6 new cases asserting the emitted args, the allow-all suppression, native enable-mcp emission, and
Start-Copilot -PassThruforwarding), README, and CHANGELOG updated. Full Copilot suite: 65 passed, 0 failed;Test-Modules.ps1green.Closes #117
Closes #118
Closes #119
Closes #120