chore: remove support for custom host parameter

Author: juliandescottes

src/cli.ts

@@ -110,13 +110,6 @@ export const cliOptions = {
     description: 'Marionette port to connect to when using --connect-existing (default: 2828)',
     default: Number(process.env.MARIONETTE_PORT ?? '2828'),
   },
-  marionetteHost: {
-    type: 'string',
-    description:
-      'Marionette host to connect to when using --connect-existing (default: 127.0.0.1).' +
-      ' Also used as the BiDi WebSocket connect address when different from 127.0.0.1.',
-    default: process.env.MARIONETTE_HOST ?? '127.0.0.1',
-  },
   env: {
     type: 'array',
     description:

src/firefox/core.ts

@@ -155,7 +155,6 @@ export class FirefoxCore {
 
     if (this.options.connectExisting) {
       const port = this.options.marionettePort ?? 2828;
-      const host = this.options.marionetteHost ?? '127.0.0.1';
 
       // Find geckodriver binary (--driver avoids downloading Firefox via selenium-manager)
       const geckodriverPath = await findGeckodriver();
@@ -180,20 +179,6 @@ export class FirefoxCore {
       // which would otherwise invoke selenium-manager with --browser firefox.
       const seleniumDriver = firefox.Driver.createSession(caps, serviceBuilder.build());
       this.driver = seleniumDriver as unknown as IDriver;
-
-      // For remote Firefox, rewrite webSocketUrl hostname in the session capabilities.
-      // Selenium only rewrites localhost→127.0.0.1; arbitrary hosts need explicit patching.
-      // Session.getCapabilities() returns the same Capabilities object every time, so
-      // calling caps.set() here is seen by the lazy getBidi() call that comes later.
-      if (host !== '127.0.0.1') {
-        const caps = await seleniumDriver.getCapabilities();
-        const wsUrl = caps.get('webSocketUrl') as string | undefined;
-        if (wsUrl) {
-          const parsed = new URL(wsUrl);
-          parsed.hostname = host;
-          caps.set('webSocketUrl', parsed.toString());
-        }
-      }
     } else {
       // Set up output file for capturing Firefox stdout/stderr
       if (this.options.logFile) {

src/firefox/types.ts

@@ -60,7 +60,6 @@ export interface FirefoxLaunchOptions {
   acceptInsecureCerts?: boolean | undefined;
   connectExisting?: boolean | undefined;
   marionettePort?: number | undefined;
-  marionetteHost?: string | undefined;
   env?: Record<string, string> | undefined;
   logFile?: string | undefined;
   /** Firefox preferences to set at startup via moz:firefoxOptions */

src/index.ts

@@ -141,7 +141,6 @@ export async function getFirefox(): Promise<FirefoxDevTools> {
       acceptInsecureCerts: args.acceptInsecureCerts,
       connectExisting: args.connectExisting,
       marionettePort: args.marionettePort,
-      marionetteHost: args.marionetteHost,
       env: envVars,
       logFile: args.outputFile ?? undefined,
       prefs,

tests/firefox/connect-existing.test.ts

@@ -4,20 +4,6 @@
 
 import { describe, it, expect, vi } from 'vitest';
 
-describe('FirefoxCore connect-existing with marionetteHost', () => {
-  it('should pass marionetteHost to options', async () => {
-    const { FirefoxCore } = await import('@/firefox/core.js');
-    const core = new FirefoxCore({
-      headless: true,
-      connectExisting: true,
-      marionettePort: 2828,
-      marionetteHost: '192.168.1.100',
-    });
-
-    expect(core.getOptions().marionetteHost).toBe('192.168.1.100');
-  });
-});
-
 describe('getFirefox() reconnect behavior', () => {
   it('should reconnect when connection is lost instead of throwing', async () => {
     vi.resetModules();