AGENTS.md points at a nonexistent path: codex-mcp/src/mcp_connection_manager.rs
The MCP guidance in AGENTS.md points at a file that does not exist:
When working with MCP tool calls, prefer using codex-rs/codex-mcp/src/mcp_connection_manager.rs to handle mutation of tools and tool calls.
codex-rs/codex-mcp/src/mcp_connection_manager.rs is not present on main. The module is codex-rs/codex-mcp/src/connection_manager.rs — the mcp_ filename prefix is gone.
This matters because AGENTS.md is read by coding agents: an agent following that line looks for a file that isn't there and then either stops or guesses at the right one.
Suggested fix, one line:
-- When working with MCP tool calls, prefer using `codex-rs/codex-mcp/src/mcp_connection_manager.rs` to handle mutation of tools and tool calls.
+- When working with MCP tool calls, prefer using `codex-rs/codex-mcp/src/connection_manager.rs` to handle mutation of tools and tool calls.I have this staged on a branch but could not open a PR — CreatePullRequest returns a permissions error for my account on this repo, so I am filing it here instead. Happy to submit a PR if outside contributions are enabled, or feel free to apply the one-line change directly.
Found with unrot and verified by hand against main.
2 Comments
yo, that line's the only place the old name appears -- repo-wide grep for mcp_connection_manager returns just AGENTS.md:35
& the file it means is codex-rs/codex-mcp/src/connection_manager.rs (the mcp_ prefix is gone). checked against the API rather than a local clone, since a sparse checkout will happily report both paths missing
so it's a one-word fix on that line, no other refs to chase
Scouted using Fable, verified and written by hand
Still true on current main.
AGENTS.mdstill points atcodex-rs/codex-mcp/src/mcp_connection_manager.rs. The file on disk iscodex-rs/codex-mcp/src/connection_manager.rs. The old name only lives on that instruction line.Agents that treat backtick paths as lookups will miss the module. One-line rename.