Skip to content

fix: follow the Codex approval policy and drop the approvals option - #3

Merged
malhashemi merged 2 commits into
mainfrom
fix/remove-approvals-option
Sep 25, 2026
Merged

malhashemi merged 2 commits into
mainfrom
fix/remove-approvals-option

Conversation

@malhashemi

Copy link
Copy Markdown
Owner

What and why

The approvals option offered three ways to answer Computer Use's "Allow Computer Use to use App?" prompts, but OpenCode's plugin API cannot show those prompts to the user (plugins can list, read and reply to permission requests, not create them). Every mode was a workaround: accept-session approved everything, decline made the plugin unusable, and codex declined forwarded prompts silently.

App access now always follows the Codex approval_policy and the approvals stored in ChatGPT/Codex:

  • A prompt Codex forwards is declined with a note that names the app and tells the user how to allow it.
  • /computer-use-doctor reports the effective policy (read with the app-server's config/read).
  • A leftover approvals setting is ignored with a warning instead of failing.
  • The README gains an "App access" section explaining the behavior.

Once OpenCode lets plugins raise permission requests, these prompts can become OpenCode permission questions.

How I tested it

  • bun run check (43 tests, including the actionable decline note, config/read parsing and the removed option).
  • bun run doctor against the real engine on macOS 26.6.2: reports approval_policy = "never".

Checklist

  • The title follows Conventional Commits
  • bun run check passes
  • Docs updated

The approvals option offered three ways to answer Computer Use's
"Allow Computer Use to use <app>?" prompts, but OpenCode's plugin API
cannot show those prompts to the user, so every mode was a workaround:
accept-session approved everything and decline broke the plugin.

App access now always follows the Codex approval_policy and the
approvals stored in ChatGPT/Codex. A prompt Codex forwards is declined
with a note that names the app and tells the user how to allow it.
/computer-use-doctor reports the effective policy. A leftover approvals
setting is ignored with a warning.
deny hides the tools; ask does not prompt yet for external plugin tools
(anomalyco/opencode#50652). Link the plugin permission API PR that will
let app prompts appear in OpenCode.
@malhashemi
malhashemi merged commit 1afbeaa into main Sep 25, 2026
3 checks passed
@malhashemi
malhashemi deleted the fix/remove-approvals-option branch September 25, 2026 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant