Skip to content

Expose MCP read-only hints in tool call items - #36055

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/dddfe905146075e5137a1094da485b86b99807f0
Jul 30, 2026
Merged

Expose MCP read-only hints in tool call items#36055
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/dddfe905146075e5137a1094da485b86b99807f0

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 30, 2026

Copy link
Copy Markdown

Expose MCP read-only hints in tool call items

What changed

  • Propagate MCP tool readOnlyHint annotations through tool-call start and
    completion events.
  • Include the optional hint in persisted thread history and app-server
    mcpToolCall items, preserving compatibility when the annotation is absent.
  • Document that the hint describes tool capability, not the outcome of a
    particular invocation.

Testing

  • Cover read-only and write-capable tools in live events and persisted rollout
    data.
  • Verify the hint survives thread reads and resumes for both in-progress and
    completed MCP calls.

## What changed

- Propagate MCP tool `readOnlyHint` annotations through tool-call start and
  completion events.
- Include the optional hint in persisted thread history and app-server
  `mcpToolCall` items, preserving compatibility when the annotation is absent.
- Document that the hint describes tool capability, not the outcome of a
  particular invocation.

## Testing

- Cover read-only and write-capable tools in live events and persisted rollout
  data.
- Verify the hint survives thread reads and resumes for both in-progress and
  completed MCP calls.

GitOrigin-RevId: dddfe905146075e5137a1094da485b86b99807f0
@github-actions

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/dddfe905146075e5137a1094da485b86b99807f0 branch from 78a32f8 to 406dc92 Compare July 30, 2026 01:38
@copyberry
copyberry Bot merged commit 406dc92 into main Jul 30, 2026
30 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/dddfe905146075e5137a1094da485b86b99807f0 branch July 30, 2026 01:39
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 2026
@aofei
aofei temporarily deployed to issue-triage July 30, 2026 03:31 — with GitHub Actions Inactive
@aofei
aofei temporarily deployed to issue-triage July 30, 2026 03:31 — with GitHub Actions Inactive
@aofei
aofei temporarily deployed to issue-triage July 30, 2026 03:31 — with GitHub Actions Inactive
@aofei
aofei temporarily deployed to issue-triage July 30, 2026 03:32 — with GitHub Actions Inactive
@daizw
daizw temporarily deployed to issue-triage July 30, 2026 03:49 — with GitHub Actions Inactive
@daizw
daizw temporarily deployed to issue-triage July 30, 2026 03:49 — with GitHub Actions Inactive
@daizw
daizw temporarily deployed to issue-triage July 30, 2026 03:49 — with GitHub Actions Inactive
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.