chrome-relay 0.5.6 → 0.5.8

package/dist/cli.js

@@ -4,6 +4,15 @@
 import { Command } from "commander";
 import { writeFileSync } from "fs";
 
+// src/index.ts
+var CHROME_RELAY_VERSION = true ? "0.5.8" : "0.0.0-dev";
+
+// src/install/install.ts
+import os from "os";
+import path from "path";
+import { chmod, mkdir, readFile, stat, writeFile } from "fs/promises";
+import { fileURLToPath } from "url";
+
 // ../protocol/dist/index.js
 var NATIVE_HOST_NAME = "dev.chrome_relay.native_host";
 var DEFAULT_HTTP_PORT = 12122;
@@ -42,14 +51,7 @@ var RelayError = class extends Error {
   }
 };
 
-// src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.6" : "0.0.0-dev";
-
 // src/install/install.ts
-import os from "os";
-import path from "path";
-import { chmod, mkdir, readFile, stat, writeFile } from "fs/promises";
-import { fileURLToPath } from "url";
 var APP_DIR = path.join(os.homedir(), ".chrome-relay");
 var KNOWN_EXTENSION_IDS = [
   ["Chrome Web Store", CHROME_WEB_STORE_EXTENSION_ID],
@@ -196,6 +198,16 @@ async function callTool(name, args) {
 
 // src/release-notes.ts
 var RELEASE_NOTES = {
+  "0.5.8": [
+    "Internal refactor (code-quality-hardening PR 6, first cut): shared CLI helpers moved out of program.ts into packages/cli/src/commands/shared.ts.",
+    "tabOpt(), makeBaseArgs(program), and runTool() are now importable from `./commands/shared.js`. program.ts dropped ~100 lines.",
+    "No behavior change \u2014 all 355 tests still pass. Future PRs can split per-domain command groups (navigation, input, capture, sessions) into their own modules without churning helpers."
+  ],
+  "0.5.7": [
+    "`chrome-relay update` returns structured verification metadata (code-quality-hardening PR 5). Output now has `install: { attempted, packageManager, status, command }`, `binary: { path, reexeced }`, `releaseNotes: { source: 'current_process' | 'updated_binary', changes }`, and a `warnings[]` array.",
+    "Surfaces the 'install said success but binary didn't change' failure mode (PATH mismatch, stale shim, cross-package-manager confusion) as `warnings[].code === 'update_not_verified'`. Agents can branch on it.",
+    "Falls back gracefully: when the re-exec can't be proven, release notes are read from the current (pre-update) process and marked `source: 'current_process'`. The agent knows the bullets may be stale."
+  ],
   "0.5.6": [
     "BEHAVIOR CHANGE \u2014 `chrome-relay network har --with-bodies` is now strict by default (code-quality-hardening PR 4). When ANY body fails to fetch, the call throws `partial_success_disallowed` with details about which entries failed.",
     "New `--best-effort-bodies` flag restores the legacy behavior: HAR still emits, missing/errored bodies are recorded per-entry in `_chrome_relay.bodyState` and `_chrome_relay.bodyError` (with code, message, and phase).",
@@ -261,6 +273,81 @@ function listReleaseNotesSince(since) {
   return Object.keys(RELEASE_NOTES).filter((v) => compareSemver(v, since) > 0).sort((a, b) => compareSemver(a, b)).map((version) => ({ version, bullets: RELEASE_NOTES[version] }));
 }
 
+// src/commands/shared.ts
+function tabOpt(cmd) {
+  return cmd.option("-t, --tab <id>", "target tab ID", (v) => Number(v)).option("--workspace <name>", "target the active tab in a named workspace window (see `chrome-relay workspace`)").option("--group <name>", "target the active tab in a named tab-group (see `chrome-relay group`)");
+}
+function makeBaseArgs(program) {
+  return function baseArgs(opts) {
+    const parentOpts = program.opts();
+    rejectIntraScopeConflict("subcommand", {
+      tab: opts.tab,
+      workspace: opts.workspace,
+      group: opts.group
+    });
+    rejectIntraScopeConflict("program-level", {
+      workspace: parentOpts.workspace,
+      group: parentOpts.group
+    });
+    if (opts.workspace && parentOpts.workspace && opts.workspace !== parentOpts.workspace) {
+      emitTargetOverride("workspace", parentOpts.workspace, opts.workspace);
+    }
+    if (opts.group && parentOpts.group && opts.group !== parentOpts.group) {
+      emitTargetOverride("group", parentOpts.group, opts.group);
+    }
+    if (opts.tab !== void 0 && (parentOpts.workspace || parentOpts.group)) {
+      const prior = parentOpts.workspace ? `workspace=${parentOpts.workspace}` : `group=${parentOpts.group}`;
+      emitTargetOverride("tab", prior, String(opts.tab));
+    }
+    const args = {};
+    if (opts.tab !== void 0) args.tabId = opts.tab;
+    const effectiveWorkspace = opts.workspace ?? parentOpts.workspace;
+    const effectiveGroup = opts.group ?? parentOpts.group;
+    if (opts.tab === void 0 && effectiveWorkspace) args.workspaceName = effectiveWorkspace;
+    if (opts.tab === void 0 && effectiveGroup) args.groupName = effectiveGroup;
+    return args;
+  };
+}
+function rejectIntraScopeConflict(scope, fields) {
+  const present = [];
+  if (fields.tab !== void 0) present.push("--tab");
+  if (fields.workspace) present.push("--workspace");
+  if (fields.group) present.push("--group");
+  if (present.length > 1) {
+    process.stderr.write(
+      `[chrome-relay] target_conflict: ${scope} flags ${present.join(" + ")} are mutually exclusive. Pass exactly one of --tab, --workspace, or --group on the same ${scope}.
+`
+    );
+    process.exit(2);
+  }
+}
+function emitTargetOverride(kind, from, to) {
+  process.stderr.write(
+    `[chrome-relay] target_overridden: ${kind} ${from} \u2192 ${to} (subcommand-level overrides program-level)
+`
+  );
+}
+async function runTool(name, args) {
+  try {
+    const result = await callTool(name, args);
+    if (typeof result === "string") {
+      process.stdout.write(result + "\n");
+    } else {
+      process.stdout.write(JSON.stringify(result, null, 2) + "\n");
+    }
+  } catch (error) {
+    if (error instanceof RelayError) {
+      process.stderr.write(error.message + "\n");
+      process.stderr.write(JSON.stringify({ relayError: error.toBridgeError() }, null, 2) + "\n");
+    } else {
+      process.stderr.write(
+        (error instanceof Error ? error.message : String(error)) + "\n"
+      );
+    }
+    process.exit(1);
+  }
+}
+
 // src/program.ts
 function buildProgram() {
   const program = new Command();
@@ -294,31 +381,75 @@ Notes:
   program.command("update").description("Update chrome-relay CLI to the latest version and print what changed (agent-readable JSON).").option("--dry-run", "skip the install; just show what changed since the current version").action(async (opts) => {
     const fromVersion = CHROME_RELAY_VERSION;
     const { spawnSync } = await import("child_process");
+    const out = {
+      updatedFrom: fromVersion,
+      updatedTo: fromVersion,
+      install: { attempted: false },
+      binary: { path: process.argv[1] ?? "", reexeced: false },
+      releaseNotes: { source: "current_process", changes: [] },
+      warnings: []
+    };
     if (!opts.dryRun) {
       const argv0 = process.argv[1] ?? "";
       const pm = /[\\/](pnpm|\.pnpm)[\\/]/.test(argv0) ? "pnpm" : /[\\/]bun[\\/]/.test(argv0) ? "bun" : "npm";
       const cmd = pm === "pnpm" ? ["pnpm", ["add", "-g", "chrome-relay@latest"]] : pm === "bun" ? ["bun", ["add", "-g", "chrome-relay@latest"]] : ["npm", ["install", "-g", "chrome-relay@latest"]];
+      out.install = {
+        attempted: true,
+        packageManager: pm,
+        command: `${cmd[0]} ${cmd[1].join(" ")}`
+      };
       process.stderr.write(`[chrome-relay] updating from ${fromVersion} via ${pm}...
 `);
       const install = spawnSync(cmd[0], cmd[1], { stdio: "inherit" });
+      out.install.status = install.status;
       if (install.status !== 0) {
-        process.stderr.write(`[chrome-relay] install failed (${pm} exited ${install.status}). Try manually: ${pm} ${cmd[1].join(" ")}
+        process.stderr.write(`[chrome-relay] install failed (${pm} exited ${install.status}). Try manually: ${cmd[0]} ${cmd[1].join(" ")}
 `);
+        out.warnings.push({
+          code: "update_install_failed",
+          message: `Package-manager exit ${install.status}. Active binary unchanged.`
+        });
+        process.stdout.write(JSON.stringify(out, null, 2) + "\n");
         process.exit(1);
       }
       const which = spawnSync("which", ["chrome-relay"]);
       const newBin = which.stdout?.toString().trim();
-      if (which.status === 0 && newBin && newBin !== argv0) {
-        spawnSync(newBin, ["release-notes", "--since", fromVersion], { stdio: "inherit" });
-        return;
+      if (which.status === 0 && newBin) {
+        const versionOut = spawnSync(newBin, ["--version"]);
+        const newVersion = (versionOut.stdout?.toString() ?? "").trim();
+        out.binary.path = newBin;
+        if (newVersion && newVersion !== fromVersion) {
+          out.updatedTo = newVersion;
+          const rn = spawnSync(newBin, ["release-notes", "--since", fromVersion]);
+          try {
+            const parsed = JSON.parse(rn.stdout?.toString() ?? "");
+            if (Array.isArray(parsed.changes)) {
+              out.releaseNotes = { source: "updated_binary", changes: parsed.changes };
+            }
+          } catch {
+            out.warnings.push({
+              code: "release_notes_parse_failed",
+              message: `Could not parse output of "${newBin} release-notes --since ${fromVersion}".`
+            });
+          }
+          out.binary.reexeced = true;
+        } else {
+          out.warnings.push({
+            code: "update_not_verified",
+            message: `Install completed but \`${newBin} --version\` still reports ${newVersion || "unknown"}. The active binary may not have changed \u2014 check your PATH or run "${cmd[0]} ${cmd[1].join(" ")}" manually and verify.`
+          });
+        }
+      } else {
+        out.warnings.push({
+          code: "update_not_verified",
+          message: `Install completed but \`which chrome-relay\` did not return a path. Could not verify the active binary changed.`
+        });
       }
     }
-    const changes = listReleaseNotesSince(fromVersion);
-    process.stdout.write(JSON.stringify({
-      updatedFrom: fromVersion,
-      updatedTo: CHROME_RELAY_VERSION,
-      changes
-    }, null, 2) + "\n");
+    if (out.releaseNotes.source === "current_process") {
+      out.releaseNotes.changes = listReleaseNotesSince(fromVersion);
+    }
+    process.stdout.write(JSON.stringify(out, null, 2) + "\n");
   });
   program.command("release-notes").description("Print release notes since a version (no install). JSON output for agents.").option("--since <version>", "show release notes for versions newer than this", "0.0.0").action((opts) => {
     const changes = listReleaseNotesSince(opts.since);
@@ -328,77 +459,8 @@ Notes:
       changes
     }, null, 2) + "\n");
   });
-  async function run(name, args) {
-    try {
-      const result = await callTool(name, args);
-      if (typeof result === "string") {
-        process.stdout.write(result + "\n");
-      } else {
-        process.stdout.write(JSON.stringify(result, null, 2) + "\n");
-      }
-    } catch (error) {
-      if (error instanceof RelayError) {
-        process.stderr.write(error.message + "\n");
-        process.stderr.write(JSON.stringify({ relayError: error.toBridgeError() }, null, 2) + "\n");
-      } else {
-        process.stderr.write(
-          (error instanceof Error ? error.message : String(error)) + "\n"
-        );
-      }
-      process.exit(1);
-    }
-  }
-  function tabOpt(cmd) {
-    return cmd.option("-t, --tab <id>", "target tab ID", (v) => Number(v)).option("--workspace <name>", "target the active tab in a named workspace window (see `chrome-relay workspace`)").option("--group <name>", "target the active tab in a named tab-group (see `chrome-relay group`)");
-  }
-  function baseArgs(opts) {
-    const parentOpts = program.opts();
-    rejectIntraScopeConflict("subcommand", {
-      tab: opts.tab,
-      workspace: opts.workspace,
-      group: opts.group
-    });
-    rejectIntraScopeConflict("program-level", {
-      workspace: parentOpts.workspace,
-      group: parentOpts.group
-    });
-    if (opts.workspace && parentOpts.workspace && opts.workspace !== parentOpts.workspace) {
-      emitTargetOverride("workspace", parentOpts.workspace, opts.workspace);
-    }
-    if (opts.group && parentOpts.group && opts.group !== parentOpts.group) {
-      emitTargetOverride("group", parentOpts.group, opts.group);
-    }
-    if (opts.tab !== void 0 && (parentOpts.workspace || parentOpts.group)) {
-      const prior = parentOpts.workspace ? `workspace=${parentOpts.workspace}` : `group=${parentOpts.group}`;
-      emitTargetOverride("tab", prior, String(opts.tab));
-    }
-    const args = {};
-    if (opts.tab !== void 0) args.tabId = opts.tab;
-    const effectiveWorkspace = opts.workspace ?? parentOpts.workspace;
-    const effectiveGroup = opts.group ?? parentOpts.group;
-    if (opts.tab === void 0 && effectiveWorkspace) args.workspaceName = effectiveWorkspace;
-    if (opts.tab === void 0 && effectiveGroup) args.groupName = effectiveGroup;
-    return args;
-  }
-  function rejectIntraScopeConflict(scope, fields) {
-    const present = [];
-    if (fields.tab !== void 0) present.push("--tab");
-    if (fields.workspace) present.push("--workspace");
-    if (fields.group) present.push("--group");
-    if (present.length > 1) {
-      process.stderr.write(
-        `[chrome-relay] target_conflict: ${scope} flags ${present.join(" + ")} are mutually exclusive. Pass exactly one of --tab, --workspace, or --group on the same ${scope}.
-`
-      );
-      process.exit(2);
-    }
-  }
-  function emitTargetOverride(kind, from, to) {
-    process.stderr.write(
-      `[chrome-relay] target_overridden: ${kind} ${from} \u2192 ${to} (subcommand-level overrides program-level)
-`
-    );
-  }
+  const run = runTool;
+  const baseArgs = makeBaseArgs(program);
   program.command("tabs [verb]").description("List open Chrome windows and tabs. (verb 'list' is accepted as alias)").action(async (verb) => {
     if (verb && verb !== "list") {
       process.stderr.write(`unknown tabs verb: ${verb}. Use 'tabs' or 'tabs list'.

package/dist/index.js

@@ -1,5 +1,5 @@
 // src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.6" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.8" : "0.0.0-dev";
 export {
   CHROME_RELAY_VERSION
 };

package/dist/native-host.js

@@ -48,7 +48,7 @@ function toBridgeError(unknownErr, fallbackTool) {
 }
 
 // src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.6" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.8" : "0.0.0-dev";
 
 // src/release-notes.ts
 function compareSemver(a, b) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chrome-relay",
-  "version": "0.5.6",
+  "version": "0.5.8",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",