chrome-relay 0.5.3 → 0.5.5

package/dist/cli.js

@@ -43,7 +43,7 @@ var RelayError = class extends Error {
 };
 
 // src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.3" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.5" : "0.0.0-dev";
 
 // src/install/install.ts
 import os from "os";
@@ -196,6 +196,20 @@ async function callTool(name, args) {
 
 // src/release-notes.ts
 var RELEASE_NOTES = {
+  "0.5.5": [
+    "BEHAVIOR CHANGE \u2014 `chrome_navigate --new` no longer silently falls back to 'wherever Chrome picks' when an explicit routing intent fails (code-quality-hardening PR 3).",
+    "`navigate --new --tab <id>` where <id> doesn't exist now throws `target_not_found` instead of silently letting Chrome drop the tab in the focused (often the user's) window.",
+    "`navigate --new --group <name>` where the group-join fails now throws `partial_success_disallowed`. The new tab IS created (we don't roll back) so the agent can clean it up; the error details include `createdTabId` and `groupName`.",
+    "Both new strict paths accept `allowPartial: true` as an arg to opt back into the legacy best-effort behavior. With allowPartial, the success result carries `partial: true` and a `warnings[]` array naming what didn't happen.",
+    "Tightened `chrome_navigate` argument errors to RelayError(invalid_arguments) \u2014 missing url, non-numeric tabId."
+  ],
+  "0.5.4": [
+    "Strict target routing (code-quality-hardening PR 2). Within a single scope, --tab / --workspace / --group are mutually exclusive \u2014 passing more than one on the same subcommand (or both at the program level) now fails with `target_conflict` and exit code 2.",
+    "Cross-scope override is still allowed but visible: `chrome-relay --workspace W <cmd> --workspace W2` works, but stderr prints a `target_overridden: workspace W \u2192 W2` notice so the agent (or user) knows what happened.",
+    "Fixed silent drops: `viewport set` and `console` previously hand-rolled their args and ignored global --workspace/--group. They now route through baseArgs() like every other targetable command.",
+    "New target-routing test matrix (55 tests) proves every targetable subcommand forwards --tab, --workspace, and --group correctly \u2014 and that the strict-conflict + override behavior holds. If you add a new targetable command to the CLI, add it to TARGETABLE_COMMANDS in packages/cli/test/target-routing.test.ts.",
+    "New TargetSelector type in @chrome-relay/protocol (future-proofing). Wire still carries the three loose fields; a future PR migrates the extension to read a single structured `target` field."
+  ],
   "0.5.3": [
     "Structured errors and notices (code-quality-hardening PR 1). New `BridgeError` and `BridgeNotice` types in @chrome-relay/protocol carry a code, tool, phase, and details \u2014 agents can branch on `errorDetails.code === 'invalid_arguments'` instead of regex-matching message strings.",
     "Tool result JSON now carries BOTH the legacy fields (`error: string`, `notice: string`) AND the new structured fields (`errorDetails: BridgeError`, `notices: BridgeNotice[]`). Old consumers keep working; new consumers prefer the structured shape.",
@@ -333,15 +347,53 @@ Notes:
     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 parentOpts = program.opts();
     const effectiveWorkspace = opts.workspace ?? parentOpts.workspace;
     const effectiveGroup = opts.group ?? parentOpts.group;
-    if (effectiveWorkspace) args.workspaceName = effectiveWorkspace;
-    if (effectiveGroup) args.groupName = effectiveGroup;
+    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)
+`
+    );
+  }
   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'.
@@ -537,7 +589,7 @@ Notes:
     viewport.command("set").description("Apply explicit viewport dimensions.").requiredOption("--width <px>", "viewport width in CSS pixels", (v) => Number(v)).requiredOption("--height <px>", "viewport height in CSS pixels", (v) => Number(v)).option("--dpr <ratio>", "device pixel ratio (1, 2, 3...)", (v) => Number(v)).option("--mobile", "set the mobile flag (affects meta viewport interpretation)").option("--touch", "enable touch event emulation").option("--user-agent <ua>", "override the User-Agent header")
   ).action(async (opts) => {
     const args = { action: "set", width: opts.width, height: opts.height };
-    if (opts.tab !== void 0) args.tabId = opts.tab;
+    Object.assign(args, baseArgs(opts));
     if (opts.dpr !== void 0) args.dpr = opts.dpr;
     if (opts.mobile) args.mobile = true;
     if (opts.touch) args.hasTouch = true;
@@ -769,8 +821,7 @@ Notes:
 `
     )
   ).action(async (opts) => {
-    const args = {};
-    if (opts.tab !== void 0) args.tabId = opts.tab;
+    const args = baseArgs(opts);
     if (opts.clear) args.action = "clear";
     if (opts.level) args.levels = opts.level;
     if (typeof opts.since === "number") args.since = opts.since;

package/dist/index.js

@@ -1,5 +1,5 @@
 // src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.3" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.5" : "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.3" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.5" : "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.3",
+  "version": "0.5.5",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",