Repository navigation
refactor(cli): drop the tool objects the agents needed; closes #331 - #657
Merged
Merged
Conversation
With the agents gone nothing used them. The CLI calls plain functions. - removed: FunctionTool objects, the registry and approval flags - removed: user price alerts (they never fired; /gtt has working alerts), portfolio overrides (nothing read them), the account display wrappers and voter explainers (commands cover them), TrailingStopCommands (never wired up) - removed: functions only an agent could reach - only the natural-language parser imports the agent framework now, and a test holds it there Closes #331 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 1, 2026
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
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.
With the agents gone (#656), nothing used this layer. The CLI calls plain functions, as it already did. The commands that replaced what only tools could do merged in #655.
Removed
FunctionToolobjects, the tool registry and the approval flags. No agent is left to ask for approval; the CLI's own prompts are unchanged./gtthas working alerts.src/trading/portfolio_override.py.TrailingStopCommands, which was never wired up;/trailingis the view now.AutoGenLLMParseralias.No data goes with them: the real
state/user.dbhas nouser_alertsorportfolio_overridestable.Guarded
Only
src/parsers/llm_parser.pyimports the agent framework now, andtest_llm_is_the_parser_only.pyholds it there. Both dependencies stay, for the parser.Testing
pytest tests/ -m "not db": 1903 passed, 1 skipped. Guarded real CLI run with temp state and a copy ofuser.db: 15 commands and 7 typed requests, 5 model calls, no blocked hosts and no order attempts. The realuser.dbwas untouched.Closes #331