cleanup: retired agent kits (claude-runtime, trading_agent_prompt, trader sprawl) - #110
Merged
Merged
Conversation
Remove the unreferenced claude-runtime tree, the stale {report} executor
prompt, and duplicate trader variants. Keep the shared playbook, four
agent personas, and the simpler discretionary trader.md. Drop the dated
static tool catalog now that tools_list is the inventory source of truth.
Co-authored-by: Emerson Gomes <emerson.gomes@gmail.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
2 participants
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.
Subtract-only cleanup of unused agent kits and prompt sprawl. No new orchestration, extras, or replacement kits.
Deleted
claude-runtime/tree (23 named agents + orchestrator +.mcp.json+settings.local.json). Repo search found zero imports or docs/README/script references. The allow-list still named deletedtrading_*/ removedfinviz_*tools.prompts/trading_agent_prompt.md— legacy{report}executor with stale APIs (tp_abs/sl_abs,trade_close(symbol=...),close_kind="pending"), Grid/Martingale allowance, and no dry-run/idempotency. Conflicts withdocs/TRADING_SAFETY.md. No in-repo{report}injector.prompts/trader_active.mdandprompts/trader_grid_scalper.md— duplicate/conflicting discretionary variants (grid/recovery risk, Playwright research for a non-catalogplaywrighttool).Kept
prompts/mtdata_tool_playbook.mdand the fourprompts/agent_*.mdpersonasprompts/trader.mdas the single discretionary trader (simpler thantrader_active.md; no Playwright section)Playbook trim
tools_listremains the inventory source of truth.gitignoreprompt allow-list to match the remaining filesDiff
30 files, +4 / −5587. No new files.
Verification
claude-runtime,trading_agent_prompt,trader_active, ortrader_grid_scalperCI note
CI / Backend testsfails on this branch for a pre-existingruffB033 onsrc/mtdata/core/output_profiles.py:2883({100, 100.0}). That line is already onmain(16efa0df) and is not in this diff. Pytest never started. Left unfixed here to keep the PR scoped to prompt-kit deletion.