feat: multi-tool MCP compatibility (Codex, Crush)#35
Merged
timeleft-- merged 5 commits intomainfrom Mar 27, 2026
Merged
Conversation
… promote, and sync guidance to MCP tool descriptions - recall: add scope discovery priority order (env var → .fava-trails.yaml → hint → ask) and session start hint - save_thought: add 'promote via propose_truth when finalized' guidance - propose_truth: add 'call sync after promoting' guidance - get_usage_guide: reposition as entry point for agents without instructions support - tests: add 5 new assertions in TestToolDescriptionEnhancements Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ail_name desc Address GPT 5.1 Codex review feedback: - _build_trail_name_desc() now includes scope resolution fallback chain when FAVA_TRAILS_SCOPE_HINT is not set, so every tool surfaces discovery guidance for Codex and other non-Claude MCP clients - Replace UTF-8 arrow in get_usage_guide description with ASCII Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… template - README: Add OpenAI Codex CLI (TOML) and generic MCP client (JSON) config examples - Create data_repo_template/AGENTS.md with tool-agnostic usage guide - cli.py: Include AGENTS.md in bootstrap template file list Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- server.py: Replace claude-code/claude-desktop examples with codex-cli/my-agent - server.py: Generalize MCP stderr logging comment - orchestrator.py: Replace "Claude Sonnet agents" with "mapper agents" - CLAUDE.md: Add pointer to AGENTS.md for non-Claude tools Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Schema-driven MCP clients (Codex) may omit optional params. trail_name is enforced at runtime but was not in most inputSchema required arrays, causing silent failures for non-Claude clients. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Make fava-trails usable by OpenAI Codex CLI and other non-Claude MCP clients (Crush/OpenCode) — not just Claude Code/Desktop.
instructionsfield support can self-discover the full lifecycleAGENTS.mdin data repo template — tool-agnostic onboarding doc (Codex reads AGENTS.md)Key technical decision
Neither Codex nor Crush reads the MCP
instructionsfield from InitializeResult. Tool descriptions are the only channel for guidance. We enriched descriptions and madeget_usage_guidethe explicit entry point for new agents.Consensus review
Test plan
uv run pytest -v)uv run ruff check src/)🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com