Skip to content

Fix: Tools Paths Standardization and Tool Registry Improvements#476

Merged
dwash96 merged 19 commits intocecli-dev:v0.98.2from
szmania:cli-4-fix-tool-paths
Apr 5, 2026
Merged

Fix: Tools Paths Standardization and Tool Registry Improvements#476
dwash96 merged 19 commits intocecli-dev:v0.98.2from
szmania:cli-4-fix-tool-paths

Conversation

@szmania
Copy link
Copy Markdown

@szmania szmania commented Apr 4, 2026

Summary

This PR addresses issues related to tool paths standardization and tool registry improvements in the cecli codebase. The changes focus on:

  1. Standardizing tool_paths to tools_paths in agent configurations to maintain consistency across the codebase.
  2. Improving the tool registry to correctly handle tool registration, execution, and error logging.
  3. Enhancing local tool dispatch to ensure tools are correctly identified and executed.
  4. Announcing loaded custom tools in the TUI for better user visibility.
  5. Fixing test failures related to the tool registry and repo map.

Key Changes

Tool Paths Standardization

  • Allow both tool_paths and tools_paths in agent configurations for backward compatibility.
  • Rename tool_paths to tools_paths in agent configurations to standardize naming.
  • Update agent config validation to handle the new tools_paths key.

Tool Registry Improvements

  • Fix tool registration to use the schema name for consistency.
  • Improve error logging in the tool registry and base coder to provide better debugging information.
  • Add checks for None or empty strings in the plugin manager to prevent runtime errors.
  • Correctly call tool execute methods in the agent coder to ensure tools are executed as expected.
  • Adjust tool registry tests to account for tuple return values and ensure compatibility.

Local Tool Dispatch

  • Fix local tool dispatch logic in AgentCoder to correctly identify and execute local tools.
  • Improve error handling for missing or invalid tools.

TUI Improvements

  • Announce loaded custom tools in the TUI to provide visibility into which tools are available during a session.

Test Fixes

  • Resolve test failures in the tool registry and repo map.
  • Enable enhanced map for C# repo map tests to ensure accurate test results.
  • Fix merge conflicts from upstream changes.

Files Modified

  • Agent configuration handling (cecli/coders/base_coder.py)
  • Tool registry (cecli/tools/utils/registry.py)
  • Plugin manager (cecli/helpers/plugin_manager.py)
  • Agent coder (cecli/coders/agent_coder.py)
  • Test files (tests/tools/test_tool_registry.py, tests/test_repomap.py)

Impact

  • Improved consistency in agent configurations by standardizing tools_paths.
  • Better debugging with enhanced error logging in the tool registry.
  • More reliable tool execution with fixes to local tool dispatch logic.
  • Enhanced user experience by announcing loaded custom tools in the TUI.
  • Stable test suite with fixes for tool registry and repo map tests.

Testing

  • All changes have been tested locally and verified against the existing test suite.
  • New tests have been added to cover edge cases in the tool registry and local tool dispatch logic.

szmania added 19 commits April 1, 2026 11:19
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
Co-authored-by: cecli (openai/gemini_cli/gemini-2.5-pro)
@dwash96 dwash96 changed the base branch from main to v0.98.2 April 5, 2026 11:36
@dwash96 dwash96 merged commit 2bc7a46 into cecli-dev:v0.98.2 Apr 5, 2026
12 checks passed
@dwash96 dwash96 mentioned this pull request Apr 5, 2026
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.

2 participants