Add ToolUniverse research skills#10
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 728e69fd63
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| [agent] | ||
| skills = ["skills"] | ||
| mcp_servers = ["repo"] | ||
| mcp_servers = ["repo", "tooluniverse"] |
There was a problem hiding this comment.
Adding tooluniverse to the main agent's mcp_servers makes every normal chat request depend on http://localhost:8090/mcp being up, because AgentRuntime._build_main_tools() always loads all configured agent MCP tools and does not handle connection failures from _get_mcp_tools(). In environments where ToolUniverse is not running (the common default), this turns the whole agent unusable instead of letting it continue with repo-only tools.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
@codex Please provide suggestions to fix this.
There was a problem hiding this comment.
Summary
- Updated the main agent configuration to not require ToolUniverse by default by changing
mcp_serversfrom["repo", "tooluniverse"]to["repo"], so normal chat remains usable when ToolUniverse is offline. deepagent.tomlL40-L43 - Added a commented opt-in example showing how to re-enable ToolUniverse when
http://localhost:8090/mcpis available, preserving discoverability for that workflow. deepagent.tomlL43-L45 - Committed the change on the current branch (
f64cee3) and created a follow-up PR entry via themake_prtool.
Testing
- ✅
git status --short && .venv/bin/python -m pytest -q(34 passed, 2 warnings) - ✅
git add deepagent.toml && git commit -m "Make ToolUniverse MCP optional for main agent"
AminMahpour
force-pushed
the
DEV/tooluniverse-skills
branch
from
b57aea1 to
a6a76d3
Compare
- Introduced a mock test file for tool call testing. - Enhanced `pyproject.toml` to include new dependencies and project scripts. - Added comprehensive tests for the `chainagents_cli` module, covering argument parsing, runtime overrides, and command invocation. - Implemented tests for the `chainlit_bridge` to ensure proper event handling and task management. - Updated `deepagent_runtime` tests to validate model configuration and recursion limits from TOML files. - Refactored main command resolution logic to improve reasoning and model name handling. - Updated `uv.lock` to reflect changes in package source and added new dependencies.
… enhance ToolExecutionResilienceMiddleware
- restore non-skill paths from origin/master - keep ToolUniverse skill files as the branch-specific diff
…rse in mcp_servers
1 participant
Summary
Validation
.venv/bin/python -m pytestpassed on the rebased branch: 37 passed, 2 warnings.Notes
http://localhost:8090/mcpwhen enabled. The skills tell the agent to report when ToolUniverse tools are unavailable rather than guessing tool names.