Skip to content

Include app link metadata in MCP approval elicitations - #42134

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

Include app link metadata in MCP approval elicitations#42134
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/a76731d9f5f3640318187511ccdc0dc82967a9c5

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Include app link metadata in MCP approval elicitations

What changed

  • Preserve the selected app account link_id when building native MCP tool
    approval requests.
  • Add link_id and link_is_implicit to the elicitation metadata, recognizing
    IDs with the reserved implicit_link:: prefix as implicit links.

Testing

  • Extend the app link policy integration test to verify elicitation metadata
    for default, explicitly selected, and implicit account links.

## What changed

- Preserve the selected app account `link_id` when building native MCP tool
  approval requests.
- Add `link_id` and `link_is_implicit` to the elicitation metadata, recognizing
  IDs with the reserved `implicit_link::` prefix as implicit links.

## Testing

- Extend the app link policy integration test to verify elicitation metadata
  for default, explicitly selected, and implicit account links.

GitOrigin-RevId: a76731d9f5f3640318187511ccdc0dc82967a9c5
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/a76731d9f5f3640318187511ccdc0dc82967a9c5 branch from 45acaa0 to a30fa3d Compare September 1, 2026 20:55
@copyberry
copyberry Bot merged commit a30fa3d into main Sep 1, 2026
24 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/a76731d9f5f3640318187511ccdc0dc82967a9c5 branch September 1, 2026 20:56
@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.

1 participant