Skip to content

feat(provider-codex): support MCP elicitation requests (fixes Computer Use perms, etc) - #3113

Open
smsunarto wants to merge 6 commits into
get-bb:mainfrom
smsunarto:scott/codex-computer-use-approval
Open

feat(provider-codex): support MCP elicitation requests (fixes Computer Use perms, etc)#3113
smsunarto wants to merge 6 commits into
get-bb:mainfrom
smsunarto:scott/codex-computer-use-approval

Conversation

@smsunarto

@smsunarto smsunarto commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Human comments

What was wrong

BB did not handle Codex's mcpServer/elicitation/request, so app-access requests were silently declined and other MCP elicitations could not collect answers.

What changed

  • Computer Use: native app icons, session/persistent approval, and Decline. High-risk warnings remain visible. Codex owns permission storage.
  • Forms: typed text, numbers, booleans, and choices with defaults and validation. Unsupported schemas cannot be accepted.
  • URLs: show the destination and open only on user action. Acceptance means the URL opened, not that sign-in completed.
  • Shared BB styling: compact composer cards, visible actions, and a subtle scroll fade. CLI/SDK responses and the provider guide cover all request kinds.
  • Contracts: experimental plugin-owned headings, SDK 0.4.48, and host daemon protocol 183.

How you verified

  • Decoder regressions failed before the fix. Provider, contract, renderer, SDK, and guide tests/typechecks pass.
  • Live BB: real Calculator request and decline, typed form submission, required-field errors, scrolling, and URL actions. Live grant persistence remains unverified.
  • CI: 12 checks passed, 2 skipped.

Screenshots

Computer Use

Computer Use

Structured form

Structured form

Validation

Form validation

URL request

URL request

AGENT GENERATED

@smsunarto smsunarto changed the title feat(provider-codex): support MCP elicitation requests feat(provider-codex): support MCP elicitation requests (fixes Computer Use perms, etc) Sep 5, 2026
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