@appium/base-driver 10.7.0 → 10.7.2

package/lib/basedriver/commands/find.ts

@@ -1,5 +1,6 @@
 /* eslint-disable @typescript-eslint/no-unused-vars */
 import type {Constraints, Element, IFindCommands} from '@appium/types';
+
 import {errors} from '../../protocol';
 import type {BaseDriver} from '../driver';
 import {mixin} from './mixin';
@@ -71,19 +72,11 @@ async function findElOrElsWithProcessing<C extends Constraints>(
 }
 
 const FindCommands: IFindCommands = {
-  async findElement<C extends Constraints>(
-    this: BaseDriver<C>,
-    strategy: string,
-    selector: string,
-  ) {
+  async findElement<C extends Constraints>(this: BaseDriver<C>, strategy: string, selector: string) {
     return await this.findElOrElsWithProcessing(strategy, selector, false);
   },
 
-  async findElements<C extends Constraints>(
-    this: BaseDriver<C>,
-    strategy: string,
-    selector: string,
-  ) {
+  async findElements<C extends Constraints>(this: BaseDriver<C>, strategy: string, selector: string) {
     return await this.findElOrElsWithProcessing(strategy, selector, true);
   },
 

package/lib/basedriver/commands/log.ts

@@ -1,4 +1,5 @@
 import type {Constraints, Driver, ILogCommands} from '@appium/types';
+
 import type {BaseDriver} from '../driver';
 import {mixin} from './mixin';
 
@@ -23,8 +24,7 @@ const LogCommands: ILogCommands = {
         Object.entries(this.supportedLogTypes).map(([key, value]) => [key, value.description]),
       );
       throw new Error(
-        `Unsupported log type '${String(logType)}'. ` +
-          `Supported types: ${JSON.stringify(logsTypesWithDescriptions)}`,
+        `Unsupported log type '${String(logType)}'. ` + `Supported types: ${JSON.stringify(logsTypesWithDescriptions)}`,
       );
     }
 

package/lib/basedriver/commands/mixin.ts

@@ -1,4 +1,5 @@
 import type {Constraints} from '@appium/types';
+
 import {BaseDriver} from '../driver';
 
 /**

package/lib/basedriver/commands/timeout.ts

@@ -1,8 +1,9 @@
-import {waitForCondition} from 'asyncbox';
 import {util} from '@appium/support';
+import type {Constraints, ITimeoutCommands} from '@appium/types';
+import {waitForCondition} from 'asyncbox';
+
 import {errors} from '../../protocol';
 import type {BaseDriver} from '../driver';
-import type {Constraints, ITimeoutCommands} from '@appium/types';
 import {mixin} from './mixin';
 
 declare module '../driver' {
@@ -40,9 +41,7 @@ const TimeoutCommands: ITimeoutCommands = {
 
     this.log.debug(`W3C timeout argument: ${JSON.stringify({script, pageLoad, implicit})}}`);
     if ([script, pageLoad, implicit].every((value) => value == null)) {
-      throw new errors.InvalidArgumentError(
-        'W3C protocol expects any of script, pageLoad or implicit to be set',
-      );
+      throw new errors.InvalidArgumentError('W3C protocol expects any of script, pageLoad or implicit to be set');
     }
     if (util.hasValue(script)) {
       await this.scriptTimeoutW3C(script);
@@ -110,10 +109,7 @@ const TimeoutCommands: ITimeoutCommands = {
     }
   },
 
-  async implicitWaitForCondition<C extends Constraints>(
-    this: BaseDriver<C>,
-    condFn: (...args: any[]) => Promise<any>,
-  ) {
+  async implicitWaitForCondition<C extends Constraints>(this: BaseDriver<C>, condFn: (...args: any[]) => Promise<any>) {
     this.log.debug(`Waiting up to ${this.implicitWaitMs} ms for condition`);
     const wrappedCondFn = async (...args: any[]) => {
       // reset command timeout

package/lib/basedriver/core.ts

@@ -1,3 +1,6 @@
+import os from 'node:os';
+
+import {util} from '@appium/support';
 import type {
   AppiumLogger,
   Constraints,
@@ -12,13 +15,12 @@ import type {
   StringRecord,
 } from '@appium/types';
 import AsyncLock from 'async-lock';
-import {util} from '@appium/support';
-import os from 'node:os';
+
 import {DEFAULT_BASE_PATH, PROTOCOLS} from '../constants';
 import {errors} from '../protocol';
 import {DeviceSettings} from './device-settings';
-import * as helpers from './helpers';
 import {ExtensionCore} from './extension-core';
+import * as helpers from './helpers';
 
 const NEW_COMMAND_TIMEOUT_MS = 60 * 1000;
 
@@ -113,7 +115,7 @@ export class DriverCore<const C extends Constraints, Settings extends StringReco
     this.shouldValidateCaps = shouldValidateCaps;
 
     // keeping track of initial opts
-    this.initialOpts = structuredClone(opts);
+    this.initialOpts = {...opts};
 
     this.sessionId = null;
     this.helpers = helpers;
@@ -275,17 +277,12 @@ export class DriverCore<const C extends Constraints, Settings extends StringReco
             `the feature name split by a colon. Got '${fullName}' instead`,
         );
       }
-      return [
-        fullName.substring(0, separatorPos).toLowerCase(),
-        fullName.substring(separatorPos + 1),
-      ];
+      return [fullName.substring(0, separatorPos).toLowerCase(), fullName.substring(separatorPos + 1)];
     };
     const parseFullNames = (fullNames: string[]) => fullNames.map(parseFullName);
     const matches = (pair: string[]) => {
       const [automationName, featureName] = pair;
-      return (
-        [currentAutomationName, ALL_DRIVERS_MATCH].includes(automationName) && featureName === name
-      );
+      return [currentAutomationName, ALL_DRIVERS_MATCH].includes(automationName) && featureName === name;
     };
 
     // if we have explicitly denied this feature, return false immediately
@@ -334,9 +331,7 @@ export class DriverCore<const C extends Constraints, Settings extends StringReco
     }
 
     if (!validStrategies.includes(strategy)) {
-      throw new errors.InvalidSelectorError(
-        `Locator Strategy '${strategy}' is not supported for this session`,
-      );
+      throw new errors.InvalidSelectorError(`Locator Strategy '${strategy}' is not supported for this session`);
     }
   }
 
@@ -390,7 +385,6 @@ export class DriverCore<const C extends Constraints, Settings extends StringReco
   }
 
   /**
-   *
    * @param {Driver} driver
    */
   addManagedDriver(driver: Driver) {

package/lib/basedriver/device-settings.ts

@@ -1,7 +1,8 @@
-import {util, node} from '@appium/support';
-import {log} from './logger';
+import {node, util} from '@appium/support';
+import type {IDeviceSettings, SettingsUpdateListener, StringRecord} from '@appium/types';
+
 import {errors} from '../protocol/errors';
-import type {StringRecord, IDeviceSettings, SettingsUpdateListener} from '@appium/types';
+import {log} from './logger';
 
 /**
  * Maximum size (in bytes) of a given driver's settings object (which is internal to {@linkcode DeviceSettings}).
@@ -21,10 +22,7 @@ export class DeviceSettings<T extends StringRecord = StringRecord> implements ID
    * @param defaultSettings - Initial settings (shallow-copied).
    * @param onSettingsUpdate - Called when a setting is changed; receives (prop, newValue, curValue).
    */
-  constructor(
-    defaultSettings: T = {} as T,
-    onSettingsUpdate: SettingsUpdateListener<T> = async () => {},
-  ) {
+  constructor(defaultSettings: T = {} as T, onSettingsUpdate: SettingsUpdateListener<T> = async () => {}) {
     this._settings = {...defaultSettings};
     this._onSettingsUpdate = onSettingsUpdate;
   }
@@ -37,8 +35,7 @@ export class DeviceSettings<T extends StringRecord = StringRecord> implements ID
   async update(newSettings: T): Promise<void> {
     if (!util.isPlainObject(newSettings)) {
       throw new errors.InvalidArgumentError(
-        `Settings update should be called with valid JSON. Got ` +
-          `${JSON.stringify(newSettings)} instead`,
+        `Settings update should be called with valid JSON. Got ` + `${JSON.stringify(newSettings)} instead`,
       );
     }
 

package/lib/basedriver/driver.ts

@@ -1,35 +1,35 @@
-import type AsyncLock from 'async-lock';
 import {util} from '@appium/support';
 import {
-  BASE_DESIRED_CAP_CONSTRAINTS,
   type AppiumServer,
+  BASE_DESIRED_CAP_CONSTRAINTS,
   type BaseDriverCapConstraints,
   type Capabilities,
   type Constraints,
   type DefaultCreateSessionResult,
+  type DefaultDeleteSessionResult,
   type Driver,
   type DriverCaps,
   type DriverData,
+  type InitialOpts,
   type ServerArgs,
+  type SessionCapabilities,
+  type SingularSessionData,
   type StringRecord,
   type W3CDriverCaps,
-  type InitialOpts,
-  type DefaultDeleteSessionResult,
-  type SingularSessionData,
-  type SessionCapabilities,
 } from '@appium/types';
+import type AsyncLock from 'async-lock';
+
 import {fixCaps, isW3cCaps} from '../helpers/capabilities';
+import {resolveExecuteExtensionName} from '../helpers/extension-command-name';
 import {getLevenshteinSuggestion} from '../helpers/levenshtein-match';
 import {calcSignature} from '../helpers/session';
 import {DELETE_SESSION_COMMAND, determineProtocol, errors} from '../protocol';
+import {mergePlainObjects} from '../utils';
 import {processCapabilities, validateCaps} from './capabilities';
 import {DriverCore} from './core';
 import * as helpers from './helpers';
-import {mergePlainObjects} from '../utils';
-import {resolveExecuteExtensionName} from '../helpers/extension-command-name';
 
-type CommandInvoker<C extends Constraints> = BaseDriver<C> &
-  Record<string, ((...args: any[]) => any) | undefined>;
+type CommandInvoker<C extends Constraints> = BaseDriver<C> & Record<string, ((...args: any[]) => any) | undefined>;
 
 const EVENT_SESSION_INIT = 'newSessionRequested';
 const EVENT_SESSION_START = 'newSessionStarted';
@@ -73,11 +73,7 @@ export class BaseDriver<
    */
   protected get _desiredCapConstraints(): Readonly<BaseDriverCapConstraints & C> {
     return Object.freeze(
-      mergePlainObjects(
-        {},
-        BASE_DESIRED_CAP_CONSTRAINTS,
-        this.desiredCapConstraints,
-      ) as BaseDriverCapConstraints & C,
+      mergePlainObjects({}, BASE_DESIRED_CAP_CONSTRAINTS, this.desiredCapConstraints) as BaseDriverCapConstraints & C,
     );
   }
 
@@ -149,11 +145,7 @@ export class BaseDriver<
         // automatic session deletion in this.onCommandTimeout. Of course we don't
         // want to trigger the timer when the user is shutting down the session
         // intentionally
-        if (
-          !wasSessionShutdownUnexpectedly &&
-          this.isCommandsQueueEnabled &&
-          cmd !== DELETE_SESSION_COMMAND
-        ) {
+        if (!wasSessionShutdownUnexpectedly && this.isCommandsQueueEnabled && cmd !== DELETE_SESSION_COMMAND) {
           // resetting existing timeout
           await this.startNewCommandTimeout();
         }
@@ -205,9 +197,7 @@ export class BaseDriver<
     return cmd;
   }
 
-  async startUnexpectedShutdown(
-    err: Error = new errors.NoSuchDriverError('The driver was unexpectedly shut down!'),
-  ) {
+  async startUnexpectedShutdown(err: Error = new errors.NoSuchDriverError('The driver was unexpectedly shut down!')) {
     this.eventEmitter.emit(ON_UNEXPECTED_SHUTDOWN_EVENT, err); // allow others to listen for this
     this.shutdownUnexpectedly = true;
     try {
@@ -227,10 +217,7 @@ export class BaseDriver<
     if (!this.newCommandTimeoutMs) return; // eslint-disable-line curly
 
     this.noCommandTimer = setTimeout(async () => {
-      this.log.warn(
-        `Shutting down because we waited ` +
-          `${this.newCommandTimeoutMs / 1000.0} seconds for a command`,
-      );
+      this.log.warn(`Shutting down because we waited ` + `${this.newCommandTimeoutMs / 1000.0} seconds for a command`);
       const errorMessage =
         `New Command Timeout of ` +
         `${this.newCommandTimeoutMs / 1000.0} seconds ` +
@@ -277,7 +264,6 @@ export class BaseDriver<
   }
 
   /**
-   *
    * Historically the first two arguments were reserved for JSONWP capabilities.
    * Appium 2 has dropped the support of these, so now we only accept capability
    * objects in W3C format and thus allow any of the three arguments to represent
@@ -291,16 +277,12 @@ export class BaseDriver<
     driverData?: DriverData[],
   ): Promise<CreateResult> {
     if (this.sessionId !== null) {
-      throw new errors.SessionNotCreatedError(
-        'Cannot create a new session while one is in progress',
-      );
+      throw new errors.SessionNotCreatedError('Cannot create a new session while one is in progress');
     }
 
     this.log.debug();
 
-    const originalCaps = structuredClone(
-      [w3cCapabilities, w3cCapabilities1, w3cCapabilities2].find(isW3cCaps),
-    );
+    const originalCaps = structuredClone([w3cCapabilities, w3cCapabilities1, w3cCapabilities2].find(isW3cCaps));
     if (!originalCaps) {
       throw new errors.SessionNotCreatedError(
         'Appium only supports W3C-style capability objects. ' +
@@ -311,17 +293,11 @@ export class BaseDriver<
     this.setProtocolW3C();
 
     this.originalCaps = originalCaps;
-    this.log.debug(
-      `Creating session with W3C capabilities: ${JSON.stringify(originalCaps, null, 2)}`,
-    );
+    this.log.debug(`Creating session with W3C capabilities: ${JSON.stringify(originalCaps, null, 2)}`);
 
     let caps: DriverCaps<C>;
     try {
-      caps = processCapabilities(
-        originalCaps,
-        this._desiredCapConstraints,
-        this.shouldValidateCaps,
-      ) as DriverCaps<C>;
+      caps = processCapabilities(originalCaps, this._desiredCapConstraints, this.shouldValidateCaps) as DriverCaps<C>;
       caps = fixCaps(caps, this._desiredCapConstraints, this.log) as DriverCaps<C>;
     } catch (e) {
       const message = e instanceof Error ? e.message : String(e);
@@ -334,7 +310,7 @@ export class BaseDriver<
     this.sessionCreationTimestampMs = Date.now();
     this.caps = caps;
     // merge caps onto opts so we don't need to worry about what's where
-    this.opts = {...structuredClone(this.initialOpts), ...this.caps};
+    this.opts = {...this.initialOpts, ...this.caps};
 
     // deal with resets
     // some people like to do weird things by setting noReset and fullReset
@@ -382,9 +358,10 @@ export class BaseDriver<
    * Use {@linkcode EventCommands.getLogEvents} instead to get the event history.
    */
   async getSession() {
-    return (
-      this.caps.eventTimings ? {...this.caps, events: this.eventHistory} : this.caps
-    ) as SingularSessionData<C, SessionData>;
+    return (this.caps.eventTimings ? {...this.caps, events: this.eventHistory} : this.caps) as SingularSessionData<
+      C,
+      SessionData
+    >;
   }
 
   /**
@@ -432,8 +409,7 @@ export class BaseDriver<
       const capError = e instanceof Error ? e : new Error(String(e));
       throw this.log.errorWithException(
         new errors.SessionNotCreatedError(
-          `Session capabilities were not valid for the ` +
-            `following reason(s): ${capError.message}`,
+          `Session capabilities were not valid for the ` + `following reason(s): ${capError.message}`,
           capError,
         ),
       );

package/lib/basedriver/extension-core.ts

@@ -1,5 +1,6 @@
-import {logger, util} from '@appium/support';
 import {EventEmitter} from 'node:events';
+
+import {logger, util} from '@appium/support';
 import type {
   AppiumLogger,
   BidiModuleMap,
@@ -9,6 +10,7 @@ import type {
   IpcData,
   StringRecord,
 } from '@appium/types';
+
 import {MAX_LOG_BODY_LENGTH} from '../constants';
 import {errors} from '../protocol';
 import {BIDI_COMMANDS} from '../protocol/bidi-commands';
@@ -43,9 +45,7 @@ export class ExtensionCore {
   updateBidiCommands(cmds: BidiModuleMap): void {
     const overlappingKeys = Object.keys(cmds).filter((key) => key in this.bidiCommands);
     if (overlappingKeys.length) {
-      this.log.warn(
-        `Overwriting existing bidi modules: ${JSON.stringify(overlappingKeys)}. This may not be intended!`,
-      );
+      this.log.warn(`Overwriting existing bidi modules: ${JSON.stringify(overlappingKeys)}. This may not be intended!`);
     }
     this.bidiCommands = {
       ...this.bidiCommands,
@@ -107,9 +107,7 @@ export class ExtensionCore {
     if (params?.required?.length) {
       for (const requiredParam of params.required) {
         if (bidiParams[requiredParam] === undefined) {
-          throw new errors.InvalidArgumentError(
-            `The ${requiredParam} parameter was required but you omitted it`,
-          );
+          throw new errors.InvalidArgumentError(`The ${requiredParam} parameter was required but you omitted it`);
         }
         args.push(bidiParams[requiredParam]);
       }
@@ -127,15 +125,12 @@ export class ExtensionCore {
         `method '${command}'`,
     );
     // call the handler with the signature appropriate to extension type (plugin or driver)
-    const commandHandler = (
-      this as unknown as Record<string, (...handlerArgs: any[]) => Promise<unknown>>
-    )[command];
+    const commandHandler = (this as unknown as Record<string, (...handlerArgs: any[]) => Promise<unknown>>)[command];
     const response =
       next && driver
         ? await commandHandler.call(this, next, driver, ...args)
         : await commandHandler.call(this, ...args);
-    const finalResponse: BiDiResultData =
-      response === undefined ? {} : (response as BiDiResultData);
+    const finalResponse: BiDiResultData = response === undefined ? {} : (response as BiDiResultData);
     this.log.debug(
       `Responding to bidi command '${bidiCmd}' with ` +
         `${util.truncateString(JSON.stringify(finalResponse), {length: MAX_LOG_BODY_LENGTH})}`,

package/lib/basedriver/helpers.ts

@@ -1,21 +1,17 @@
 import nodeFs from 'node:fs';
 import path from 'node:path';
-import {log as logger} from './logger';
-import {tempDir, fs, util, timing, node} from '@appium/support';
-import {LRUCache} from 'lru-cache';
+import type {Readable} from 'node:stream';
+
+import {fs, node, tempDir, timing, util} from '@appium/support';
+import type {CachedAppInfo, ConfigureAppOptions, DriverHelpers, HTTPHeaders, PostProcessOptions} from '@appium/types';
 import AsyncLock from 'async-lock';
 import axios from 'axios';
-import type {
-  ConfigureAppOptions,
-  CachedAppInfo,
-  PostProcessOptions,
-  HTTPHeaders,
-  DriverHelpers,
-} from '@appium/types';
 import type {AxiosResponseHeaders, RawAxiosRequestHeaders} from 'axios';
-import type {Readable} from 'node:stream';
+import {LRUCache} from 'lru-cache';
 import type {PackageJson} from 'type-fest';
 
+import {log as logger} from './logger';
+
 // for compat with running tests transpiled and in-place
 export const BASEDRIVER_VER = readBaseDriverVersion();
 
@@ -31,10 +27,7 @@ const APPLICATIONS_CACHE = new LRUCache<string, CachedAppInfoEntry>({
   ttl: CACHED_APPS_MAX_AGE_MS, // expire after 24 hours
   updateAgeOnGet: true,
   dispose: ({fullPath}, app) => {
-    logger.info(
-      `The application '${app}' cached at '${fullPath}' has ` +
-        `expired after ${CACHED_APPS_MAX_AGE_MS}ms`,
-    );
+    logger.info(`The application '${app}' cached at '${fullPath}' has ` + `expired after ${CACHED_APPS_MAX_AGE_MS}ms`);
     if (fullPath) {
       void fs.rimraf(fullPath);
     }
@@ -52,9 +45,7 @@ process.on('exit', () => {
   }
 
   const appPaths = [...APPLICATIONS_CACHE.values()].map(({fullPath}) => fullPath);
-  logger.debug(
-    `Performing cleanup of ${util.pluralize('cached application', appPaths.length, true)}`,
-  );
+  logger.debug(`Performing cleanup of ${util.pluralize('cached application', appPaths.length, true)}`);
   for (const appPath of appPaths) {
     if (!appPath) {
       continue;
@@ -285,8 +276,7 @@ export async function configureApp(
     }
 
     verifyAppExtension(newApp, supportedAppExtensions);
-    return appCacheKey !== toCacheKey(newApp) &&
-      (packageHash || Object.values(remoteAppProps).some(Boolean))
+    return appCacheKey !== toCacheKey(newApp) && (packageHash || Object.values(remoteAppProps).some(Boolean))
       ? await storeAppInCache(newApp)
       : newApp;
   });
@@ -386,9 +376,7 @@ export function generateDriverLogPrefix(obj: object | null, _sessionId?: string
 
 // #region Private helpers
 
-function parseAppLink(
-  appLink: string,
-): URL | {protocol?: string; pathname?: string; href?: string; search?: string} {
+function parseAppLink(appLink: string): URL | {protocol?: string; pathname?: string; href?: string; search?: string} {
   try {
     return new URL(appLink);
   } catch {
@@ -438,10 +426,7 @@ function toCacheKey(app: string): string {
   return app;
 }
 
-async function queryAppLink(
-  appLink: string,
-  reqHeaders: RawAxiosRequestHeaders,
-): Promise<RemoteAppData> {
+async function queryAppLink(appLink: string, reqHeaders: RawAxiosRequestHeaders): Promise<RemoteAppData> {
   const url = new URL(appLink);
   // Extract credentials, then remove them from the URL for axios
   const {username, password} = url;
@@ -454,8 +439,7 @@ async function queryAppLink(
     auth: axiosAuth,
     responseType: 'stream' as const,
     timeout: APP_DOWNLOAD_TIMEOUT_MS,
-    validateStatus: (status: number) =>
-      (status >= 200 && status < 300) || status === HTTP_STATUS_NOT_MODIFIED,
+    validateStatus: (status: number) => (status >= 200 && status < 300) || status === HTTP_STATUS_NOT_MODIFIED,
     headers: reqHeaders,
   };
   try {
@@ -510,10 +494,7 @@ function determineFilename(
     replacement: SANITIZE_REPLACEMENT,
   });
   const extname = path.extname(basename);
-  if (
-    headers['content-disposition'] &&
-    /^attachment/i.test(String(headers['content-disposition']))
-  ) {
+  if (headers['content-disposition'] && /^attachment/i.test(String(headers['content-disposition']))) {
     logger.debug(`Content-Disposition: ${headers['content-disposition']}`);
     const match = /filename="([^"]+)/i.exec(String(headers['content-disposition']));
     if (match) {
@@ -522,13 +503,9 @@ function determineFilename(
   }
 
   // assign the default file name and the extension if none has been detected
-  const resultingName = basename
-    ? basename.substring(0, basename.length - extname.length)
-    : DEFAULT_BASENAME;
+  const resultingName = basename ? basename.substring(0, basename.length - extname.length) : DEFAULT_BASENAME;
   let resultingExt = extname;
-  if (
-    !supportedAppExtensions.map((ext) => ext.toLowerCase()).includes(resultingExt.toLowerCase())
-  ) {
+  if (!supportedAppExtensions.map((ext) => ext.toLowerCase()).includes(resultingExt.toLowerCase())) {
     logger.info(
       `The current file extension '${resultingExt}' is not supported. ` +
         `Defaulting to '${supportedAppExtensions[0]}'`,
@@ -539,9 +516,7 @@ function determineFilename(
 }
 
 function verifyAppExtension(app: string, supportedAppExtensions: string[]): string {
-  if (
-    supportedAppExtensions.map((ext) => ext.toLowerCase()).includes(path.extname(app).toLowerCase())
-  ) {
+  if (supportedAppExtensions.map((ext) => ext.toLowerCase()).includes(path.extname(app).toLowerCase())) {
     return app;
   }
   throw new Error(
@@ -592,9 +567,7 @@ function readBaseDriverVersion(): string {
   if (!pkgRoot) {
     throw new Error('Cannot find the @appium/base-driver package root');
   }
-  const pkg = JSON.parse(
-    nodeFs.readFileSync(path.join(pkgRoot, 'package.json'), 'utf8'),
-  ) as PackageJson;
+  const pkg = JSON.parse(nodeFs.readFileSync(path.join(pkgRoot, 'package.json'), 'utf8')) as PackageJson;
   if (typeof pkg.version !== 'string') {
     throw new Error('Invalid `package.json` for @appium/base-driver');
   }

package/lib/basedriver/ipc.ts

@@ -1,16 +1,18 @@
-import {log} from './logger';
+import EventEmitter from 'node:events';
+
+import {node} from '@appium/support';
 import type {
-  StringRecord,
-  IIpcSubscription,
-  IAppiumIpc,
-  IpcMessage,
-  IpcEvent,
   AppiumLogger,
+  IAppiumIpc,
+  IIpcSubscription,
   IpcData,
+  IpcEvent,
+  IpcMessage,
+  StringRecord,
 } from '@appium/types';
-import EventEmitter from 'node:events';
 import {sleep} from 'asyncbox';
-import {node} from '@appium/support';
+
+import {log} from './logger';
 
 const DEF_MAX_OBJ_SIZE_BYTES = 1024 * 1024; // 1mb seems like plenty for any plugin to pass a message
 const DEF_MAX_TOPICS = 1000;
@@ -26,10 +28,7 @@ export type AppiumIpcOpts = {
 
 const ASYNC_ITERATOR_STOP = Symbol('asyncIteratorStop');
 
-export class IpcSubscription<T extends IpcData>
-  extends EventEmitter<IpcEvent<T>>
-  implements IIpcSubscription<T>
-{
+export class IpcSubscription<T extends IpcData> extends EventEmitter<IpcEvent<T>> implements IIpcSubscription<T> {
   constructor(
     public readonly subscriber: string,
     public readonly topic: string,
@@ -110,9 +109,7 @@ export class AppiumIpc implements IAppiumIpc {
   subscribe<T extends IpcData>(topic: string, subscriber: string): IpcSubscription<T> {
     this.log.info(`Subscribing ${subscriber} to topic '${topic}'`);
     if (this.subscriptionExists(topic, subscriber)) {
-      throw new Error(
-        `Subscription already exists for topic "${topic}" and subscriber "${subscriber}"`,
-      );
+      throw new Error(`Subscription already exists for topic "${topic}" and subscriber "${subscriber}"`);
     }
 
     this.ensureTopic(topic);
@@ -157,9 +154,7 @@ export class AppiumIpc implements IAppiumIpc {
 
     this.messageByTopic[topic] = message;
 
-    const subs = this.subs[topic]
-      ? this.subs[topic].filter((sub) => sub.subscriber !== publisher)
-      : [];
+    const subs = this.subs[topic] ? this.subs[topic].filter((sub) => sub.subscriber !== publisher) : [];
 
     for (const sub of subs) {
       sub.emit(EVT_MESSAGE, structuredClone(message));

package/lib/basedriver/validation.ts

@@ -1,12 +1,10 @@
-import type {Constraint} from '@appium/types';
 import {util} from '@appium/support';
+import type {Constraint} from '@appium/types';
+
 import {log} from './logger';
 
 export class Validator {
-  private readonly _validators: Record<
-    keyof Constraint,
-    (value: any, options?: any, key?: string) => string | null
-  > = {
+  private readonly _validators: Record<keyof Constraint, (value: any, options?: any, key?: string) => string | null> = {
     isString: (value: any, options?: any): string | null => {
       if (value === undefined || options == null) {
         return null;
@@ -108,8 +106,7 @@ export class Validator {
       }
       if (
         !options?.allowEmpty &&
-        ((value !== undefined && util.isEmpty(value)) ||
-          (typeof value === 'string' && !value.trim()))
+        ((value !== undefined && util.isEmpty(value)) || (typeof value === 'string' && !value.trim()))
       ) {
         return 'must not be empty or blank';
       }
@@ -117,10 +114,7 @@ export class Validator {
     },
   };
 
-  validate(
-    values: Record<string, any>,
-    constraints: Record<string, Constraint>,
-  ): Record<string, string[]> | null {
+  validate(values: Record<string, any>, constraints: Record<string, Constraint>): Record<string, string[]> | null {
     const result: Record<string, string[]> = {};
     for (const [key, constraint] of Object.entries(constraints)) {
       const value = values[key];
@@ -129,11 +123,7 @@ export class Validator {
           continue;
         }
 
-        const validationError = this._validators[validatorName as keyof Constraint](
-          value,
-          options,
-          key,
-        );
+        const validationError = this._validators[validatorName as keyof Constraint](value, options, key);
         if (validationError == null) {
           continue;
         }