chore(webapp): name the tenant in failure logs

Name the tenant on failure. Across 72 api, dashboard and resource
routes, the catch-path log that reports a failure now includes the
project, organization, environment or user the request was for, taken
from the route params or the authenticated context already in scope.
When one of these routes errors, the log line says whose request it was,
which is what you need to trace an incident to a customer rather than
reading it as an anonymous 500.

Mono-RevId: 401cdce0cf3af4d56827815b11b0cf66ab6cd94f

Author: 1stvamp

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.branches/route.tsx

@@ -127,7 +127,10 @@ export const loader = dashboardLoader(
           ability.can("write", { type: "deployments", envType: "PREVIEW" }),
       });
     } catch (error) {
-      logger.error("Error loading preview branches page", { error });
+      logger.error("Error loading preview branches page", {
+        error,
+        organizationSlug: params.organizationSlug,
+      });
       throw new Response(undefined, {
         status: 400,
         statusText: "Something went wrong, if this problem persists please contact support.",

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.dev-branches/route.tsx

@@ -81,7 +81,10 @@ export const loader = dashboardLoader(
           ability.can("write", { type: "deployments", envType: "DEVELOPMENT" }),
       });
     } catch (error) {
-      logger.error("Error loading dev branches page", { error });
+      logger.error("Error loading dev branches page", {
+        error,
+        organizationSlug: params.organizationSlug,
+      });
       throw new Response(undefined, {
         status: 400,
         statusText: "Something went wrong, if this problem persists please contact support.",

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues/route.tsx

@@ -253,7 +253,10 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
           };
         }
       } catch (error) {
-        logger.warn("Queue list metrics unavailable, rendering without them", { error });
+        logger.warn("Queue list metrics unavailable, rendering without them", {
+          error,
+          organizationSlug: params.organizationSlug,
+        });
       }
     }
 
@@ -264,7 +267,10 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
       try {
         allocation = await new QueueAllocationPresenter().call({ environment });
       } catch (error) {
-        logger.warn("Queue allocation summary unavailable, rendering without it", { error });
+        logger.warn("Queue allocation summary unavailable, rendering without it", {
+          error,
+          organizationSlug: params.organizationSlug,
+        });
       }
     }
 

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.runs.$runParam/route.tsx

@@ -259,7 +259,7 @@ async function getRunsListFromTableState({
 
     return runsList;
   } catch (error) {
-    logger.error("Error loading runs list from tableState:", { error });
+    logger.error("Error loading runs list from tableState:", { error, organizationSlug });
     return null;
   }
 }

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.test.tasks.$taskParam/route.tsx

@@ -153,6 +153,7 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
       taskParam,
       error: error instanceof Error ? error.message : error,
       stack: error instanceof Error ? error.stack : undefined,
+      organizationSlug,
     });
 
     return redirectWithErrorMessage(
@@ -202,7 +203,10 @@ export const action: ActionFunction = async ({ request, params }) => {
           formAction,
         });
       } catch (e) {
-        logger.error("Failed to create template", { error: e instanceof Error ? e.message : e });
+        logger.error("Failed to create template", {
+          error: e instanceof Error ? e.message : e,
+          organizationSlug,
+        });
         return redirectBackWithErrorMessage(request, "Failed to create template");
       }
     }
@@ -226,7 +230,10 @@ export const action: ActionFunction = async ({ request, params }) => {
           formAction,
         });
       } catch (e) {
-        logger.error("Failed to delete template", { error: e instanceof Error ? e.message : e });
+        logger.error("Failed to delete template", {
+          error: e instanceof Error ? e.message : e,
+          organizationSlug,
+        });
         return redirectBackWithErrorMessage(request, "Failed to delete template");
       }
     }
@@ -275,7 +282,10 @@ export const action: ActionFunction = async ({ request, params }) => {
           );
         }
 
-        logger.error("Failed to start a test run", { error: e instanceof Error ? e.message : e });
+        logger.error("Failed to start a test run", {
+          error: e instanceof Error ? e.message : e,
+          organizationSlug,
+        });
 
         return redirectBackWithErrorMessage(
           request,

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam/route.tsx

@@ -136,7 +136,7 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
       ]);
     } catch (error) {
       // The dashboard must load even when the agent's store doesn't answer.
-      logger.error("Failed to read dashboard agent wake activity", { error });
+      logger.error("Failed to read dashboard agent wake activity", { error, organizationSlug });
     }
   }
 

apps/webapp/app/routes/_app.orgs.$organizationSlug.settings._index/route.tsx

@@ -245,6 +245,7 @@ export const action = dashboardAction(
             const errorMessage = error instanceof Error ? error.message : JSON.stringify(error);
             logger.error("Organization could not be deleted", {
               error: errorMessage,
+              organizationSlug,
             });
             return redirectWithErrorMessage(
               organizationSettingsPath({ slug: organizationSlug }),

apps/webapp/app/routes/account.authorization-code.$authorizationCode/route.tsx

@@ -81,7 +81,7 @@ export const action = async ({ request, params }: ActionFunctionArgs) => {
       return typedjson({ success: false as const, error: error.message, source, clientName });
     }
 
-    logger.error(JSON.stringify(error));
+    logger.error(JSON.stringify(error), { userId });
     throw new Response(undefined, {
       status: 400,
       statusText: "Something went wrong, if this problem persists please contact support.",

apps/webapp/app/routes/api.v1.batches.$batchParam.results.ts

@@ -47,11 +47,15 @@ export async function loader({ request, params }: LoaderFunctionArgs) {
       if (unroutable) {
         logger.warn("Unroutable batch id on batch results", {
           error: error instanceof Error ? error.message : error,
+          environmentId: authenticationResult.environment.id,
         });
         return unroutable;
       }
 
-      logger.error("Failed to load batch results", { error });
+      logger.error("Failed to load batch results", {
+        error,
+        environmentId: authenticationResult.environment.id,
+      });
       return json({ error: "Something went wrong, please try again." }, { status: 500 });
     }
   } catch (error) {

apps/webapp/app/routes/api.v1.bulk-actions.$bulkActionId.abort.ts

@@ -39,7 +39,10 @@ const { action } = createActionApiRoute(
         return json({ error: error.message }, { status: error.status ?? 400 });
       }
 
-      logger.error("Failed to abort API bulk action", { error });
+      logger.error("Failed to abort API bulk action", {
+        error,
+        environmentId: authentication.environment.id,
+      });
       return json({ error: "Failed to abort bulk action" }, { status: 500 });
     }
   }