update dependencies

Author: jochen-testingbot

package-lock.json

@@ -60,18 +60,18 @@
     "tracer": "^1.3.0"
   },
   "devDependencies": {
-    "@eslint/js": "^9.39.1",
+    "@eslint/js": "^10.0.1",
     "@tsconfig/node20": "^20.1.8",
     "@types/archiver": "^7.0.0",
-    "@types/jest": "^29.5.14",
+    "@types/jest": "^30.0.0",
     "@types/js-yaml": "^4.0.9",
-    "@types/node": "^20.19.0",
+    "@types/node": "^25.4.0",
     "@types/progress-stream": "^2.0.5",
-    "babel-jest": "^29.7.0",
-    "eslint": "^9.39.1",
+    "babel-jest": "^30.3.0",
+    "eslint": "^10.0.3",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.4",
-    "jest": "^29.7.0",
+    "jest": "^30.3.0",
     "prettier": "^3.7.4",
     "ts-jest": "^29.4.6",
     "typescript": "^5.9.3",

package.json

@@ -120,7 +120,8 @@ export default class MaestroOptions {
     this._report = options?.report;
     this._reportOutputDir = options?.reportOutputDir;
     // IPA files can only be tested on real iOS devices, so automatically enable realDevice
-    this._realDevice = MaestroOptions.isIpaFile(app) || (options?.realDevice ?? false);
+    this._realDevice =
+      MaestroOptions.isIpaFile(app) || (options?.realDevice ?? false);
     this._downloadArtifacts = options?.downloadArtifacts;
     this._artifactsOutputDir = options?.artifactsOutputDir;
     this._ignoreChecksumCheck = options?.ignoreChecksumCheck ?? false;

src/models/maestro_options.ts

@@ -378,7 +378,6 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
     const flowsPaths = this.options.flows;
 
     let zipPath: string;
-    let shouldCleanup = false;
 
     // Special case: single zip file - upload directly
     if (flowsPaths.length === 1) {
@@ -482,12 +481,9 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
         realDevice: this.options.realDevice,
         device: this.options.device,
         version: this.options.version,
-        flowCount: allFlowFiles.filter(
-          (f) =>
-            f.endsWith('.yaml') || f.endsWith('.yml'),
-        ).filter(
-          (f) => !this.isConfigFile(f),
-        ).length,
+        flowCount: allFlowFiles
+          .filter((f) => f.endsWith('.yaml') || f.endsWith('.yml'))
+          .filter((f) => !this.isConfigFile(f)).length,
         shardSplit: this.options.shardSplit,
       });
     }
@@ -496,14 +492,12 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
     const missingReferences = await this.findMissingReferences(
       allFlowFiles,
       allFlowFiles,
-      baseDir,
     );
     if (!this.options.quiet && missingReferences.length > 0) {
       this.logMissingReferences(missingReferences, baseDir);
     }
 
     zipPath = await this.createFlowsZip(allFlowFiles, baseDir);
-    shouldCleanup = true;
 
     try {
       await this.upload.upload({
@@ -514,9 +508,7 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
         showProgress: !this.options.quiet,
       });
     } finally {
-      if (shouldCleanup) {
-        await fs.promises.unlink(zipPath).catch(() => {});
-      }
+      await fs.promises.unlink(zipPath).catch(() => {});
     }
 
     return true;
@@ -535,7 +527,10 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
     for (const configName of ['config.yaml', 'config.yml']) {
       const candidatePath = path.join(directory, configName);
       try {
-        const configContent = await fs.promises.readFile(candidatePath, 'utf-8');
+        const configContent = await fs.promises.readFile(
+          candidatePath,
+          'utf-8',
+        );
         config = yaml.load(configContent) as MaestroConfig;
         configPath = candidatePath;
         break; // Use the first config file found
@@ -891,10 +886,11 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
   public async findMissingReferences(
     flowFiles: string[],
     allIncludedFiles: string[],
-    baseDir?: string,
   ): Promise<MissingFileReference[]> {
     const missingReferences: MissingFileReference[] = [];
-    const includedFilesSet = new Set(allIncludedFiles.map((f) => path.resolve(f)));
+    const includedFilesSet = new Set(
+      allIncludedFiles.map((f) => path.resolve(f)),
+    );
 
     for (const flowFile of flowFiles) {
       const ext = path.extname(flowFile).toLowerCase();
@@ -1294,7 +1290,9 @@ export default class Maestro extends BaseProvider<MaestroOptions> {
         utils.checkForUpdate(latestVersion);
 
         if (this.options.debug) {
-          logger.debug(`API response: ${JSON.stringify(response.data, null, 2)}`);
+          logger.debug(
+            `API response: ${JSON.stringify(response.data, null, 2)}`,
+          );
         }
 
         return response.data;

src/providers/maestro.ts

@@ -37,7 +37,12 @@ export default {
   isWildcardDevice(device: string | undefined): boolean {
     if (!device) return true;
     // Check for common wildcard/regex characters
-    return device === '*' || device.includes('*') || device.includes('?') || device.includes('.*');
+    return (
+      device === '*' ||
+      device.includes('*') ||
+      device.includes('?') ||
+      device.includes('.*')
+    );
   },
 
   /**
@@ -46,7 +51,12 @@ export default {
   isWildcardVersion(version: string | undefined): boolean {
     if (!version) return true;
     // Check for common wildcard/regex characters
-    return version === '*' || version.includes('*') || version.includes('?') || version.includes('.*');
+    return (
+      version === '*' ||
+      version.includes('*') ||
+      version.includes('?') ||
+      version.includes('.*')
+    );
   },
 
   /**
@@ -91,7 +101,9 @@ export default {
       ),
     );
     logger.info(pc.cyan(''));
-    logger.info(pc.cyan('   Consider these alternatives for faster execution:'));
+    logger.info(
+      pc.cyan('   Consider these alternatives for faster execution:'),
+    );
     logger.info(
       pc.cyan(
         `   • Use ${pc.white('--shard-split <n>')} to run multiple flows in the same session`,

src/utils.ts

@@ -112,7 +112,7 @@ export async function checkInternetConnectivity(): Promise<ConnectivityCheckResu
       endpointResults: [successResult],
       message: `Internet connectivity verified via ${successResult.endpoint} (${successResult.latencyMs}ms)`,
     };
-  } catch (aggregateError) {
+  } catch {
     // All endpoints failed - collect all results
     const endpointResults = await Promise.all(endpointPromises);
     const testedEndpoints = endpointResults.map((r) => r.endpoint).join(', ');

src/utils/connectivity.ts

@@ -67,7 +67,9 @@ describe('Espresso', () => {
     // Default mock for fs.promises.open - returns valid zip magic bytes
     fs.promises.open = jest
       .fn()
-      .mockResolvedValue(createMockFileHandle() as unknown as fs.promises.FileHandle);
+      .mockResolvedValue(
+        createMockFileHandle() as unknown as fs.promises.FileHandle,
+      );
   });
 
   describe('Validation', () => {

tests/providers/espresso.test.ts

@@ -100,7 +100,9 @@ describe('Maestro', () => {
     // Default mock for fs.promises.open - returns valid zip magic bytes
     fs.promises.open = jest
       .fn()
-      .mockResolvedValue(createMockFileHandle() as unknown as fs.promises.FileHandle);
+      .mockResolvedValue(
+        createMockFileHandle() as unknown as fs.promises.FileHandle,
+      );
   });
 
   describe('Validation', () => {
@@ -2945,7 +2947,8 @@ flows:
 `;
       const projectDir = path.resolve(path.sep, 'project');
       // First call (config.yaml) fails, second call (config.yml) succeeds
-      fs.promises.readFile = jest.fn()
+      fs.promises.readFile = jest
+        .fn()
         .mockRejectedValueOnce(new Error('ENOENT: no such file'))
         .mockResolvedValueOnce(configContent);
       fs.promises.readdir = jest.fn().mockResolvedValue([]);
@@ -2987,11 +2990,12 @@ flows:
     it('should exclude config.yml from flow files when no config exists', async () => {
       const projectDir = path.resolve(path.sep, 'project');
       // Both config.yaml and config.yml fail
-      fs.promises.readFile = jest.fn()
+      fs.promises.readFile = jest
+        .fn()
         .mockRejectedValue(new Error('ENOENT: no such file'));
       fs.promises.readdir = jest.fn().mockResolvedValue([
         { name: 'flow1.yaml', isFile: () => true },
-        { name: 'config.yml', isFile: () => true },  // Should be excluded as a config file
+        { name: 'config.yml', isFile: () => true }, // Should be excluded as a config file
       ]);
       fs.promises.access = jest.fn().mockResolvedValue(undefined);
 
@@ -4123,7 +4127,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath], // Only flow file in included files
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(1);
@@ -4151,7 +4154,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(1);
@@ -4172,7 +4174,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(1);
@@ -4195,7 +4196,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(2);
@@ -4223,7 +4223,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath, scriptPath], // Script is included
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(0);
@@ -4244,7 +4243,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath], // Script not in included files
-        projectDir,
       );
 
       // File is not in included files, so it should be reported
@@ -4266,7 +4264,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath, scriptPath], // Script is in included files
-        projectDir,
       );
 
       // File is in included files, so no missing reference reported
@@ -4290,7 +4287,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(4);
@@ -4318,7 +4314,6 @@ flows:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(1);
@@ -4341,7 +4336,6 @@ onFlowStart:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(1);
@@ -4360,7 +4354,6 @@ onFlowStart:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath],
         [flowPath],
-        projectDir,
       );
 
       // Should not throw, just return empty
@@ -4376,7 +4369,6 @@ onFlowStart:
       const missingRefs = await maestro['findMissingReferences'](
         [jsPath],
         [jsPath],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(0);
@@ -4403,7 +4395,6 @@ onFlowStart:
       const missingRefs = await maestro['findMissingReferences'](
         [flowPath1, flowPath2],
         [flowPath1, flowPath2],
-        projectDir,
       );
 
       expect(missingRefs).toHaveLength(2);

tests/providers/maestro.test.ts

@@ -67,7 +67,9 @@ describe('XCUITest', () => {
     // Default mock for fs.promises.open - returns valid zip magic bytes
     fs.promises.open = jest
       .fn()
-      .mockResolvedValue(createMockFileHandle() as unknown as fs.promises.FileHandle);
+      .mockResolvedValue(
+        createMockFileHandle() as unknown as fs.promises.FileHandle,
+      );
   });
 
   describe('Validation', () => {

tests/providers/xcuitest.test.ts

@@ -74,7 +74,9 @@ describe('Upload', () => {
     jest
       .spyOn(fs.promises, 'open')
       .mockResolvedValue(
-        createMockFileHandle(ZIP_MAGIC_BYTES) as unknown as fs.promises.FileHandle,
+        createMockFileHandle(
+          ZIP_MAGIC_BYTES,
+        ) as unknown as fs.promises.FileHandle,
       );
   });
 
@@ -190,7 +192,9 @@ describe('Upload', () => {
       const invalidFileHandle = createMockFileHandle([0x00, 0x00, 0x00, 0x00]);
       jest
         .spyOn(fs.promises, 'open')
-        .mockResolvedValue(invalidFileHandle as unknown as fs.promises.FileHandle);
+        .mockResolvedValue(
+          invalidFileHandle as unknown as fs.promises.FileHandle,
+        );
 
       const options = createUploadOptions({
         filePath: '/path/to/invalid.apk',