@essential-apps/shopify-test-runner 1.0.27 → 1.0.29

package/src/scripts/{runOfflineFullTests.ts → runOfflineTests.ts}

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * runOfflineFullTests — full-stack offline orchestrator.
+ * runOfflineTests — full-stack offline orchestrator.
  *
  * Boots the entire offline stack in ONE Node process:
  *
@@ -13,14 +13,14 @@
  *     reaches our Admin GraphQL mock
  *   - Playwright as a child process, with mock URLs in env
  *
- * The Playwright fixture (`offlineFullTest`, orchestrated mode)
+ * The Playwright fixture (`offlineTest`, orchestrated mode)
  * reads the mock URLs from env and routes browser requests via
  * `page.route()`. Tests mutate ShopState via control-plane HTTP
  * endpoints exposed by the storefront mock (mounted under
  * `/__test__/state/*`).
  *
  * This is the offline counterpart to runTests.ts (online mode).
- * Online tests still go via runTests.ts; offline-full tests via this.
+ * Online tests still go via runTests.ts; offline tests via this.
  *
  * Env it sets in the spawned Remix backend:
  *   SHOPIFY_API_SECRET  = mock JWT secret (so JWTs validate)
@@ -135,7 +135,7 @@ function dropOrphanDatabases(prefix: string): void {
       .map((s) => s.trim())
       .filter((s) => s.length > 0);
     if (orphans.length === 0) return;
-    console.log(`[runOfflineFull] cleanup: dropping ${orphans.length} orphan DB(s)…`);
+    console.log(`[runOfflineTests] cleanup: dropping ${orphans.length} orphan DB(s)…`);
     for (const name of orphans) {
       try {
         // `--force` (psql 13+) terminates any leftover connections
@@ -152,7 +152,7 @@ function dropOrphanDatabases(prefix: string): void {
   } catch (err) {
     // If psql isn't available at all we just skip cleanup; the
     // primary createdb below will fail loudly anyway.
-    console.warn(`[runOfflineFull] orphan-DB cleanup failed: ${(err as Error).message}`);
+    console.warn(`[runOfflineTests] orphan-DB cleanup failed: ${(err as Error).message}`);
   }
 }
 
@@ -289,7 +289,7 @@ function ensureOfflineSetup(): void {
   try {
     hostsContent = readFileSync(hostsPath, 'utf8');
   } catch {
-    console.warn(`[runOfflineFull] could not read ${hostsPath}; skipping /etc/hosts setup`);
+    console.warn(`[runOfflineTests] could not read ${hostsPath}; skipping /etc/hosts setup`);
     return;
   }
   const hostsToAdd: string[] = [];
@@ -302,6 +302,10 @@ function ensureOfflineSetup(): void {
     // `font_face` filter output (which contains real
     // fonts.shopifycdn.com URLs) just works in-browser.
     'fonts.shopifycdn.com',
+    // Partner API — so in-VM Node test code (the issue-credit helper)
+    // reaches the mock through the edge proxy, same as the backend's
+    // hard-coded https://partners.shopify.com fetch. No app-side override.
+    'partners.shopify.com',
   ]) {
     if (!new RegExp(`^127\\.0\\.0\\.1\\s+${host.replace(/\./g, '\\.')}\\s*$`, 'm').test(hostsContent)) {
       hostsToAdd.push(`127.0.0.1 ${host}`);
@@ -312,10 +316,10 @@ function ensureOfflineSetup(): void {
   if (hostsToAdd.length > 0) {
     try {
       writeFileSync(hostsPath, hostsContent + '\n' + hostsToAdd.join('\n') + '\n');
-      console.log(`[runOfflineFull] wrote ${hostsToAdd.length} /etc/hosts entries`);
+      console.log(`[runOfflineTests] wrote ${hostsToAdd.length} /etc/hosts entries`);
     } catch (err) {
       console.warn(
-        `[runOfflineFull] could not write ${hostsPath}: ${(err as Error).message}. ` +
+        `[runOfflineTests] could not write ${hostsPath}: ${(err as Error).message}. ` +
           `Browser may resolve Shopify hostnames to real Shopify on the internet.`,
       );
     }
@@ -336,12 +340,12 @@ function ensureOfflineSetup(): void {
     if (existing !== TEST_CA_CERT_PEM) {
       writeFileSync(systemCertDest, TEST_CA_CERT_PEM);
       execSync('update-ca-certificates >/dev/null 2>&1 || true', { stdio: 'ignore' });
-      console.log('[runOfflineFull] installed edge CA into system trust store');
+      console.log('[runOfflineTests] installed edge CA into system trust store');
     }
     process.env['NODE_EXTRA_CA_CERTS'] = systemCertDest;
   } catch (err) {
     console.warn(
-      `[runOfflineFull] could not install edge CA: ${(err as Error).message}. ` +
+      `[runOfflineTests] could not install edge CA: ${(err as Error).message}. ` +
         `TLS connections to the edge may fail.`,
     );
   }
@@ -381,10 +385,10 @@ function ensureOfflineSetup(): void {
       { stdio: 'ignore' },
     );
     unlinkSync(tmpCaPath);
-    console.log('[runOfflineFull] installed edge CA into NSS DB (Chrome)');
+    console.log('[runOfflineTests] installed edge CA into NSS DB (Chrome)');
   } catch (err) {
     console.warn(
-      `[runOfflineFull] could not install edge CA into NSS DB: ${(err as Error).message}. ` +
+      `[runOfflineTests] could not install edge CA into NSS DB: ${(err as Error).message}. ` +
         `Chrome will reject the edge's cert. Verify libnss3-tools is installed in the image.`,
     );
   }
@@ -421,7 +425,7 @@ async function startOfflineDnsRedirect(): Promise<OfflineResolver | null> {
     resolver = await startOfflineResolver({ upstream });
   } catch (err) {
     console.warn(
-      `[runOfflineFull] offline DNS resolver failed to bind 127.0.0.1:53: ` +
+      `[runOfflineTests] offline DNS resolver failed to bind 127.0.0.1:53: ` +
         `${(err as Error).message}. page.request may resolve Shopify ` +
         `hostnames to real Shopify on the internet.`,
     );
@@ -437,12 +441,12 @@ async function startOfflineDnsRedirect(): Promise<OfflineResolver | null> {
     // so an aggressive timeout would spuriously fail real lookups.
     writeFileSync('/etc/resolv.conf', 'nameserver 127.0.0.1\n');
     console.log(
-      '[runOfflineFull] offline DNS resolver on 127.0.0.1:53 ' +
+      '[runOfflineTests] offline DNS resolver on 127.0.0.1:53 ' +
         `(Shopify → loopback${upstream ? `, else → ${upstream}` : ', else → SERVFAIL'})`,
     );
   } catch (err) {
     console.warn(
-      `[runOfflineFull] could not repoint /etc/resolv.conf: ` +
+      `[runOfflineTests] could not repoint /etc/resolv.conf: ` +
         `${(err as Error).message}. c-ares clients may bypass the resolver.`,
     );
   }
@@ -548,7 +552,7 @@ function seedSession(env: Record<string, string>, shopDomain: string): void {
  *   - Supervisor (no `TEST_OFFLINE_SHARD_INDEX` set, N>1):
  *       1. Build cache check + build once (so shards don't race)
  *       2. Orphan-DB cleanup once
- *       3. Spawn N child processes (`node ... runOfflineFullTests.ts`)
+ *       3. Spawn N child processes (`node ... runOfflineTests.ts`)
  *          each with `TEST_OFFLINE_SHARD_INDEX=k`, `TEST_OFFLINE_SKIP_BUILD=true`
  *       4. Stream child stdout/stderr prefixed with [shard k]
  *       5. Wait for all, exit with worst exit code
@@ -580,14 +584,14 @@ async function main(): Promise<void> {
   // strategy or invasive Chromium/Node bypasses. We tried that path
   // (page.route() + host-resolver-rules + a Node socket-layer shim);
   // it worked but maintained ~400 lines of bypass code in parallel.
-  // Use `npm run test:offline-full` from the consuming app — it
+  // Use `npm run test:offline` from the consuming app — it
   // routes through `runDockerOffline.ts` which spawns the
   // `essential-upsell-test:latest` image where TEST_IN_CONTAINER=true
   // is set by docker/entrypoint.sh.
   if (process.env['TEST_IN_CONTAINER'] !== 'true') {
     console.error(
-      '[runOfflineFull] FATAL: must run inside the offline container.\n' +
-        '  Run: `npm run test:offline-full` (which invokes runDockerOffline.ts)\n' +
+      '[runOfflineTests] FATAL: must run inside the offline container.\n' +
+        '  Run: `npm run test:offline` (which invokes runDockerOffline.ts)\n' +
         '  If you are seeing this from inside what should be a container, the\n' +
         "  entrypoint script (tests/offline/docker/entrypoint.sh) didn't run or\n" +
         "  didn't export TEST_IN_CONTAINER=true.",
@@ -642,7 +646,7 @@ async function main(): Promise<void> {
   const scopes = process.env['SCOPES'] ?? 'write_products,read_products,write_discounts,read_discounts';
   const playwrightConfig =
     process.env['TEST_PLAYWRIGHT_CONFIG'] ??
-    'tests/offline/playwright.offline-full.config.ts';
+    'tests/offline/playwright.offline.config.ts';
   const isChildShard = shardIndex !== null;
   // Backend port: shard k uses BASE + (k-1); single-worker uses BASE.
   const baseBackendPort = Number(process.env['TEST_OFFLINE_BACKEND_PORT'] ?? '8181');
@@ -650,9 +654,9 @@ async function main(): Promise<void> {
     isChildShard ? baseBackendPort + (shardIndex - 1) : baseBackendPort,
   );
   // Production Remix serves via plain HTTP (remix-serve has no TLS).
-  // Vite dev had self-signed HTTPS; offline-full uses HTTP. The mock
+  // Vite dev had self-signed HTTPS; offline uses HTTP. The mock
   // admin shell serves the iframe pointing here; mixed-content
-  // warnings are silenced via Chrome flags in the offlineFullStack
+  // warnings are silenced via Chrome flags in the offlineStack
   // fixture.
   const backendUrl = `http://localhost:${backendPort}`;
   // Skip the build if TEST_OFFLINE_SKIP_BUILD=true and a recent build
@@ -667,8 +671,8 @@ async function main(): Promise<void> {
   const dbConn = dbUrl(dbName);
   const preserveDb = process.env['TEST_PRESERVE_DB'] === 'true';
 
-  console.log(`[runOfflineFull] Test DB: ${dbConn}`);
-  console.log(`[runOfflineFull] Backend: ${backendUrl}`);
+  console.log(`[runOfflineTests] Test DB: ${dbConn}`);
+  console.log(`[runOfflineTests] Backend: ${backendUrl}`);
 
   // Drop orphan DBs from previous crashed runs. The supervisor
   // already did this once — children skip.
@@ -676,7 +680,7 @@ async function main(): Promise<void> {
     dropOrphanDatabases(dbPrefix);
   }
 
-  console.log('[runOfflineFull] createdb…');
+  console.log('[runOfflineTests] createdb…');
   execSync(`createdb -h ${PG_HOST} -p ${PG_PORT} ${dbName}`, { stdio: 'inherit' });
 
   // ── Mock servers (in-process, sharing one ShopState) ─────
@@ -702,7 +706,7 @@ async function main(): Promise<void> {
     seedExploreProducts(state);
   }
 
-  console.log('[runOfflineFull] booting mock servers…');
+  console.log('[runOfflineTests] booting mock servers…');
   // 127.0.0.1 binding is the safe default — orchestrator and child
   // both reach the mocks on loopback. Used to be subtly broken
   // because we ran Playwright via execSync (sync, blocks the event
@@ -732,7 +736,7 @@ async function main(): Promise<void> {
     : undefined;
   if (postPurchaseExtensionPath && !existsSync(postPurchaseExtensionPath)) {
     throw new Error(
-      `[runOfflineFull] TEST_OFFLINE_POST_PURCHASE_EXT points at ${postPurchaseExtensionPath} but the file doesn't exist. ` +
+      `[runOfflineTests] TEST_OFFLINE_POST_PURCHASE_EXT points at ${postPurchaseExtensionPath} but the file doesn't exist. ` +
         `Build the extension first — the bundled .js must exist on disk before the test run.`,
     );
   }
@@ -778,7 +782,7 @@ async function main(): Promise<void> {
     startOpts,
   );
   console.log(
-    `[runOfflineFull] mocks ready:\n` +
+    `[runOfflineTests] mocks ready:\n` +
       `   shell:        ${adminShell.baseUrl}\n` +
       `   admin GQL:    ${adminApi.baseUrl}\n` +
       `   storefront GQL: ${storefrontApi.baseUrl}\n` +
@@ -796,9 +800,9 @@ async function main(): Promise<void> {
   for (const p of probes) {
     try {
       const r = await fetch(p.url, p.init);
-      console.log(`[runOfflineFull] probe ${p.name}: ${r.status}`);
+      console.log(`[runOfflineTests] probe ${p.name}: ${r.status}`);
     } catch (e) {
-      console.log(`[runOfflineFull] probe ${p.name}: FAILED ${(e as Error).message}`);
+      console.log(`[runOfflineTests] probe ${p.name}: FAILED ${(e as Error).message}`);
     }
   }
 
@@ -831,6 +835,7 @@ async function main(): Promise<void> {
       adminApi: adminApi.baseUrl,
       storefrontApi: storefrontApi.baseUrl,
       adminShell: adminShell.baseUrl,
+      partnerApi: partnerApi.baseUrl,
     },
     shopDomain: DEFAULT_SHOP_DOMAIN,
     // Previously bound dual-stack `::` so /etc/hosts entries for
@@ -842,7 +847,7 @@ async function main(): Promise<void> {
     hostname: '0.0.0.0',
     port: 443,
   });
-  console.log(`[runOfflineFull] edge proxy: ${edge.baseUrl}`);
+  console.log(`[runOfflineTests] edge proxy: ${edge.baseUrl}`);
 
   // ── Neon-local WS proxy (zero-app-touch) ──────────────────
   // Bridge the app's @neondatabase/serverless driver to THIS run's local
@@ -851,7 +856,7 @@ async function main(): Promise<void> {
   // the proxy reads the driver's `?address=` and forwards to Postgres.
   const neonProxy = await startNeonWsProxy({ targetHost: PG_HOST, targetPort: Number(PG_PORT) });
   console.log(
-    `[runOfflineFull] neon ws-proxy: 127.0.0.1:${neonProxy.port} → Postgres ${PG_HOST}:${PG_PORT}`,
+    `[runOfflineTests] neon ws-proxy: 127.0.0.1:${neonProxy.port} → Postgres ${PG_HOST}:${PG_PORT}`,
   );
 
   // ── Backend env ───────────────────────────────────────────
@@ -951,7 +956,7 @@ async function main(): Promise<void> {
       : {}),
   };
 
-  console.log('[runOfflineFull] prisma migrate deploy…');
+  console.log('[runOfflineTests] prisma migrate deploy…');
   // Strip the MSW loader from NODE_OPTIONS for prisma's invocation.
   // MSW shouldn't intercept Postgres connections (it's HTTP-only),
   // but loading it from inside `npx prisma` triggers a silent
@@ -974,19 +979,19 @@ async function main(): Promise<void> {
   // a no-op for apps that don't need it.
   const extraSchema = resolve(process.cwd(), 'tests/offline/offline-extra-schema.sql');
   if (existsSync(extraSchema)) {
-    console.log('[runOfflineFull] applying extra schema (tests/offline/offline-extra-schema.sql)…');
+    console.log('[runOfflineTests] applying extra schema (tests/offline/offline-extra-schema.sql)…');
     execSync(
       `psql -h ${PG_HOST} -p ${PG_PORT} -d ${dbName} -v ON_ERROR_STOP=1 -f ${JSON.stringify(extraSchema)}`,
       { stdio: 'inherit', env: prismaEnv },
     );
   }
 
-  console.log('[runOfflineFull] seeding session row…');
+  console.log('[runOfflineTests] seeding session row…');
   try {
     seedSession(prismaEnv as Record<string, string>, DEFAULT_SHOP_DOMAIN);
   } catch (err) {
     console.warn(
-      `[runOfflineFull] session seed failed (continuing): ${(err as Error).message}`,
+      `[runOfflineTests] session seed failed (continuing): ${(err as Error).message}`,
     );
   }
 
@@ -1044,7 +1049,7 @@ async function main(): Promise<void> {
    * the conventional exit code (`128 + signal`).
    */
   const onSignal = (sig: 'SIGINT' | 'SIGTERM'): void => {
-    console.log(`\n[runOfflineFull] received ${sig}, tearing down…`);
+    console.log(`\n[runOfflineTests] received ${sig}, tearing down…`);
     cleanup();
     // 128 + signal number is the POSIX convention.
     process.exit(sig === 'SIGINT' ? 130 : 143);
@@ -1082,15 +1087,15 @@ async function main(): Promise<void> {
     const skipBuildBecauseCache = cachedHash === currentBuildHash;
 
     if (skipBuildBecauseExplicit) {
-      console.log('[runOfflineFull] skipping build (TEST_OFFLINE_SKIP_BUILD=true)');
+      console.log('[runOfflineTests] skipping build (TEST_OFFLINE_SKIP_BUILD=true)');
     } else if (skipBuildBecauseCache) {
       console.log(
-        `[runOfflineFull] build cache HIT (${currentBuildHash.slice(0, 12)}) — skipping npm run build. ` +
+        `[runOfflineTests] build cache HIT (${currentBuildHash.slice(0, 12)}) — skipping npm run build. ` +
           `Force with TEST_OFFLINE_FORCE_BUILD=true.`,
       );
     } else {
       console.log(
-        `[runOfflineFull] build cache MISS — running npm run build. ` +
+        `[runOfflineTests] build cache MISS — running npm run build. ` +
           `(${cachedHash ? `was ${cachedHash.slice(0, 12)}, now ` : 'no cache, '}${currentBuildHash.slice(0, 12)})`,
       );
       execSync('npm run build', {
@@ -1101,7 +1106,7 @@ async function main(): Promise<void> {
       // crashed build leaves stale `build/` we shouldn't endorse.
       writeFileSync(buildCachePath, currentBuildHash);
     }
-    console.log('[runOfflineFull] booting Remix prod server (remix-serve)…');
+    console.log('[runOfflineTests] booting Remix prod server (remix-serve)…');
     // Resolve remix-serve's CLI path directly — `npx remix-serve`
     // on some systems alters NODE_OPTIONS (it spawns a child via
     // npm-cli with PATH manipulation), which silently drops our
@@ -1140,9 +1145,9 @@ async function main(): Promise<void> {
         env: backendRuntimeEnv,
       },
     );
-    console.log(`[runOfflineFull] waiting for backend at ${backendUrl}…`);
+    console.log(`[runOfflineTests] waiting for backend at ${backendUrl}…`);
     await waitForReady(backendUrl, READY_TIMEOUT_MS);
-    console.log('[runOfflineFull] backend ready.');
+    console.log('[runOfflineTests] backend ready.');
 
     // ── Pre-warm Remix routes ──────────────────────────────────
     // remix-serve lazy-compiles each route's loader/action module on
@@ -1162,7 +1167,7 @@ async function main(): Promise<void> {
     //
     // Failures are non-fatal — the next real test request will
     // surface a clearer error than "warm-up failed".
-    console.log('[runOfflineFull] pre-warming hot Remix routes…');
+    console.log('[runOfflineTests] pre-warming hot Remix routes…');
     const warmupToken = await mintIdToken({
       shop: DEFAULT_SHOP_DOMAIN,
       clientId: MOCK_CLIENT_ID,
@@ -1197,7 +1202,7 @@ async function main(): Promise<void> {
               authorization: `Bearer ${warmupToken}`,
               // shopify-app-remix's `isbot` check returns 410 for
               // Node's default User-Agent. Match the same UA the
-              // browser context uses in tests (see offlineFullStack
+              // browser context uses in tests (see offlineStack
               // fixture) so the request reaches the loader.
               'user-agent':
                 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 ' +
@@ -1218,7 +1223,7 @@ async function main(): Promise<void> {
       }),
     );
     console.log(
-      `[runOfflineFull] route pre-warm done (${Date.now() - t0}ms):`,
+      `[runOfflineTests] route pre-warm done (${Date.now() - t0}ms):`,
     );
     for (const r of warmupResults) {
       const status =
@@ -1246,7 +1251,7 @@ async function main(): Promise<void> {
       const adminUrl =
         process.env['TEST_OFFLINE_EXPLORE_URL'] ??
         `https://admin.shopify.com/store/test/apps/${DEFAULT_APP_HANDLE}/app`;
-      console.log('[runOfflineFull] explore mode — launching Chrome at:');
+      console.log('[runOfflineTests] explore mode — launching Chrome at:');
       console.log(`   ${adminUrl}`);
       console.log('');
       console.log('   Useful URLs you can navigate to (all routed via the edge proxy):');
@@ -1356,7 +1361,7 @@ async function main(): Promise<void> {
         await sfPage.goto(storefrontUrl, { waitUntil: 'domcontentloaded' });
       } catch (err) {
         console.error(
-          `[runOfflineFull] explore: storefront tab failed to load (${(err as Error).message}) — continuing`,
+          `[runOfflineTests] explore: storefront tab failed to load (${(err as Error).message}) — continuing`,
         );
       }
       // Leave the admin tab focused as the active one.
@@ -1381,7 +1386,7 @@ async function main(): Promise<void> {
     }
 
     // ── Playwright ─────────────────────────────────────────
-    console.log('[runOfflineFull] running Playwright…');
+    console.log('[runOfflineTests] running Playwright…');
     // Headed-under-Xvfb is the DEFAULT for offline runs (production-
     // faithful, ~4% slower than headless). Opt OUT with
     // TEST_OFFLINE_HEADLESS=true. Full-screen recording (TEST_OFFLINE_VIDEO)
@@ -1406,7 +1411,7 @@ async function main(): Promise<void> {
       }
       if (!hasFfmpeg) {
         console.error(
-          '[runOfflineFull] TEST_OFFLINE_VIDEO set but `ffmpeg` is missing from the ' +
+          '[runOfflineTests] TEST_OFFLINE_VIDEO set but `ffmpeg` is missing from the ' +
             'VM image — rebuild it (it predates video support): ' +
             '`container image rm essential-apps/shopify-test-vm:latest` then re-run. ' +
             'Continuing headed WITHOUT recording.',
@@ -1433,7 +1438,7 @@ async function main(): Promise<void> {
       headedReady = existsSync('/tmp/.X11-unix/X99');
       if (!headedReady) {
         console.error(
-          '[runOfflineFull] WARNING: Xvfb :99 did not start — falling back to headless',
+          '[runOfflineTests] WARNING: Xvfb :99 did not start — falling back to headless',
         );
         videoOn = false;
         headedOn = false;
@@ -1441,7 +1446,7 @@ async function main(): Promise<void> {
         process.env['TEST_OFFLINE_HEADED'] = 'false';
       } else {
         console.error(
-          `[runOfflineFull] headed under Xvfb :99 (${videoOn ? 'recording' : 'no recording'})`,
+          `[runOfflineTests] headed under Xvfb :99 (${videoOn ? 'recording' : 'no recording'})`,
         );
       }
     }
@@ -1487,19 +1492,13 @@ async function main(): Promise<void> {
       // (backendEnv sets it too). Without it the helper's credit comes back
       // test:false and the assertion fails.
       BILLING_IS_TEST: process.env['BILLING_IS_TEST'] ?? 'true',
-      // Point the app's Partner API client at the in-VM partner mock.
-      //
-      // The app fetches `https://partners.shopify.com/<org>/api/<ver>/
-      // graphql.json`; the backend reaches the mock because its preloaded
-      // MSW rewrites that origin. We deliberately do NOT preload MSW here:
-      // MSW globally patches Node networking and corrupts Playwright's
-      // `page.request` (APIRequestContext) — it broke unrelated specs that
-      // hit `page.request.post(...)`. Instead the app's PartnerApiClient
-      // honours this env override for the base origin (a prod-safe test
-      // seam; unset in prod → real URL). The mock serves the identical
-      // `/:org/api/:ver/graphql.json` path, so only the origin changes —
-      // the real client / GraphQL / response-parsing all still run.
-      SHOPIFY_PARTNER_API_BASE_URL: partnerApi.baseUrl,
+      // NB: the app's Partner API client fetches its hard-coded
+      // https://partners.shopify.com endpoint with NO override. That host
+      // is routed to the in-VM partner mock via /etc/hosts → the edge
+      // proxy (see ensureOfflineSetup + edgeProxy's partners.shopify.com
+      // case), so the issue-credit helper's Node fetch reaches the mock
+      // with no app-side change and no MSW preload (which would corrupt
+      // Playwright's page.request).
     };
     // Use async spawn (NOT execSync) — execSync blocks the
     // orchestrator's event loop, so the in-process mock servers
@@ -1526,7 +1525,7 @@ async function main(): Promise<void> {
     // a file path, `--repeat-each`, etc. The docker runner already
     // forwards user args verbatim into this script's argv; we just
     // pass them through to playwright so single-test iteration is
-    // possible (`npm run test:offline-full -- --grep "PDP form"`).
+    // possible (`npm run test:offline -- --grep "PDP form"`).
     // Skip our own known flags (currently none after the launcher
     // strips them — process.argv[0] is `node`, argv[1] is this
     // script).
@@ -1549,9 +1548,9 @@ async function main(): Promise<void> {
     }
     exitCode = 0;
   } catch (err) {
-    console.error(`[runOfflineFull] failed: ${(err as Error).message}`);
+    console.error(`[runOfflineTests] failed: ${(err as Error).message}`);
   } finally {
-    console.log('[runOfflineFull] tearing down…');
+    console.log('[runOfflineTests] tearing down…');
     // `cleanup()` kills the backend AND drops the DB (with
     // preserveDb honored). It's the same function the signal
     // handlers call, so the teardown path is identical whether
@@ -1571,7 +1570,7 @@ async function main(): Promise<void> {
     ]);
 
     if (preserveDb) {
-      console.log(`[runOfflineFull] DB preserved: ${dbName}`);
+      console.log(`[runOfflineTests] DB preserved: ${dbName}`);
     }
   }
 
@@ -1600,28 +1599,28 @@ function loadExtensions(): ExtensionConfig[] {
     parsed = JSON.parse(raw);
   } catch (err) {
     throw new Error(
-      `[runOfflineFull] TEST_OFFLINE_EXTENSIONS_JSON is not valid JSON: ${(err as Error).message}`,
+      `[runOfflineTests] TEST_OFFLINE_EXTENSIONS_JSON is not valid JSON: ${(err as Error).message}`,
     );
   }
   if (!Array.isArray(parsed)) {
     throw new Error(
-      `[runOfflineFull] TEST_OFFLINE_EXTENSIONS_JSON must be a JSON array; got ${typeof parsed}`,
+      `[runOfflineTests] TEST_OFFLINE_EXTENSIONS_JSON must be a JSON array; got ${typeof parsed}`,
     );
   }
   const out: ExtensionConfig[] = [];
   for (const entry of parsed) {
     if (typeof entry !== 'object' || entry === null) {
-      throw new Error(`[runOfflineFull] extension entry must be an object: ${JSON.stringify(entry)}`);
+      throw new Error(`[runOfflineTests] extension entry must be an object: ${JSON.stringify(entry)}`);
     }
     const e = entry as { name?: unknown; rootDir?: unknown; enabled?: unknown };
     if (typeof e.name !== 'string' || typeof e.rootDir !== 'string') {
       throw new Error(
-        `[runOfflineFull] each extension needs { name: string, rootDir: string }; got ${JSON.stringify(entry)}`,
+        `[runOfflineTests] each extension needs { name: string, rootDir: string }; got ${JSON.stringify(entry)}`,
       );
     }
     const rootDir = resolve(process.cwd(), e.rootDir);
     if (!existsSync(rootDir)) {
-      throw new Error(`[runOfflineFull] extension rootDir does not exist: ${rootDir}`);
+      throw new Error(`[runOfflineTests] extension rootDir does not exist: ${rootDir}`);
     }
     out.push({
       name: e.name,
@@ -1631,7 +1630,7 @@ function loadExtensions(): ExtensionConfig[] {
   }
   if (out.length > 0) {
     console.log(
-      `[runOfflineFull] loaded ${out.length} extension(s): ${out.map((e) => `${e.name}=${e.rootDir}`).join(', ')}`,
+      `[runOfflineTests] loaded ${out.length} extension(s): ${out.map((e) => `${e.name}=${e.rootDir}`).join(', ')}`,
     );
   }
   return out;
@@ -1755,7 +1754,7 @@ function seedExploreProducts(state: ShopState): void {
 }
 
 async function runSupervisor(workers: number): Promise<number> {
-  console.log(`[runOfflineFull] supervisor: spawning ${workers} shard(s)…`);
+  console.log(`[runOfflineTests] supervisor: spawning ${workers} shard(s)…`);
 
   // Pre-build: each child would otherwise duplicate the build (or
   // race on the cache). Build here once, then pass SKIP_BUILD=true
@@ -1769,9 +1768,9 @@ async function runSupervisor(workers: number): Promise<number> {
       ? readFileSync(buildCachePath, 'utf8').trim()
       : null;
   if (cachedHash === currentBuildHash) {
-    console.log(`[runOfflineFull] supervisor: build cache HIT (${currentBuildHash.slice(0, 12)})`);
+    console.log(`[runOfflineTests] supervisor: build cache HIT (${currentBuildHash.slice(0, 12)})`);
   } else {
-    console.log(`[runOfflineFull] supervisor: build cache MISS — running npm run build…`);
+    console.log(`[runOfflineTests] supervisor: build cache MISS — running npm run build…`);
     execSync('npm run build', {
       stdio: 'inherit',
       // `TEST_OFFLINE=true` switches the consuming app's
@@ -1807,7 +1806,7 @@ async function runSupervisor(workers: number): Promise<number> {
     children.push(
       new Promise<number>((resolveChild, rejectChild) => {
         // Re-exec ourselves through `node --import tsx` so TS still
-        // parses. argv[1] points at the user's `runOfflineFullTests.ts`
+        // parses. argv[1] points at the user's `runOfflineTests.ts`
         // by the time we're invoked from the consuming app's
         // npm script.
         const proc = spawn(
@@ -1823,7 +1822,7 @@ async function runSupervisor(workers: number): Promise<number> {
   const exitCodes = await Promise.all(children);
   const worst = exitCodes.reduce((a, b) => Math.max(a, b), 0);
   console.log(
-    `[runOfflineFull] supervisor: shard exit codes [${exitCodes.join(', ')}]; overall=${worst}`,
+    `[runOfflineTests] supervisor: shard exit codes [${exitCodes.join(', ')}]; overall=${worst}`,
   );
   return worst;
 }

package/src/scripts/runTests.ts

@@ -164,7 +164,7 @@ async function main(): Promise<void> {
   // from .env.test only.
   const backendEnv: NodeJS.ProcessEnv = {
     PATH: process.env['PATH'] ?? '',
-    // See runOfflineFullTests.ts for the long-form note — empty HOME
+    // See runOfflineTests.ts for the long-form note — empty HOME
     // makes npm write its cache to literal `~/.npm` in cwd.
     HOME: process.env['HOME'] || '/root',
     USER: process.env['USER'] ?? '',

package/src/scripts/runVm.ts

@@ -10,7 +10,7 @@
  *   ─────────                        ────────────
  *   amd64 image (Rosetta)            arm64 image (native)
  *   ~10 s cold boot                  ~5 s cold bake / ~1 s warm restore
- *   ~30 s/test wall-clock            ~3× faster end-to-end on offline-full
+ *   ~30 s/test wall-clock            ~3× faster end-to-end on offline
  *   Google Chrome amd64 in Xvfb      Playwright bundled chromium in
  *                                    Xvfb (patchright stealth still works)
  *
@@ -553,7 +553,7 @@ async function main(): Promise<void> {
     });
 
     // Stream + collect output. Hard 10-min cap by default, with a
-    // 60 s no-output stall guard (mirrors the offline-full bench's
+    // 60 s no-output stall guard (mirrors the offline bench's
     // proven pattern for live processes).
     const maxMs = Number(process.env['TEST_ONLINE_VM_TIMEOUT_MS'] ?? 10 * 60 * 1000);
     // Stall guard: kill the test process if no output for N seconds.

package/src/vite/onlineConfig.ts

@@ -5,7 +5,7 @@ import { loadConfigFromFile, mergeConfig, type ConfigEnv, type UserConfig } from
  * suite — `runTests.ts` boots the dev server with
  * `vite --config <this file>`. It is never referenced by an app's own
  * `vite dev` / `vite build`, so it cannot change any app's production
- * or local-dev behaviour. (Offline-full serves a production build via
+ * or local-dev behaviour. (Offline serves a production build via
  * remix-serve; conformance runs VM probes — neither uses Vite dev, so
  * this is online-only.)
  *

package/dist/scripts/runOfflineFullTests.d.ts

@@ -1,3 +0,0 @@
-#!/usr/bin/env node
-export {};
-//# sourceMappingURL=runOfflineFullTests.d.ts.map

package/dist/scripts/runOfflineFullTests.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"runOfflineFullTests.d.ts","sourceRoot":"","sources":["../../src/scripts/runOfflineFullTests.ts"],"names":[],"mappings":""}