♻️ Remove unused remote browser code from Ember SDK

Removes browserWSEndpoint, connectOverCDP, and rewriteUrlForDocker.

These were for a remote browser approach that we're not using.
The Docker workflow runs everything in the container, so we only
need VIZZLY_SERVER_URL env var support (already in place).

Author: Robdel12

clients/ember/bin/vizzly-testem-launcher.js

@@ -33,12 +33,6 @@ let args = process.argv.slice(2);
 let browserType = args[0];
 let testUrl = args[args.length - 1];
 
-// If using remote browser (Docker), rewrite localhost to host.docker.internal
-// so the containerized browser can reach the host's test server
-function rewriteUrlForDocker(url) {
-  return url.replace(/localhost|127\.0\.0\.1/g, 'host.docker.internal');
-}
-
 // Validate arguments
 if (!browserType || !testUrl || !testUrl.startsWith('http')) {
   console.error('Usage: vizzly-testem-launcher <browser> <url>');
@@ -49,14 +43,10 @@ if (!browserType || !testUrl || !testUrl.startsWith('http')) {
 
 // Read Playwright launch options from config file (written by configure())
 let playwrightOptions = { headless: true }; // Default to headless
-let browserWSEndpoint = null;
 let configPath = join(process.cwd(), '.vizzly', 'playwright.json');
 if (existsSync(configPath)) {
   try {
     let config = JSON.parse(readFileSync(configPath, 'utf8'));
-    // Extract browserWSEndpoint separately (not a Playwright launch option)
-    browserWSEndpoint = config.browserWSEndpoint;
-    delete config.browserWSEndpoint;
     playwrightOptions = {
       headless: true, // Default
       ...config,
@@ -122,19 +112,10 @@ async function main() {
       }
     }
 
-    // 3. Launch browser with Playwright (or connect to remote browser)
-    // If remote browser, rewrite URL so container can reach host
-    let actualTestUrl = browserWSEndpoint ? rewriteUrlForDocker(testUrl) : testUrl;
-
-    if (browserWSEndpoint) {
-      console.log(`[vizzly] Connecting to remote browser: ${browserWSEndpoint}`);
-      console.log(`[vizzly] Test URL rewritten: ${testUrl} -> ${actualTestUrl}`);
-    }
-
-    browserInstance = await launchBrowser(browserType, actualTestUrl, {
+    // 3. Launch browser with Playwright
+    browserInstance = await launchBrowser(browserType, testUrl, {
       screenshotUrl,
       failOnDiff,
-      browserWSEndpoint,
       playwrightOptions,
       onPageCreated: page => {
         setPage(page);

clients/ember/src/launcher/browser.js

@@ -61,7 +61,6 @@ function getDefaultChromiumArgs() {
  * @param {Object} options - Launch options
  * @param {string} options.screenshotUrl - URL of the screenshot HTTP server
  * @param {boolean} [options.failOnDiff] - Whether tests should fail on visual diffs
- * @param {string} [options.browserWSEndpoint] - WebSocket endpoint to connect to remote browser
  * @param {Object} [options.playwrightOptions] - Playwright launch options (headless, slowMo, timeout, etc.)
  * @param {Function} [options.onPageCreated] - Callback when page is created (before navigation)
  * @param {Function} [options.onBrowserDisconnected] - Callback when browser disconnects unexpectedly
@@ -71,7 +70,6 @@ export async function launchBrowser(browserType, testUrl, options = {}) {
   let {
     screenshotUrl,
     failOnDiff,
-    browserWSEndpoint,
     playwrightOptions = {},
     onPageCreated,
     onBrowserDisconnected,
@@ -85,38 +83,27 @@ export async function launchBrowser(browserType, testUrl, options = {}) {
     );
   }
 
-  let browser;
-
-  // Connect to remote browser if endpoint provided
-  if (browserWSEndpoint) {
-    // Use connectOverCDP for remote browsers (e.g., browserless, Docker)
-    // This connects via Chrome DevTools Protocol, which is what most
-    // remote browser services expose (vs Playwright's own server protocol)
-    let cdpEndpoint = browserWSEndpoint.replace(/^ws:/, 'http:').replace(/^wss:/, 'https:');
-    browser = await factory.connectOverCDP(cdpEndpoint);
-  } else {
-    // Build args: our defaults + user's args (user can override)
-    let args = [];
-    if (browserType === 'chromium') {
-      args = [...getDefaultChromiumArgs()];
-    }
-
-    // Merge user's args if provided
-    if (playwrightOptions.args) {
-      args.push(...playwrightOptions.args);
-    }
-
-    // Build Playwright launch options
-    // User's playwrightOptions take precedence, but we ensure args are merged
-    let launchOptions = {
-      headless: true, // Default to headless
-      ...playwrightOptions,
-      args,
-    };
-
-    browser = await factory.launch(launchOptions);
+  // Build args: our defaults + user's args (user can override)
+  let args = [];
+  if (browserType === 'chromium') {
+    args = [...getDefaultChromiumArgs()];
   }
 
+  // Merge user's args if provided
+  if (playwrightOptions.args) {
+    args.push(...playwrightOptions.args);
+  }
+
+  // Build Playwright launch options
+  // User's playwrightOptions take precedence, but we ensure args are merged
+  let launchOptions = {
+    headless: true, // Default to headless
+    ...playwrightOptions,
+    args,
+  };
+
+  let browser = await factory.launch(launchOptions);
+
   // Listen for unexpected browser disconnection (crash, killed, etc.)
   if (onBrowserDisconnected) {
     browser.on('disconnected', onBrowserDisconnected);

clients/ember/src/testem-config.js

@@ -105,7 +105,6 @@ function writePlaywrightConfig(options) {
  * @param {number} [playwrightOptions.slowMo] - Slow down operations by this many milliseconds
  * @param {number} [playwrightOptions.timeout] - Browser launch timeout in milliseconds
  * @param {Object} [playwrightOptions.proxy] - Proxy settings
- * @param {string} [playwrightOptions.browserWSEndpoint] - WebSocket endpoint to connect to remote browser (e.g., Docker)
  * @returns {Object} Modified configuration with Vizzly launchers
  *
  * @example