chrome-relay 0.5.7 → 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.7" : "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,11 @@ 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.",
@@ -266,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();
@@ -377,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.7" : "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.7" : "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.7",
+  "version": "0.5.8",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",