chore(webapp): Logging cleanup

Author: 0ski

apps/webapp/app/models/vercelIntegration.server.ts

@@ -511,14 +511,6 @@ export class VercelIntegrationRepository {
           }
 
           try {
-            logger.debug("Fetching decrypted value for shared env var", {
-              teamId,
-              envId: env.id,
-              envKey: env.key,
-              envType: env.type,
-              envTarget: env.target,
-            });
-
             // Get the decrypted value for this shared env var
             const getResponse = await client.environment.getSharedEnvVar({
               id: env.id as string,
@@ -655,12 +647,6 @@ export class VercelIntegrationRepository {
         raw: params.raw,
       };
 
-      logger.debug("Updating Vercel secret", {
-        integrationId: params.integrationId,
-        teamId: params.teamId,
-        installationId: params.installationId,
-      });
-
       await secretStore.setSecret(integration.tokenReference.key, secretValue);
 
       await tx.organizationIntegration.update({
@@ -702,11 +688,6 @@ export class VercelIntegrationRepository {
         raw: params.raw,
       };
 
-      logger.debug("Storing Vercel secret", {
-        teamId: params.teamId,
-        installationId: params.installationId,
-      });
-
       await secretStore.setSecret(integrationFriendlyId, secretValue);
 
       const reference = await tx.secretReference.create({
@@ -852,10 +833,6 @@ export class VercelIntegrationRepository {
       }
 
       if (envVarsToSync.length === 0) {
-        logger.debug("No API keys to sync to Vercel", {
-          projectId: params.projectId,
-          vercelProjectId: params.vercelProjectId,
-        });
         return { success: true, errors: [] };
       }
 
@@ -1022,15 +999,6 @@ export class VercelIntegrationRepository {
         });
       }
 
-      logger.info("Vercel pullEnvVarsFromVercel: environment mapping", {
-        projectId: params.projectId,
-        vercelProjectId: params.vercelProjectId,
-        envMappingCount: envMapping.length,
-        envMapping: envMapping.map(m => ({ type: m.triggerEnvType, target: m.vercelTarget })),
-        runtimeEnvironmentsCount: runtimeEnvironments.length,
-        runtimeEnvironments: runtimeEnvironments.map(e => e.type),
-      });
-
       if (envMapping.length === 0) {
         logger.warn("No environments to sync for Vercel integration", {
           projectId: params.projectId,
@@ -1089,16 +1057,6 @@ export class VercelIntegrationRepository {
             continue;
           }
 
-          logger.info("Vercel pullEnvVarsFromVercel: fetched env vars for target", {
-            projectId: params.projectId,
-            vercelTarget: mapping.vercelTarget,
-            triggerEnvType: mapping.triggerEnvType,
-            projectEnvVarsCount: projectEnvVars.length,
-            sharedEnvVarsCount: filteredSharedEnvVars.length,
-            mergedEnvVarsCount: mergedEnvVars.length,
-            mergedEnvVarKeys: mergedEnvVars.map(v => v.key),
-          });
-
           const varsToSync = mergedEnvVars.filter((envVar) => {
             if (envVar.isSecret) {
               return false;
@@ -1113,14 +1071,6 @@ export class VercelIntegrationRepository {
             );
           });
 
-          logger.info("Vercel pullEnvVarsFromVercel: filtered vars to sync", {
-            projectId: params.projectId,
-            vercelTarget: mapping.vercelTarget,
-            triggerEnvType: mapping.triggerEnvType,
-            varsToSyncCount: varsToSync.length,
-            varsToSyncKeys: varsToSync.map(v => v.key),
-          });
-
           if (varsToSync.length === 0) {
             continue;
           }
@@ -1181,24 +1131,9 @@ export class VercelIntegrationRepository {
           });
 
           if (changedVars.length === 0) {
-            logger.info("Vercel pullEnvVarsFromVercel: no changes detected, skipping", {
-              projectId: params.projectId,
-              vercelTarget: mapping.vercelTarget,
-              triggerEnvType: mapping.triggerEnvType,
-              skippedCount: varsToSync.length,
-            });
             continue;
           }
 
-          logger.info("Vercel pullEnvVarsFromVercel: updating changed vars", {
-            projectId: params.projectId,
-            vercelTarget: mapping.vercelTarget,
-            triggerEnvType: mapping.triggerEnvType,
-            changedCount: changedVars.length,
-            unchangedCount: varsToSync.length - changedVars.length,
-            changedKeys: changedVars.map((v) => v.key),
-          });
-
           const secretVars = changedVars.filter((v) => existingSecretKeys.has(v.key));
           const nonSecretVars = changedVars.filter((v) => !existingSecretKeys.has(v.key));
 
@@ -1520,11 +1455,6 @@ export class VercelIntegrationRepository {
         },
       });
 
-      logger.info("Disabled autoAssignCustomDomains on Vercel project", {
-        vercelProjectId,
-        teamId,
-      });
-
       return { success: true };
     } catch (error) {
       const errorMessage = `Failed to disable autoAssignCustomDomains: ${error instanceof Error ? error.message : "Unknown error"}`;

apps/webapp/app/presenters/v3/VercelSettingsPresenter.server.ts

@@ -1,6 +1,7 @@
 import { type PrismaClient } from "@trigger.dev/database";
 import { fromPromise, ok, ResultAsync } from "neverthrow";
 import { env } from "~/env.server";
+import { logger } from "~/services/logger.server";
 import { OrgIntegrationRepository } from "~/models/orgIntegration.server";
 import {
   VercelIntegrationRepository,
@@ -280,12 +281,12 @@ export class VercelSettingsPresenter extends BasePresenter {
           } as VercelSettingsResult));
         }).mapErr((error) => {
           // Log the error and return a safe fallback
-          console.error("Error in VercelSettingsPresenter.call:", error);
+          logger.error("Error in VercelSettingsPresenter.call", { error });
           return error;
         });
       } catch (syncError) {
         // Handle any synchronous errors that might occur
-        console.error("Synchronous error in VercelSettingsPresenter.call:", syncError);
+        logger.error("Synchronous error in VercelSettingsPresenter.call", { error: syncError });
         return ok({
           enabled: true,
           hasOrgIntegration: false,
@@ -299,7 +300,7 @@ export class VercelSettingsPresenter extends BasePresenter {
       }
     } catch (error) {
       // If there's an unexpected error, log it and return a safe error result
-      console.error("Unexpected error in VercelSettingsPresenter.call:", error);
+      logger.error("Unexpected error in VercelSettingsPresenter.call", { error });
       return ok({
         enabled: true,
         hasOrgIntegration: false,
@@ -566,7 +567,7 @@ export class VercelSettingsPresenter extends BasePresenter {
         isGitHubConnected,
       };
     } catch (error) {
-      console.error("Error in getOnboardingData:", error);
+      logger.error("Error in getOnboardingData", { error });
       return null;
     }
   }

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.vercel.tsx

@@ -287,16 +287,6 @@ export async function action({ request, params }: ActionFunctionArgs) {
       }
     }
 
-    logger.info("Vercel complete-onboarding action: received params", {
-      projectId: project.id,
-      vercelStagingEnvironment,
-      pullEnvVarsBeforeBuild,
-      atomicBuilds,
-      discoverEnvVars,
-      syncEnvVarsMappingRaw: syncEnvVarsMapping,
-      parsedMappingKeys: Object.keys(parsedMapping),
-    });
-
     const parsedStagingEnv = parseVercelStagingEnvironment(vercelStagingEnvironment);
 
     const result = await vercelService.completeOnboarding(project.id, {