refactor: split driver interface into DeviceAllocator/MobilewrightSession, drop auto-reporter wiring

docs/src/test/configuration.md

@@ -77,7 +77,28 @@ driver: new MobileNextDriver({
 })
 ```
 
-With the `MobileNextDriver`, test results are uploaded to mobilenext.ai automatically unless you set `testResult: { uploadReport: 'off' }`.
+## Reporting
+
+With the `MobileNextDriver`, test results are uploaded to mobilenext.ai automatically after
+the run — no reporter configuration needed. Control it through the driver's `testResult`
+option:
+
+```ts
+import { defineConfig } from 'mobilewright';
+import { MobileNextDriver } from '@mobilewright/driver-mobilenext';
+
+export default defineConfig({
+  driver: new MobileNextDriver({
+    apiKey: process.env.MOBILENEXT_API_KEY,
+    testResult: { uploadReport: 'on' }, // 'on' | 'off' | 'on-failure' (default: 'on')
+    uploadTimeout: 60_000, // ms, default: none
+  }),
+});
+```
+
+Your own `reporter:` entries are preserved — Mobilewright appends what it needs alongside
+them. If your config already includes a `json` reporter with an `outputFile`, that report is
+reused for the upload instead of writing a second one.
 
 ## Test runner
 

docs/src/test/timeouts.md

@@ -154,7 +154,7 @@ export default defineConfig({
 
 ### Upload timeout
 
-When `testResult` is configured, Mobilewright uploads the test report to mobilenext.ai after the run. This timeout limits how long that upload may take.
+With the `MobileNextDriver`, the test report is uploaded to mobilenext.ai after the run (see [Reporting](./configuration.md#reporting)). This timeout limits how long that upload may take.
 
 ```ts
 import { defineConfig } from 'mobilewright';
@@ -163,7 +163,6 @@ import { MobileNextDriver } from '@mobilewright/driver-mobilenext';
 export default defineConfig({
   driver: new MobileNextDriver({
     apiKey: process.env.MOBILENEXT_API_KEY,
-    testResult: { uploadReport: 'on' },
     uploadTimeout: 2 * 60_000, // 2 minutes
   }),
 });
@@ -196,7 +195,6 @@ export default defineConfig({
     apiKey: process.env.MOBILENEXT_API_KEY,
     allocationTimeout: 15 * 60_000,
     uploadTimeout: 2 * 60_000,
-    testResult: { uploadReport: 'on-failure' },
   }),
 });
 ```

e2e/README.md

@@ -0,0 +1,23 @@
+# End-to-end tests
+
+End-to-end tests that verify mobilewright's locators, assertions, and other
+functionality against real emulators, simulators, and physical devices.
+
+Tests under `src/conformance` run on both platforms; tests under `src/ios` and
+`src/android` are platform-specific.
+
+## Prerequisites
+
+Download the Playground app from
+[github.com/mobile-next/playground/releases/latest](https://github.com/mobile-next/playground/releases/latest)
+and install it on the target device.
+
+## Running
+
+```sh
+# against a local device via mobilecli
+npm run test:mobilecli
+
+# against a Mobile Next cloud device (requires MOBILENEXT_API_KEY)
+npm run test:mobilenext
+```

packages/driver-mobilecli/src/driver.ts

@@ -15,7 +15,8 @@ import type {
   HardwareButton,
   LaunchOptions,
   ListDevicesOptions,
-  MobilewrightDriver,
+  MobilewrightSession,
+  DeviceAllocator,
   Orientation,
   Platform,
   RecordingOptions,
@@ -258,7 +259,7 @@ class MobilecliWebViewSession implements WebViewSession {
   }
 }
 
-export class MobilecliDriver implements MobilewrightDriver {
+export class MobilecliDriver implements MobilewrightSession, DeviceAllocator {
   private session: { deviceId: string; deviceName: string; platform: Platform; deviceType: DeviceType; rpc: RpcClient } | null = null;
   private readonly serverUrl: string;
   private readonly mobilecliPath?: string;

packages/driver-mobilenext/package.json

@@ -14,10 +14,6 @@
     ".": {
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
-    },
-    "./reporter": {
-      "types": "./dist/reporter.d.ts",
-      "default": "./dist/reporter.js"
     }
   },
   "scripts": {

packages/driver-mobilenext/src/driver.ts

@@ -1,10 +1,7 @@
 import { createReadStream, openSync, readSync, closeSync } from 'node:fs';
 import { stat } from 'node:fs/promises';
-import { basename, join } from 'node:path';
+import { basename } from 'node:path';
 import { Transform } from 'node:stream';
-import { createRequire } from 'node:module';
-import { randomUUID } from 'node:crypto';
-import os from 'node:os';
 import createDebug from 'debug';
 import type {
   AllocatedDevice,
@@ -18,27 +15,26 @@ import type {
   HardwareButton,
   LaunchOptions,
   ListDevicesOptions,
-  MobilewrightDriver,
+  MobilewrightSession,
+  DeviceAllocator,
   Orientation,
   Platform,
   RecordingOptions,
   RecordingResult,
-  ReporterEntry,
   ScreenSize,
   ScreenshotOptions,
   Session,
   SwipeDirection,
   SwipeOptions,
+  TestObserver,
   ViewNode,
 } from '@mobilewright/protocol';
 import { RpcClient } from './rpc-client.js';
 import { FleetApiClient, type DeviceFilter } from './fleet-api.js';
-import type { MobileNextTestResultConfig } from './reporter.js';
+import { MobileNextTestObserver, type MobileNextTestResultConfig } from './observer.js';
 
 export const DEFAULT_URL = 'wss://api.mobilenext.ai/ws';
 
-const _require = createRequire(import.meta.url);
-
 // ─── RPC response types ───────────────────────────────────────
 
 interface MobileNextElement {
@@ -106,9 +102,9 @@ export interface MobileNextDriverOptions {
   apiUrl?: string;
   /** Timeout waiting for a cloud device to be allocated from the pool, in ms. Default: 300000 (5 min). */
   allocationTimeout?: number;
-  /** Test-result upload options for the auto-injected upload reporter. Omit to use defaults ('on'). */
+  /** Controls automatic test-result upload to mobilenext after each run. Omit to upload on every run. */
   testResult?: MobileNextTestResultConfig;
-  /** Timeout for uploading test results to mobilenext.ai, in ms. Default: none. */
+  /** Timeout for the entire report-upload operation, in ms. */
   uploadTimeout?: number;
 }
 
@@ -205,13 +201,15 @@ interface ActiveSession {
 
 const debug = createDebug('mw:driver-mobilenext');
 
-export class MobileNextDriver implements MobilewrightDriver {
+export class MobileNextDriver implements MobilewrightSession, DeviceAllocator {
   private session: ActiveSession | null = null;
   private readonly options: MobileNextDriverOptions;
   private readonly fleetClient: FleetApiClient;
   private fleetSessionPromise: Promise<string> | null = null;
   // serial -> the fleet session it was allocated in, needed to release it later.
   private readonly fleetSessionBySerial = new Map<string, string>();
+  /** Test-lifecycle observer that uploads results to mobilenext; undefined when uploading is disabled. */
+  readonly observer: TestObserver | undefined;
 
   constructor(options: MobileNextDriverOptions = {}) {
     if (options.apiKey && options.apiUrl && !options.apiUrl.startsWith('https://')) {
@@ -223,6 +221,13 @@ export class MobileNextDriver implements MobilewrightDriver {
       apiUrl: options.apiUrl,
       allocationTimeout: options.allocationTimeout,
     });
+    this.observer = options.testResult?.uploadReport === 'off'
+      ? undefined
+      : new MobileNextTestObserver({
+        apiKey: options.apiKey ?? '',
+        testResult: options.testResult ?? {},
+        uploadTimeout: options.uploadTimeout,
+      });
   }
 
   // ─── Connection ──────────────────────────────────────────────
@@ -550,29 +555,6 @@ export class MobileNextDriver implements MobilewrightDriver {
     return this.fleetSessionPromise;
   }
 
-  // ─── Reporting ──────────────────────────────────────────────
-
-  configureReporting(): { reporters: ReporterEntry[]; captureGitInfo?: boolean } | undefined {
-    if (this.options.testResult?.uploadReport === 'off') {
-      return undefined;
-    }
-    const jsonResultsPath = join(os.tmpdir(), `mobilewright-results-${randomUUID()}.json`);
-    const uploadReporterPath = _require.resolve('./reporter.js');
-
-    return {
-      captureGitInfo: true,
-      reporters: [
-        ['json', { outputFile: jsonResultsPath }],
-        [uploadReporterPath, {
-          apiKey: this.options.apiKey ?? '',
-          jsonResultsPath,
-          testResult: this.options.testResult ?? {},
-          uploadTimeout: this.options.uploadTimeout,
-        }],
-      ],
-    };
-  }
-
   // ─── Helpers ────────────────────────────────────────────────
 
   private call<T = unknown>(method: string, params?: Record<string, unknown>): Promise<T> {

packages/driver-mobilenext/src/index.ts

@@ -9,4 +9,4 @@ export {
   type DeviceStatus,
 } from './fleet-api.js';
 export { uploadTestResult, extractGitInfoFromReport, type UploadTestResultParams, type GitInfo } from './upload-client.js';
-export { default as MobileNextUploadReporter, type MobileNextTestResultConfig } from './reporter.js';
+export { MobileNextTestObserver, type MobileNextTestResultConfig } from './observer.js';