CM-59284: remove sentry and bump min CLI version (#137)

Author: gotbadger

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## [Unreleased]
 
+- Remove Sentry integration
+- Increase required CLI version to 3.9.2
+
 ## [v1.17.0]
 
 - Increase required CLI version to 3.6.0

package.json

@@ -463,7 +463,6 @@
   },
   "devDependencies": {
     "@eslint/js": "9.20.0",
-    "@sentry/webpack-plugin": "3.1.2",
     "@stylistic/eslint-plugin": "3.1.0",
     "@types/decompress": "4.2.7",
     "@types/eslint__js": "8.42.3",
@@ -486,7 +485,6 @@
     "webpack-cli": "6.0.1"
   },
   "dependencies": {
-    "@sentry/node": "8.49.0",
     "class-transformer": "0.5.1",
     "decompress": "4.2.1",
     "reflect-metadata": "0.2.2",

src/cli/cli-wrapper.ts

@@ -12,7 +12,6 @@ import { CliError, isCliError } from './models/cli-error';
 import { ExitCode } from './exit-code';
 import { CommandParameters } from './constants';
 import { getUserAgentArg } from './user-agent';
-import { captureException } from '../sentry';
 
 export class CliWrapper {
   public workDirectory?: string;
@@ -35,8 +34,7 @@ export class CliWrapper {
     let camelCasedObj;
     try {
       camelCasedObj = JSON_.parse(out);
-    } catch (e) {
-      captureException(e);
+    } catch {
       this.logger.debug('Failed to parse output as JSON. Returning CliResultPanic');
       return new CliResultPanic(exitCode, out);
     }

src/constants.ts

@@ -128,7 +128,7 @@ export const getScanTypeDisplayName = (scanType: string): string => {
   return _SCAN_TYPE_TO_DISPLAY_NAME[scanType];
 };
 
-export const REQUIRED_CLI_VERSION = '3.6.0';
+export const REQUIRED_CLI_VERSION = '3.9.2';
 
 export const CLI_GITHUB = {
   OWNER: 'cycodehq',
@@ -159,9 +159,3 @@ export const getDefaultCliPath = (): string => {
 export const CYCODE_DOMAIN = 'cycode.com';
 export const DEFAULT_CYCODE_API_URL = `https://api.${CYCODE_DOMAIN}`;
 export const DEFAULT_CYCODE_APP_URL = `https://app.${CYCODE_DOMAIN}`;
-
-export const SENTRY_DSN = 'https://d9527c2348300201235516c0c33a5824@o1026942.ingest.us.sentry.io/4507543875813376';
-export const SENTRY_DEBUG = false;
-export const SENTRY_SAMPLE_RATE = 1.0;
-export const SENTRY_SEND_DEFAULT_PII = false;
-export const SENTRY_INCLUDE_LOCAL_VARIABLES = false;

src/extension.ts

@@ -8,7 +8,6 @@ import * as vscode from 'vscode';
 import { extensionName } from './utils/texts';
 import statusBar from './utils/status-bar';
 import { config } from './utils/config';
-import { captureException, initSentry } from './sentry';
 import { container } from 'tsyringe';
 import { ICycodeService } from './services/cycode-service';
 import {
@@ -33,8 +32,6 @@ import { registerOnDidChangeActiveTextEditor } from './listeners/on-did-change-a
 import { showCliInstallFailed } from './utils/tray-notifications';
 
 export function activate(context: vscode.ExtensionContext) {
-  initSentry();
-
   const logger = container.resolve<ILoggerService>(LoggerServiceSymbol);
   logger.initLogger();
   logger.info('Cycode extension is now active');
@@ -88,7 +85,6 @@ const postActivate = async () => {
     const cycode = container.resolve<ICycodeService>(CycodeServiceSymbol);
     await cycode.installCliIfNeededAndCheckAuthentication();
   } catch (error) {
-    captureException(error);
     logger.error(`Cycode CLI is not installed: ${error}`);
     showCliInstallFailed();
   }

src/sentry.ts

@@ -1,6 +1,5 @@
 import * as vscode from 'vscode';
 import * as fs from 'node:fs';
-import { setSentryUser } from '../sentry';
 import { inject, singleton } from 'tsyringe';
 import { ExtensionServiceSymbol, LoggerServiceSymbol, ScanResultsServiceSymbol, StateServiceSymbol } from '../symbols';
 import { GlobalExtensionState, IStateService, TemporaryExtensionState } from './state-service';
@@ -156,9 +155,6 @@ export class CliService implements ICliService {
       this.showErrorNotification('You are not authenticated in Cycode. Please authenticate');
     } else {
       statusBar.showDefault();
-      if (processedResult.result.userId && processedResult.result.tenantId) {
-        setSentryUser(processedResult.result.userId, processedResult.result.tenantId);
-      }
     }
 
     return;

src/services/cli-service.ts

@@ -12,7 +12,6 @@ import { ProgressOptions } from 'vscode';
 import { ILoggerService } from './logger-service';
 import { getScanTypeDisplayName } from '../constants';
 import statusBar from '../utils/status-bar';
-import { captureException } from '../sentry';
 import { CliIgnoreType } from '../cli/models/cli-ignore-type';
 import { IScanResultsService } from './scan-results-service';
 import { IExtensionService } from './extension-service';
@@ -52,7 +51,6 @@ export class CycodeService implements ICycodeService {
           progress.report({ message });
           return await fn(cancellationToken);
         } catch (error: unknown) {
-          captureException(error);
           if (error instanceof Error) {
             this.logger.error(`Error during progress: ${error.message}. FN: ${fn}`);
             vscode.window.showErrorMessage(`Cycode error: ${error.message}`);

src/services/cycode-service.ts

@@ -1,7 +1,6 @@
 import * as fs from 'fs';
 import * as crypto from 'crypto';
 import * as path from 'path';
-import { captureException } from '../sentry';
 import { container } from 'tsyringe';
 import { ILoggerService } from '../services/logger-service';
 import { LoggerServiceSymbol } from '../symbols';
@@ -24,7 +23,6 @@ export const verifyFileChecksum = (filePath: string, checksum: string): boolean
   try {
     return getFileShaHash(filePath).toLowerCase() === checksum.toLowerCase();
   } catch (error) {
-    captureException(error);
     logger.error(`Failed to verify file checksum ${error}`);
   }
 

src/utils/file-checksum.ts

@@ -3,7 +3,6 @@
 "use strict";
 
 const path = require("path");
-const { sentryWebpackPlugin } = require("@sentry/webpack-plugin");
 
 //@ts-check
 /** @typedef {import('webpack').Configuration} WebpackConfig **/
@@ -42,14 +41,6 @@ const extensionConfig = {
     ],
   },
   devtool: "source-map",
-    plugins: [
-        sentryWebpackPlugin({
-            org: "cycode",
-            project: "vscode-extension",
-            telemetry: false,
-            authToken: process.env.SENTRY_AUTH_TOKEN,
-        }),
-    ],
   infrastructureLogging: {
     level: "log", // enables logging required for problem matchers
   },