chrome-relay 0.5.10 → 0.5.12

package/dist/cli.js

@@ -9,6 +9,234 @@ var __export = (target, all) => {
     __defProp(target, name, { get: all[name], enumerable: true });
 };
 
+// ../protocol/dist/args/shared.js
+function asObject(input, tool) {
+  if (!input || typeof input !== "object" || Array.isArray(input)) {
+    throw new RelayError({
+      code: "invalid_arguments",
+      message: `${tool}: arguments must be a JSON object.`,
+      tool,
+      phase: "parse_arguments",
+      details: { received: input },
+      retryable: false
+    });
+  }
+  return input;
+}
+function requireString(obj, key, tool) {
+  const v = obj[key];
+  if (typeof v !== "string" || !v) {
+    throw new RelayError({
+      code: "invalid_arguments",
+      message: `${tool}: \`${key}\` is required and must be a non-empty string.`,
+      tool,
+      phase: "parse_arguments",
+      details: { field: key, received: v },
+      retryable: false
+    });
+  }
+  return v;
+}
+function optString(obj, key) {
+  const v = obj[key];
+  return typeof v === "string" && v ? v : void 0;
+}
+function optNumber(obj, key) {
+  const v = obj[key];
+  return typeof v === "number" && Number.isFinite(v) ? v : void 0;
+}
+function optBool(obj, key) {
+  const v = obj[key];
+  return typeof v === "boolean" ? v : void 0;
+}
+function parseTargetArgs(obj) {
+  const out = {};
+  if (typeof obj.tabId === "number")
+    out.tabId = obj.tabId;
+  if (typeof obj.workspaceName === "string" && obj.workspaceName)
+    out.workspaceName = obj.workspaceName;
+  if (typeof obj.groupName === "string" && obj.groupName)
+    out.groupName = obj.groupName;
+  return out;
+}
+var init_shared = __esm({
+  "../protocol/dist/args/shared.js"() {
+    "use strict";
+    init_dist();
+  }
+});
+
+// ../protocol/dist/args/navigate.js
+function parseChromeNavigateArgs(input) {
+  const obj = asObject(input, TOOL_NAMES.NAVIGATE);
+  const out = {
+    url: requireString(obj, "url", TOOL_NAMES.NAVIGATE),
+    ...parseTargetArgs(obj)
+  };
+  if (typeof obj.tabId === "string" && obj.tabId) {
+    const n = Number(obj.tabId);
+    if (Number.isFinite(n))
+      out.tabId = n;
+  } else {
+    const n = optNumber(obj, "tabId");
+    if (n !== void 0)
+      out.tabId = n;
+  }
+  const newTab = optBool(obj, "newTab");
+  if (newTab !== void 0)
+    out.newTab = newTab;
+  const active = optBool(obj, "active");
+  if (active !== void 0)
+    out.active = active;
+  const allowPartial = optBool(obj, "allowPartial");
+  if (allowPartial !== void 0)
+    out.allowPartial = allowPartial;
+  void optString;
+  return out;
+}
+var init_navigate = __esm({
+  "../protocol/dist/args/navigate.js"() {
+    "use strict";
+    init_dist();
+    init_shared();
+  }
+});
+
+// ../protocol/dist/args/hover.js
+function parseChromeHoverArgs(input) {
+  const obj = asObject(input, TOOL_NAMES.HOVER);
+  const target = parseTargetArgs(obj);
+  const x = optNumber(obj, "x");
+  const y = optNumber(obj, "y");
+  if (x !== void 0 && y !== void 0) {
+    return { ...target, kind: "coords", x, y };
+  }
+  const selector = optString(obj, "selector");
+  if (selector) {
+    return { ...target, kind: "selector", selector };
+  }
+  throw new RelayError({
+    code: "invalid_arguments",
+    message: "chrome_hover requires either a selector or x AND y.",
+    tool: TOOL_NAMES.HOVER,
+    phase: "parse_arguments",
+    details: { received: { selector: obj.selector, x: obj.x, y: obj.y } },
+    retryable: false
+  });
+}
+var init_hover = __esm({
+  "../protocol/dist/args/hover.js"() {
+    "use strict";
+    init_dist();
+    init_shared();
+  }
+});
+
+// ../protocol/dist/args/network.js
+function parseFilter(obj) {
+  const out = {};
+  const filter = optString(obj, "filter");
+  if (filter)
+    out.filter = filter;
+  const method = optString(obj, "method");
+  if (method)
+    out.method = method;
+  const limit = optNumber(obj, "limit");
+  if (limit !== void 0)
+    out.limit = limit;
+  const status = obj.status;
+  if (status !== void 0 && status !== null) {
+    if (typeof status !== "string" || !VALID_STATUSES.includes(status)) {
+      throw new RelayError({
+        code: "invalid_arguments",
+        message: `chrome_network: invalid status ${JSON.stringify(status)}. Expected one of: ${VALID_STATUSES.join(", ")}.`,
+        tool: TOOL_NAMES.NETWORK,
+        phase: "parse_status",
+        details: { received: status, validChoices: VALID_STATUSES },
+        retryable: false
+      });
+    }
+    out.status = status;
+  }
+  return out;
+}
+function parseChromeNetworkArgs(input) {
+  const obj = asObject(input, TOOL_NAMES.NETWORK);
+  const target = parseTargetArgs(obj);
+  const rawAction = obj.action;
+  const action = typeof rawAction === "string" ? rawAction : "read";
+  if (action === "clear") {
+    return { ...target, action: "clear" };
+  }
+  if (action === "body") {
+    const requestId = optString(obj, "requestId");
+    if (!requestId) {
+      throw new RelayError({
+        code: "invalid_arguments",
+        message: "chrome_network body requires `requestId` (a non-empty string).",
+        tool: TOOL_NAMES.NETWORK,
+        phase: "parse_arguments",
+        details: { field: "requestId", received: obj.requestId },
+        retryable: false
+      });
+    }
+    const out = {
+      ...target,
+      action: "body",
+      requestId
+    };
+    const full = optBool(obj, "full");
+    if (full !== void 0)
+      out.full = full;
+    const head = optNumber(obj, "head");
+    if (head !== void 0)
+      out.head = head;
+    return out;
+  }
+  if (action === "har") {
+    const withBodies = optBool(obj, "withBodies");
+    const bestEffortBodies = optBool(obj, "bestEffortBodies");
+    return {
+      ...target,
+      action: "har",
+      ...withBodies !== void 0 ? { withBodies } : {},
+      ...bestEffortBodies !== void 0 ? { bestEffortBodies } : {},
+      ...parseFilter(obj)
+    };
+  }
+  if (action === "read") {
+    return { ...target, action: "read", ...parseFilter(obj) };
+  }
+  throw new RelayError({
+    code: "invalid_arguments",
+    message: `chrome_network: unknown action "${action}". Expected read | clear | har | body.`,
+    tool: TOOL_NAMES.NETWORK,
+    phase: "parse_action",
+    details: { received: action, validChoices: ["read", "clear", "har", "body"] },
+    retryable: false
+  });
+}
+var VALID_STATUSES;
+var init_network = __esm({
+  "../protocol/dist/args/network.js"() {
+    "use strict";
+    init_dist();
+    init_shared();
+    VALID_STATUSES = ["ok", "redirect", "client_error", "server_error", "failed"];
+  }
+});
+
+// ../protocol/dist/args/index.js
+var init_args = __esm({
+  "../protocol/dist/args/index.js"() {
+    "use strict";
+    init_shared();
+    init_navigate();
+    init_hover();
+    init_network();
+  }
+});
+
 // ../protocol/dist/index.js
 var dist_exports = {};
 __export(dist_exports, {
@@ -21,6 +249,15 @@ __export(dist_exports, {
   NATIVE_HOST_NAME: () => NATIVE_HOST_NAME,
   RelayError: () => RelayError,
   TOOL_NAMES: () => TOOL_NAMES,
+  asObject: () => asObject,
+  optBool: () => optBool,
+  optNumber: () => optNumber,
+  optString: () => optString,
+  parseChromeHoverArgs: () => parseChromeHoverArgs,
+  parseChromeNavigateArgs: () => parseChromeNavigateArgs,
+  parseChromeNetworkArgs: () => parseChromeNetworkArgs,
+  parseTargetArgs: () => parseTargetArgs,
+  requireString: () => requireString,
   toBridgeError: () => toBridgeError
 });
 function toBridgeError(unknownErr, fallbackTool) {
@@ -39,6 +276,7 @@ var NATIVE_HOST_NAME, DEFAULT_HTTP_PORT, CHROME_WEB_STORE_EXTENSION_ID, LEGACY_D
 var init_dist = __esm({
   "../protocol/dist/index.js"() {
     "use strict";
+    init_args();
     NATIVE_HOST_NAME = "dev.chrome_relay.native_host";
     DEFAULT_HTTP_PORT = 12122;
     CHROME_WEB_STORE_EXTENSION_ID = "cpdiapbifblhlcpnmlmfpgfjlacebokb";
@@ -130,7 +368,7 @@ var init_dist = __esm({
 import { Command } from "commander";
 
 // src/index.ts
-var CHROME_RELAY_VERSION = true ? "0.5.10" : "0.0.0-dev";
+var CHROME_RELAY_VERSION = true ? "0.5.12" : "0.0.0-dev";
 
 // src/commands/shared.ts
 init_dist();
@@ -364,6 +602,16 @@ async function runDoctor() {
 
 // src/release-notes.ts
 var RELEASE_NOTES = {
+  "0.5.12": [
+    "Protocol-owned tool arg parsers (code-quality-hardening Risk 1). New @chrome-relay/protocol exports: `parseChromeNavigateArgs`, `parseChromeHoverArgs`, `parseChromeNetworkArgs`. Each is the single source of truth for what its tool accepts \u2014 CLI and extension consume the same parser so silent shape drift can't happen.",
+    "Pattern established with 3 representative tools (navigate, hover, network \u2014 the doc-followup explicitly named these). Remaining 19 tools are mechanical follow-up (~20 lines + tests each). Each parser throws `RelayError(invalid_arguments)` with field/received/validChoices in details.",
+    "chrome_hover handler refactored to use the new parser end-to-end. Hover args now collapse into a discriminated union (`{kind:'selector', selector}` | `{kind:'coords', x, y}`) so the handler branches without re-doing the typeof dance.",
+    "19 new tests in packages/protocol/test/args.test.ts. Total now 397."
+  ],
+  "0.5.11": [
+    "Tests-only: 6 new edge-case tests for `chrome-relay update`. Covers --dry-run, install failure, install-success-but-binary-version-unchanged (PATH mismatch / stale shim), install-success-but-which-fails, install-success-but-release-notes-parse-fails, and the happy path. Locks in the structured-metadata contract from 0.5.7.",
+    "Total tests now 378 (was 372)."
+  ],
   "0.5.10": [
     "Direct /call target conflict enforcement. Third-party callers posting to /call with multiple loose target fields (tabId + workspaceName, etc.) now throw `target_conflict` instead of silently applying precedence. Matches the CLI rule the CLI itself enforced since 0.5.4.",
     "All useful plain Error throws in extension handlers converted to RelayError(invalid_arguments). Affected tools: chrome_click_element, chrome_fill_or_select, chrome_keyboard, chrome_type, chrome_evaluate, chrome_switch_tab, chrome_close_tabs, chrome_viewport (preset name + width/height), chrome_workspace (create/close), chrome_group (create/close/add/remove), chrome_network (body without --request-id), chrome_hover (no selector or x,y), chrome_click_ax (no --node), and bbox parser. Agents can now branch on `errorDetails.code === 'invalid_arguments'` for all of these.",

package/dist/index.js

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