docs: update Claude Code approve mode in cli-providers guide#7448
docs: update Claude Code approve mode in cli-providers guide#7448codefromthecrypt wants to merge 1 commit intomainfrom
Conversation
Signed-off-by: Adrian Cole <adrian@tetrate.io>
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 13d21d2d37
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| | `auto` | `--dangerously-skip-permissions` | Bypasses all permission prompts | | ||
| | `smart-approve` | `--permission-mode acceptEdits` | Auto-accepts edits, prompts for other actions | | ||
| | `approve` | Not supported | Returns an error | | ||
| | `approve` | `--permission-prompt-tool stdio` | Routes permission checks through the control protocol | |
There was a problem hiding this comment.
Mark Claude Code approve mode as unsupported
This row documents approve as supported via --permission-prompt-tool stdio, but in the same commit tree the Claude Code provider still rejects GooseMode::Approve with a ProviderError in crates/goose/src/providers/claude_code.rs (apply_permission_flags, lines 296-303). Users who follow this guide and set GOOSE_MODE=approve will hit an immediate runtime error, so the docs now describe behavior that the implementation does not provide.
Useful? React with 👍 / 👎.
Summary
Updates cli-providers documentation to reflect that Claude Code now supports approve mode via
--permission-prompt-tool stdio.Type of Change
AI Assistance
Related Issues
Docs from #7420