Repository navigation
🟡 [DESIGN] threshold lives in infrastructure - should be SearchOptions #14
Description
Problem
Search threshold is configured in repository constructor:
def __init__(..., threshold: float = 0.3):
self.threshold = thresholdThis is retrieval POLICY, not infrastructure config.
Why It Matters
- Can't vary threshold per query
- Policy mixed with implementation
- Hard to A/B test different thresholds
Solution
Move to Application layer as SearchOptions:
@dataclass
class SearchOptions:
threshold: float = 0.3
filters: dict | None = None
async def semantic_search(
self, vector: List[float], limit: int,
options: SearchOptions | None = None
) -> List[SearchHit]:Acceptance Criteria
- threshold removed from repository constructor
- SearchOptions created in application layer
- semantic_search accepts options parameter
Activity
FullFran commentedon Jan 26, 2026
Sigue abierta. El todavía vive en infraestructura () y se usa como default en ; no está integrado un en el flujo de aplicación. Dejo abierta.
FullFran commentedon Jan 26, 2026
Corrijo comentario anterior (el backtick se interpretó en shell).
Sigue abierta. El
thresholdtodavía vive en infraestructura (SupabaseRepository.__init__) y se usa como default ensemantic_search; no está integrado unSearchOptionsen el flujo de aplicación.refactor(core)!: separar acceso destructivo, mover el umbral a la apl…