appium 3.5.0 → 3.5.2

package/lib/extension/manifest.ts

@@ -1,12 +1,17 @@
-import {env, fs} from '@appium/support';
-import _ from 'lodash';
+import {env, fs, util} from '@appium/support';
 import path from 'node:path';
 import * as YAML from 'yaml';
 import type {DriverType, ExtensionType, PluginType} from '@appium/types';
-import type {ExtManifest, ExtPackageJson, ExtRecord, InternalMetadata, ManifestData} from 'appium/types';
+import type {
+  ExtManifest,
+  ExtPackageJson,
+  ExtRecord,
+  InternalMetadata,
+  ManifestData,
+} from 'appium/types';
 import {CURRENT_SCHEMA_REV, DRIVER_TYPE, PLUGIN_TYPE} from '../constants';
 import {INSTALL_TYPE_DEV, INSTALL_TYPE_NPM} from './extension-config';
-import {packageDidChange} from './package-changed';
+import {packageDidChange} from '../utils';
 import {migrate} from './manifest-migrations';
 
 const CONFIG_DATA_DRIVER_KEY = `${DRIVER_TYPE}s` as const;
@@ -29,7 +34,7 @@ export class Manifest {
    *
    * @param appiumHome - `APPIUM_HOME` path used as the cache key
    */
-  static getInstance = _.memoize((appiumHome: string): Manifest => new Manifest(appiumHome));
+  static getInstance = util.memoize((appiumHome: string): Manifest => new Manifest(appiumHome));
 
   #data!: ManifestData;
   readonly #appiumHome: string;
@@ -39,7 +44,7 @@ export class Manifest {
 
   private constructor(appiumHome: string) {
     this.#appiumHome = appiumHome;
-    this.#data = _.cloneDeep(INITIAL_MANIFEST_DATA) as ManifestData;
+    this.#data = structuredClone(INITIAL_MANIFEST_DATA) as ManifestData;
   }
 
   /** `APPIUM_HOME` directory this manifest is tied to. */
@@ -110,7 +115,7 @@ export class Manifest {
         data = YAML.parse(yaml) as ManifestData;
       } catch (err: any) {
         if (err.code === 'ENOENT') {
-          data = _.cloneDeep(INITIAL_MANIFEST_DATA) as ManifestData;
+          data = structuredClone(INITIAL_MANIFEST_DATA) as ManifestData;
           shouldWrite = true;
         } else {
           throw new Error(
@@ -118,7 +123,7 @@ export class Manifest {
               `cache file (${manifestPathResolved}). It may be invalid YAML. Specific error: ${
                 err.message
               }`,
-            {cause: err}
+            {cause: err},
           );
         }
       }
@@ -165,9 +170,9 @@ export class Manifest {
       } catch (err: any) {
         throw new Error(
           `Appium could not create the directory for the manifest file: ${path.dirname(
-            manifestPathResolved
+            manifestPathResolved,
           )}. Original error: ${err.message}`,
-          {cause: err}
+          {cause: err},
         );
       }
       try {
@@ -178,7 +183,7 @@ export class Manifest {
           `Appium could not write to manifest at ${manifestPathResolved} using APPIUM_HOME ${
             this.#appiumHome
           }. Please ensure it is writable. Original error: ${err.message}`,
-          {cause: err}
+          {cause: err},
         );
       }
     })();
@@ -211,9 +216,7 @@ export class Manifest {
       }
     };
 
-    const queue: Promise<void>[] = [
-      onMatch(path.join(this.#appiumHome, 'package.json'), true),
-    ];
+    const queue: Promise<void>[] = [onMatch(path.join(this.#appiumHome, 'package.json'), true)];
 
     const filepaths = await fs.glob('node_modules/{*,@*/*}/package.json', {
       cwd: this.#appiumHome,
@@ -240,7 +243,7 @@ export class Manifest {
   addExtensionFromPackage(
     pkgJson: ExtPackageJson<ExtensionType>,
     pkgPath: string,
-    installType: typeof INSTALL_TYPE_NPM | typeof INSTALL_TYPE_DEV = INSTALL_TYPE_NPM
+    installType: typeof INSTALL_TYPE_NPM | typeof INSTALL_TYPE_DEV = INSTALL_TYPE_NPM,
   ): boolean {
     const extensionPath = path.dirname(pkgPath);
 
@@ -254,31 +257,31 @@ export class Manifest {
     };
 
     if (isDriver(pkgJson)) {
-      const driverName = pkgJson.appium.driverName;
+      const {driverName, ...appiumDriverMeta} = pkgJson.appium;
       const value = {
-        ..._.omit(pkgJson.appium, 'driverName'),
+        ...appiumDriverMeta,
         ...internal,
       };
-      if (!_.isEqual(value, this.#data.drivers[driverName])) {
+      if (!util.isEqual(value, this.#data.drivers[driverName])) {
         this.setExtension(DRIVER_TYPE, driverName, value);
         return true;
       }
       return false;
     }
     if (isPlugin(pkgJson)) {
-      const pluginName = pkgJson.appium.pluginName;
+      const {pluginName, ...appiumPluginMeta} = pkgJson.appium;
       const value = {
-        ..._.omit(pkgJson.appium, 'pluginName'),
+        ...appiumPluginMeta,
         ...internal,
       };
-      if (!_.isEqual(value, this.#data.plugins[pluginName])) {
+      if (!util.isEqual(value, this.#data.plugins[pluginName])) {
         this.setExtension(PLUGIN_TYPE, pluginName, value);
         return true;
       }
       return false;
     }
     throw new TypeError(
-      `The extension in ${extensionPath} is neither a valid ${DRIVER_TYPE} nor a valid ${PLUGIN_TYPE}.`
+      `The extension in ${extensionPath} is neither a valid ${DRIVER_TYPE} nor a valid ${PLUGIN_TYPE}.`,
     );
   }
 
@@ -293,9 +296,9 @@ export class Manifest {
   setExtension<ExtType extends ExtensionType>(
     extType: ExtType,
     extName: string,
-    extData: ExtManifest<ExtType>
+    extData: ExtManifest<ExtType>,
   ): ExtManifest<ExtType> {
-    const data = _.cloneDeep(extData) as ExtManifest<ExtType>;
+    const data = structuredClone(extData) as ExtManifest<ExtType>;
     if (extType === DRIVER_TYPE) {
       this.#data.drivers[extName] = data as unknown as ExtManifest<DriverType>;
     } else {
@@ -336,7 +339,7 @@ export class Manifest {
         throw new Error(
           `Mismatch between location of APPIUM_HOME and manifest file. APPIUM_HOME: ${
             this.appiumHome
-          }, manifest file: ${resolved}`
+          }, manifest file: ${resolved}`,
         );
       }
     }
@@ -347,10 +350,10 @@ export class Manifest {
 
 function isExtension(value: unknown): value is ExtPackageJson<ExtensionType> {
   return (
-    _.isPlainObject(value) &&
-    _.isPlainObject((value as {appium?: unknown}).appium) &&
-    _.isString((value as {name?: unknown}).name) &&
-    _.isString((value as {version?: unknown}).version)
+    util.isPlainObject(value) &&
+    util.isPlainObject((value as {appium?: unknown}).appium) &&
+    typeof (value as {name?: unknown}).name === 'string' &&
+    typeof (value as {version?: unknown}).version === 'string'
   );
 }
 
@@ -358,7 +361,7 @@ function isDriver(value: unknown): value is ExtPackageJson<DriverType> {
   return (
     isExtension(value) &&
     'driverName' in value.appium &&
-    _.isString((value.appium as {driverName?: unknown}).driverName)
+    typeof (value.appium as {driverName?: unknown}).driverName === 'string'
   );
 }
 
@@ -366,6 +369,6 @@ function isPlugin(value: unknown): value is ExtPackageJson<PluginType> {
   return (
     isExtension(value) &&
     'pluginName' in value.appium &&
-    _.isString((value.appium as {pluginName?: unknown}).pluginName)
+    typeof (value.appium as {pluginName?: unknown}).pluginName === 'string'
   );
 }

package/lib/extension/plugin-config.ts

@@ -1,4 +1,4 @@
-import _ from 'lodash';
+import {util} from '@appium/support';
 import type {PluginType} from '@appium/types';
 import type {ExtManifest, ExtName, ExtRecord} from 'appium/types';
 import {PLUGIN_TYPE} from '../constants';
@@ -17,7 +17,7 @@ export class PluginConfig extends ExtensionConfig<PluginType> {
     const instance = new PluginConfig(manifest);
     if (PluginConfig.getInstance(manifest)) {
       throw new Error(
-        `Manifest with APPIUM_HOME ${manifest.appiumHome} already has a PluginConfig; use PluginConfig.getInstance() to retrieve it.`
+        `Manifest with APPIUM_HOME ${manifest.appiumHome} already has a PluginConfig; use PluginConfig.getInstance() to retrieve it.`,
       );
     }
     PluginConfig._instances.set(manifest, instance);
@@ -34,7 +34,7 @@ export class PluginConfig extends ExtensionConfig<PluginType> {
 
   public override extensionDesc(
     pluginName: ExtName<PluginType>,
-    {version}: ExtManifest<PluginType>
+    {version}: ExtManifest<PluginType>,
   ): string {
     return `${String(pluginName)}@${version}`;
   }
@@ -42,21 +42,26 @@ export class PluginConfig extends ExtensionConfig<PluginType> {
   override print(activeNames: ExtName<PluginType>[] = []): void {
     const pluginNames = Object.keys(this.installedExtensions);
 
-    if (_.isEmpty(pluginNames)) {
+    if (util.isEmpty(pluginNames)) {
       log.info(
-        `No plugins have been installed. Use the "appium plugin" ` + 'command to install the one(s) you want to use.'
+        `No plugins have been installed. Use the "appium plugin" ` +
+          'command to install the one(s) you want to use.',
       );
       return;
     }
 
     log.info(`Available plugins:`);
-    for (const [pluginName, pluginData] of _.toPairs(this.installedExtensions)) {
-      const activeTxt = _.includes(activeNames, pluginName as ExtName<PluginType>) ? ' (ACTIVE)' : '';
-      log.info(`  - ${this.extensionDesc(pluginName as ExtName<PluginType>, pluginData)}${activeTxt}`);
+    for (const [pluginName, pluginData] of Object.entries(this.installedExtensions)) {
+      const activeTxt = activeNames.includes(pluginName as ExtName<PluginType>) ? ' (ACTIVE)' : '';
+      log.info(
+        `  - ${this.extensionDesc(pluginName as ExtName<PluginType>, pluginData)}${activeTxt}`,
+      );
     }
 
-    if (_.isEmpty(activeNames)) {
-      log.info('No plugins activated. Use the --use-plugins flag with names of plugins to activate');
+    if (util.isEmpty(activeNames)) {
+      log.info(
+        'No plugins activated. Use the --use-plugins flag with names of plugins to activate',
+      );
     }
   }
 }

package/lib/helpers/build.ts

@@ -1,17 +1,15 @@
-import _ from 'lodash';
 import axios from 'axios';
 import {exec} from 'teen_process';
-import {system, fs} from '@appium/support';
+import {system, fs, util} from '@appium/support';
 import type {BuildInfo} from 'appium/types';
 import {npmPackage} from '../utils';
 
 export const APPIUM_VER = npmPackage.version;
-export const rootDir = fs.findRoot(__dirname);
 
 const GIT_BINARY = `git${system.isWindows() ? '.exe' : ''}`;
 const GITHUB_API = 'https://api.github.com/repos/appium/appium';
 
-const getFullGitPath = _.memoize(async function getFullGitPath(): Promise<string | null> {
+const getFullGitPath = util.memoize(async function getFullGitPath(): Promise<string | null> {
   try {
     return await fs.which(GIT_BINARY);
   } catch {
@@ -54,7 +52,10 @@ export async function getGitRev(useGithubApiFallback = false): Promise<string |
   return null;
 }
 
-async function getGitTimestamp(commitSha: string, useGithubApiFallback = false): Promise<string | null> {
+async function getGitTimestamp(
+  commitSha: string,
+  useGithubApiFallback = false,
+): Promise<string | null> {
   const fullGitPath = await getFullGitPath();
   if (fullGitPath) {
     try {

package/lib/helpers/capability.ts

@@ -5,14 +5,10 @@ import type {
   NSCapabilities,
   W3CCapabilities,
 } from '@appium/types';
-import _ from 'lodash';
+import {util} from '@appium/support';
 import {log as logger} from '../logger';
-import {
-  processCapabilities,
-  STANDARD_CAPS,
-  errors,
-  isW3cCaps,
-} from '@appium/base-driver';
+import {processCapabilities, STANDARD_CAPS, errors, isW3cCaps} from '@appium/base-driver';
+import {mapKeys} from '../utils';
 
 const W3C_APPIUM_PREFIX = 'appium';
 const STANDARD_CAPS_LOWERCASE = new Set([...STANDARD_CAPS].map((cap) => cap.toLowerCase()));
@@ -36,7 +32,7 @@ export interface InvalidCaps<C extends Constraints = BaseDriverCapConstraints> {
 export function makeNonW3cCapsError(): Error {
   return new errors.SessionNotCreatedError(
     'Session capabilities format must comply to the W3C standard. Make sure your client is up to date. ' +
-      'See https://www.w3.org/TR/webdriver/#new-session for more details.'
+      'See https://www.w3.org/TR/webdriver/#new-session for more details.',
   );
 }
 
@@ -48,7 +44,7 @@ export function makeNonW3cCapsError(): Error {
 export function parseCapsForInnerDriver<C extends Constraints = BaseDriverCapConstraints>(
   w3cCapabilities: W3CCapabilities<C>,
   constraints: C = {} as C,
-  defaultCapabilities: NSCapabilities<C> = {}
+  defaultCapabilities: NSCapabilities<C> = {},
 ): ParsedDriverCaps<C> | InvalidCaps<C> {
   if (!isW3cCaps(w3cCapabilities)) {
     return {error: makeNonW3cCapsError()};
@@ -58,16 +54,16 @@ export function parseCapsForInnerDriver<C extends Constraints = BaseDriverCapCon
   // eslint-disable-next-line prefer-const -- assigned in success path after try
   let processedW3CCapabilities: W3CCapabilities<C> | undefined;
 
-  w3cCapabilities = _.cloneDeep(w3cCapabilities);
-  defaultCapabilities = _.cloneDeep(defaultCapabilities);
+  w3cCapabilities = structuredClone(w3cCapabilities);
+  defaultCapabilities = structuredClone(defaultCapabilities);
 
-  if (!_.isEmpty(defaultCapabilities)) {
-    for (const [defaultCapKey, defaultCapValue] of _.toPairs(defaultCapabilities)) {
+  if (!util.isEmpty(defaultCapabilities)) {
+    for (const [defaultCapKey, defaultCapValue] of Object.entries(defaultCapabilities)) {
       let isCapAlreadySet = false;
       for (const firstMatchEntry of w3cCapabilities.firstMatch ?? []) {
         if (
-          _.isPlainObject(firstMatchEntry) &&
-          _.has(removeAppiumPrefixes(firstMatchEntry as NSCapabilities<C>), removeAppiumPrefix(defaultCapKey))
+          util.isPlainObject(firstMatchEntry) &&
+          hasOwnCapability(firstMatchEntry as Record<string, unknown>, defaultCapKey)
         ) {
           isCapAlreadySet = true;
           break;
@@ -75,17 +71,16 @@ export function parseCapsForInnerDriver<C extends Constraints = BaseDriverCapCon
       }
       isCapAlreadySet =
         isCapAlreadySet ||
-        (_.isPlainObject(w3cCapabilities.alwaysMatch) &&
-          _.has(
-            removeAppiumPrefixes(w3cCapabilities.alwaysMatch),
-            removeAppiumPrefix(defaultCapKey)
-          ));
+        (util.isPlainObject(w3cCapabilities.alwaysMatch) &&
+          hasOwnCapability(w3cCapabilities.alwaysMatch as Record<string, unknown>, defaultCapKey));
       if (isCapAlreadySet) {
         continue;
       }
 
-      if (_.isEmpty(w3cCapabilities.firstMatch)) {
-        w3cCapabilities.firstMatch = [{[defaultCapKey]: defaultCapValue}] as W3CCapabilities<C>['firstMatch'];
+      if (util.isEmpty(w3cCapabilities.firstMatch)) {
+        w3cCapabilities.firstMatch = [
+          {[defaultCapKey]: defaultCapValue},
+        ] as W3CCapabilities<C>['firstMatch'];
       } else {
         (w3cCapabilities.firstMatch[0] as Record<string, unknown>)[defaultCapKey] = defaultCapValue;
       }
@@ -118,12 +113,12 @@ export function parseCapsForInnerDriver<C extends Constraints = BaseDriverCapCon
  * Prefixes capability keys with `appium:` where appropriate.
  */
 export function insertAppiumPrefixes<C extends Constraints = BaseDriverCapConstraints>(
-  caps: Capabilities<C>
+  caps: Capabilities<C>,
 ): NSCapabilities<C> {
-  return _.mapKeys(caps, (_, key) =>
-    STANDARD_CAPS_LOWERCASE.has(key.toLowerCase()) || key.includes(':')
-      ? key
-      : `${W3C_APPIUM_PREFIX}:${key}`
+  return mapKeys(caps as Record<string, unknown>, (_, key) =>
+    STANDARD_CAPS_LOWERCASE.has(String(key).toLowerCase()) || String(key).includes(':')
+      ? String(key)
+      : `${W3C_APPIUM_PREFIX}:${key}`,
   ) as NSCapabilities<C>;
 }
 
@@ -131,9 +126,11 @@ export function insertAppiumPrefixes<C extends Constraints = BaseDriverCapConstr
  * Removes `appium:` prefix from capability keys.
  */
 export function removeAppiumPrefixes<C extends Constraints = BaseDriverCapConstraints>(
-  caps: NSCapabilities<C>
+  caps: NSCapabilities<C>,
 ): Capabilities<C> {
-  return _.mapKeys(caps, (_, key) => removeAppiumPrefix(key)) as Capabilities<C>;
+  return mapKeys(caps as Record<string, unknown>, (_, key) =>
+    removeAppiumPrefix(String(key)),
+  ) as Capabilities<C>;
 }
 
 /**
@@ -142,16 +139,18 @@ export function removeAppiumPrefixes<C extends Constraints = BaseDriverCapConstr
  *
  * @returns Parsed settings object; empty if none found.
  */
-export function pullSettings(caps: Record<string, unknown> | null | undefined): Record<string, unknown> {
-  if (!_.isPlainObject(caps) || _.isEmpty(caps)) {
+export function pullSettings(
+  caps: Record<string, unknown> | null | undefined,
+): Record<string, unknown> {
+  if (!util.isPlainObject(caps) || util.isEmpty(caps)) {
     return {};
   }
 
   const result: Record<string, unknown> = {};
   const singleSettings: Record<string, unknown> = {};
-  for (const [key, value] of _.toPairs(caps)) {
+  for (const [key, value] of Object.entries(caps)) {
     let match: RegExpExecArray | null;
-    if (/^(s|appium:s)ettings$/.test(key) && _.isPlainObject(value)) {
+    if (/^(s|appium:s)ettings$/.test(key) && util.isPlainObject(value)) {
       Object.assign(result, value);
       delete caps[key];
     } else if ((match = /^(s|appium:s)ettings\[(\S+)\]$/.exec(key))) {
@@ -159,7 +158,7 @@ export function pullSettings(caps: Record<string, unknown> | null | undefined):
       delete caps[key];
     }
   }
-  if (!_.isEmpty(singleSettings)) {
+  if (!util.isEmpty(singleSettings)) {
     Object.assign(result, singleSettings);
   }
   return result;
@@ -167,5 +166,11 @@ export function pullSettings(caps: Record<string, unknown> | null | undefined):
 
 function removeAppiumPrefix(key: string): string {
   const prefix = `${W3C_APPIUM_PREFIX}:`;
-  return _.startsWith(key, prefix) ? key.substring(prefix.length) : key;
+  return key.startsWith(prefix) ? key.substring(prefix.length) : key;
+}
+
+/** True when `caps` has an own property matching `capKey` (with or without `appium:` prefix). */
+function hasOwnCapability(caps: Record<string, unknown>, capKey: string): boolean {
+  const targetKey = removeAppiumPrefix(capKey);
+  return Object.keys(caps).some((key) => removeAppiumPrefix(key) === targetKey);
 }

package/lib/helpers/network.ts

@@ -1,4 +1,3 @@
-import _ from 'lodash';
 import os from 'node:os';
 
 export const V4_BROADCAST_IP = '0.0.0.0';
@@ -16,10 +15,12 @@ export function fetchInterfaces(family: 4 | 6 | null = null): os.NetworkInterfac
   } else if (family === 6) {
     familyValue = [6, 'IPv6'];
   }
-  const ifaces = _.values(os.networkInterfaces()).filter(Boolean) as os.NetworkInterfaceInfo[][];
-  return _.flatMap(ifaces).filter(
-    (info) => !familyValue || familyValue.includes(info.family as 4 | 6 | string)
-  );
+  const ifaces = Object.values(os.networkInterfaces()).filter(
+    Boolean,
+  ) as os.NetworkInterfaceInfo[][];
+  return ifaces
+    .flat()
+    .filter((info) => !familyValue || familyValue.includes(info.family as 4 | 6 | string));
 }
 
 /**

package/lib/insecure-features.ts

@@ -1,4 +1,4 @@
-import _ from 'lodash';
+import {util} from '@appium/support';
 import {log as logger} from './logger';
 
 import type {AppiumDriver} from './appium';
@@ -20,20 +20,20 @@ export function configureGlobalFeatures(this: AppiumDriver) {
         `enabled unless explicitly disabled by --deny-insecure`,
     );
     this.relaxedSecurityEnabled = true;
-  } else if (!_.isEmpty(this.args.allowInsecure)) {
+  } else if (!util.isEmpty(this.args.allowInsecure)) {
     this.allowInsecure = validateFeatures(this.args.allowInsecure);
     const globalAllowedFeatures = filterInsecureFeatures(this.allowInsecure);
-    if (!_.isEmpty(globalAllowedFeatures)) {
+    if (!util.isEmpty(globalAllowedFeatures)) {
       logger.info('Explicitly enabling insecure features:');
       globalAllowedFeatures.forEach((a) => logger.info(`    ${a}`));
     }
   }
-  if (_.isEmpty(this.args.denyInsecure)) {
+  if (util.isEmpty(this.args.denyInsecure)) {
     return;
   }
   this.denyInsecure = validateFeatures(this.args.denyInsecure);
   const globalDeniedFeatures = filterInsecureFeatures(this.denyInsecure);
-  if (_.isEmpty(globalDeniedFeatures)) {
+  if (util.isEmpty(globalDeniedFeatures)) {
     return;
   }
   logger.info('Explicitly disabling insecure features:');
@@ -60,19 +60,20 @@ export function configureDriverFeatures(
     driver.relaxedSecurityEnabled = true;
   }
   const allowedDriverFeatures = filterInsecureFeatures(this.allowInsecure, driverName);
-  if (!_.isEmpty(allowedDriverFeatures)) {
-    this.log.info('Explicitly enabling insecure features for this session ' +
-      'as per the server configuration:',
+  if (!util.isEmpty(allowedDriverFeatures)) {
+    this.log.info(
+      'Explicitly enabling insecure features for this session ' +
+        'as per the server configuration:',
     );
     allowedDriverFeatures.forEach((a) => this.log.info(`    ${a}`));
     driver.allowInsecure = allowedDriverFeatures;
   }
   const deniedDriverFeatures = filterInsecureFeatures(this.denyInsecure, driverName);
-  if (_.isEmpty(deniedDriverFeatures)) {
+  if (util.isEmpty(deniedDriverFeatures)) {
     return;
   }
-  this.log.info('Explicitly disabling insecure features for this session ' +
-    'as per the server configuration:',
+  this.log.info(
+    'Explicitly disabling insecure features for this session ' + 'as per the server configuration:',
   );
   deniedDriverFeatures.forEach((a) => this.log.info(`    ${a}`));
   driver.denyInsecure = deniedDriverFeatures;
@@ -85,7 +86,8 @@ export function configureDriverFeatures(
  */
 function validateFeatures(features: string[]): string[] {
   const validator = (fullName: string) => {
-    const errMsg = `The full feature name must include both the destination automation name or the ` +
+    const errMsg =
+      `The full feature name must include both the destination automation name or the ` +
       `'${ALL_DRIVERS_MATCH}' wildcard to apply the feature to all installed drivers, and ` +
       `the feature name split by a colon. Got '${fullName}' instead`;
 
@@ -95,7 +97,7 @@ function validateFeatures(features: string[]): string[] {
     }
     const [automationName, featureName] = [
       fullName.substring(0, separatorPos),
-      fullName.substring(separatorPos + 1)
+      fullName.substring(separatorPos + 1),
     ];
     if (!automationName || !featureName) {
       throw new Error(errMsg);
@@ -115,7 +117,7 @@ function validateFeatures(features: string[]): string[] {
  */
 function filterInsecureFeatures(
   features: string[],
-  driverName: string = ALL_DRIVERS_MATCH
+  driverName: string = ALL_DRIVERS_MATCH,
 ): string[] {
   const filterFn = (fullName: string) => {
     const separatorPos = fullName.indexOf(FEATURE_NAME_SEPARATOR);