Skip to content

169 chore cleanup dead code#172

Merged
nwang783 merged 5 commits into
mainfrom
169-chore-cleanup-dead-code
Jan 20, 2026
Merged

169 chore cleanup dead code#172
nwang783 merged 5 commits into
mainfrom
169-chore-cleanup-dead-code

Conversation

@nwang783
Copy link
Copy Markdown
Owner

No description provided.

nwang added 5 commits January 20, 2026 09:10
- Delete testplan_tools.py (unused test plan management tools)
- Remove unused create_sandboxed_tools/create_local_tools from tool_factory.py
- Clean up __init__.py exports
- Remove associated test classes from test_meta.py

~592 lines of dead code removed
- Delete event_bus.py (never used - no .emit() or .subscribe() calls)
- Remove EventBus and AgentEvent exports from __init__.py

~133 lines of dead code removed
- Delete subagent_tracker.py (only used in tests, not production)
- Delete test_subagent_tracker.py

~300 lines of dead code removed
- Remove TestPlanManager protocol from infra/protocols.py
- Remove test plan methods from context/store.py
  (add_test_plan_item, complete_test_plan_item, get_test_plan, set_test_plan)

~115 lines of orphaned code removed (testplan_tools was previously deleted)
- Remove unused list_sessions() and delete_session() from persistence.py
- Remove orphaned test_plan field from AgentState in frame.py
- Remove corresponding test classes from test_persistence.py

~115 lines of dead code removed
@nwang783 nwang783 linked an issue Jan 20, 2026 that may be closed by this pull request
@nwang783 nwang783 merged commit 2e049d6 into main Jan 20, 2026
3 checks passed
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.

Chore: Cleanup Dead Code

1 participant