Skip to content

[mcp] Add dummy tools for previously called but currently missing tools.#17853

Merged
mzeng-openai merged 13 commits into
mainfrom
dev/mzeng/plan-dummy-mcp-tools
Apr 15, 2026
Merged

[mcp] Add dummy tools for previously called but currently missing tools.#17853
mzeng-openai merged 13 commits into
mainfrom
dev/mzeng/plan-dummy-mcp-tools

Conversation

@mzeng-openai

@mzeng-openai mzeng-openai commented Apr 14, 2026

Copy link
Copy Markdown
Collaborator
  • Add dummy tools for previously called but currently missing tools. Currently supporting MCP tools only.

@mzeng-openai mzeng-openai marked this pull request as ready for review April 14, 2026 23:57
Comment thread codex-rs/core/src/mcp_tool_exposure.rs Outdated
@mzeng-openai mzeng-openai changed the title [mcp] Add dummy tools for previously called but currently missing mcp tools. [mcp] Add dummy tools for previously called but currently missing tools. Apr 15, 2026
Comment thread codex-rs/core/src/tools/spec_tests.rs Outdated
Comment thread codex-rs/core/src/unavailable_tool.rs Outdated
Comment thread codex-rs/core/src/unavailable_tool.rs Outdated
Comment thread codex-rs/core/tests/suite/tools.rs
Comment thread codex-rs/core/src/tools/spec.rs Outdated
Comment thread codex-rs/core/src/tools/spec.rs Outdated

@pakrym-oai pakrym-oai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good direction, few details

@mzeng-openai mzeng-openai requested a review from pakrym-oai April 15, 2026 19:29
@mzeng-openai mzeng-openai enabled auto-merge (squash) April 15, 2026 20:56
@mzeng-openai mzeng-openai merged commit 28b76d1 into main Apr 15, 2026
35 of 36 checks passed
@mzeng-openai mzeng-openai deleted the dev/mzeng/plan-dummy-mcp-tools branch April 15, 2026 21:48
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 15, 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