Add Sentry error tracking for environment load/transform

src/renderer/actions/local-sync/fs-utils.ts

@@ -1,4 +1,5 @@
 import { v4 as uuidv4 } from "uuid";
+import * as Sentry from "@sentry/browser";
 import {
   API,
   APIEntity,
@@ -275,17 +276,30 @@ export async function writeContent(
         };
       }
     }
-
+    
     const parsedContentResult = parseRaw(content, fileType.validator);
     if (parsedContentResult.type === "error") {
+      // Send analytics event for validation failure
+      Sentry.captureEvent({
+        message: `Environment validation error: ${parsedContentResult.error.message}`,
+        level: "error",
+        tags: {
+          fileType: fileType.type,
+          errorType: "validation_failed",
+        },
+        extra: {
+          path: resource.path,
+          error: parsedContentResult.error.message,
+          content,
+        },
+      });
       return createFileSystemError(
         { message: parsedContentResult.error.message },
         resource.path,
         fileType.type
       );
     }
 
-    console.log("writing at", resource.path);
     const serializedContent = serializeContentForWriting(content);
     if (writeWithElevatedAccess) {
       await FsService.writeFileWithElevatedAccess(
@@ -307,6 +321,20 @@ export async function writeContent(
       },
     };
   } catch (e: any) {
+    // Send analytics event for write exception
+    Sentry.captureEvent({
+      message: `Environment write error: ${e.message}`,
+      level: "error",
+      tags: {
+        fileType: fileType.type,
+        errorType: "write_exception",
+      },
+      extra: {
+        path: resource.path,
+        error: e.message,
+        stack: e.stack,
+      },
+    });
     return createFileSystemError(e, resource.path, fileType.type);
   }
 }
@@ -1060,55 +1088,144 @@ export function sanitizeFsResourceList(
 export async function parseFileToEnv(
   file: FileResource
 ): Promise<FileSystemResult<Environment>> {
-  const parsedFileResult = await parseFile({
-    resource: file,
-    fileType: new EnvironmentRecordFileType(),
-  });
-
-  if (parsedFileResult.type === "error") {
-    return parsedFileResult;
-  }
+  try {
+    const parsedFileResult = await parseFile({
+      resource: file,
+      fileType: new EnvironmentRecordFileType(),
+    });
 
-  const { content } = parsedFileResult;
+    if (parsedFileResult.type === "error") {
+      Sentry.captureEvent({
+        message: `Environment not found: ${parsedFileResult.error.message}`,
+        level: "error",
+        tags: {
+          fileType: "environment",
+          errorType: "environment_load_failed",
+        },
+        extra: {
+          path: file.path,
+          error: parsedFileResult.error.message,
+          errorCode: parsedFileResult.error.code,
+        },
+      });
 
-  const isGlobal = file.path.endsWith(`/${GLOBAL_ENV_FILE}`);
-  const environment: Environment = {
-    type: "environment",
-    id: getIdFromPath(file.path),
-    name: content.name,
-    variables: content.variables,
-    isGlobal,
-  };
+      return parsedFileResult;
+    }
 
-  const result: FileSystemResult<Environment> = {
-    type: "success",
-    content: environment,
-  };
+    const { content } = parsedFileResult;
+    const isGlobal = file.path.endsWith(`/${GLOBAL_ENV_FILE}`);
+    const environment: Environment = {
+      type: "environment",
+      id: getIdFromPath(file.path),
+      name: content.name,
+      variables: content.variables,
+      isGlobal,
+    };
 
-  return result;
+    return {
+      type: "success",
+      content: environment,
+    };
+  } catch (e: any) {
+    Sentry.captureEvent({
+      message: `Environment load exception: ${e.message}`,
+      level: "error",
+      tags: {
+        fileType: "environment",
+        errorType: "environment_load_exception",
+      },
+      extra: {
+        path: file.path,
+        error: e.message,
+        stack: e.stack,
+      },
+    });
+    return createFileSystemError(e, file.path, FileTypeEnum.ENVIRONMENT);
+  }
 }
 
 export function parseToEnvironmentEntity(
   variables: Record<string, EnvironmentVariableValue>
 ) {
-  const newVariables: Record<
-    string,
-    Static<(typeof EnvironmentRecord)["variables"]>
-  > = {};
-  // eslint-disable-next-line
-  for (const key in variables) {
-    newVariables[key] = {
-      value: variables[key].syncValue,
-      type:
-        variables[key].type === EnvironmentVariableType.Secret
-          ? EnvironmentVariableType.String
-          : variables[key].type,
-      id: variables[key].id,
-      isSecret: variables[key].type === EnvironmentVariableType.Secret,
-    };
-  }
+  try {
+    const newVariables: Record<
+      string,
+      Static<(typeof EnvironmentRecord)["variables"]>
+    > = {};
+    const missingFieldsPerVariable: Record<string, string[]> = {};
+
+    // eslint-disable-next-line
+    for (const key in variables) {
+      const variable = variables[key];
+      const missingFields: string[] = [];
+
+      const resolvedValue =
+        variable.localValue !== undefined
+          ? variable.localValue
+          : variable.syncValue !== undefined
+          ? variable.syncValue
+          : (variable as any).value !== undefined
+          ? (variable as any).value
+          : undefined;
+
+      if (!variable.id) {
+        missingFields.push("id");
+      }
+      if (resolvedValue === undefined) {
+        missingFields.push("value");
+      }
+      if (!variable.type) {
+        missingFields.push("type");
+      }
 
-  return newVariables;
+      if (missingFields.length > 0) {
+        missingFieldsPerVariable[key] = missingFields;
+      }
+
+      newVariables[key] = {
+        value: resolvedValue,
+        type:
+          variable.type === EnvironmentVariableType.Secret
+            ? EnvironmentVariableType.String
+            : variable.type,
+        id: variable.id,
+        isSecret: variable.type === EnvironmentVariableType.Secret,
+      };
+    }
+
+    // If any variables have missing fields, send analytics event
+    if (Object.keys(missingFieldsPerVariable).length > 0) {
+      Sentry.captureEvent({
+        message: `Environment variables with missing fields: ${Object.keys(missingFieldsPerVariable).length}`,
+        level: "warning",
+        tags: {
+          fileType: "environment",
+          errorType: "invalid_variable_structure",
+        },
+        extra: {
+          totalVariables: Object.keys(variables).length,
+          variablesWithIssues: Object.keys(missingFieldsPerVariable).length,
+          missingFields: missingFieldsPerVariable,
+        },
+      });
+    }
+
+    return newVariables;
+  } catch (e: any) {
+    Sentry.captureEvent({
+      message: `Environment variable transform exception: ${e.message}`,
+      level: "error",
+      tags: {
+        fileType: "environment",
+        errorType: "variable_transform_exception",
+      },
+      extra: {
+        error: e.message,
+        stack: e.stack,
+      },
+    });
+    throw e;
+  }
 }
 
 export function getFileNameFromPath(filePath: string) {