deps: Upgrade dependencies to address vurnabilities.

build/venv-test-ipywidgets8-requirements.txt

@@ -1,5 +1,5 @@
 # List of requirements for ipython tests
-numpy
+numpy>=1.22.2
 pandas
 # Install jupyter itself so we end up with a kernel
 jupyter
@@ -8,3 +8,17 @@ ipywidgets
 anywidget
 matplotlib
 ipympl
+
+# Security updates for transitive dependencies
+pillow>=10.2.0
+setuptools>=78.1.1
+zipp>=3.19.1
+tornado>=6.4.1
+anyio>=4.4.0
+requests>=2.32.4
+fonttools>=4.43.0
+jupyter-server>=2.14.1
+jupyter-core>=5.8.0
+ipython>=8.10.0
+urllib3>=2.2.2
+jupyterlab>=4.4.8

package.json

@@ -2196,7 +2196,7 @@
         "fast-deep-equal": "^2.0.1",
         "format-util": "^1.0.5",
         "fs-extra": "^4.0.3",
-        "glob": "^7.1.2",
+        "glob": "^9.3.5",
         "iconv-lite": "^0.6.3",
         "immer": "^10.1.3",
         "inversify": "^6.0.1",
@@ -2230,7 +2230,6 @@
         "slickgrid": "^2.4.17",
         "stack-trace": "0.0.10",
         "strip-comments": "^2.0.1",
-        "styled-components": "^5.2.1",
         "svg-to-pdfkit": "^0.1.8",
         "tailwind-merge": "^3.3.1",
         "tcp-port-used": "^1.0.1",
@@ -2309,6 +2308,7 @@
         "@vscode/zeromq": "^0.2.3",
         "acorn": "^8.9.0",
         "autoprefixer": "^10.4.21",
+        "bare-events": "^2.8.1",
         "buffer": "^6.0.3",
         "bufferutil": "^4.0.6",
         "chai": "^4.3.10",

src/telemetryGenerator.node.ts

@@ -771,15 +771,7 @@ function generateTelemetryGdpr(output: TelemetryEntry[]) {
 }
 
 async function generateTelemetryOutput() {
-    const files = await new Promise<string[]>((resolve, reject) => {
-        glob(path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'src/**/*.ts'), (ex, res) => {
-            if (ex) {
-                reject(ex);
-            } else {
-                resolve(res);
-            }
-        });
-    });
+    const files = await glob(path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'src/**/*.ts'));
     // Print out the source tree
     return generateDocumentation(files);
 }

src/test/extension.serviceRegistry.vscode.test.ts

@@ -114,16 +114,8 @@ async function getInjectableClasses(fileNames: string[], options: ts.CompilerOpt
 }
 
 async function getSourceFiles() {
-    const files = await new Promise<string[]>((resolve, reject) => {
-        const globPattern = path.join(__dirname, '..', '..', 'src', '**', '*.ts').replace(/\\/g, '/');
-        glob(globPattern, (ex, res) => {
-            if (ex) {
-                reject(ex);
-            } else {
-                resolve(res);
-            }
-        });
-    });
+    const globPattern = path.join(__dirname, '..', '..', 'src', '**', '*.ts').replace(/\\/g, '/');
+    const files = await glob(globPattern);
     return files;
 }
 

src/test/index.node.ts

@@ -187,21 +187,13 @@ export async function run(): Promise<void> {
         default:
             break;
     }
-    const testFiles = await new Promise<string[]>((resolve, reject) => {
-        // If we have mulitple patterns, then turn into regex from `a,b,c` to `(a|b|c)`
-        const pattern = options.testFilesSuffix.includes(',')
-            ? `(${options.testFilesSuffix.split(',').join('|')})`
-            : options.testFilesSuffix;
-        glob(
-            `**/*${pattern}.js`,
-            { ignore: ['**/**.unit.test.js'].concat(ignoreGlob), cwd: testsRoot },
-            (error, files) => {
-                if (error) {
-                    return reject(error);
-                }
-                resolve(files);
-            }
-        );
+    // If we have mulitple patterns, then turn into regex from `a,b,c` to `(a|b|c)`
+    const pattern = options.testFilesSuffix.includes(',')
+        ? `(${options.testFilesSuffix.split(',').join('|')})`
+        : options.testFilesSuffix;
+    const testFiles = await glob(`**/*${pattern}.js`, {
+        ignore: ['**/**.unit.test.js'].concat(ignoreGlob),
+        cwd: testsRoot
     });
 
     // Setup test files that need to be run.

src/test/testRunner.ts

@@ -71,26 +71,17 @@ export async function run(): Promise<void> {
         return promise;
     }
     // Run the tests.
+    const files = await glob(`**/**.${testFilesGlob}.js`, { ignore: ['**/**.unit.test.js'], cwd: testsRoot });
+    files.forEach((file) => mocha.addFile(path.join(testsRoot, file)));
+
     await new Promise<void>((resolve, reject) => {
-        glob(`**/**.${testFilesGlob}.js`, { ignore: ['**/**.unit.test.js'], cwd: testsRoot }, (error, files) => {
-            if (error) {
-                return reject(error);
-            }
-            try {
-                files.forEach((file) => mocha.addFile(path.join(testsRoot, file)));
-                initializationScript()
-                    .then(() =>
-                        mocha.run((failures) =>
-                            failures > 0 ? reject(new Error(`${failures} total failures`)) : resolve()
-                        )
-                    )
-                    .finally(() => {
-                        stopJupyterServer().catch(noop);
-                    })
-                    .catch(reject);
-            } catch (error) {
-                return reject(error);
-            }
-        });
+        initializationScript()
+            .then(() =>
+                mocha.run((failures) => (failures > 0 ? reject(new Error(`${failures} total failures`)) : resolve()))
+            )
+            .finally(() => {
+                stopJupyterServer().catch(noop);
+            })
+            .catch(reject);
     });
 }