Mz/disable apex ls error telemetry

packages/salesforcedx-utils/src/types/localization/localizationProvider.ts

@@ -5,6 +5,5 @@
  * For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 export type LocalizationProvider = {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   localize(label: string, ...args: any[]): string;
 };

packages/salesforcedx-visualforce-language-server/src/visualforceServer.ts

@@ -25,7 +25,6 @@ import {
   ColorInformation,
   ColorPresentationRequest,
   DocumentColorRequest,
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   ServerCapabilities as CPServerCapabilities
 } from 'vscode-languageserver-protocol';
 import { ConfigurationParams, ConfigurationRequest } from 'vscode-languageserver-protocol';

packages/salesforcedx-vscode-apex-debugger/src/adapter/debugConfigurationProvider.ts

@@ -26,7 +26,7 @@ export class DebugConfigurationProvider implements vscode.DebugConfigurationProv
 
   public provideDebugConfigurations(
     folder: vscode.WorkspaceFolder | undefined,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+
     token?: vscode.CancellationToken
   ): vscode.ProviderResult<vscode.DebugConfiguration[]> {
     return [DebugConfigurationProvider.getConfig(folder)];
@@ -35,7 +35,7 @@ export class DebugConfigurationProvider implements vscode.DebugConfigurationProv
   public resolveDebugConfiguration(
     folder: vscode.WorkspaceFolder | undefined,
     config: vscode.DebugConfiguration,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+
     token?: vscode.CancellationToken
   ): vscode.ProviderResult<vscode.DebugConfiguration> {
     return this.asyncDebugConfig(folder, config).catch(async err => {

packages/salesforcedx-vscode-apex-debugger/src/context/isvContext.ts

@@ -27,11 +27,11 @@ export const registerIsvAuthWatcher = (extensionContext: vscode.ExtensionContext
   if (vscode.workspace.workspaceFolders instanceof Array && vscode.workspace.workspaceFolders.length > 0) {
     const configPath = projectPaths.salesforceProjectConfig();
     const isvAuthWatcher = vscode.workspace.createFileSystemWatcher(configPath);
-    /* eslint-disable @typescript-eslint/no-unused-vars */
+
     isvAuthWatcher.onDidChange(uri => setupGlobalDefaultUserIsvAuth());
     isvAuthWatcher.onDidCreate(uri => setupGlobalDefaultUserIsvAuth());
     isvAuthWatcher.onDidDelete(uri => setupGlobalDefaultUserIsvAuth());
-    /* eslint-enable @typescript-eslint/no-unused-vars */
+
     extensionContext.subscriptions.push(isvAuthWatcher);
   }
 };

packages/salesforcedx-vscode-apex-debugger/src/index.ts

@@ -185,15 +185,15 @@ export const getExceptionBreakpointCache = (): Map<string, ExceptionBreakpointIt
 
 const registerFileWatchers = (): vscode.Disposable => {
   const clsWatcher = vscode.workspace.createFileSystemWatcher('**/*.cls');
-  /* eslint-disable @typescript-eslint/no-unused-vars */
+
   clsWatcher.onDidChange(uri => notifyDebuggerSessionFileChanged());
   clsWatcher.onDidCreate(uri => notifyDebuggerSessionFileChanged());
   clsWatcher.onDidDelete(uri => notifyDebuggerSessionFileChanged());
   const trgWatcher = vscode.workspace.createFileSystemWatcher('**/*.trigger');
   trgWatcher.onDidChange(uri => notifyDebuggerSessionFileChanged());
   trgWatcher.onDidCreate(uri => notifyDebuggerSessionFileChanged());
   trgWatcher.onDidDelete(uri => notifyDebuggerSessionFileChanged());
-  /* eslint-enable @typescript-eslint/no-unused-vars */
+
   return vscode.Disposable.from(clsWatcher, trgWatcher);
 };
 

packages/salesforcedx-vscode-apex/package.json

@@ -387,6 +387,11 @@
           "type": "boolean",
           "default": false,
           "description": "%apex_code_disable-warnings-for-missing-coverage%"
+        },
+        "salesforcedx-vscode-apex.enable-apex-ls-error-to-telemetry": {
+          "type": "boolean",
+          "default": false,
+          "description": "%enable-apex-ls-error-to-telemetry%"
         }
       }
     },

packages/salesforcedx-vscode-apex/package.nls.ja.json

@@ -19,6 +19,7 @@
   "apex_verbose_level_trace_description": "Output everything, including details about notifications and responses received by the client, and requests sent by the server.",
   "configuration_title": "Salesforce Apex Configuration",
   "collapse_tests_title": "SFDX: Apex テストを隠す",
+  "enable-apex-ls-error-to-telemetry": "Allow the Apex Language Server to collect telemetry of errors",
   "go_to_definition_title": "定義に移動",
   "java_home_description": "Specifies the folder path to the Java 11, Java 17, or Java 21 runtime used to launch the Apex Language Server. Note on Windows the backslashes must be escaped.\n\nMac Example: `/Library/Java/JavaVirtualMachines/openjdk-11.jdk/Contents/Home`\n\nWindows Example: `C:\\\\Program Files\\\\Zulu\\\\zulu-17`\n\nLinux Example: `/usr/lib/jvm/java-21-openjdk-amd64`",
   "refresh_test_title": "テストを更新",

packages/salesforcedx-vscode-apex/package.nls.json

@@ -19,6 +19,7 @@
   "apex_verbose_level_trace_description": "Output everything, including details about notifications and responses received by the client, and requests sent by the server.",
   "configuration_title": "Salesforce Apex Configuration",
   "collapse_tests_title": "SFDX: Collapse All Apex Tests",
+  "enable-apex-ls-error-to-telemetry": "Allow the Apex Language Server to collect telemetry of errors",
   "go_to_definition_title": "Go to Definition",
   "java_home_description": "Specifies the folder path to the Java 11, Java 17, or Java 21 runtime used to launch the Apex Language Server. Note on Windows the backslashes must be escaped.\n\nMac Example: `/Library/Java/JavaVirtualMachines/openjdk-11.jdk/Contents/Home`\n\nWindows Example: `C:\\\\Program Files\\\\Zulu\\\\zulu-17`\n\nLinux Example: `/usr/lib/jvm/java-21-openjdk-amd64`",
   "java_memory_description": "Specifies the amount of memory allocation to the Apex Language Server in MB, or null to use the system default value.",

packages/salesforcedx-vscode-apex/src/languageServer.ts

@@ -14,7 +14,7 @@ import { LSP_ERR, UBER_JAR_NAME } from './constants';
 import { soqlMiddleware } from './embeddedSoql';
 import { nls } from './messages';
 import * as requirements from './requirements';
-import { retrieveEnableSyncInitJobs } from './settings';
+import { retrieveEnableApexLSErrorToTelemetry, retrieveEnableSyncInitJobs } from './settings';
 import { getTelemetryService } from './telemetry/telemetry';
 
 const JDWP_DEBUG_PORT = 2739;
@@ -153,7 +153,8 @@ export const buildClientOptions = (): LanguageClientOptions => {
     },
     initializationOptions: {
       enableEmbeddedSoqlCompletion: soqlExtensionInstalled,
-      enableSynchronizedInitJobs: retrieveEnableSyncInitJobs()
+      enableSynchronizedInitJobs: retrieveEnableSyncInitJobs(),
+      enableErrorToTelemetry: retrieveEnableApexLSErrorToTelemetry()
     },
     ...(soqlExtensionInstalled ? { middleware: soqlMiddleware } : {}),
     errorHandler: new ApexErrorHandler()

packages/salesforcedx-vscode-apex/src/settings.ts

@@ -17,3 +17,9 @@ export const retrieveTestCodeCoverage = (): boolean => {
 export const retrieveEnableSyncInitJobs = (): boolean => {
   return vscode.workspace.getConfiguration().get<boolean>('salesforcedx-vscode-apex.wait-init-jobs', true);
 };
+
+export const retrieveEnableApexLSErrorToTelemetry = (): boolean => {
+  return vscode.workspace
+    .getConfiguration()
+    .get<boolean>('salesforcedx-vscode-apex.enable-apex-ls-error-to-telemetry', false);
+};

packages/salesforcedx-vscode-apex/src/views/testOutlineProvider.ts

@@ -14,7 +14,6 @@ import { nls } from '../messages';
 import { IconsEnum, iconHelpers } from './icons';
 import { ApexTestMethod } from './lspConverter';
 
-// eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
 const safeLocalize = (val: string) => nls.localize(val);
 
 // Message
@@ -161,7 +160,7 @@ export class ApexTestOutlineProvider implements vscode.TreeDataProvider<TestNode
           this.apexTestMap.set(test.definingType, apexGroup);
         }
         const apexTest = new ApexTestNode(test.methodName, test.location);
-        // eslint-disable-next-line @typescript-eslint/restrict-plus-operands
+
         apexTest.name = apexGroup.label + '.' + apexTest.label;
         this.apexTestMap.set(apexTest.name, apexTest);
         apexGroup.children.push(apexTest);
@@ -240,7 +239,7 @@ export abstract class TestNode extends vscode.TreeItem {
   };
 
   // TODO: create a ticket to address this particular issue.
-  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+
   // @ts-ignore
   get tooltip(): string {
     return this.description;

packages/salesforcedx-vscode-apex/test/vscode-integration/commands/apexTestRunCodeAction.test.ts

@@ -375,33 +375,30 @@ describe('Apex Test Run - Code Action', () => {
         testLevel: TestLevel.RunSpecifiedTests
       });
       const apexLibExecutor = new ApexLibraryTestRunExecutor(['testClass', 'secondTestClass'], 'path/to/dir', false);
-      runTestStub.callsFake(
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        (payload, codecoverage, exitEarly, progressReporter, token) => {
-          progressReporter.report({
-            type: 'StreamingClientProgress',
-            value: 'streamingTransportUp',
-            message: 'Listening for streaming state changes...'
-          });
-          progressReporter.report({
-            type: 'StreamingClientProgress',
-            value: 'streamingProcessingTestRun',
-            message: 'Processing test run 707500000000000001',
-            testRunId: '707500000000000001'
-          });
-          progressReporter.report({
-            type: 'FormatTestResultProgress',
-            value: 'retrievingTestRunSummary',
-            message: 'Retrieving test run summary record'
-          });
-          progressReporter.report({
-            type: 'FormatTestResultProgress',
-            value: 'queryingForAggregateCodeCoverage',
-            message: 'Querying for aggregate code coverage results'
-          });
-          return passingResult;
-        }
-      );
+      runTestStub.callsFake((payload, codecoverage, exitEarly, progressReporter, token) => {
+        progressReporter.report({
+          type: 'StreamingClientProgress',
+          value: 'streamingTransportUp',
+          message: 'Listening for streaming state changes...'
+        });
+        progressReporter.report({
+          type: 'StreamingClientProgress',
+          value: 'streamingProcessingTestRun',
+          message: 'Processing test run 707500000000000001',
+          testRunId: '707500000000000001'
+        });
+        progressReporter.report({
+          type: 'FormatTestResultProgress',
+          value: 'retrievingTestRunSummary',
+          message: 'Retrieving test run summary record'
+        });
+        progressReporter.report({
+          type: 'FormatTestResultProgress',
+          value: 'queryingForAggregateCodeCoverage',
+          message: 'Querying for aggregate code coverage results'
+        });
+        return passingResult;
+      });
 
       await apexLibExecutor.run(undefined, progress, cancellationToken);
 

packages/salesforcedx-vscode-core/src/commands/aliasList.ts

@@ -10,7 +10,6 @@ import { nls } from '../messages';
 import { EmptyParametersGatherer, SfCommandlet, SfCommandletExecutor, SfWorkspaceChecker } from './util';
 
 export class AliasList extends SfCommandletExecutor<{}> {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('alias_list_text'))

packages/salesforcedx-vscode-core/src/commands/auth/orgLoginAccessToken.ts

@@ -19,15 +19,13 @@ export class OrgLoginAccessTokenExecutor extends LibraryCommandletExecutor<Acces
     super(nls.localize('org_login_access_token_text'), 'org_login_access_token', OUTPUT_CHANNEL);
   }
 
-  /* eslint-disable @typescript-eslint/no-unused-vars */
   public async run(
     response: ContinueResponse<AccessTokenParams>,
     progress?: vscode.Progress<{
       message?: string | undefined;
       increment?: number | undefined;
     }>,
     token?: vscode.CancellationToken
-    /* eslint-enable @typescript-eslint/no-unused-vars */
   ): Promise<boolean> {
     const { instanceUrl, accessToken, alias } = response.data;
     try {

packages/salesforcedx-vscode-core/src/commands/auth/orgLogout.ts

@@ -35,7 +35,6 @@ export class OrgLogoutAll extends SfCommandletExecutor<{}> {
     return instance;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('org_logout_all_text'))
@@ -61,15 +60,13 @@ export class OrgLogoutDefault extends LibraryCommandletExecutor<string> {
     super(nls.localize('org_logout_default_text'), 'org_logout_default', OUTPUT_CHANNEL);
   }
 
-  /* eslint-disable @typescript-eslint/no-unused-vars */
   public async run(
     response: ContinueResponse<string>,
     progress?: Progress<{
       message?: string | undefined;
       increment?: number | undefined;
     }>,
     token?: CancellationToken
-    /* eslint-enable @typescript-eslint/no-unused-vars */
   ): Promise<boolean> {
     try {
       await removeUsername(response.data);

packages/salesforcedx-vscode-core/src/commands/configList.ts

@@ -10,7 +10,6 @@ import { nls } from '../messages';
 import { EmptyParametersGatherer, SfCommandlet, SfCommandletExecutor, SfWorkspaceChecker } from './util';
 
 export class ConfigList extends SfCommandletExecutor<{}> {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('config_list_text'))

packages/salesforcedx-vscode-core/src/commands/configSet.ts

@@ -33,7 +33,6 @@ export class ConfigSetExecutor extends LibraryCommandletExecutor<{}> {
     this.usernameOrAlias = `${usernameOrAlias}`.split(',')[0];
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public async run(response: ContinueResponse<string>): Promise<boolean> {
     let result: boolean;
     let message: string | undefined;

packages/salesforcedx-vscode-core/src/commands/dataQuery.ts

@@ -87,7 +87,6 @@ export enum ApiType {
 
 const workspaceChecker = new SfWorkspaceChecker();
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
 export const dataQuery = (explorerDir?: any): void => {
   const parameterGatherer = new GetQueryAndApiInputs();
   const commandlet = new SfCommandlet(workspaceChecker, parameterGatherer, new DataQueryExecutor());

packages/salesforcedx-vscode-core/src/commands/debuggerStop.ts

@@ -58,7 +58,6 @@ export class DebuggerSessionDetachExecutor extends SfCommandletExecutor<IdSelect
 }
 
 export class StopActiveDebuggerSessionExecutor extends SfCommandletExecutor<{}> {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withArg('data:query')

packages/salesforcedx-vscode-core/src/commands/describeMetadata.ts

@@ -23,7 +23,6 @@ export class DescribeMetadataExecutor extends SfCommandletExecutor<string> {
     super();
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withArg('org:list:metadata-types')

packages/salesforcedx-vscode-core/src/commands/isvdebugging/bootstrapCmd.ts

@@ -35,7 +35,7 @@ import {
 } from '../projectGenerate';
 import { CompositeParametersGatherer, EmptyPreChecker, SfCommandlet, SfCommandletExecutor } from '../util';
 // below uses require due to bundling restrictions
-// eslint-disable-next-line @typescript-eslint/no-var-requires, @typescript-eslint/no-unused-vars
+// eslint-disable-next-line @typescript-eslint/no-var-requires
 const AdmZip = require('adm-zip');
 
 export type InstalledPackageInfo = {
@@ -56,7 +56,6 @@ export class IsvDebugBootstrapExecutor extends SfCommandletExecutor<{}> {
   public readonly relativeApexPackageXmlPath = path.join(this.relativeMetadataTempPath, PACKAGE_XML);
   public readonly relativeInstalledPackagesPath = path.join(projectPaths.relativeToolsFolder(), INSTALLED_PACKAGES);
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     throw new Error('not in use');
   }
@@ -349,12 +348,10 @@ export class IsvDebugBootstrapExecutor extends SfCommandletExecutor<{}> {
     return result;
   }
 
-  /* eslint-disable @typescript-eslint/no-unused-vars */
   protected attachExecution(
     execution: CommandExecution,
     cancellationTokenSource: vscode.CancellationTokenSource,
     cancellationToken: vscode.CancellationToken
-    /* eslint-enable @typescript-eslint/no-unused-vars */
   ) {
     channelService.streamCommandOutput(execution);
     channelService.showChannelOutput();

packages/salesforcedx-vscode-core/src/commands/listMetadata.ts

@@ -28,7 +28,6 @@ export class ListMetadataExecutor extends SfCommandletExecutor<string> {
     this.folder = folder;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     const builder = new SfCommandBuilder()
       .withArg('org:list:metadata')

packages/salesforcedx-vscode-core/src/commands/orgCreate.ts

@@ -74,7 +74,6 @@ export class OrgCreateExecutor extends SfCommandletExecutor<AliasAndFileSelectio
       stdOut += realData.toString();
     });
 
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     execution.processExitSubject.subscribe(async exitCode => {
       this.logMetric(execution.command.logName, startTime);
       try {

packages/salesforcedx-vscode-core/src/commands/orgList.ts

@@ -10,7 +10,6 @@ import { nls } from '../messages';
 import { PromptConfirmGatherer, SfCommandlet, SfCommandletExecutor, SfWorkspaceChecker } from './util';
 
 export class OrgListExecutor extends SfCommandletExecutor<{}> {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: { choice?: string }): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('org_list_clean_text'))

packages/salesforcedx-vscode-core/src/commands/orgOpen.ts

@@ -24,7 +24,6 @@ import { workspaceUtils } from '../util';
 import { EmptyParametersGatherer, SfCommandlet, SfCommandletExecutor, SfWorkspaceChecker } from './util';
 
 export class OrgOpenContainerExecutor extends SfCommandletExecutor<{}> {
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('org_open_default_scratch_org_text'))
@@ -94,7 +93,7 @@ export class OrgOpenContainerExecutor extends SfCommandletExecutor<{}> {
 
 export class OrgOpenExecutor extends SfCommandletExecutor<{}> {
   protected showChannelOutput = false;
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('org_open_default_scratch_org_text'))

packages/salesforcedx-vscode-core/src/commands/projectGenerateManifest.ts

@@ -30,15 +30,14 @@ export class GenerateManifestExecutor extends LibraryCommandletExecutor<string>
     this.sourcePaths = sourcePaths;
     this.responseText = responseText;
   }
-  /* eslint-disable @typescript-eslint/no-unused-vars */
+
   public async run(
     response: ContinueResponse<string>,
     progress?: vscode.Progress<{
       message?: string | undefined;
       increment?: number | undefined;
     }>,
     token?: vscode.CancellationToken
-    /* eslint-enable @typescript-eslint/no-unused-vars */
   ): Promise<boolean> {
     if (this.sourcePaths) {
       const packageXML = await ComponentSet.fromSource(this.sourcePaths).getPackageXml();

packages/salesforcedx-vscode-core/src/commands/projectRetrieveStart.ts

@@ -50,7 +50,6 @@ export class ProjectRetrieveStartExecutor extends SfCommandletExecutor<{}> {
     this.flag = flag;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   public build(data: {}): Command {
     const builder = new SfCommandBuilder()
       .withDescription(nls.localize(this.params.description.default))

packages/salesforcedx-vscode-core/src/commands/refreshSObjects.ts

@@ -83,7 +83,7 @@ export class RefreshSObjectsExecutor extends SfCommandletExecutor<{}> {
   public static readonly onRefreshSObjectsCommandCompletion =
     RefreshSObjectsExecutor.refreshSObjectsCommandCompletionEventEmitter.event;
   private static isActive = false;
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+
   public build(data: {}): Command {
     return new SfCommandBuilder()
       .withDescription(nls.localize('sobjects_refresh'))