Update tas client (#13636)

* Update experiments framework

* Reduce logging

package.json

@@ -2082,7 +2082,7 @@
         "vscode-languageclient": "8.0.2-next.5",
         "vscode-languageserver": "8.0.2-next.5",
         "vscode-languageserver-protocol": "3.17.2-next.6",
-        "vscode-tas-client": "^0.1.27",
+        "vscode-tas-client": "^0.1.63",
         "ws": "^6.2.2",
         "zeromq": "^6.0.0-beta.16",
         "zeromqold": "npm:zeromq@^6.0.0-beta.6"

src/extension.node.ts

@@ -334,7 +334,6 @@ async function activateLegacy(
     const experimentService = serviceContainer.get<IExperimentService>(IExperimentService);
     // This must be done first, this guarantees all experiment information has loaded & all telemetry will contain experiment info.
     await experimentService.activate();
-    experimentService.logExperiments();
 
     const applicationEnv = serviceManager.get<IApplicationEnvironment>(IApplicationEnvironment);
     const configuration = serviceManager.get<IConfigurationService>(IConfigurationService);

src/extension.web.ts

@@ -315,7 +315,6 @@ async function activateLegacy(
     const experimentService = serviceContainer.get<IExperimentService>(IExperimentService);
     // This must be done first, this guarantees all experiment information has loaded & all telemetry will contain experiment info.
     await experimentService.activate();
-    experimentService.logExperiments();
 
     const applicationEnv = serviceManager.get<IApplicationEnvironment>(IApplicationEnvironment);
     const configuration = serviceManager.get<IConfigurationService>(IConfigurationService);

src/kernels/jupyter/finder/remoteKernelFinder.ts

@@ -271,8 +271,6 @@ export class RemoteKernelFinder implements IRemoteKernelFinder, IDisposable {
 
     private async getFromCache(cancelToken?: CancellationToken): Promise<RemoteKernelConnectionMetadata[]> {
         try {
-            traceVerbose('UniversalRemoteKernelFinder: get from cache');
-
             let results: RemoteKernelConnectionMetadata[] = this.cache;
             const key = this.cacheKey;
 

src/kernels/raw/finder/interpreterKernelSpecFinderHelper.node.ts

@@ -95,7 +95,9 @@ export async function findKernelSpecsInInterpreter(
         }
     });
     results = results.filter((r) => !r.specFile || !originalSpecFiles.has(r.specFile));
-    traceVerbose(`Kernel Specs found in interpreter ${interpreter.id} are ${JSON.stringify(results)}`);
+    if (results.length) {
+        traceVerbose(`Kernel Specs found in interpreter ${interpreter.id} are ${JSON.stringify(results)}`);
+    }
     // There was also an old bug where the same item would be registered more than once. Eliminate these dupes
     // too.
     const uniqueKernelSpecs: IJupyterKernelSpec[] = [];
@@ -169,7 +171,7 @@ export class InterpreterSpecificKernelSpecsFinder implements IDisposable {
     private async listKernelSpecsImpl() {
         const cancelToken = this.cancelToken.token;
 
-        traceVerbose(`Listing Python & non-Python kernels for Interpreter ${getDisplayPath(this.interpreter.uri)}`);
+        traceVerbose(`Search for KernelSpecs in Interpreter ${getDisplayPath(this.interpreter.uri)}`);
         const [kernelSpecsBelongingToPythonEnvironment, tempDirForKernelSpecs] = await Promise.all([
             findKernelSpecsInInterpreter(this.interpreter, cancelToken, this.jupyterPaths, this.kernelSpecFinder),
             this.jupyterPaths.getKernelSpecTempRegistrationFolder()
@@ -215,7 +217,6 @@ export class InterpreterSpecificKernelSpecsFinder implements IDisposable {
                       id: getKernelId(k, this.interpreter)
                   });
 
-            traceVerbose(`Found kernel spec at end of discovery ${kernelSpec?.id}`);
             // Check if we have already seen this.
             if (kernelSpec && !distinctKernelMetadata.has(kernelSpec.id)) {
                 distinctKernelMetadata.set(kernelSpec.id, kernelSpec);
@@ -233,7 +234,6 @@ export class InterpreterSpecificKernelSpecsFinder implements IDisposable {
             interpreter: this.interpreter,
             id: getKernelId(spec, this.interpreter)
         });
-        traceVerbose(`Kernel for interpreter ${this.interpreter.id} is ${result.id}`);
         if (!distinctKernelMetadata.has(result.id)) {
             distinctKernelMetadata.set(result.id, result);
         }

src/kernels/raw/finder/localKernelSpecFinderBase.node.ts

@@ -155,7 +155,6 @@ export class LocalKernelSpecFinder implements IDisposable {
                 const files = await this.fs.searchLocal(`**/kernel.json`, kernelSearchPath.fsPath, true);
                 return files.map((item) => uriPath.joinPath(kernelSearchPath, item));
             } else {
-                traceVerbose(`Not Searching for kernels as path does not exist, ${getDisplayPath(kernelSearchPath)}`);
                 return [];
             }
         })();
@@ -320,8 +319,8 @@ export async function loadKernelSpec(
     let kernelJson: ReadWrite<IJupyterKernelSpec>;
     try {
         traceVerbose(
-            `Loading kernelspec from ${getDisplayPath(specPath)} for ${
-                interpreter?.uri ? getDisplayPath(interpreter.uri) : ''
+            `Loading kernelspec from ${getDisplayPath(specPath)} ${
+                interpreter?.uri ? `for ${getDisplayPath(interpreter.uri)}` : ''
             }`
         );
         kernelJson = JSON.parse(await fs.readFile(specPath));

src/kernels/raw/finder/localPythonAndRelatedNonPythonKernelSpecFinder.node.ts

@@ -86,7 +86,6 @@ export class LocalPythonAndRelatedNonPythonKernelSpecFinder extends LocalKernelS
         this.disposables.push(this._onDidChangeKernels);
         interpreterService.onDidChangeInterpreters(
             () => {
-                traceVerbose(`refreshData after detecting changes to interpreters`);
                 this.refreshCancellation?.cancel();
                 this.refreshData().catch(noop);
             },