Skip to content

Comments

Fix Codex executor MCP task tools access#464

Open
bborn wants to merge 1 commit intomainfrom
task/1195-fix-codex-executor-mcp-task-tools-access
Open

Fix Codex executor MCP task tools access#464
bborn wants to merge 1 commit intomainfrom
task/1195-fix-codex-executor-mcp-task-tools-access

Conversation

@bborn
Copy link
Owner

@bborn bborn commented Feb 18, 2026

Summary

  • Fixed TaskYou MCP tools (taskyou_complete, taskyou_get_project_context, etc.) not being available to non-Claude executors (Codex, Gemini, OpenClaw, etc.)
  • The MCP config was only written to ~/.claude.json inside runClaude/runClaudeResume, so Codex and other CLIs never got the tools configured
  • Now writes MCP config to both ~/.claude.json (for Claude Code) and .mcp.json in the worktree (for all other CLIs) from the centralized executeTask function

Test plan

  • New tests for writeWorktreeMCPConfig
  • Existing MCP config tests still pass
  • Full executor test suite passes
  • Manual: run a task with Codex executor and verify taskyou_get_project_context works

🤖 Generated with Claude Code

The TaskYou MCP server config (taskyou_complete, taskyou_get_project_context, etc.)
was only written to ~/.claude.json inside runClaude/runClaudeResume, so non-Claude
executors (Codex, Gemini, etc.) never got MCP tools configured.

Changes:
- Move writeWorkflowMCPConfig call from runClaude/runClaudeResume to executeTask
  so it runs for ALL executors (centralized)
- Add writeWorktreeMCPConfig that writes taskyou MCP server to .mcp.json in the
  worktree directory, which non-Claude CLIs read for MCP discovery
- Handle symlinked .mcp.json by breaking the symlink before writing (preserves
  the project's original file)
- Add comprehensive tests for writeWorktreeMCPConfig

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant