Skip to content

PR #13: prefer streamable-http + fresher checks for reachable ranking#13

Merged
khalidsaidi merged 1 commit intomainfrom
feat/reachable-ranking-preference
Mar 1, 2026
Merged

PR #13: prefer streamable-http + fresher checks for reachable ranking#13
khalidsaidi merged 1 commit intomainfrom
feat/reachable-ranking-preference

Conversation

@khalidsaidi
Copy link
Owner

Summary

  • add reachability tie-break helpers in apps/api/src/rag/search.ts:
    • remoteTypeWeight: streamable-http > sse > unknown
    • checkedAtMs: parse reachability check timestamps
  • apply these tie-breakers only when filters.reachable === true
  • keep relevance primary for /rag/search:
    • keyword/semantic score remains the first sort key
    • then apply reachability preferences
    • then existing quality signals (ragScore, updatedAt, name)
  • update /rag/top ordering when reachable=true to prefer:
    • streamable-http over sse
    • more recently checked entries
    • then existing quality signals

Tests

Added deterministic integration tests in apps/api/tests/api.test.ts:

  • rag top prefers reachable streamable-http over sse when reachable=true
  • rag search prefers more recently checked entries when remote type ties

Validation

  • pnpm -r test
  • pnpm -r typecheck
  • pnpm -r lint

@khalidsaidi khalidsaidi merged commit 25761dc into main Mar 1, 2026
1 check 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.

1 participant