Skip to content

Commit 42ee901

Browse files
committed
refactor + tweak
1 parent 47ad447 commit 42ee901

File tree

3 files changed

+9
-7
lines changed

3 files changed

+9
-7
lines changed

packages/web/src/components/repos/RepoStatusMessages.tsx renamed to packages/web/src/components/search/SearchResultRepoStatusMessages.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { AlertCircleIcon } from "lucide-react";
33

44
import { Button } from "@/components/ui/button";
55

6-
import { RepoPreview, type RepoPreviewProps } from "./RepoPreview";
6+
import { RepoPreview, type RepoPreviewProps } from "../repos/RepoPreview";
77

88
const MessageLayout = ({ children }: { children: React.ReactNode }) => (
99
<div className="rounded-lg border bg-background p-4 text-mobile-base sm:p-6 sm:text-base">
@@ -107,6 +107,8 @@ export const OnGithubMessage = ({ error, preview }: OnGithubMessageProps) => (
107107
owner: preview.owner.login,
108108
repo: preview.name,
109109
}}
110+
// important, or we will load all repos immediately
111+
preload={false}
110112
>
111113
Load repo in SemHub
112114
</Link>

packages/web/src/lib/hooks/useRepoStatus.ts renamed to packages/web/src/lib/hooks/useSearchResultRepoStatus.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ type UseRepoStatusResult = {
2020
isLoading: boolean;
2121
};
2222

23-
export function useRepoStatus(
23+
export function useSearchResultRepoStatus(
2424
owner: string | null,
2525
repo: string | null,
2626
): UseRepoStatusResult {

packages/web/src/routes/search.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,20 +8,20 @@ import { useMemo } from "react";
88

99
import { extractOwnerAndRepo } from "@/core/semsearch/util";
1010
import { publicSearchIssues } from "@/lib/api/search";
11-
import { useRepoStatus } from "@/lib/hooks/useRepoStatus";
11+
import { useSearchResultRepoStatus } from "@/lib/hooks/useSearchResultRepoStatus";
1212
import { queryKeys } from "@/lib/queryClient";
1313
import { Button } from "@/components/ui/button";
1414
import { Skeleton } from "@/components/ui/skeleton";
1515
import { RepoPreviewSkeleton } from "@/components/repos/RepoPreview";
16+
import { IssueCard } from "@/components/search/IssueCard";
17+
import { ResultsSearchBar } from "@/components/search/PublicSearchBars";
1618
import {
1719
ErrorMessage,
1820
InitializingMessage,
1921
NoMatchesMessage,
2022
NotFoundMessage,
2123
OnGithubMessage,
22-
} from "@/components/repos/RepoStatusMessages";
23-
import { IssueCard } from "@/components/search/IssueCard";
24-
import { ResultsSearchBar } from "@/components/search/PublicSearchBars";
24+
} from "@/components/search/SearchResultRepoStatusMessages";
2525
import {
2626
publicSearchSchema,
2727
type PublicSearchSchema,
@@ -61,7 +61,7 @@ export const Route = createFileRoute("/search")({
6161
function NothingMatched({ query }: { query: string }) {
6262
const extracted = useMemo(() => extractOwnerAndRepo(query), [query]);
6363

64-
const { repoStatus, error, preview, isLoading } = useRepoStatus(
64+
const { repoStatus, error, preview, isLoading } = useSearchResultRepoStatus(
6565
extracted?.owner ?? null,
6666
extracted?.repo ?? null,
6767
);

0 commit comments

Comments
 (0)