Merge pull request #15 from percy/release/v1.1.0-beta.1

release v1.1.0-beta.1

Author: AkashBrowserStack

CHANGELOG.md

@@ -4,6 +4,19 @@ All notable changes to `@percy/maestro-app` are documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.0-beta.1] — 2026-06-25
+
+Second pre-release of the `1.1.0` self-hosted line. No customer-facing behaviour change over `1.1.0-beta.0` — this is a release-hygiene bump that completes the RELEASING.md "Bump checklist" the `beta.0` cut had left partially applied.
+
+### Changed
+
+- **`percy/scripts/percy-screenshot.js`** — `clientInfo` telemetry string bumps from `percy-maestro-app/1.1.0-beta.0` to `percy-maestro-app/1.1.0-beta.1`, keeping the embedded literal in lockstep with `package.json` per the bump checklist.
+
+### Fixed
+
+- **`package-lock.json`** — synced the root and `packages[""]` `version` (stale `1.0.0`) to the current `package.json` version.
+- **`test/unit/percy-screenshot.test.mjs`** — the `clientInfo` assertion now reads the expected version from `package.json` instead of hard-coding it, so it tracks future bumps automatically. Updated the missing-`PERCY_SESSION_ID` test to assert the current self-hosted upload behaviour (the upload-gate relaxation shipped in `1.1.0-beta.0`); it had still been asserting the pre-`beta.0` skip-and-log behaviour.
+
 ## [1.1.0-beta.0] — 2026-06-24
 
 First pre-release of self-hosted (non-BrowserStack) Maestro support. Promotes the GraalJS-side changes from PR #7 — together with `@percy/cli@1.32.3-beta.0` — to make `percy app:exec -- maestro test <flow>.yaml` work end-to-end on customer-managed devices (local dev machines, Maestro Cloud, customer CI). Backward-compatible: BrowserStack App Automate runs are byte-identical to `1.0.0`.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@percy/maestro-app",
-  "version": "1.1.0-beta.0",
+  "version": "1.1.0-beta.1",
   "description": "Percy SDK for Maestro mobile testing — visual testing for Android and iOS Maestro flows on BrowserStack",
   "license": "MIT",
   "author": "Perceptual Inc.",

package.json

@@ -296,7 +296,7 @@ try {
       }
 
       payload.platform = maestro.platform;
-      payload.clientInfo = "percy-maestro-app/1.1.0-beta.0";
+      payload.clientInfo = "percy-maestro-app/1.1.0-beta.1";
       payload.environmentInfo = "percy-maestro";
 
       // POST to the relay endpoint — Percy CLI reads the file from disk

percy/scripts/percy-screenshot.js

@@ -45,6 +45,24 @@ test('PERCY_SERVER env override is honoured', () => {
   assert.equal(httpCalls.get[0][0], 'http://localhost:9999' + HC);
 });
 
+test('PERCY_SERVER_ADDRESS used when set; PERCY_SERVER takes precedence when both set', () => {
+  // PERCY_SERVER_ADDRESS is the self-hosted `percy app:exec` export; it wins
+  // over the default but loses to an explicit PERCY_SERVER.
+  const addrOnly = runScript('healthcheck', {
+    platform: 'android',
+    env: { PERCY_SERVER_ADDRESS: 'http://addr:1' },
+    http: { get: [OK_RESPONSE({})] },
+  });
+  assert.equal(addrOnly.httpCalls.get[0][0], 'http://addr:1' + HC);
+
+  const both = runScript('healthcheck', {
+    platform: 'android',
+    env: { PERCY_SERVER_ADDRESS: 'http://addr:1', PERCY_SERVER: 'http://explicit:2' },
+    http: { get: [OK_RESPONSE({})] },
+  });
+  assert.equal(both.httpCalls.get[0][0], 'http://explicit:2' + HC);
+});
+
 test('4xx response → reachable-but-rejected banner, disabled', () => {
   const { output, logs } = runScript('healthcheck', {
     platform: 'android',

test/unit/percy-healthcheck.test.mjs

@@ -75,6 +75,22 @@ test('inline healthcheck: PERCY_SERVER override and no-version success', () => {
   assert.ok(logs.some((l) => l === '[percy] Percy CLI healthcheck passed.'));
 });
 
+test('inline healthcheck: PERCY_SERVER_ADDRESS used; PERCY_SERVER wins when both set', () => {
+  const addrOnly = runScript('prepare', {
+    platform: 'ios',
+    env: { SCREENSHOT_NAME: 'x', PERCY_SERVER_ADDRESS: 'http://addr:3' },
+    http: { get: [OK_RESPONSE({})] },
+  });
+  assert.equal(addrOnly.httpCalls.get[0][0], 'http://addr:3/percy/healthcheck');
+
+  const both = runScript('prepare', {
+    platform: 'ios',
+    env: { SCREENSHOT_NAME: 'x', PERCY_SERVER_ADDRESS: 'http://addr:3', PERCY_SERVER: 'http://explicit:4' },
+    http: { get: [OK_RESPONSE({})] },
+  });
+  assert.equal(both.httpCalls.get[0][0], 'http://explicit:4/percy/healthcheck');
+});
+
 test('inline healthcheck: 4xx disables with rejected banner', () => {
   const { output, logs } = runScript('prepare', {
     platform: 'android',

test/unit/percy-prepare-screenshot.test.mjs

@@ -1,10 +1,25 @@
 // test/unit/percy-screenshot.test.mjs
 import { test } from 'node:test';
 import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { runScript, OK_RESPONSE, NOT_OK_RESPONSE, THROW } from './harness.mjs';
 
 const SS = '/percy/maestro-screenshot';
 
+// Read the version from package.json so the clientInfo assertion tracks the
+// release bump automatically and never goes stale. The shipped
+// percy-screenshot.js embeds the clientInfo string as a literal (GraalJS in
+// Maestro can't require package.json at runtime); the RELEASING.md "Bump
+// checklist" keeps the literal and package.json in lockstep, and this test is
+// the guard that they actually match.
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const PKG = JSON.parse(
+  fs.readFileSync(path.resolve(__dirname, '..', '..', 'package.json'), 'utf8')
+);
+const EXPECTED_CLIENT_INFO = 'percy-maestro-app/' + PKG.version;
+
 // Helper: an "enabled" output so we skip self-init and go straight to the
 // upload path. percyServer set so the `output.percyServer || default` reads
 // the truthy side.
@@ -65,6 +80,27 @@ test('self-init: PERCY_SERVER override used for healthcheck and upload', () => {
   assert.equal(httpCalls.post[0][0], 'http://srv:1' + SS);
 });
 
+test('self-init: PERCY_SERVER_ADDRESS used; PERCY_SERVER wins when both set', () => {
+  const addrOnly = runScript('screenshot', {
+    platform: 'android',
+    env: { SCREENSHOT_NAME: 'home', PERCY_SESSION_ID: 's', PERCY_SERVER_ADDRESS: 'http://addr:5' },
+    http: { get: [OK_RESPONSE({})], post: [OK_RESPONSE()] },
+  });
+  assert.equal(addrOnly.httpCalls.get[0][0], 'http://addr:5/percy/healthcheck');
+  assert.equal(addrOnly.httpCalls.post[0][0], 'http://addr:5' + SS);
+
+  const both = runScript('screenshot', {
+    platform: 'android',
+    env: {
+      SCREENSHOT_NAME: 'home', PERCY_SESSION_ID: 's',
+      PERCY_SERVER_ADDRESS: 'http://addr:5', PERCY_SERVER: 'http://explicit:6',
+    },
+    http: { get: [OK_RESPONSE({})], post: [OK_RESPONSE()] },
+  });
+  assert.equal(both.httpCalls.get[0][0], 'http://explicit:6/percy/healthcheck');
+  assert.equal(both.httpCalls.post[0][0], 'http://explicit:6' + SS);
+});
+
 test('self-init: unsupported platform disables and skips upload', () => {
   const { output, httpCalls, logs } = runScript('screenshot', {
     platform: 'tvos',
@@ -181,14 +217,22 @@ test('enabled with invalid SCREENSHOT_NAME → regex throw, caught, logged', ()
   assert.ok(logs.some((l) => l.includes('SCREENSHOT_NAME must match')));
 });
 
-test('enabled, valid name, MISSING session id → logs and does not upload', () => {
-  const { httpCalls, logs } = runScript('screenshot', {
+test('enabled, valid name, MISSING session id → uploads as selfhosted (no session-id gate)', () => {
+  // The PERCY_SESSION_ID upload gate was relaxed in 1.1.0-beta.0 (see
+  // CHANGELOG "Changed" + the runtime-field plan): a missing session id no
+  // longer skips the upload. The POST now fires with runtime "selfhosted" and
+  // no sessionId field; the CLI relay's runtime-aware handler picks the
+  // self-hosted file-find path.
+  const { httpCalls } = runScript('screenshot', {
     platform: 'android',
     output: enabled(),
     env: { SCREENSHOT_NAME: 'home' },
+    http: { post: [OK_RESPONSE()] },
   });
-  assert.equal(httpCalls.post.length, 0);
-  assert.ok(logs.some((l) => l.includes('PERCY_SESSION_ID not set')));
+  assert.equal(httpCalls.post.length, 1, 'self-hosted upload fires without a session id');
+  const payload = JSON.parse(httpCalls.post[0][1].body);
+  assert.equal(payload.runtime, 'selfhosted');
+  assert.equal(payload.sessionId, undefined, 'no sessionId field when PERCY_SESSION_ID absent');
 });
 
 // ---------------------------------------------------------------------------
@@ -208,7 +252,7 @@ test('android default tag/name/dimensions when env vars absent', () => {
   assert.equal(payload.statusBarHeight, 120);
   assert.equal(payload.navBarHeight, 100);
   assert.equal(payload.platform, 'android');
-  assert.equal(payload.clientInfo, 'percy-maestro-app/1.0.0');
+  assert.equal(payload.clientInfo, EXPECTED_CLIENT_INFO);
   assert.equal(payload.environmentInfo, 'percy-maestro');
 });