Get telemetry client and LS middleware working in the browser (#17010)

Author: jakebailey

.vscode/settings.json

@@ -2,6 +2,7 @@
 {
     "files.exclude": {
         "out": true, // set this to true to hide the "out" folder with the compiled JS files
+        "dist": true,
         "**/*.pyc": true,
         ".nyc_output": true,
         "obj": true,
@@ -15,6 +16,7 @@
     },
     "search.exclude": {
         "out": true, // set this to false to include "out" folder in search results
+        "dist": true,
         "**/node_modules": true,
         "coverage": true,
         "languageServer*/**": true,

build/webpack/webpack.extension.config.js

@@ -53,7 +53,15 @@ const config = {
             },
         ],
     },
-    externals: ['vscode', 'commonjs', ...existingModulesInOutDir],
+    externals: [
+        'vscode',
+        'commonjs',
+        ...existingModulesInOutDir,
+        // These dependencies are ignored because we don't use them, and App Insights has try-catch protecting their loading if they don't exist
+        // See: https://github.com/microsoft/vscode-extension-telemetry/issues/41#issuecomment-598852991
+        'applicationinsights-native-metrics',
+        '@opentelemetry/tracing',
+    ],
     plugins: [...common.getDefaultPlugins('extension')],
     resolve: {
         alias: {

gulpfile.js

@@ -201,7 +201,11 @@ function getAllowedWarningsForWebPack(buildConfig) {
                 'WARNING in ./node_modules/applicationinsights/out/AutoCollection/NativePerformance.js',
             ];
         case 'browser':
-            return [];
+            return [
+                'WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).',
+                'WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.',
+                'WARNING in webpack performance recommendations:',
+            ];
         default:
             throw new Error('Unknown WebPack Configuration');
     }

news/3 Code Health/16871.md

@@ -0,0 +1 @@
+Update telemetry client to support browser, plumb to Pylance.

news/3 Code Health/16872.md

@@ -0,0 +1 @@
+Refactor language server middleware to work in the browser.

package-lock.json

@@ -2265,7 +2265,7 @@
         "untildify": "^3.0.2",
         "vscode-debugadapter": "^1.28.0",
         "vscode-debugprotocol": "^1.28.0",
-        "vscode-extension-telemetry": "0.1.4",
+        "vscode-extension-telemetry": "0.2.8",
         "vscode-jsonrpc": "6.0.0",
         "vscode-languageclient": "7.0.0",
         "vscode-languageserver": "7.0.0",