Skip to content

Remove legacy query facades (query_facade.py, pathfinding_facade.py, query_service.py) #279

@aaronsb

Description

@aaronsb

Context

With GraphFacade (PR #274) and batch hydration (PR #276) merged, the old pre-acceleration query layer is dead code:

File Lines Status
api/app/lib/query_facade.py 681 Superseded by graph_facade.py
api/app/lib/pathfinding_facade.py 654 Superseded by graph_facade.find_paths()
api/app/lib/query_service.py ~200 Superseded by direct client.graph.* calls

These were the intermediate abstraction layer between routes and AGEClient. Routes now call client.graph.* directly (for topology) and _hydrate_grounding_batch() (for grounding/confidence).

Steps

  1. Audit all imports of QueryFacade, PathfindingFacade, QueryService across the codebase
  2. Verify no remaining callers (routes, services, tests)
  3. Delete the files
  4. Remove any test files that only test the old facades
  5. Run full test suite to confirm

~1,500 lines removed

References

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestrefactorCode refactoring

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions