duplicate shortcut and user session fetching fix

Author: mpcgrid

apps/webapp/app/components/logs/LogDetailView.tsx

@@ -117,8 +117,6 @@ export function LogDetailView({ logId, initialLog, onClose, searchTerm }: LogDet
 
       if (e.key === "Escape") {
         onClose();
-      } else if ((e.key === "v" || e.key === "V") && !e.ctrlKey && !e.metaKey && !e.altKey && !e.shiftKey && !isLoading && log) {
-        window.open(runPath, "_blank");
       }
     };
     window.addEventListener("keydown", handleKeyDown);

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.logs.ts

@@ -19,7 +19,10 @@ function parseLevelsFromUrl(url: URL): LogLevel[] | undefined {
 }
 
 export const loader = async ({ request, params }: LoaderFunctionArgs) => {
-  const userId = await requireUserId(request);
+  const user = await requireUser(request);
+  const userId = user.id;
+  const isAdmin = user?.admin || user?.isImpersonating;
+
   const { projectParam, organizationSlug, envParam } = EnvironmentParamSchema.parse(params);
 
   const project = await findProjectBySlug(organizationSlug, projectParam, userId);
@@ -32,9 +35,6 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
     throw new Response("Environment not found", { status: 404 });
   }
 
-  const user = await requireUser(request);
-  const isAdmin = user?.admin || user?.isImpersonating;
-
   // Get the user's plan to determine log retention limit
   const plan = await getCurrentPlan(project.organizationId);
   const retentionLimitDays = plan?.v3Subscription?.plan?.limits.logRetentionDays.number ?? 30;