Skip to content

Honor explicit account selectors for Apps tool calls - #42054

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/667fd04102a3934d53ded020da6d1bf68a1ca3e5
Sep 1, 2026
Merged

Honor explicit account selectors for Apps tool calls#42054
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/667fd04102a3934d53ded020da6d1bf68a1ca3e5

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Honor explicit account selectors for Apps tool calls

What changed

  • Resolve link_id from tool-call arguments when Apps metadata sets
    requires_explicit_link_id to true.
  • Reject the call before approval or execution when the required selector is
    missing, empty, or not a string.
  • Preserve catalog-provided account metadata for legacy Apps tools and leave
    non-Apps MCP tools unchanged.

Testing

  • Add unit coverage for required selectors, malformed values, legacy
    fallbacks, and non-Apps tools.
  • Add end-to-end coverage for execution, approval prompts, and rejection when
    an Apps call omits link_id.

## What changed

- Resolve `link_id` from tool-call arguments when Apps metadata sets
  `requires_explicit_link_id` to `true`.
- Reject the call before approval or execution when the required selector is
  missing, empty, or not a string.
- Preserve catalog-provided account metadata for legacy Apps tools and leave
  non-Apps MCP tools unchanged.

## Testing

- Add unit coverage for required selectors, malformed values, legacy
  fallbacks, and non-Apps tools.
- Add end-to-end coverage for execution, approval prompts, and rejection when
  an Apps call omits `link_id`.

GitOrigin-RevId: 667fd04102a3934d53ded020da6d1bf68a1ca3e5
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/667fd04102a3934d53ded020da6d1bf68a1ca3e5 branch from 76b458c to 0e37d83 Compare September 1, 2026 13:29
@copyberry
copyberry Bot merged commit 0e37d83 into main Sep 1, 2026
24 of 30 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/667fd04102a3934d53ded020da6d1bf68a1ca3e5 branch September 1, 2026 13:30
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants