fix: Fix repo carousel thrashing (#294)

brendan-kellam · web-flow · commit b280e2ddbb69 · 2025-05-06T10:44:14.000-07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- Fixes bug with repos not being visible in the homepage carousel when re-indexing. [#294](https://github.com/sourcebot-dev/sourcebot/pull/294)
+
 ### Added
 - Added special `*` value for `rev:` to allow searching across all branches. [#281](https://github.com/sourcebot-dev/sourcebot/pull/281)
 
diff --git a/packages/web/src/app/[domain]/components/repositorySnapshot.tsx b/packages/web/src/app/[domain]/components/repositorySnapshot.tsx
@@ -15,14 +15,24 @@ import {
 } from "@/components/ui/carousel";
 import { RepoIndexingStatus } from "@sourcebot/db";
 import { SymbolIcon } from "@radix-ui/react-icons";
+import { RepositoryQuery } from "@/lib/types";
 
-export function RepositorySnapshot({ authEnabled }: { authEnabled: boolean }) {
+interface RepositorySnapshotProps {
+    authEnabled: boolean;
+    repos: RepositoryQuery[];
+}
+
+export function RepositorySnapshot({
+    authEnabled,
+    repos: initialRepos,
+}: RepositorySnapshotProps) {
     const domain = useDomain();
 
     const { data: repos, isPending, isError } = useQuery({
         queryKey: ['repos', domain],
         queryFn: () => unwrapServiceError(getRepos(domain)),
         refetchInterval: env.NEXT_PUBLIC_POLLING_INTERVAL_MS,
+        placeholderData: initialRepos,
     });
 
     if (isPending || isError || !repos) {
@@ -33,22 +43,30 @@ export function RepositorySnapshot({ authEnabled }: { authEnabled: boolean }) {
         )
     }
 
-    const numIndexedRepos = repos.filter((repo) => repo.repoIndexingStatus === RepoIndexingStatus.INDEXED).length;
-    const numIndexingRepos = repos.filter((repo) => repo.repoIndexingStatus === RepoIndexingStatus.INDEXING || repo.repoIndexingStatus === RepoIndexingStatus.IN_INDEX_QUEUE).length;
-    if (numIndexedRepos === 0 && numIndexingRepos > 0) {
-        return (
-            <div className="flex flex-row items-center gap-3">
-                <SymbolIcon className="h-4 w-4 animate-spin" />
-                <span className="text-sm">indexing in progress...</span>
-            </div>
-        )
-    } else if (numIndexedRepos == 0) {
-        return (
-            <EmptyRepoState domain={domain} authEnabled={authEnabled} />
-        )
+    // Use `indexedAt` to determine if a repo has __ever__ been indexed.
+    // The repo indexing status only tells us the repo's current indexing status.
+    const indexedRepos = repos.filter((repo) => repo.indexedAt !== undefined);
+
+    // If there are no indexed repos...
+    if (indexedRepos.length === 0) {
+
+        // ... show a loading state if repos are being indexed now
+        if (repos.some((repo) => repo.repoIndexingStatus === RepoIndexingStatus.INDEXING || repo.repoIndexingStatus === RepoIndexingStatus.IN_INDEX_QUEUE)) {
+            return (
+                <div className="flex flex-row items-center gap-3">
+                    <SymbolIcon className="h-4 w-4 animate-spin" />
+                    <span className="text-sm">indexing in progress...</span>
+                </div>
+            )
+
+        // ... otherwise, show the empty state.
+        } else {
+            return (
+                <EmptyRepoState domain={domain} authEnabled={authEnabled} />
+            )
+        }
     }
  
-    const indexedRepos = repos.filter((repo) => repo.repoIndexingStatus === RepoIndexingStatus.INDEXED);
     return (
         <div className="flex flex-col items-center gap-3">
             <span className="text-sm">
@@ -57,7 +75,7 @@ export function RepositorySnapshot({ authEnabled }: { authEnabled: boolean }) {
                     href={`${domain}/repos`}
                     className="text-blue-500"
                 >
-                    {repos.length > 1 ? 'repositories' : 'repository'}
+                    {indexedRepos.length > 1 ? 'repositories' : 'repository'}
                 </Link>
             </span>
             <RepositoryCarousel repos={indexedRepos} />
diff --git a/packages/web/src/app/[domain]/page.tsx b/packages/web/src/app/[domain]/page.tsx
@@ -10,13 +10,17 @@ import { SourcebotLogo } from "../components/sourcebotLogo";
 import { RepositorySnapshot } from "./components/repositorySnapshot";
 import { SyntaxReferenceGuideHint } from "./components/syntaxReferenceGuideHint";
 import { env } from '@/env.mjs';
+import { getRepos } from "@/actions";
+import { isServiceError } from "@/lib/utils";
 
 export default async function Home({ params: { domain } }: { params: { domain: string } }) {
     const org = await getOrgFromDomain(domain);
     if (!org) {
         return <PageNotFound />
     }
 
+    const repos = await getRepos(domain);
+
     return (
         <div className="flex flex-col items-center overflow-hidden min-h-screen">
             <NavigationMenu
@@ -34,7 +38,10 @@ export default async function Home({ params: { domain } }: { params: { domain: s
                     className="mt-4 w-full max-w-[800px]"
                 />
                 <div className="mt-8">
-                    <RepositorySnapshot authEnabled={env.SOURCEBOT_AUTH_ENABLED === 'true'} />
+                    <RepositorySnapshot
+                        authEnabled={env.SOURCEBOT_AUTH_ENABLED === 'true'}
+                        repos={isServiceError(repos) ? [] : repos}
+                    />
                 </div>
                 <div className="flex flex-col items-center w-fit gap-6">
                     <Separator className="mt-5" />
