Problem
Six Python files in agent/ are skeleton stubs that no other file imports. They predate the TypeScript pivot and now mislead readers, fail ruff check exemptions, and waste reviewer time when grep'ing the agent surface.
Verified unreferenced (no import sites in agent/, mcp/, or tests/):
| File |
Lines |
Status |
agent/main.py |
201 |
All TODO comments — class TXLookup body is print(...) |
agent/executor.py |
157 |
Stub |
agent/memory.py |
48 |
Stub with TODO: swap to Supabase |
agent/tools/browser.py |
52 |
4x # TODO: Codex — implement |
agent/tools/search.py |
26 |
2x # TODO: Codex — implement |
agent/tools/writer.py |
23 |
1x # TODO: Codex — implement |
mcp/tools.py |
(small) |
# TODO: Codex — implement shared utilities |
The live runtime uses app/lib/agent.ts for orchestration, app/lib/specialists.ts for delegation, and app/api/agent/route.ts for SSE. The Python mcp/server.py only depends on agent/planner.py, agent/tools/data.py, agent/tools/miro.py.
Fix
git rm the six unreferenced files above
- Update
AGENTS.md directory-structure block (paired with the docs-drift issue) to drop them
- Confirm
ruff check agent/ mcp/ still passes
- Confirm
pytest tests/ still passes (none of the deleted files have associated tests)
If any of these are wanted as future placeholders, leave a single agent/README.md that says "browser/search/writer tools are unimplemented — see app/lib/agent.ts for the live runtime."
Code pointers
agent/main.py, agent/executor.py, agent/memory.py
agent/tools/browser.py, agent/tools/search.py, agent/tools/writer.py
mcp/tools.py
- Verification:
grep -rn 'from agent.main\|from agent.executor\|from agent.memory\|from agent.tools.browser\|from agent.tools.search\|from agent.tools.writer' . returns nothing
Acceptance
Problem
Six Python files in
agent/are skeleton stubs that no other file imports. They predate the TypeScript pivot and now mislead readers, failruff checkexemptions, and waste reviewer time when grep'ing the agent surface.Verified unreferenced (no import sites in
agent/,mcp/, ortests/):agent/main.pyclass TXLookupbody isprint(...)agent/executor.pyagent/memory.pyTODO: swap to Supabaseagent/tools/browser.py# TODO: Codex — implementagent/tools/search.py# TODO: Codex — implementagent/tools/writer.py# TODO: Codex — implementmcp/tools.py# TODO: Codex — implement shared utilitiesThe live runtime uses
app/lib/agent.tsfor orchestration,app/lib/specialists.tsfor delegation, andapp/api/agent/route.tsfor SSE. The Pythonmcp/server.pyonly depends onagent/planner.py,agent/tools/data.py,agent/tools/miro.py.Fix
git rmthe six unreferenced files aboveAGENTS.mddirectory-structure block (paired with the docs-drift issue) to drop themruff check agent/ mcp/still passespytest tests/still passes (none of the deleted files have associated tests)If any of these are wanted as future placeholders, leave a single
agent/README.mdthat says "browser/search/writer tools are unimplemented — see app/lib/agent.ts for the live runtime."Code pointers
agent/main.py,agent/executor.py,agent/memory.pyagent/tools/browser.py,agent/tools/search.py,agent/tools/writer.pymcp/tools.pygrep -rn 'from agent.main\|from agent.executor\|from agent.memory\|from agent.tools.browser\|from agent.tools.search\|from agent.tools.writer' .returns nothingAcceptance
mcp/server.pybootstests/test_mcp_boot.pystill passes