@happier-dev/stack 0.1.0-preview.17.1 → 0.1.0-preview.21.1

package/scripts/mobile.mjs

@@ -14,6 +14,7 @@ import { expoExec } from './utils/expo/command.mjs';
 import { ensureDevExpoServer } from './utils/dev/expo_dev.mjs';
 import { resolveMobileReachableServerUrl } from './utils/server/mobile_api_url.mjs';
 import { patchIosXcodeProjectsForSigningAndIdentity, resolveIosAppXcodeProjects } from './utils/mobile/ios_xcodeproj_patch.mjs';
+import { pickMetroPort, resolveStablePortStart } from './utils/expo/metro_ports.mjs';
 
 /**
  * Mobile dev helper for the Happier UI Expo app (typically `apps/ui`).
@@ -75,6 +76,7 @@ async function main() {
   const uiRepoDir = getComponentDir(rootDir, 'happier-ui');
   await requireDir('happier-ui', uiRepoDir);
   await ensureDepsInstalled(uiRepoDir, 'happier-ui');
+  const happyDir = uiRepoDir;
 
   // Happy monorepo layouts (historical):
   // - legacy: <happyDir>/expo-app (split-repo era)
@@ -104,7 +106,9 @@ async function main() {
   // (Info.plist includes `dev.happier.app.dev`), so iOS will open the dev build instead of the App Store app.
   const appEnv = process.env.APP_ENV ?? kv.get('--app-env') ?? 'development';
   const host = kv.get('--host') ?? process.env.HAPPIER_STACK_MOBILE_HOST ?? 'lan';
-  const portRaw = kv.get('--port') ?? process.env.HAPPIER_STACK_MOBILE_PORT ?? '8081';
+  const portFromArg = kv.get('--port') ?? '';
+  const portFromEnv = process.env.HAPPIER_STACK_MOBILE_PORT ?? '';
+  const portRaw = portFromArg || portFromEnv || '8081';
   // Default behavior:
   // - `hstack mobile` starts Metro and keeps running.
   // - `hstack mobile --run-ios` / `hstack mobile:ios` just builds/installs and exits (unless --metro is provided).
@@ -126,6 +130,29 @@ async function main() {
   const stackCtx = resolveStackContext({ env, autostart });
   const { stackMode, runtimeStatePath, stackName, envPath } = stackCtx;
 
+  // Expo CLI resolves a Metro port early, but it won't actually bind it until late in the native build.
+  // If we stick to the default 8081, builds are much more likely to fail late if another Metro/Expo claims 8081 mid-build.
+  //
+  // Strategy:
+  // - If the user explicitly sets --port or HAPPIER_STACK_MOBILE_PORT, honor it.
+  // - Otherwise pick a stable, collision-resistant port in a higher range for build-only steps.
+  const needsNativeBuildPort = flags.has('--prebuild') || flags.has('--run-ios');
+  if (needsNativeBuildPort) {
+    const forcedPort = (portFromArg || portFromEnv).toString().trim();
+    const stableKey = (stackMode && stackName ? stackName : '') || iosBundleId || scheme || 'happier';
+    const startPort = resolveStablePortStart({
+      env,
+      stackName: stableKey,
+      baseKey: 'HAPPIER_STACK_MOBILE_BUILD_PORT_BASE',
+      rangeKey: 'HAPPIER_STACK_MOBILE_BUILD_PORT_RANGE',
+      defaultBase: 19000,
+      defaultRange: 10000,
+    });
+    const metroPort = await pickMetroPort({ startPort, forcedPort, host: '127.0.0.1' });
+    env.RCT_METRO_PORT = String(metroPort);
+    env.EXPO_PACKAGER_PORT = String(metroPort);
+  }
+
   // Ensure the built iOS app registers the same scheme we use for dev-client QR links.
   // (Happy app reads EXPO_APP_SCHEME in app.config.js; default remains unchanged when unset.)
   env.EXPO_APP_SCHEME = scheme;
@@ -306,7 +333,8 @@ async function main() {
     }
 
     const configuration = kv.get('--configuration') ?? 'Debug';
-    const args = ['run:ios', '--no-bundler', '--no-build-cache', '--configuration', configuration];
+    const metroPort = String(env.RCT_METRO_PORT ?? portRaw ?? '8081');
+    const args = ['run:ios', '--port', metroPort, '--no-build-cache', '--configuration', configuration];
     if (device) {
       args.push('-d', device);
     }

package/scripts/mobile_dev_client.mjs

@@ -3,11 +3,10 @@ import { parseArgs } from './utils/cli/args.mjs';
 import { printResult, wantsHelp, wantsJson } from './utils/cli/cli.mjs';
 import { run } from './utils/proc/proc.mjs';
 import { getRootDir } from './utils/paths/paths.mjs';
-import { join } from 'node:path';
 import { banner, cmd, sectionTitle } from './utils/ui/layout.mjs';
 import { cyan, dim, yellow } from './utils/ui/ansi.mjs';
 
-import { defaultDevClientIdentity } from './utils/mobile/identifiers.mjs';
+import { buildMobileDevClientInstallInvocation } from './utils/mobile/dev_client_install_invocation.mjs';
 
 async function main() {
   const argv = process.argv.slice(2);
@@ -18,13 +17,13 @@ async function main() {
     printResult({
       json,
       data: {
-        flags: ['--device=<id-or-name>', '--clean', '--configuration=Debug|Release', '--json'],
+        flags: ['--device=<id-or-name>', '--port=<port>', '--clean', '--configuration=Debug|Release', '--json'],
       },
       text: [
         banner('mobile-dev-client', { subtitle: 'Install the shared iOS dev-client app (one-time).' }),
         '',
         sectionTitle('usage:'),
-        `  ${cyan('hstack mobile-dev-client')} --install [--device=...] [--clean] [--configuration=Debug|Release] [--json]`,
+        `  ${cyan('hstack mobile-dev-client')} --install [--device=...] [--port=...] [--clean] [--configuration=Debug|Release] [--json]`,
         '',
         sectionTitle('notes:'),
         `- Installs a dedicated ${cyan('hstack Dev')} Expo dev-client app on your iPhone.`,
@@ -45,39 +44,15 @@ async function main() {
   }
 
   const rootDir = getRootDir(import.meta.url);
-  const mobileScript = join(rootDir, 'scripts', 'mobile.mjs');
-
-  const device = kv.get('--device') ?? '';
-  const clean = flags.has('--clean');
-  const configuration = kv.get('--configuration') ?? 'Debug';
-
-  const id = defaultDevClientIdentity({ user: process.env.USER ?? process.env.USERNAME ?? 'user' });
-
-  const args = [
-    mobileScript,
-    '--app-env=development',
-    `--ios-app-name=${id.iosAppName}`,
-    `--ios-bundle-id=${id.iosBundleId}`,
-    `--scheme=${id.scheme}`,
-    '--prebuild',
-    ...(clean ? ['--clean'] : []),
-    '--run-ios',
-    `--configuration=${configuration}`,
-    '--no-metro',
-    ...(device ? [`--device=${device}`] : []),
-  ];
+  const invocation = buildMobileDevClientInstallInvocation({ rootDir, argv, baseEnv: process.env });
 
   const env = {
-    ...process.env,
-    // Ensure Expo app config uses the dev-client scheme.
-    EXPO_APP_SCHEME: id.scheme,
-    // Ensure per-stack storage isolation is available during dev-client usage.
-    EXPO_PUBLIC_HAPPY_STORAGE_SCOPE: process.env.EXPO_PUBLIC_HAPPY_STORAGE_SCOPE ?? '',
+    ...invocation.env,
   };
 
-  const out = await run(process.execPath, args, { cwd: rootDir, env });
+  const out = await run(process.execPath, invocation.nodeArgs, { cwd: rootDir, env });
   if (json) {
-    printResult({ json, data: { ok: true, installed: true, identity: id, out } });
+    printResult({ json, data: { ok: true, installed: true, identity: invocation.identity, out } });
   }
 }
 

package/scripts/mobile_dev_client_help_smoke.test.mjs

@@ -0,0 +1,24 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { dirname, join } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+import { runNodeCapture } from './testkit/auth_testkit.mjs';
+
+test('mobile-dev-client --help runs without syntax errors', async () => {
+  const scriptsDir = dirname(fileURLToPath(import.meta.url));
+  const rootDir = dirname(scriptsDir);
+  const script = join(rootDir, 'scripts', 'mobile_dev_client.mjs');
+
+  const env = {
+    ...process.env,
+    // Prevent env.mjs from selecting a real stack env file (keeps the test fast and hermetic).
+    HAPPIER_STACK_ENV_FILE: join(rootDir, 'scripts', 'nonexistent-env'),
+  };
+
+  const res = await runNodeCapture([script, '--help'], { cwd: rootDir, env });
+  assert.equal(res.code, 0, `expected exit 0, got ${res.code}\nstderr:\n${res.stderr}\nstdout:\n${res.stdout}`);
+  assert.match(res.stdout, /\bmobile-dev-client\b/, `expected help to mention command name\nstdout:\n${res.stdout}`);
+  assert.match(res.stdout, /--port(?:=|\b)/, `expected help to mention --port\nstdout:\n${res.stdout}`);
+});
+

package/scripts/mobile_prebuild_happyDir_defined.test.mjs

@@ -0,0 +1,47 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdtemp, mkdir, rm } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+import { getStackRootFromMeta, runNodeCapture } from './testkit/auth_testkit.mjs';
+
+test('hstack mobile --prebuild does not crash with undefined happyDir', async () => {
+  const rootDir = getStackRootFromMeta(import.meta.url);
+  const mobileScript = join(rootDir, 'scripts', 'mobile.mjs');
+
+  const tmp = await mkdtemp(join(tmpdir(), 'hstack-mobile-'));
+  const repoDir = join(tmp, 'repo');
+  const storageDir = join(tmp, 'storage');
+
+  try {
+    // Minimal fixture: the script expects a "happier-ui" dir to exist at <repo>/apps/ui.
+    // Avoid creating package.json to keep the run hermetic (ensureDepsInstalled will no-op).
+    await mkdir(join(repoDir, 'apps', 'ui'), { recursive: true });
+
+    // Ensure stack storage stays within this tmp dir (avoid touching real ~/.happier paths).
+    await mkdir(join(storageDir, 'main'), { recursive: true });
+
+    const env = {
+      ...process.env,
+      HAPPIER_STACK_REPO_DIR: repoDir,
+      HAPPIER_STACK_HOME_DIR: join(tmp, 'home'),
+      HAPPIER_STACK_STORAGE_DIR: storageDir,
+      HAPPIER_STACK_STACK: 'main',
+      // Keep resolveServerUrls from probing tailscale in tests.
+      HAPPIER_STACK_TAILSCALE_PREFER_PUBLIC_URL: '0',
+      HAPPIER_STACK_TAILSCALE_SERVE: '0',
+      // Prevent env auto-loading from a real machine stack env file.
+      HAPPIER_STACK_ENV_FILE: join(tmp, 'nonexistent-env'),
+    };
+
+    const res = await runNodeCapture([mobileScript, '--prebuild', '--no-metro'], { cwd: rootDir, env });
+    assert.doesNotMatch(
+      res.stderr,
+      /\bhappyDir is not defined\b/,
+      `expected prebuild to fail for a real reason (e.g. missing expo), not a ReferenceError\nstderr:\n${res.stderr}\nstdout:\n${res.stdout}`
+    );
+  } finally {
+    await rm(tmp, { recursive: true, force: true });
+  }
+});

package/scripts/mobile_prebuild_sets_rct_metro_port.test.mjs

@@ -0,0 +1,81 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { chmod, mkdtemp, mkdir, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+import { getStackRootFromMeta, runNodeCapture } from './testkit/auth_testkit.mjs';
+
+function parseKeyValueLines(text) {
+  const out = {};
+  for (const line of String(text ?? '').split(/\r?\n/)) {
+    const idx = line.indexOf('=');
+    if (idx === -1) continue;
+    const key = line.slice(0, idx).trim();
+    const value = line.slice(idx + 1).trim();
+    if (key) out[key] = value;
+  }
+  return out;
+}
+
+test('hstack mobile --prebuild sets RCT_METRO_PORT for native build steps', async () => {
+  const rootDir = getStackRootFromMeta(import.meta.url);
+  const mobileScript = join(rootDir, 'scripts', 'mobile.mjs');
+
+  const tmp = await mkdtemp(join(tmpdir(), 'hstack-mobile-port-'));
+  const repoDir = join(tmp, 'repo');
+  const storageDir = join(tmp, 'storage');
+
+  try {
+    const uiDir = join(repoDir, 'apps', 'ui');
+    const expoBin = join(uiDir, 'node_modules', '.bin', 'expo');
+    await mkdir(join(uiDir, 'node_modules', '.bin'), { recursive: true });
+
+    // Stub Expo CLI so the test doesn't require installing or running the real toolchain.
+    await writeFile(
+      expoBin,
+      `#!/usr/bin/env node
+console.log('RCT_METRO_PORT=' + (process.env.RCT_METRO_PORT ?? ''));
+console.log('EXPO_PACKAGER_PORT=' + (process.env.EXPO_PACKAGER_PORT ?? ''));
+process.exit(0);
+`,
+      'utf-8'
+    );
+    if (process.platform !== 'win32') {
+      await chmod(expoBin, 0o755);
+    }
+
+    // Ensure stack storage stays within this tmp dir (avoid touching real ~/.happier paths).
+    await mkdir(join(storageDir, 'main'), { recursive: true });
+
+    const env = {
+      ...process.env,
+      HAPPIER_STACK_REPO_DIR: repoDir,
+      HAPPIER_STACK_HOME_DIR: join(tmp, 'home'),
+      HAPPIER_STACK_STORAGE_DIR: storageDir,
+      HAPPIER_STACK_STACK: 'main',
+      // Keep resolveServerUrls from probing tailscale in tests.
+      HAPPIER_STACK_TAILSCALE_PREFER_PUBLIC_URL: '0',
+      HAPPIER_STACK_TAILSCALE_SERVE: '0',
+      // Prevent env auto-loading from a real machine stack env file.
+      HAPPIER_STACK_ENV_FILE: join(tmp, 'nonexistent-env'),
+    };
+
+    const res = await runNodeCapture([mobileScript, '--prebuild', '--platform=android', '--no-metro'], { cwd: rootDir, env });
+    const kv = parseKeyValueLines(res.stdout);
+    assert.ok(kv.RCT_METRO_PORT, `expected stub expo to print RCT_METRO_PORT\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.equal(
+      kv.EXPO_PACKAGER_PORT,
+      kv.RCT_METRO_PORT,
+      `expected EXPO_PACKAGER_PORT to match RCT_METRO_PORT\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`
+    );
+    assert.notEqual(
+      kv.RCT_METRO_PORT,
+      '8081',
+      `expected native build steps to avoid default Metro port 8081 (more collision-prone)\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`
+    );
+  } finally {
+    await rm(tmp, { recursive: true, force: true });
+  }
+});
+

package/scripts/mobile_run_ios_passes_port.integration.test.mjs

@@ -0,0 +1,103 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { chmod, mkdtemp, mkdir, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+import { getStackRootFromMeta, runNodeCapture } from './testkit/auth_testkit.mjs';
+
+function parseKeyValueLines(text) {
+  const out = {};
+  for (const line of String(text ?? '').split(/\r?\n/)) {
+    const idx = line.indexOf('=');
+    if (idx === -1) continue;
+    const key = line.slice(0, idx).trim();
+    const value = line.slice(idx + 1).trim();
+    if (key) out[key] = value;
+  }
+  return out;
+}
+
+test('hstack mobile --run-ios passes --port to Expo so the native build and dev server use the same Metro port', async () => {
+  const rootDir = getStackRootFromMeta(import.meta.url);
+  const mobileScript = join(rootDir, 'scripts', 'mobile.mjs');
+
+  const tmp = await mkdtemp(join(tmpdir(), 'hstack-mobile-runios-port-'));
+  const repoDir = join(tmp, 'repo');
+  const storageDir = join(tmp, 'storage');
+
+  try {
+    const binDir = join(tmp, 'bin');
+    await mkdir(binDir, { recursive: true });
+    const xcrunStub = join(binDir, 'xcrun');
+    await writeFile(
+      xcrunStub,
+      `#!/bin/bash
+set -euo pipefail
+if [[ "\${1:-}" == "xcdevice" && "\${2:-}" == "list" ]]; then
+  echo "[]"
+  exit 0
+fi
+echo "xcrun stub: unsupported args: $*" >&2
+exit 1
+`,
+      'utf-8'
+    );
+    if (process.platform !== 'win32') {
+      await chmod(xcrunStub, 0o755);
+    }
+
+    const uiDir = join(repoDir, 'apps', 'ui');
+    const expoBin = join(uiDir, 'node_modules', '.bin', 'expo');
+    await mkdir(join(uiDir, 'node_modules', '.bin'), { recursive: true });
+
+    // Stub Expo CLI: print argv + env-derived ports and exit successfully.
+    // Use an absolute node shebang so the test can safely clear PATH.
+    await writeFile(
+      expoBin,
+      `#!${process.execPath}
+console.log('ARGS=' + JSON.stringify(process.argv.slice(2)));
+console.log('RCT_METRO_PORT=' + (process.env.RCT_METRO_PORT ?? ''));
+console.log('EXPO_PACKAGER_PORT=' + (process.env.EXPO_PACKAGER_PORT ?? ''));
+process.exit(0);
+`,
+      'utf-8'
+    );
+    if (process.platform !== 'win32') {
+      await chmod(expoBin, 0o755);
+    }
+
+    await mkdir(join(storageDir, 'main'), { recursive: true });
+
+    const env = {
+      ...process.env,
+      // Ensure xcrun runs fast/deterministically in tests.
+      // Include system paths so env.mjs won't prepend /usr/bin ahead of our stub.
+      PATH: `${binDir}:/usr/bin:/bin`,
+      HAPPIER_STACK_REPO_DIR: repoDir,
+      HAPPIER_STACK_HOME_DIR: join(tmp, 'home'),
+      HAPPIER_STACK_STORAGE_DIR: storageDir,
+      HAPPIER_STACK_STACK: 'main',
+      HAPPIER_STACK_TAILSCALE_PREFER_PUBLIC_URL: '0',
+      HAPPIER_STACK_TAILSCALE_SERVE: '0',
+      HAPPIER_STACK_ENV_FILE: join(tmp, 'nonexistent-env'),
+    };
+
+    const res = await runNodeCapture([mobileScript, '--run-ios', '--no-metro'], { cwd: rootDir, env });
+    const kv = parseKeyValueLines(res.stdout);
+
+    const args = JSON.parse(kv.ARGS ?? '[]');
+    const port = kv.RCT_METRO_PORT ?? '';
+
+    assert.ok(port, `expected RCT_METRO_PORT to be set\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.equal(kv.EXPO_PACKAGER_PORT, port, `expected EXPO_PACKAGER_PORT to match\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.ok(!args.includes('-p'), `expected expo args to not include -p\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.ok(!args.includes('--no-bundler'), `expected expo args to not include --no-bundler\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.ok(args.includes('--port'), `expected expo args to include --port\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    const portIdx = args.indexOf('--port');
+    assert.ok(portIdx >= 0 && args[portIdx + 1] === port, `expected --port to match RCT_METRO_PORT\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.notEqual(port, '8081', `expected non-default port to reduce collisions\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+  } finally {
+    await rm(tmp, { recursive: true, force: true });
+  }
+});

package/scripts/mobile_run_ios_uses_long_port_flag.test.mjs

@@ -0,0 +1,106 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { chmod, mkdtemp, mkdir, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+
+import { getStackRootFromMeta, runNodeCapture } from './testkit/auth_testkit.mjs';
+
+function parseKeyValueLines(text) {
+  const out = {};
+  for (const line of String(text ?? '').split(/\r?\n/)) {
+    const idx = line.indexOf('=');
+    if (idx === -1) continue;
+    const key = line.slice(0, idx).trim();
+    const value = line.slice(idx + 1).trim();
+    if (key) out[key] = value;
+  }
+  return out;
+}
+
+test('hstack mobile --run-ios passes --port (long flag) so Expo uses the requested Metro port', async () => {
+  const rootDir = getStackRootFromMeta(import.meta.url);
+  const mobileScript = join(rootDir, 'scripts', 'mobile.mjs');
+
+  const tmp = await mkdtemp(join(tmpdir(), 'hstack-mobile-runios-longport-'));
+  const repoDir = join(tmp, 'repo');
+  const storageDir = join(tmp, 'storage');
+
+  try {
+    const binDir = join(tmp, 'bin');
+    await mkdir(binDir, { recursive: true });
+    const xcrunStub = join(binDir, 'xcrun');
+    await writeFile(
+      xcrunStub,
+      `#!/bin/bash
+set -euo pipefail
+if [[ "\${1:-}" == "xcdevice" && "\${2:-}" == "list" ]]; then
+  echo "[]"
+  exit 0
+fi
+echo "xcrun stub: unsupported args: $*" >&2
+exit 1
+`,
+      'utf-8'
+    );
+    if (process.platform !== 'win32') {
+      await chmod(xcrunStub, 0o755);
+    }
+
+    const uiDir = join(repoDir, 'apps', 'ui');
+    const expoBin = join(uiDir, 'node_modules', '.bin', 'expo');
+    await mkdir(join(uiDir, 'node_modules', '.bin'), { recursive: true });
+
+    await writeFile(
+      expoBin,
+      `#!${process.execPath}
+console.log('ARGS=' + JSON.stringify(process.argv.slice(2)));
+console.log('RCT_METRO_PORT=' + (process.env.RCT_METRO_PORT ?? ''));
+console.log('EXPO_PACKAGER_PORT=' + (process.env.EXPO_PACKAGER_PORT ?? ''));
+process.exit(0);
+`,
+      'utf-8'
+    );
+    if (process.platform !== 'win32') {
+      await chmod(expoBin, 0o755);
+    }
+
+    await mkdir(join(storageDir, 'main'), { recursive: true });
+
+    const env = {
+      ...process.env,
+      // Ensure our xcrun stub wins over /usr/bin/xcrun even after env.mjs normalizes PATH.
+      PATH: `${binDir}:/usr/bin:/bin`,
+      HAPPIER_STACK_REPO_DIR: repoDir,
+      HAPPIER_STACK_HOME_DIR: join(tmp, 'home'),
+      HAPPIER_STACK_STORAGE_DIR: storageDir,
+      HAPPIER_STACK_STACK: 'main',
+      HAPPIER_STACK_TAILSCALE_PREFER_PUBLIC_URL: '0',
+      HAPPIER_STACK_TAILSCALE_SERVE: '0',
+      HAPPIER_STACK_ENV_FILE: join(tmp, 'nonexistent-env'),
+    };
+
+    const res = await runNodeCapture([mobileScript, '--run-ios', '--no-metro', '--port=14362'], { cwd: rootDir, env });
+    assert.equal(res.code, 0, `expected exit 0, got ${res.code}\nstderr:\n${res.stderr}\nstdout:\n${res.stdout}`);
+    const kv = parseKeyValueLines(res.stdout);
+    const args = JSON.parse(kv.ARGS ?? '[]');
+
+    assert.ok(!args.includes('--no-bundler'), `expected expo args to not include --no-bundler\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.ok(args.includes('--port'), `expected expo args to include --port\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.ok(!args.includes('-p'), `expected expo args to not include -p\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    const portIdx = args.indexOf('--port');
+    assert.ok(portIdx >= 0 && args[portIdx + 1] === '14362', `expected --port 14362\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`);
+    assert.equal(
+      kv.RCT_METRO_PORT,
+      '14362',
+      `expected RCT_METRO_PORT to be set from hstack --port\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`
+    );
+    assert.equal(
+      kv.EXPO_PACKAGER_PORT,
+      '14362',
+      `expected EXPO_PACKAGER_PORT to match\nstdout:\n${res.stdout}\nstderr:\n${res.stderr}`
+    );
+  } finally {
+    await rm(tmp, { recursive: true, force: true });
+  }
+});