rn-iso 0.4.4 → 0.4.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rn-iso",
-  "version": "0.4.4",
+  "version": "0.4.6",
   "description": "Isolated React Native dev environments per project/worktree",
   "type": "module",
   "bin": {

package/skill/SKILL.md

@@ -30,7 +30,7 @@ From the project root (or any subdirectory):
 ## CRITICAL rules
 
 - **Pass `--auto` for non-interactive use** of `ios` or `android`. Without it, the command will prompt with an arrow-key picker if multiple unclaimed sims/AVDs exist. `--auto` is also implied automatically when stdin isn't a TTY (e.g., when an agent pipes the command), so under most agent harnesses you don't have to remember the flag — but passing it explicitly is harmless and clearer.
-- **Forward extra flags to the build CLI with `--`.** `npx rn-iso ios -- --variant=release` (or `android -- --mode=diaRelease`) appends those flags to the underlying `react-native run-*` / `expo run:*` invocation. Useful for release-mode builds, custom terminals, etc. Last-wins semantics, so extras can override defaults rn-iso set earlier in the command.
+- **Forward extra flags to the build CLI with `--`.** `npx rn-iso ios -- --variant=release` (or `android -- --mode=diaRelease`) appends those flags to the underlying `react-native run-*` / `expo run:*` invocation. Useful for release-mode builds, custom terminals, etc. Last-wins semantics, so extras can override defaults rn-iso set earlier in the command. `start` accepts the same `--` extras and forwards them to `expo start` / `react-native start` — e.g. `npx rn-iso start -- --reset-cache`. If Metro is already running, extras are not applied (run `rn-iso stop` first and re-run).
 - **`--auto` will NOT take over a claimed sim/AVD.** If every device is claimed by other rn-iso projects, `--auto` errors. To take one over, run the command interactively (no `--auto`, with a real TTY) and confirm at the prompt — only do this if the user explicitly asks.
 - **Always use `npx rn-iso device` to discover your target.** Never assume `booted` is your sim — another project's simulator might be booted too.
 - **Always pass the UDID/serial explicitly** to `xcrun simctl` and `adb -s`. Examples:
@@ -81,7 +81,7 @@ Reserve binds the sim to the current project the same way `ios` does, but skips
 ## Other useful commands
 
 - `npx rn-iso status` — show all projects, their assignments, and Metro state.
-- `npx rn-iso start` — start Metro detached on the project's assigned port WITHOUT building/installing. Useful to keep Metro alive across builds.
+- `npx rn-iso start [-- <extras...>]` — start Metro detached on the project's assigned port WITHOUT building/installing. Useful to keep Metro alive across builds. Extras after `--` are forwarded to `expo start` / `react-native start` (e.g. `--reset-cache`).
 - `npx rn-iso stop [<port>|<shortcut>|<path>]` — kill Metro. No arg = current project. Passing a port (e.g. `8083`) kills whatever is on it; a project shortcut (label or unique basename) or absolute path targets that project. Finds the process by port, so it works whether Metro was started by `npx rn-iso start` or by the build CLI.
 - `npx rn-iso release [<port>|<shortcut>|<path>] [--platform <p>] [--shutdown]` — free a project's sim assignment. Defaults to the current project. Target can also be a Metro port (`8083`) or a shortcut (label / unique basename). `--shutdown` also stops the sim/emulator.
 - `npx rn-iso shutdown [<shortcut>|<path>] [-y] [--keep-sims]` — kill Metro, shut down sims/emulators, and clear device assignments. With no arg, scopes to **every** registered project (end-of-day reset); pass a project shortcut (label or unique basename) or absolute path to scope to one. Note this does NOT default to the current project (deliberate — `shutdown` is the explicit "tear it all down" command). Prompts unless `-y` / non-TTY; `--keep-sims` only kills Metro and clears assignments without touching the sims. Project entries themselves stay registered, so `metroPort` allocations and labels survive.

package/src/commands/start.js

@@ -8,8 +8,9 @@ import { ensureMetro } from '../metro.js';
 export default function startCommand(program) {
   program
     .command('start')
-    .description('Ensure Metro is running for the current project (no platform action)')
-    .action(async () => {
+    .description('Ensure Metro is running for the current project (no platform action). Pass extra flags to `expo start` / `react-native start` after `--`, e.g. `rn-iso start -- --reset-cache`.')
+    .argument('[extras...]', 'Flags forwarded as-is to expo/react-native start (after `--`)')
+    .action(async (extras) => {
       const root = findProjectRoot(process.cwd());
       if (!root) {
         console.error(chalk.red('Not in a React Native project (no package.json found).'));
@@ -32,9 +33,16 @@ export default function startCommand(program) {
         proj = getProject(root);
       }
 
-      const metro = await ensureMetro({ projectPath: root, isExpo, port: proj.metroPort });
+      const metro = await ensureMetro({ projectPath: root, isExpo, port: proj.metroPort, extras });
       if (metro.alreadyRunning) {
-        console.log(chalk.dim(`Metro already running on port ${proj.metroPort}`));
+        if (extras?.length) {
+          console.log(chalk.yellow(
+            `Metro already running on port ${proj.metroPort}; extras (${extras.join(' ')}) were not applied. ` +
+            `Run \`rn-iso stop\` first and re-run to apply them.`
+          ));
+        } else {
+          console.log(chalk.dim(`Metro already running on port ${proj.metroPort}`));
+        }
       } else {
         setMetro(root, proj.metroPort, metro.pid);
         console.log(chalk.green(`Metro started (pid ${metro.pid}, port ${proj.metroPort})`));

package/src/metro.js

@@ -15,22 +15,23 @@ export function logFileFor(projectPath) {
   return join(dir, `${projectHash(projectPath)}.log`);
 }
 
-export function buildMetroSpawnArgs({ isExpo, port }) {
+export function buildMetroSpawnArgs({ isExpo, port, extras = [] }) {
+  const base = isExpo
+    ? ['expo', 'start', '--port', String(port)]
+    : ['react-native', 'start', '--port', String(port)];
   return {
     cmd: 'npx',
-    args: isExpo
-      ? ['expo', 'start', '--port', String(port)]
-      : ['react-native', 'start', '--port', String(port)],
+    args: [...base, ...extras],
   };
 }
 
-export async function ensureMetro({ projectPath, isExpo, port, detach = true }) {
+export async function ensureMetro({ projectPath, isExpo, port, extras = [], detach = true }) {
   if (await isMetroRunning(port)) return { alreadyRunning: true, pid: null };
 
   const log = logFileFor(projectPath);
   const fd = openSync(log, 'a');
 
-  const { cmd, args } = buildMetroSpawnArgs({ isExpo, port });
+  const { cmd, args } = buildMetroSpawnArgs({ isExpo, port, extras });
   const exec = getExecutor();
   const child = exec.spawn(cmd, args, {
     cwd: projectPath,

package/src/runner.js

@@ -66,7 +66,7 @@ export function buildScriptCommand(packageManager, scriptName, extraArgs = []) {
 
 // Decide which CLI a script invokes. Affects flag names:
 //   iOS:     Expo `--device <UDID>`     | RN `--udid <UDID>`
-//   Android: Expo `--device <AVD-name>` | RN `--deviceId <serial>`
+//   Android: Expo `--device <AVD-name>` | RN `--device <serial>`
 // Expo's run:android resolves --device by name (not by serial), so we pass
 // the AVD name there even though we boot/track by serial.
 export function detectScriptCli(scriptBody) {
@@ -113,8 +113,8 @@ export function buildAndroidCommand({ projectRoot, packageManager, scriptName, i
     const script = getProjectScript(projectRoot, scriptName);
     if (script) {
       const cli = detectScriptCli(script);
-      // Expo: --device <AVD name | serial>; RN: --deviceId <serial>.
-      const deviceFlag = cli === 'expo' ? `--device "${expoDeviceArg}"` : `--deviceId ${serial}`;
+      // Expo: --device <AVD name | serial>; RN: --device <serial>.
+      const deviceFlag = cli === 'expo' ? `--device "${expoDeviceArg}"` : `--device ${serial}`;
       return buildScriptCommand(packageManager, scriptName, [
         deviceFlag,
         `--port ${port}`,
@@ -126,7 +126,7 @@ export function buildAndroidCommand({ projectRoot, packageManager, scriptName, i
   if (isExpo) {
     return `npx expo run:android --device "${expoDeviceArg}" --port ${port}${tailStr}`;
   }
-  return `RCT_METRO_PORT=${port} npx react-native run-android --deviceId ${serial}${tailStr}`;
+  return `RCT_METRO_PORT=${port} npx react-native run-android --device ${serial}${tailStr}`;
 }
 
 // POSIX-safe single-quote shell escape. Leaves "safe" tokens (alnum and a