fix(web): avoid bundling search API in client; address review

Author: h30s

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 ### Added
+- Search results show an informational alert when a query may be incomplete (for example due to match limits, skipped index shards, or streaming flush), and non-exhaustive Zoekt searches are logged for operators. [#1098](https://github.com/sourcebot-dev/sourcebot/pull/1098)
 - Linear issue links in chat responses now render as a rich card-style UI showing the Linear logo, issue identifier, and title instead of plain hyperlinks. [#1060](https://github.com/sourcebot-dev/sourcebot/pull/1060)
 
 ### Changed

packages/web/src/app/(app)/search/components/searchResultsPage.tsx

@@ -11,7 +11,8 @@ import {
 } from "@/components/ui/resizable";
 import { Separator } from "@/components/ui/separator";
 import { Tooltip, TooltipContent, TooltipTrigger } from "@/components/ui/tooltip";
-import { getSearchLimitExplanation, RepositoryInfo, SearchResultFile, SearchStats } from "@/features/search";
+import { getSearchLimitExplanation } from "@/features/search/searchLimitExplanation";
+import type { RepositoryInfo, SearchResultFile, SearchStats } from "@/features/search/types";
 import { Alert, AlertDescription, AlertTitle } from "@/components/ui/alert";
 import useCaptureEvent from "@/hooks/useCaptureEvent";
 import { useNonEmptyQueryParam } from "@/hooks/useNonEmptyQueryParam";

packages/web/src/app/(app)/search/useStreamedSearch.ts

@@ -1,6 +1,6 @@
 'use client';
 
-import { RepositoryInfo, SearchRequest, SearchResultFile, SearchStats, StreamedSearchResponse } from '@/features/search';
+import type { RepositoryInfo, SearchRequest, SearchResultFile, SearchStats, StreamedSearchResponse } from '@/features/search/types';
 import { ServiceErrorException } from '@/lib/serviceError';
 import { isServiceError } from '@/lib/utils';
 import * as Sentry from '@sentry/nextjs';
@@ -116,6 +116,7 @@ export const useStreamedSearch = ({ query, matches, contextLines, whole, isRegex
                 timeToSearchCompletionMs: 0,
                 timeToFirstSearchResultMs: 0,
                 numMatches: 0,
+                stats: undefined,
             });
 
             try {

packages/web/src/features/search/searchLimitExplanation.test.ts

@@ -81,3 +81,15 @@ test('flushReason timer when no higher-priority signal', () => {
     );
     expect(out.summary).toContain('streaming timer');
 });
+
+test('flushReason max size when no higher-priority signal', () => {
+    const out = getSearchLimitExplanation(
+        stats({
+            flushReason: 'FLUSH_REASON_MAX_SIZE',
+            totalMatchCount: 10,
+            actualMatchCount: 10,
+        }),
+        100,
+    );
+    expect(out.summary).toContain('size limit');
+});

packages/web/src/features/search/searchLimitExplanation.ts

@@ -56,6 +56,8 @@ export function getSearchLimitExplanation(
         };
     }
 
+    // Defensive fallback: non-exhaustive searches should usually hit a branch above
+    // (e.g. totalMatchCount vs display cap, skipped shards/files, or flush reason).
     return {
         summary: 'More matches may exist than are shown.',
         detail: 'Increase the match limit, narrow your query, or scope to a repository.',