wip: move permissions check to Prisma extension

Author: brendan-kellam

packages/backend/src/index.ts

@@ -74,7 +74,7 @@ await repoManager.validateIndexedReposHaveShards();
 
 const connectionManagerInterval = connectionManager.startScheduler();
 const repoManagerInterval = repoManager.startScheduler();
-const permissionSyncerInterval = env.EXPERIMENT_PERMISSION_SYNC_ENABLED ? permissionSyncer.startScheduler() : null;
+const permissionSyncerInterval = env.EXPERIMENT_PERMISSION_SYNC_ENABLED === 'true' ? permissionSyncer.startScheduler() : null;
 
 
 const cleanup = async (signal: string) => {

packages/backend/src/permissionSyncer.ts

@@ -59,7 +59,7 @@ export class RepoPermissionSyncer {
             }
 
         // @todo: make this configurable
-        }, 1000 * 5);
+        }, 1000 * 60);
     }
 
     public dispose() {

packages/web/src/actions.ts

@@ -20,7 +20,7 @@ export const CodePreviewPanel = async ({ path, repoName, revisionName, domain }:
             repository: repoName,
             branch: revisionName,
         }, domain),
-        getRepoInfoByName(repoName, domain),
+        getRepoInfoByName(repoName),
     ]);
 
     if (isServiceError(fileSourceResponse) || isServiceError(repoInfoResponse)) {

packages/web/src/app/[domain]/browse/[...path]/components/codePreviewPanel.tsx

@@ -10,17 +10,16 @@ interface TreePreviewPanelProps {
     path: string;
     repoName: string;
     revisionName?: string;
-    domain: string;
 }
 
-export const TreePreviewPanel = async ({ path, repoName, revisionName, domain }: TreePreviewPanelProps) => {
+export const TreePreviewPanel = async ({ path, repoName, revisionName }: TreePreviewPanelProps) => {
     const [repoInfoResponse, folderContentsResponse] = await Promise.all([
-        getRepoInfoByName(repoName, domain),
+        getRepoInfoByName(repoName),
         getFolderContents({
             repoName,
             revisionName: revisionName ?? 'HEAD',
             path,
-        }, domain)
+        })
     ]);
 
     if (isServiceError(folderContentsResponse) || isServiceError(repoInfoResponse)) {

packages/web/src/app/[domain]/browse/[...path]/components/treePreviewPanel.tsx

@@ -42,7 +42,6 @@ export default async function BrowsePage(props: BrowsePageProps) {
                         path={path}
                         repoName={repoName}
                         revisionName={revisionName}
-                        domain={domain}
                     />
                 )}
             </Suspense>

packages/web/src/app/[domain]/browse/[...path]/page.tsx

@@ -6,7 +6,6 @@ import { useHotkeys } from "react-hotkeys-hook";
 import { useQuery } from "@tanstack/react-query";
 import { unwrapServiceError } from "@/lib/utils";
 import { FileTreeItem, getFiles } from "@/features/fileTree/actions";
-import { useDomain } from "@/hooks/useDomain";
 import { Dialog, DialogContent, DialogDescription, DialogTitle } from "@/components/ui/dialog";
 import { useBrowseNavigation } from "../hooks/useBrowseNavigation";
 import { useBrowseState } from "../hooks/useBrowseState";
@@ -28,7 +27,6 @@ type SearchResult = {
 
 export const FileSearchCommandDialog = () => {
     const { repoName, revisionName } = useBrowseParams();
-    const domain = useDomain();
     const { state: { isFileSearchOpen }, updateBrowseState } = useBrowseState();
 
     const commandListRef = useRef<HTMLDivElement>(null);
@@ -57,8 +55,8 @@ export const FileSearchCommandDialog = () => {
     }, [isFileSearchOpen]);
 
     const { data: files, isLoading, isError } = useQuery({
-        queryKey: ['files', repoName, revisionName, domain],
-        queryFn: () => unwrapServiceError(getFiles({ repoName, revisionName: revisionName ?? 'HEAD' }, domain)),
+        queryKey: ['files', repoName, revisionName],
+        queryFn: () => unwrapServiceError(getFiles({ repoName, revisionName: revisionName ?? 'HEAD' })),
         enabled: isFileSearchOpen,
     });
 

packages/web/src/app/[domain]/browse/components/fileSearchCommandDialog.tsx

@@ -8,7 +8,6 @@ import { zodResolver } from "@hookform/resolvers/zod";
 import { useForm } from "react-hook-form";
 import { z } from "zod";
 import { experimental_addGithubRepositoryByUrl } from "@/actions";
-import { useDomain } from "@/hooks/useDomain";
 import { isServiceError } from "@/lib/utils";
 import { useToast } from "@/components/hooks/use-toast";
 import { useRouter } from "next/navigation";
@@ -37,7 +36,6 @@ const formSchema = z.object({
 });
 
 export const AddRepositoryDialog = ({ isOpen, onOpenChange }: AddRepositoryDialogProps) => {
-    const domain = useDomain();
     const { toast } = useToast();
     const router = useRouter();
 
@@ -52,7 +50,7 @@ export const AddRepositoryDialog = ({ isOpen, onOpenChange }: AddRepositoryDialo
 
     const onSubmit = async (data: z.infer<typeof formSchema>) => {
 
-        const result = await experimental_addGithubRepositoryByUrl(data.repositoryUrl.trim(), domain);
+        const result = await experimental_addGithubRepositoryByUrl(data.repositoryUrl.trim());
         if (isServiceError(result)) {
             toast({
                 title: "Error adding repository",

packages/web/src/app/[domain]/repos/components/addRepositoryDialog.tsx

@@ -3,18 +3,18 @@ import { isServiceError } from "@/lib/utils";
 import { NextRequest } from "next/server";
 
 export async function GET(
-    request: NextRequest,
+    _request: NextRequest,
     props: { params: Promise<{ domain: string; repoId: string }> }
 ) {
     const params = await props.params;
-    const { domain, repoId } = params;
+    const { repoId } = params;
     const repoIdNum = parseInt(repoId);
 
     if (isNaN(repoIdNum)) {
         return new Response("Invalid repo ID", { status: 400 });
     }
 
-    const result = await getRepoImage(repoIdNum, domain);
+    const result = await getRepoImage(repoIdNum);
     if (isServiceError(result)) {
         return new Response(result.message, { status: result.statusCode });
     }