[research] Duplicate research_handler.py in services/research/ and src/ — 565 lines of diff #773

Closed
opened 2026-06-03 00:28:12 +02:00 by sleepy · 0 comments
Owner

Problem

Two copies of research_handler exist:

  • services/research/research_handler.py — 463 lines
  • src/research_handler.py — 823 lines (360 lines longer)

The src/ version has additional methods including synthesize_query(), _bounded_int(), and imports from src.research_utils. The services/ version is a simpler/older copy.

AGENTS.md violations

  • Duplicate code across two locations
  • src/research_handler.py at 823 lines and src/deep_research.py at 816 lines both exceed ~400 line limit
  • No README.md in services/research/

Fix

  1. Consolidate into a single canonical location
  2. Split src/research_handler.py — query synthesis, legacy engine, and task registry are separate concerns
  3. Split src/deep_research.py — the DeepResearcher class with 21 methods spans multiple responsibilities
  4. Add README.md to the research feature directory
## Problem Two copies of research_handler exist: - `services/research/research_handler.py` — 463 lines - `src/research_handler.py` — 823 lines (360 lines longer) The `src/` version has additional methods including `synthesize_query()`, `_bounded_int()`, and imports from `src.research_utils`. The `services/` version is a simpler/older copy. ## AGENTS.md violations - Duplicate code across two locations - `src/research_handler.py` at 823 lines and `src/deep_research.py` at 816 lines both exceed ~400 line limit - No `README.md` in `services/research/` ## Fix 1. Consolidate into a single canonical location 2. Split `src/research_handler.py` — query synthesis, legacy engine, and task registry are separate concerns 3. Split `src/deep_research.py` — the DeepResearcher class with 21 methods spans multiple responsibilities 4. Add `README.md` to the research feature directory
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sleepy/odysseus#773
No description provided.