feat: Integrate oxc and release configs

.editorconfig

@@ -0,0 +1,14 @@
+# editorconfig.org
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+max_line_length = 120
+
+[*.md]
+trim_trailing_whitespace = false

.github/workflows/unit-test.yml

@@ -39,6 +39,6 @@ jobs:
     - run: npm run lint
       name: Run linter
     - run: npm run format:check
-      name: Run Prettier check
+      name: Run format check
     - run: npm run test
       name: Run unit tests

lib/devicectl.ts

@@ -1,10 +1,11 @@
-import {exec, SubProcess} from 'teen_process';
 import {log as logger} from '@appium/logger';
-import type {DevicectlOptions, ExecuteOptions, ExecuteResult} from './types.js';
-import * as processMixins from './mixins/process.js';
-import * as infoMixins from './mixins/info.js';
+import {exec, SubProcess} from 'teen_process';
+
 import * as copyMixins from './mixins/copy.js';
+import * as infoMixins from './mixins/info.js';
 import * as listMixins from './mixins/list.js';
+import * as processMixins from './mixins/process.js';
+import type {DevicectlOptions, ExecuteOptions, ExecuteResult} from './types.js';
 
 const XCRUN = 'xcrun';
 const LOG_TAG = 'Devicectl';
@@ -55,10 +56,7 @@ export class Devicectl {
    * @param opts - Execution options
    * @returns Promise that resolves to the command result
    */
-  async execute<T extends ExecuteOptions>(
-    subcommand: string[],
-    opts?: T,
-  ): Promise<ExecuteResult<T>> {
+  async execute<T extends ExecuteOptions>(subcommand: string[], opts?: T): Promise<ExecuteResult<T>> {
     const {
       logStdout = false,
       asynchronous = false,
@@ -72,9 +70,7 @@ export class Devicectl {
     const finalArgs = ['devicectl', ...subcommand, ...(noDevice ? [] : ['--device', this.udid])];
 
     if (subcommandOptions && subcommandOptions.length > 0) {
-      finalArgs.push(
-        ...(Array.isArray(subcommandOptions) ? subcommandOptions : [subcommandOptions]),
-      );
+      finalArgs.push(...(Array.isArray(subcommandOptions) ? subcommandOptions : [subcommandOptions]));
     }
 
     if (asJson) {

lib/mixins/copy.ts

@@ -1,5 +1,5 @@
-import type {ListFilesOptions, PullFileOptions} from '../types.js';
 import type {Devicectl} from '../devicectl.js';
+import type {ListFilesOptions, PullFileOptions} from '../types.js';
 
 /**
  * Lists files at a specified path on the device
@@ -30,12 +30,7 @@ export async function listFiles(
 /**
  * Pulls a file from the specified path on the device to a local file system
  */
-export async function pullFile(
-  this: Devicectl,
-  from: string,
-  to: string,
-  opts: PullFileOptions,
-): Promise<string> {
+export async function pullFile(this: Devicectl, from: string, to: string, opts: PullFileOptions): Promise<string> {
   const subcommandOptions = [
     '--domain-type',
     opts.domainType,

lib/mixins/info.ts

@@ -1,5 +1,5 @@
-import type {AppInfo, ProcessInfo} from '../types.js';
 import type {Devicectl} from '../devicectl.js';
+import type {AppInfo, ProcessInfo} from '../types.js';
 
 /**
  * Retrieves the list of installed apps from the device

lib/mixins/list.ts

@@ -1,5 +1,5 @@
-import type {DeviceInfo} from '../types.js';
 import type {Devicectl} from '../devicectl.js';
+import type {DeviceInfo} from '../types.js';
 
 /**
  * Retrieves the list of connected device infos.

lib/mixins/process.ts

@@ -1,5 +1,5 @@
-import type {LaunchAppOptions, ProcessInfo, TerminateAppOptions} from '../types.js';
 import type {Devicectl} from '../devicectl.js';
+import type {LaunchAppOptions, ProcessInfo, TerminateAppOptions} from '../types.js';
 
 /**
  * Simulates memory warning for the process with the given PID
@@ -28,11 +28,7 @@ export async function sendSignalToProcess(
  * This method is over devicectl command, this it may take additional seconds to launch the app.
  * Please use via WDA or via appium-ios-device as primary method to launch app if possible.
  */
-export async function launchApp(
-  this: Devicectl,
-  bundleId: string,
-  opts: LaunchAppOptions = {},
-): Promise<void> {
+export async function launchApp(this: Devicectl, bundleId: string, opts: LaunchAppOptions = {}): Promise<void> {
   const {env, terminateExisting = false} = opts;
 
   const subcommandOptions: string[] = [];
@@ -44,9 +40,7 @@ export async function launchApp(
   if (env && Object.keys(env).length > 0) {
     subcommandOptions.push(
       '--environment-variables',
-      JSON.stringify(
-        Object.fromEntries(Object.entries(env).map(([key, value]) => [key, String(value)])),
-      ),
+      JSON.stringify(Object.fromEntries(Object.entries(env).map(([key, value]) => [key, String(value)]))),
     );
   }
 
@@ -112,10 +106,7 @@ export function escapeProcessFilterValue(value: string): string {
   return value.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
 }
 
-async function listProcessesForAppPath(
-  devicectl: Devicectl,
-  appPath: string,
-): Promise<ProcessInfo[]> {
+async function listProcessesForAppPath(devicectl: Devicectl, appPath: string): Promise<ProcessInfo[]> {
   const filter = `executable.path BEGINSWITH "${escapeProcessFilterValue(appPath)}"`;
   const {stdout} = await devicectl.execute(['device', 'info', 'processes'], {
     subcommandOptions: ['--filter', filter],

oxfmt.config.mjs

@@ -0,0 +1,6 @@
+import appiumConfig, {defineConfig, ignorePatterns} from '@appium/oxc-config/oxfmt';
+
+export default defineConfig({
+  ...appiumConfig,
+  ignorePatterns: [...ignorePatterns],
+});

oxlint.config.mjs

@@ -0,0 +1,6 @@
+import appiumConfig, {defineConfig, ignorePatterns} from '@appium/oxc-config/oxlint';
+
+export default defineConfig({
+  extends: [appiumConfig],
+  ignorePatterns: [...ignorePatterns],
+});

package.json

@@ -1,29 +1,23 @@
 {
   "name": "node-devicectl",
+  "version": "2.0.1",
   "description": "Node.js wrapper around Apple's devicectl tool",
-  "tags": [
+  "keywords": [
     "apple",
-    "ios",
-    "devicectl",
     "device",
+    "devicectl",
+    "ios",
     "xcode"
   ],
-  "version": "2.0.1",
-  "author": "Appium Contributors",
+  "bugs": {
+    "url": "https://github.com/appium/node-devicectl/issues"
+  },
   "license": "Apache-2.0",
+  "author": "Appium Contributors",
   "repository": {
     "type": "git",
     "url": "https://github.com/appium/node-devicectl.git"
   },
-  "bugs": {
-    "url": "https://github.com/appium/node-devicectl/issues"
-  },
-  "engines": {
-    "node": "^20.19.0 || ^22.12.0 || >=24.0.0",
-    "npm": ">=10"
-  },
-  "main": "./build/lib/index.js",
-  "types": "./build/lib/index.d.ts",
   "bin": {},
   "directories": {
     "lib": "./lib"
@@ -33,46 +27,42 @@
     "build/lib",
     "CHANGELOG.md"
   ],
-  "dependencies": {
-    "@appium/logger": "^2.0.0-rc.1",
-    "teen_process": "^4.1.0"
+  "type": "module",
+  "main": "./build/lib/index.js",
+  "types": "./build/lib/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./build/lib/index.d.ts",
+      "import": "./build/lib/index.js"
+    },
+    "./package.json": "./package.json"
   },
   "scripts": {
     "build": "tsc -b",
     "clean": "npm run build -- --clean",
     "rebuild": "npm run clean; npm run build",
     "dev": "npm run build -- --watch",
-    "lint": "eslint .",
-    "lint:fix": "npm run lint -- --fix",
-    "format": "prettier -w ./lib ./test",
-    "format:check": "prettier --check ./lib ./test",
+    "lint": "oxlint -c oxlint.config.mjs .",
+    "lint:fix": "oxlint -c oxlint.config.mjs --fix .",
+    "format": "oxfmt -c oxfmt.config.mjs .",
+    "format:check": "oxfmt -c oxfmt.config.mjs --check .",
     "prepare": "npm run build",
     "test": "node --enable-source-maps --test --test-timeout=60000 \"./build/test/unit/**/*.spec.js\"",
     "e2e-test": "node --enable-source-maps --test --test-force-exit --test-concurrency=1 --test-timeout=300000 \"./build/test/e2e/**/*.spec.js\""
   },
-  "prettier": {
-    "bracketSpacing": false,
-    "printWidth": 100,
-    "singleQuote": true
+  "dependencies": {
+    "@appium/logger": "^2.0.0-rc.1",
+    "teen_process": "^4.1.0"
   },
   "devDependencies": {
-    "@appium/eslint-config-appium-ts": "^3.0.1",
-    "@appium/tsconfig": "^1.0.0-rc.1",
+    "@appium/oxc-config": "^1.1.0",
+    "@appium/semantic-release-config": "^1.1.0",
+    "@appium/tsconfig": "^1.2.0",
     "@appium/types": "^1.0.0-rc.1",
-    "@semantic-release/changelog": "^6.0.1",
-    "@semantic-release/git": "^10.0.1",
-    "@types/node": "^26.1.0",
-    "conventional-changelog-conventionalcommits": "^9.0.0",
-    "prettier": "^3.0.0",
-    "semantic-release": "^25.0.2",
-    "typescript": "^6.0.2"
+    "@types/node": "^26.1.0"
   },
-  "type": "module",
-  "exports": {
-    ".": {
-      "types": "./build/lib/index.d.ts",
-      "import": "./build/lib/index.js"
-    },
-    "./package.json": "./package.json"
+  "engines": {
+    "node": "^20.19.0 || ^22.12.0 || >=24.0.0",
+    "npm": ">=10"
   }
 }

release.config.mjs

@@ -0,0 +1,3 @@
+import releaseConfig from '@appium/semantic-release-config';
+
+export default releaseConfig();

test/e2e/sudo-execution.spec.ts

@@ -1,7 +1,8 @@
 import assert from 'node:assert/strict';
-import {Devicectl} from '../../lib/devicectl.js';
 import {describe, it, type TestContext} from 'node:test';
 
+import {Devicectl} from '../../lib/devicectl.js';
+
 describe('manual sudo execution e2e', function () {
   it('runs devicectl as original non-root user under sudo', async function (ctx: TestContext) {
     if (

test/unit/devicectl.spec.ts

@@ -1,7 +1,8 @@
 import assert from 'node:assert/strict';
+import {describe, it, beforeEach} from 'node:test';
+
 import {Devicectl} from '../../lib/devicectl.js';
 import {appUrlToFilesystemPath, escapeProcessFilterValue} from '../../lib/mixins/process.js';
-import {describe, it, beforeEach} from 'node:test';
 
 describe('Devicectl', function () {
   let devicectl: Devicectl;
@@ -24,10 +25,7 @@ describe('Devicectl', function () {
   describe('sudo behavior', function () {
     it('should cache sudo user identity when available', function () {
       const localDevicectl = new Devicectl('test-device-udid');
-      assert.strictEqual(
-        (localDevicectl as any).sudoUser === null || !!(localDevicectl as any).sudoUser,
-        true,
-      );
+      assert.strictEqual((localDevicectl as any).sudoUser === null || !!(localDevicectl as any).sudoUser, true);
     });
 
     it('should keep constructor default for runAsNonRootWhenSudo behavior', function () {
@@ -100,10 +98,7 @@ describe('Devicectl', function () {
       });
 
       it('should strip both the file:// prefix and trailing slash', function () {
-        assert.strictEqual(
-          appUrlToFilesystemPath('file:///private/var/App.app/'),
-          '/private/var/App.app',
-        );
+        assert.strictEqual(appUrlToFilesystemPath('file:///private/var/App.app/'), '/private/var/App.app');
       });
 
       it('should leave paths without a file:// prefix or trailing slash unchanged', function () {
@@ -117,21 +112,15 @@ describe('Devicectl', function () {
       });
 
       it('should escape backslashes', function () {
-        assert.strictEqual(
-          escapeProcessFilterValue(String.raw`path\with\slashes`),
-          String.raw`path\\with\\slashes`,
-        );
+        assert.strictEqual(escapeProcessFilterValue(String.raw`path\with\slashes`), String.raw`path\\with\\slashes`);
       });
 
       it('should escape double quotes', function () {
         assert.strictEqual(escapeProcessFilterValue('path"with"quotes'), 'path\\"with\\"quotes');
       });
 
       it('should escape backslashes and double quotes together', function () {
-        assert.strictEqual(
-          escapeProcessFilterValue(String.raw`path\"mixed`),
-          String.raw`path\\\"mixed`,
-        );
+        assert.strictEqual(escapeProcessFilterValue(String.raw`path\"mixed`), String.raw`path\\\"mixed`);
       });
     });
   });