appium 3.5.1 → 3.6.0

package/lib/bootstrap/config-file.ts

@@ -1,12 +1,13 @@
+import {util} from '@appium/support';
+import type {AppiumConfig, NormalizedAppiumConfig} from '@appium/types';
 import type {IOutputError} from '@sidvind/better-ajv-errors';
 import type {ErrorObject, SchemaObject} from 'ajv';
-import {lilconfig, type LoaderSync, type LilconfigResult} from 'lilconfig';
-import {util} from '@appium/support';
+import {lilconfig, type LilconfigResult, type LoaderSync} from 'lilconfig';
 import * as yaml from 'yaml';
-import {camelCase, getPath, mapKeys, mapValues} from '../utils';
-import type {AppiumConfig, NormalizedAppiumConfig} from '@appium/types';
-import {getSchema, validate} from '../schema/schema';
+
 import {formatErrors} from '../schema/format-errors';
+import {getSchema, validate} from '../schema/schema';
+import {camelCase, getPath, mapKeys, mapValues} from '../utils';
 
 /**
  * A cache of the raw config file (a JSON string) at a filepath.
@@ -99,35 +100,22 @@ export async function readConfigFile(
 export function normalizeConfig(config: AppiumConfig): NormalizedAppiumConfig {
   const schema = getSchema();
 
-  const isSchemaTypeObject = (
-    schemaObj: SchemaObject | Record<string, unknown> | undefined,
-  ): boolean =>
+  const isSchemaTypeObject = (schemaObj: SchemaObject | Record<string, unknown> | undefined): boolean =>
     Boolean(
-      (schemaObj as SchemaObject | undefined)?.properties ||
-      (schemaObj as SchemaObject | undefined)?.type === 'object',
+      (schemaObj as SchemaObject | undefined)?.properties || (schemaObj as SchemaObject | undefined)?.type === 'object',
     );
 
   const normalize = (rootConfig: AppiumConfig, section?: string): Record<string, unknown> => {
     const obj =
-      section === undefined
-        ? rootConfig
-        : (getPath(rootConfig, section, rootConfig) as Record<string, unknown>);
+      section === undefined ? rootConfig : (getPath(rootConfig, section, rootConfig) as Record<string, unknown>);
 
     const mappedObj = mapKeys(obj as Record<string, unknown>, (_v, prop) =>
-      String(
-        getPath(
-          schema,
-          `properties.server.properties.${prop}.appiumCliDest`,
-          camelCase(String(prop)),
-        ),
-      ),
+      String(getPath(schema, `properties.server.properties.${prop}.appiumCliDest`, camelCase(String(prop)))),
     );
 
     return mapValues(mappedObj, (value, property) => {
       const nextSection = section ? `${section}.${property}` : property;
-      return isSchemaTypeObject((schema as any).properties?.[property])
-        ? normalize(rootConfig, nextSection)
-        : value;
+      return isSchemaTypeObject((schema as any).properties?.[property]) ? normalize(rootConfig, nextSection) : value;
     });
   };
 
@@ -141,10 +129,9 @@ function yamlLoader(filepath: string, content: string): unknown {
   try {
     return yaml.parse(content);
   } catch (e) {
-    throw new Error(
-      `The YAML config at '${filepath}' cannot be loaded. Original error: ${(e as Error).message}`,
-      {cause: e},
-    );
+    throw new Error(`The YAML config at '${filepath}' cannot be loaded. Original error: ${(e as Error).message}`, {
+      cause: e,
+    });
   }
 }
 
@@ -158,27 +145,22 @@ function jsonLoader(filepath: string, content: string): unknown {
     return JSON.parse(content);
   } catch (e) {
     rawConfig.delete(filepath);
-    throw new Error(
-      `The JSON config at '${filepath}' cannot be loaded. Original error: ${(e as Error).message}`,
-      {cause: e},
-    );
+    throw new Error(`The JSON config at '${filepath}' cannot be loaded. Original error: ${(e as Error).message}`, {
+      cause: e,
+    });
   }
 }
 
 /**
  * Loads a config file from an explicit path
  */
-async function loadConfigFile(
-  lc: LilconfigAsyncSearcher,
-  filepath: string,
-): Promise<LilconfigResult> {
+async function loadConfigFile(lc: LilconfigAsyncSearcher, filepath: string): Promise<LilconfigResult> {
   try {
     // removing "await" will cause any rejection to _not_ be caught in this block!
     return await lc.load(filepath);
   } catch (err) {
     if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
-      (err as NodeJS.ErrnoException).message =
-        `Config file not found at user-provided path: ${filepath}`;
+      (err as NodeJS.ErrnoException).message = `Config file not found at user-provided path: ${filepath}`;
       throw err;
     } else if (err instanceof SyntaxError) {
       // generally invalid JSON

package/lib/bootstrap/grid-v3-register.ts

@@ -1,6 +1,7 @@
-import axios from 'axios';
 import {fs} from '@appium/support';
 import type {StringRecord} from '@appium/types';
+import axios from 'axios';
+
 import {log as logger} from '../logger';
 
 /**
@@ -39,12 +40,7 @@ interface Grid3ProxyApiResponse {
   msg?: string;
 }
 
-export default async function registerNode(
-  data: string,
-  addr: string,
-  port: number,
-  basePath: string,
-): Promise<void>;
+export default async function registerNode(data: string, addr: string, port: number, basePath: string): Promise<void>;
 export default async function registerNode(
   data: Grid3NodeConfig,
   addr?: string,
@@ -80,9 +76,7 @@ export default async function registerNode(
       );
     }
     if (typeof port !== 'number' || !Number.isFinite(port)) {
-      throw logger.errorWithException(
-        'When registering from a node config file path, port must be a finite number.',
-      );
+      throw logger.errorWithException('When registering from a node config file path, port must be a finite number.');
     }
   }
 
@@ -103,8 +97,7 @@ export default async function registerNode(
       configHolder = JSON.parse(fileContent) as Grid3NodeConfig;
     } catch (err) {
       throw logger.errorWithException(
-        `Syntax error in Selenium Grid 3 node configuration file ${configFilePath}: ` +
-          (err as Error).message,
+        `Syntax error in Selenium Grid 3 node configuration file ${configFilePath}: ` + (err as Error).message,
       );
     }
   } else {
@@ -134,29 +127,15 @@ async function registerToGrid(
     if (status !== 200) {
       throw new Error(`Request failed with code ${status}`);
     }
-    logger.debug(
-      `Appium successfully registered with the Selenium Grid 3 hub at ` + hubUri(hubCfg),
-    );
+    logger.debug(`Appium successfully registered with the Selenium Grid 3 hub at ` + hubUri(hubCfg));
   } catch (err) {
-    logger.error(
-      `An attempt to register with the Selenium Grid 3 hub was unsuccessful: ` +
-        (err as Error).message,
-    );
+    logger.error(`An attempt to register with the Selenium Grid 3 hub was unsuccessful: ` + (err as Error).message);
   }
 }
 
-function postRequest(
-  configHolder: Grid3NodeConfig,
-  addr?: string,
-  port?: number,
-  basePath?: string,
-): void {
+function postRequest(configHolder: Grid3NodeConfig, addr?: string, port?: number, basePath?: string): void {
   // Move Selenium Grid 3 (flat) configuration properties into `configuration`
-  if (
-    configHolder != null &&
-    typeof configHolder === 'object' &&
-    !Object.hasOwn(configHolder, 'configuration')
-  ) {
+  if (configHolder != null && typeof configHolder === 'object' && !Object.hasOwn(configHolder, 'configuration')) {
     const configuration: StringRecord = {};
     const holder = configHolder as StringRecord;
     for (const property of Object.keys(holder)) {
@@ -201,11 +180,7 @@ function postRequest(
   }
 
   const registerCycleInterval = cfg.registerCycle;
-  if (
-    registerCycleInterval === undefined ||
-    isNaN(registerCycleInterval) ||
-    registerCycleInterval <= 0
-  ) {
+  if (registerCycleInterval === undefined || isNaN(registerCycleInterval) || registerCycleInterval <= 0) {
     logger.warn(
       `'registerCycle' is not a valid positive number. ` +
         `No registration request will be sent to the Selenium Grid 3 hub.`,
@@ -215,8 +190,7 @@ function postRequest(
   // initiate a new Thread
   let first = true;
   logger.debug(
-    `Starting auto-register thread for Selenium Grid 3. ` +
-      `Will try to register every ${registerCycleInterval} ms.`,
+    `Starting auto-register thread for Selenium Grid 3. ` + `Will try to register every ${registerCycleInterval} ms.`,
   );
   setInterval(async function registerRetry() {
     if (first) {
@@ -231,7 +205,7 @@ function postRequest(
 
 /** Query the Selenium Grid 3 hub to see if this node id is already registered. */
 async function isAlreadyRegistered(configHolder: Grid3NodeConfig): Promise<boolean | undefined> {
-  //check if node is already registered
+  // check if node is already registered
   const hubCfg = configHolder.configuration;
   if (!hubCfg?.id) {
     return;

package/lib/bootstrap/init-types.ts

@@ -1,4 +1,3 @@
-import type {AppiumDriver} from '../appium';
 import type {
   Args,
   CliCommand,
@@ -7,6 +6,8 @@ import type {
   CliExtensionSubcommand,
   ParsedArgs,
 } from 'appium/types';
+
+import type {AppiumDriver} from '../appium';
 import type {ExtensionConfigs} from '../extension';
 
 /** Empty object returned when `init` completes for non-server CLI flows (extension/setup). */
@@ -25,7 +26,4 @@ export type InitResult<Cmd extends CliCommand = CliCommandServer> = Cmd extends
   : ExtCommandInitResult;
 
 /** CLI + programmatic args before `init` narrows by `subcommand`. */
-export type PreConfigArgs = Args<
-  CliCommand,
-  CliExtensionSubcommand | CliCommandSetupSubcommand | void
->;
+export type PreConfigArgs = Args<CliCommand, CliExtensionSubcommand | CliCommandSetupSubcommand | void>;

package/lib/bootstrap/main-helpers.ts

@@ -1,24 +1,26 @@
+import type {NetworkInterfaceInfo} from 'node:os';
 import {inspect as dump, type InspectOptions} from 'node:util';
-import {log as logger} from '../logger';
+
 import {
-  routeConfiguringFunction as makeRouter,
   normalizeBasePath,
+  routeConfiguringFunction as makeRouter,
   server as baseServer,
   type ServerOpts,
 } from '@appium/base-driver';
 import {console as supportConsole, util} from '@appium/support';
 import type {AppiumServer, Driver, MethodMap, UpdateServerCallback} from '@appium/types';
+import type {Args, CliCommandServer, ParsedArgs} from 'appium/types';
 import {WebSocketServer} from 'ws';
-import type {NetworkInterfaceInfo} from 'node:os';
+
 import type {AppiumDriver} from '../appium';
+import {BIDI_BASE_PATH, LONG_STACKTRACE_LIMIT} from '../constants';
+import type {DriverNameMap, PluginNameMap} from '../extension';
 import {APPIUM_VER, getBuildInfo, getGitRev, updateBuildInfo} from '../helpers/build';
+import {fetchInterfaces, isBroadcastIp, V4_BROADCAST_IP} from '../helpers/network';
+import {log as logger} from '../logger';
+import {validate as validateSchema} from '../schema/schema';
 import {checkNodeOk, requireDir} from './node-helpers';
 import {getNonDefaultServerArgs} from './startup-config';
-import {validate as validateSchema} from '../schema/schema';
-import {fetchInterfaces, V4_BROADCAST_IP, isBroadcastIp} from '../helpers/network';
-import {LONG_STACKTRACE_LIMIT, BIDI_BASE_PATH} from '../constants';
-import type {Args, ParsedArgs, CliCommandServer} from 'appium/types';
-import type {DriverNameMap, PluginNameMap} from '../extension';
 
 const isStdoutTTY = process.stdout.isTTY;
 
@@ -88,10 +90,7 @@ export function logServerAddress(url: string): void {
  * @param args - Parsed server CLI args
  * @param throwInsteadOfExit - When true, rethrows failures instead of calling `process.exit(1)`
  */
-export async function preflightChecks(
-  args: ParsedArgs<CliCommandServer>,
-  throwInsteadOfExit = false,
-): Promise<void> {
+export async function preflightChecks(args: ParsedArgs<CliCommandServer>, throwInsteadOfExit = false): Promise<void> {
   try {
     checkNodeOk();
     if (args.longStacktrace) {
@@ -141,10 +140,7 @@ export async function logStartupInfo(args: ParsedArgs<CliCommandServer>): Promis
 /**
  * Collects `updateServer` hooks from active driver and plugin classes for HTTP server customization.
  */
-export function getServerUpdaters(
-  driverClasses: DriverNameMap,
-  pluginClasses: PluginNameMap,
-): UpdateServerCallback[] {
+export function getServerUpdaters(driverClasses: DriverNameMap, pluginClasses: PluginNameMap): UpdateServerCallback[] {
   return [...driverClasses.keys(), ...pluginClasses.keys()]
     .map((klass) => klass.updateServer)
     .filter(Boolean) as UpdateServerCallback[];
@@ -153,10 +149,7 @@ export function getServerUpdaters(
 /**
  * Merges `newMethodMap` contributions from all active drivers and plugins into one method map.
  */
-export function getExtraMethodMap(
-  driverClasses: DriverNameMap,
-  pluginClasses: PluginNameMap,
-): MethodMap<Driver> {
+export function getExtraMethodMap(driverClasses: DriverNameMap, pluginClasses: PluginNameMap): MethodMap<Driver> {
   return [...driverClasses.keys(), ...pluginClasses.keys()].reduce<MethodMap<Driver>>(
     (map, klass) => ({
       ...map,
@@ -218,12 +211,9 @@ function logNonDefaultArgsWarning(args: Args): void {
   inspect(args);
 }
 
-function logDefaultCapabilitiesWarning(
-  caps: ParsedArgs<CliCommandServer>['defaultCapabilities'],
-): void {
+function logDefaultCapabilitiesWarning(caps: ParsedArgs<CliCommandServer>['defaultCapabilities']): void {
   logger.info(
-    'Default capabilities, which will be added to each request ' +
-      'unless overridden by desired capabilities:',
+    'Default capabilities, which will be added to each request ' + 'unless overridden by desired capabilities:',
   );
   inspect(caps);
 }

package/lib/bootstrap/node-helpers.ts

@@ -1,10 +1,12 @@
-/* eslint-disable no-console */
-import path from 'node:path';
 import os from 'node:os';
+import path from 'node:path';
+
+/* eslint-disable no-console */
+import {fs, npm, system} from '@appium/support';
 import * as semver from 'semver';
-import {system, fs, npm} from '@appium/support';
+
+import {getBuildInfo, updateBuildInfo} from '../helpers/build';
 import {appiumPackageRoot, npmPackage} from '../utils';
-import {updateBuildInfo, getBuildInfo} from '../helpers/build';
 
 const MIN_NODE_VERSION = (npmPackage.engines as Record<string, string>).node;
 
@@ -24,9 +26,7 @@ interface DebugInfoInput {
 export function checkNodeOk(): void {
   const version = getNodeVersion();
   if (!semver.satisfies(version, MIN_NODE_VERSION)) {
-    throw new Error(
-      `Node version must be at least ${MIN_NODE_VERSION}; current is ${version.version}`,
-    );
+    throw new Error(`Node version must be at least ${MIN_NODE_VERSION}; current is ${version.version}`);
   }
 }
 
@@ -67,9 +67,7 @@ export function adjustNodePath(): void {
   if (refreshRequirePaths()) {
     process.env.APPIUM_OMIT_PEER_DEPS = '1';
   } else {
-    process.env.NODE_PATH = nodePathParts
-      .filter((p) => p !== appiumModuleSearchRoot)
-      .join(path.delimiter);
+    process.env.NODE_PATH = nodePathParts.filter((p) => p !== appiumModuleSearchRoot).join(path.delimiter);
   }
 }
 
@@ -77,17 +75,12 @@ export function adjustNodePath(): void {
  * Prints JSON debug info (OS, Node/npm, Appium build, installed drivers/plugins) to stdout.
  * Input combines extension config snapshots with the resolved Appium home path.
  */
-export async function showDebugInfo({
-  driverConfig,
-  pluginConfig,
-  appiumHome,
-}: DebugInfoInput): Promise<void> {
+export async function showDebugInfo({driverConfig, pluginConfig, appiumHome}: DebugInfoInput): Promise<void> {
   const getNpmVersion = async (): Promise<string> => {
     const {stdout} = await npm.exec('--version', [], {cwd: process.cwd()});
     return stdout.trim();
   };
-  const findNpmLocation = async (): Promise<string> =>
-    await fs.which(system.isWindows() ? 'npm.cmd' : 'npm');
+  const findNpmLocation = async (): Promise<string> => await fs.which(system.isWindows() ? 'npm.cmd' : 'npm');
 
   const [npmVersion, npmLocation] = await Promise.all([
     ...[getNpmVersion, findNpmLocation].map((f) => getSafeResult(f, 'unknown')),
@@ -131,11 +124,7 @@ export async function showDebugInfo({
  *
  * @throws {Error}
  */
-export async function requireDir(
-  root: string,
-  requireWriteable = true,
-  displayName = 'folder path',
-): Promise<void> {
+export async function requireDir(root: string, requireWriteable = true, displayName = 'folder path'): Promise<void> {
   let stat;
   try {
     stat = await fs.stat(root);

package/lib/bootstrap/startup-config.ts

@@ -1,9 +1,10 @@
 /* eslint-disable no-console */
 import {util} from '@appium/support';
-import {getDefaultsForSchema, getAllArgSpecs} from '../schema/schema';
 import type {Args} from 'appium/types';
-import type {ReadConfigFileResult} from './config-file';
+
+import {getAllArgSpecs, getDefaultsForSchema} from '../schema/schema';
 import {difference, getPath, pickBy, setPath} from '../utils';
+import type {ReadConfigFileResult} from './config-file';
 
 interface FlattenedArg {
   value: unknown;
@@ -22,16 +23,13 @@ export function getNonDefaultServerArgs(parsedArgs: Args): Args {
    */
   const flatten = (args: Args): Record<string, FlattenedArg> => {
     const argSpecs = getAllArgSpecs();
-    const flattened = [...argSpecs.values()].reduce<Record<string, FlattenedArg>>(
-      (acc, argSpec: {dest: string}) => {
-        const value = getPath(args, argSpec.dest);
-        if (value !== undefined) {
-          acc[argSpec.dest] = {value, argSpec};
-        }
-        return acc;
-      },
-      {},
-    );
+    const flattened = [...argSpecs.values()].reduce<Record<string, FlattenedArg>>((acc, argSpec: {dest: string}) => {
+      const value = getPath(args, argSpec.dest);
+      if (value !== undefined) {
+        acc[argSpec.dest] = {value, argSpec};
+      }
+      return acc;
+    }, {});
 
     return flattened;
   };
@@ -39,8 +37,7 @@ export function getNonDefaultServerArgs(parsedArgs: Args): Args {
   const args = flatten(parsedArgs);
 
   // hopefully these function names are descriptive enough
-  const typesDiffer = (dest: string): boolean =>
-    typeof args[dest].value !== typeof defaultsFromSchema[dest];
+  const typesDiffer = (dest: string): boolean => typeof args[dest].value !== typeof defaultsFromSchema[dest];
 
   const defaultValueIsArray = (dest: string): boolean => Array.isArray(defaultsFromSchema[dest]);
 
@@ -53,11 +50,9 @@ export function getNonDefaultServerArgs(parsedArgs: Args): Args {
 
   const defaultIsDefined = (dest: string): boolean => defaultsFromSchema[dest] !== undefined;
 
-  const argValueNotArrayOrArraysDiffer = (dest: string) =>
-    !argsValueIsArray(dest) || arraysDiffer(dest);
+  const argValueNotArrayOrArraysDiffer = (dest: string) => !argsValueIsArray(dest) || arraysDiffer(dest);
 
-  const defaultValueNotArrayAndValuesDiffer = (dest: string) =>
-    !defaultValueIsArray(dest) && valuesDiffer(dest);
+  const defaultValueNotArrayAndValuesDiffer = (dest: string) => !defaultValueIsArray(dest) && valuesDiffer(dest);
 
   /**
    * This used to be a hideous conditional, but it's broken up into a hideous function instead.

package/lib/cli/args.ts

@@ -1,26 +1,24 @@
 import {util} from '@appium/support';
-import type {ArgumentOptions} from 'argparse';
 import type {ExtensionType} from '@appium/types';
 import type {CliExtensionSubcommand} from 'appium/types';
+import type {ArgumentOptions} from 'argparse';
+
 import {
   DRIVER_TYPE,
-  PLUGIN_TYPE,
+  EXT_SUBCOMMAND_DOCTOR,
   EXT_SUBCOMMAND_INSTALL,
   EXT_SUBCOMMAND_LIST,
   EXT_SUBCOMMAND_RUN,
   EXT_SUBCOMMAND_UNINSTALL,
   EXT_SUBCOMMAND_UPDATE,
-  EXT_SUBCOMMAND_DOCTOR,
+  PLUGIN_TYPE,
 } from '../constants';
 import {INSTALL_TYPES} from '../extension/extension-config';
 import {toParserArgs} from '../schema/cli-args';
 const DRIVER_EXAMPLE = 'xcuitest';
 const PLUGIN_EXAMPLE = 'images';
 
-export type ArgumentDefinitions = Map<
-  [name: string] | [name: string, alias: string],
-  ArgumentOptions
->;
+export type ArgumentDefinitions = Map<[name: string] | [name: string, alias: string], ArgumentOptions>;
 
 /**
  * This is necessary because we pass the array into `argparse`. `argparse` is bad and mutates things. We don't want that.
@@ -53,10 +51,7 @@ export const getExtensionArgs = util.memoize(function getExtensionArgs(): Record
   ExtensionType,
   Record<CliExtensionSubcommand, ArgumentDefinitions>
 > {
-  const extensionArgs = {} as Record<
-    ExtensionType,
-    Record<CliExtensionSubcommand, ArgumentDefinitions>
-  >;
+  const extensionArgs = {} as Record<ExtensionType, Record<CliExtensionSubcommand, ArgumentDefinitions>>;
   for (const type of EXTENSION_TYPES) {
     extensionArgs[type] = {
       [EXT_SUBCOMMAND_LIST]: makeListArgs(type),
@@ -130,8 +125,7 @@ function makeInstallArgs(type: ExtensionType): ArgumentDefinitions {
       {
         type: 'str',
         help:
-          `Name of the ${type} to install, for example: ` +
-          (type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE),
+          `Name of the ${type} to install, for example: ` + (type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE),
       },
     ],
     [
@@ -152,9 +146,7 @@ function makeInstallArgs(type: ExtensionType): ArgumentDefinitions {
         required: false,
         default: null,
         type: 'str',
-        help:
-          `The Node.js package name of the ${type}. ` +
-          `Required if "source" is set to "git" or "github".`,
+        help: `The Node.js package name of the ${type}. ` + `Required if "source" is set to "git" or "github".`,
         dest: 'packageName',
       },
     ],
@@ -172,8 +164,7 @@ function makeUninstallArgs(type: ExtensionType): ArgumentDefinitions {
       {
         type: 'str',
         help:
-          `Name of the ${type} to uninstall, for example: ` +
-          (type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE),
+          `Name of the ${type} to uninstall, for example: ` + (type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE),
       },
     ],
   ]);

package/lib/cli/driver-command.ts

@@ -1,6 +1,8 @@
 import {console, util} from '@appium/support';
 import type {ExtMetadata, ExtRecord, InstallType} from 'appium/types';
-import ExtensionCliCommand from './extension-command';
+
+import {KNOWN_DRIVERS} from '../constants';
+import {ExtensionCliCommand} from './extension-command';
 import type {
   ExtensionArgs,
   ExtensionCommandOptions,
@@ -8,7 +10,6 @@ import type {
   PostInstallText,
   RunOutput,
 } from './extension-command';
-import {KNOWN_DRIVERS} from '../constants';
 const REQ_DRIVER_FIELDS = ['driverName', 'automationName', 'platformNames', 'mainClass'];
 type DriverInstallOpts = {driver: string; installType: InstallType; packageName?: string};
 type DriverUninstallOpts = {driver: string};
@@ -27,11 +28,7 @@ export default class DriverCliCommand extends ExtensionCliCommand<'driver'> {
    *
    * @param opts - install options
    */
-  async install({
-    driver,
-    installType,
-    packageName,
-  }: DriverInstallOpts): Promise<ExtRecord<'driver'>> {
+  async install({driver, installType, packageName}: DriverInstallOpts): Promise<ExtRecord<'driver'>> {
     return await super._install({
       installSpec: driver,
       installType,
@@ -108,13 +105,9 @@ export default class DriverCliCommand extends ExtensionCliCommand<'driver'> {
    * @param driverMetadata - `appium` metadata from extension package
    * @param installSpec - install spec from CLI
    */
-  override validateExtensionFields(
-    driverMetadata: ExtMetadata<'driver'>,
-    installSpec: string,
-  ): void {
+  override validateExtensionFields(driverMetadata: ExtMetadata<'driver'>, installSpec: string): void {
     const missingFields = REQ_DRIVER_FIELDS.reduce<string[]>(
-      (acc, field) =>
-        driverMetadata[field as keyof typeof driverMetadata] ? acc : [...acc, field],
+      (acc, field) => (driverMetadata[field as keyof typeof driverMetadata] ? acc : [...acc, field]),
       [],
     );