refactor(cli): share browser resize output helper

src/cli/browser-cli-actions-input/register.navigation.ts

@@ -1,11 +1,8 @@
 import type { Command } from "commander";
 import { danger } from "../../globals.js";
 import { defaultRuntime } from "../../runtime.js";
-import {
+import { runBrowserResizeWithOutput } from "../browser-cli-resize.js";
-  callBrowserRequest,
+import { callBrowserRequest, type BrowserParentOpts } from "../browser-cli-shared.js";
-  callBrowserResize,
-  type BrowserParentOpts,
-} from "../browser-cli-shared.js";
 import { requireRef, resolveBrowserActionContext } from "./shared.js";
 
 export function registerBrowserNavigationCommands(
@@ -52,27 +49,16 @@ export function registerBrowserNavigationCommands(
     .option("--target-id <id>", "CDP target id (or unique prefix)")
     .action(async (width: number, height: number, opts, cmd) => {
       const { parent, profile } = resolveBrowserActionContext(cmd, parentOpts);
-      if (!Number.isFinite(width) || !Number.isFinite(height)) {
-        defaultRuntime.error(danger("width and height must be numbers"));
-        defaultRuntime.exit(1);
-        return;
-      }
       try {
-        const result = await callBrowserResize(
+        await runBrowserResizeWithOutput({
           parent,
-          {
+          profile,
-            profile,
+          width,
-            width,
+          height,
-            height,
+          targetId: opts.targetId,
-            targetId: opts.targetId,
+          timeoutMs: 20000,
-          },
+          successMessage: `resized to ${width}x${height}`,
-          { timeoutMs: 20000 },
+        });
-        );
-        if (parent?.json) {
-          defaultRuntime.log(JSON.stringify(result, null, 2));
-          return;
-        }
-        defaultRuntime.log(`resized to ${width}x${height}`);
       } catch (err) {
         defaultRuntime.error(danger(String(err)));
         defaultRuntime.exit(1);

src/cli/browser-cli-resize.ts

@@ -0,0 +1,37 @@
+import { danger } from "../globals.js";
+import { defaultRuntime } from "../runtime.js";
+import { callBrowserResize, type BrowserParentOpts } from "./browser-cli-shared.js";
+
+export async function runBrowserResizeWithOutput(params: {
+  parent: BrowserParentOpts;
+  profile?: string;
+  width: number;
+  height: number;
+  targetId?: string;
+  timeoutMs?: number;
+  successMessage: string;
+}): Promise<void> {
+  const { width, height } = params;
+  if (!Number.isFinite(width) || !Number.isFinite(height)) {
+    defaultRuntime.error(danger("width and height must be numbers"));
+    defaultRuntime.exit(1);
+    return;
+  }
+
+  const result = await callBrowserResize(
+    params.parent,
+    {
+      profile: params.profile,
+      width,
+      height,
+      targetId: params.targetId,
+    },
+    { timeoutMs: params.timeoutMs ?? 20000 },
+  );
+
+  if (params.parent?.json) {
+    defaultRuntime.log(JSON.stringify(result, null, 2));
+    return;
+  }
+  defaultRuntime.log(params.successMessage);
+}

src/cli/browser-cli-state.ts

@@ -2,11 +2,8 @@ import type { Command } from "commander";
 import { danger } from "../globals.js";
 import { defaultRuntime } from "../runtime.js";
 import { parseBooleanValue } from "../utils/boolean.js";
-import {
+import { runBrowserResizeWithOutput } from "./browser-cli-resize.js";
-  callBrowserRequest,
+import { callBrowserRequest, type BrowserParentOpts } from "./browser-cli-shared.js";
-  callBrowserResize,
-  type BrowserParentOpts,
-} from "./browser-cli-shared.js";
 import { registerBrowserCookiesAndStorageCommands } from "./browser-cli-state.cookies-storage.js";
 import { runCommandWithRuntime } from "./cli-utils.js";
 
@@ -39,27 +36,16 @@ export function registerBrowserStateCommands(
     .action(async (width: number, height: number, opts, cmd) => {
       const parent = parentOpts(cmd);
       const profile = parent?.browserProfile;
-      if (!Number.isFinite(width) || !Number.isFinite(height)) {
-        defaultRuntime.error(danger("width and height must be numbers"));
-        defaultRuntime.exit(1);
-        return;
-      }
       await runBrowserCommand(async () => {
-        const result = await callBrowserResize(
+        await runBrowserResizeWithOutput({
           parent,
-          {
+          profile,
-            profile,
+          width,
-            width,
+          height,
-            height,
+          targetId: opts.targetId,
-            targetId: opts.targetId,
+          timeoutMs: 20000,
-          },
+          successMessage: `viewport set: ${width}x${height}`,
-          { timeoutMs: 20000 },
+        });
-        );
-        if (parent?.json) {
-          defaultRuntime.log(JSON.stringify(result, null, 2));
-          return;
-        }
-        defaultRuntime.log(`viewport set: ${width}x${height}`);
       });
     });