tabwright 1.0.0 → 1.0.1

package/src/test-utils.ts

@@ -1,566 +0,0 @@
-import { exec } from 'node:child_process'
-import { promisify } from 'node:util'
-import http from 'node:http'
-import net from 'node:net'
-import { chromium, type BrowserContext } from '@xmorse/playwright-core'
-import path from 'node:path'
-import fs from 'node:fs'
-import os from 'node:os'
-import { startTabwrightCDPRelayServer, type RelayServer } from './cdp-relay.js'
-import { createFileLogger } from './create-logger.js'
-import { killPortProcess } from './kill-port.js'
-
-const execAsync = promisify(exec)
-const extensionBuildQueues: Map<string, Promise<void>> = new Map()
-const EXTENSION_SERVICE_WORKER_TIMEOUT_MS = 15_000
-
-export function getLocalChromeExecutable(): string | undefined {
-  const candidates = [
-    process.env.PLAYWRIGHT_CHROME_EXECUTABLE,
-    '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome',
-    '/Applications/Chromium.app/Contents/MacOS/Chromium',
-    '/usr/bin/google-chrome',
-    '/usr/bin/google-chrome-stable',
-    '/usr/bin/chromium',
-    '/usr/bin/chromium-browser',
-    'C:\\Program Files\\Google\\Chrome\\Application\\chrome.exe',
-    'C:\\Program Files (x86)\\Google\\Chrome\\Application\\chrome.exe',
-  ].filter((candidate): candidate is string => {
-    return Boolean(candidate)
-  })
-
-  return candidates.find((candidate) => {
-    return fs.existsSync(candidate)
-  })
-}
-
-export function getLegacyExtensionLaunchArgs({ extensionPaths }: { extensionPaths: string[] }): string[] {
-  const resolvedExtensionPaths = extensionPaths.map((extensionPath) => {
-    return path.resolve(extensionPath)
-  })
-  const legacyExtensionPaths = resolvedExtensionPaths.join(',')
-
-  return [
-    `--disable-extensions-except=${legacyExtensionPaths}`,
-    `--load-extension=${legacyExtensionPaths}`,
-  ]
-}
-
-export function isExtensionLoadUnavailableError(error: unknown): boolean {
-  const message = error instanceof Error ? error.message : String(error)
-  return (
-    message.includes('Method not found') ||
-    message.includes("'Extensions.loadUnpacked' wasn't found") ||
-    message.includes('-32601')
-  )
-}
-
-async function loadUnpackedExtensionsThroughCdp({
-  browserContext,
-  extensionPaths,
-}: {
-  browserContext: BrowserContext
-  extensionPaths: string[]
-}): Promise<void> {
-  const browser = browserContext.browser()
-  if (!browser) {
-    throw new Error('Persistent browser context has no browser connection')
-  }
-
-  const session = await browser.newBrowserCDPSession()
-  try {
-    await Promise.all(
-      extensionPaths.map(async (extensionPath) => {
-        await session.send('Extensions.loadUnpacked', { path: path.resolve(extensionPath) })
-      }),
-    )
-  } finally {
-    await session.detach().catch((error: unknown) => {
-      console.error('Failed to detach test extension CDP session:', error)
-    })
-  }
-}
-
-async function launchTestBrowser({
-  userDataDir,
-  args,
-}: {
-  userDataDir: string
-  args: string[]
-}): Promise<BrowserContext> {
-  const chromeExecutable = getLocalChromeExecutable()
-  return await chromium.launchPersistentContext(userDataDir, {
-    ...(chromeExecutable ? { executablePath: chromeExecutable } : { channel: 'chromium' }),
-    headless: !process.env.HEADFUL,
-    colorScheme: 'dark',
-    ignoreDefaultArgs: ['--disable-extensions'],
-    args,
-  })
-}
-
-export async function launchPersistentContextWithExtensions({
-  userDataDir,
-  extensionPaths,
-}: {
-  userDataDir: string
-  extensionPaths: string[]
-}): Promise<BrowserContext> {
-  const browserContext = await launchTestBrowser({
-    userDataDir,
-    // Chrome 137+ ignores --load-extension for branded builds. Its supported
-    // Extensions.loadUnpacked replacement requires this opt-in switch.
-    args: ['--enable-unsafe-extension-debugging'],
-  })
-
-  try {
-    await loadUnpackedExtensionsThroughCdp({ browserContext, extensionPaths })
-    return browserContext
-  } catch (error: unknown) {
-    await browserContext.close()
-    if (!isExtensionLoadUnavailableError(error)) {
-      throw new Error(`Failed to load unpacked test extensions: ${extensionPaths.join(', ')}`, { cause: error })
-    }
-
-    // Older Chrome/Chromium does not expose Extensions.loadUnpacked. Relaunch
-    // the same isolated profile with the legacy flags it still supports.
-    return await launchTestBrowser({
-      userDataDir,
-      args: getLegacyExtensionLaunchArgs({ extensionPaths }),
-    })
-  }
-}
-
-async function buildExtension({ port, distDir }: { port: number; distDir: string }): Promise<void> {
-  const previous = extensionBuildQueues.get(distDir) || Promise.resolve()
-  const buildPromise = previous
-    .catch((error) => {
-      console.error('Previous extension build failed:', error)
-    })
-    .then(async () => {
-      // Build into a per-port dist to avoid parallel test runs overwriting each other.
-      await execAsync(`TESTING=1 TABWRIGHT_PORT=${port} TABWRIGHT_EXTENSION_DIST=${distDir} pnpm build`, {
-        cwd: '../extension',
-      })
-    })
-
-  extensionBuildQueues.set(
-    distDir,
-    buildPromise.finally(() => {}),
-  )
-  await buildPromise
-}
-
-export async function getExtensionServiceWorker(context: BrowserContext) {
-  const serviceWorkers = context.serviceWorkers().filter((sw) => sw.url().startsWith('chrome-extension://'))
-  if (serviceWorkers.length === 0) {
-    await context
-      .waitForEvent('serviceworker', {
-        predicate: (sw) => sw.url().startsWith('chrome-extension://'),
-        timeout: EXTENSION_SERVICE_WORKER_TIMEOUT_MS,
-      })
-      .catch((error: unknown) => {
-        throw new Error(
-          `No extension service worker appeared within ${EXTENSION_SERVICE_WORKER_TIMEOUT_MS}ms`,
-          { cause: error },
-        )
-      })
-  }
-
-  // Check all chrome-extension service workers for the Tabwright one (the one
-  // that exposes toggleExtensionForActiveTab). This handles cases where
-  // additional test fixture extensions are loaded alongside Tabwright.
-  for (let i = 0; i < 50; i++) {
-    const allSws = context.serviceWorkers().filter((sw) => sw.url().startsWith('chrome-extension://'))
-    for (const sw of allSws) {
-      try {
-        const isReady = await sw.evaluate(() => {
-          // @ts-ignore
-          return typeof globalThis.toggleExtensionForActiveTab === 'function'
-        })
-        if (isReady) {
-          return sw
-        }
-      } catch {
-        // Service worker might not be ready yet
-      }
-    }
-    await new Promise((r) => setTimeout(r, 100))
-  }
-
-  const extensionWorkerUrls = context
-    .serviceWorkers()
-    .filter((sw) => {
-      return sw.url().startsWith('chrome-extension://')
-    })
-    .map((sw) => {
-      return sw.url()
-    })
-  throw new Error(
-    `Tabwright extension service worker did not become ready within ${EXTENSION_SERVICE_WORKER_TIMEOUT_MS}ms. Visible extension workers: ${extensionWorkerUrls.join(', ') || 'none'}`,
-  )
-}
-
-export interface TestContext {
-  browserContext: BrowserContext
-  userDataDir: string
-  relayServer: RelayServer
-}
-
-export async function setupTestContext({
-  port,
-  tempDirPrefix,
-  toggleExtension = false,
-  additionalExtensions = [],
-}: {
-  port: number
-  tempDirPrefix: string
-  /** Create initial page and toggle extension on it */
-  toggleExtension?: boolean
-  /** Additional extension paths to load alongside the main Tabwright extension */
-  additionalExtensions?: string[]
-}): Promise<TestContext> {
-  await killPortProcess({ port }).catch(() => {})
-
-  // Use a port-scoped dist folder so parallel tests don't replace each other's extension builds.
-  const distDir = `dist-${port}`
-
-  console.log('Building extension...')
-  await buildExtension({ port, distDir })
-  console.log('Extension built')
-
-  const localLogPath = path.join(process.cwd(), 'relay-server.log')
-  const logger = createFileLogger({ logFilePath: localLogPath })
-  const relayServer = await startTabwrightCDPRelayServer({ port, logger })
-
-  const userDataDir = fs.mkdtempSync(path.join(os.tmpdir(), tempDirPrefix))
-  const extensionPath = path.resolve('../extension', distDir)
-  try {
-    const browserContext = await launchPersistentContextWithExtensions({
-      userDataDir,
-      extensionPaths: [extensionPath, ...additionalExtensions],
-    })
-
-    try {
-      if (toggleExtension) {
-        const serviceWorker = await getExtensionServiceWorker(browserContext)
-        const page = await browserContext.newPage()
-        await page.goto('about:blank')
-        await serviceWorker.evaluate(async () => {
-          await (globalThis as any).toggleExtensionForActiveTab()
-        })
-      }
-
-      return { browserContext, userDataDir, relayServer }
-    } catch (error) {
-      await browserContext.close().catch((closeError: unknown) => {
-        console.error('Failed to close browser after test setup error:', closeError)
-      })
-      throw error
-    }
-  } catch (error) {
-    relayServer.close()
-    try {
-      fs.rmSync(userDataDir, { recursive: true, force: true })
-    } catch (cleanupError) {
-      console.error('Failed to clean test profile after setup error:', cleanupError)
-    }
-    throw error
-  }
-}
-
-export async function cleanupTestContext(
-  ctx: TestContext | null,
-  cleanup?: (() => Promise<void>) | null,
-): Promise<void> {
-  if (ctx?.browserContext) {
-    await ctx.browserContext.close()
-  }
-  if (ctx?.relayServer) {
-    ctx.relayServer.close()
-  }
-
-  if (ctx?.userDataDir) {
-    try {
-      fs.rmSync(ctx.userDataDir, { recursive: true, force: true })
-    } catch (e) {
-      console.error('Failed to cleanup user data dir:', e)
-    }
-  }
-  if (cleanup) {
-    await cleanup()
-  }
-}
-
-export type SseServerState = {
-  connected: boolean
-  finished: boolean
-  writeCount: number
-  closed: boolean
-}
-
-export type SseServer = {
-  baseUrl: string
-  getState: () => SseServerState
-  close: () => Promise<void>
-}
-
-export async function createSseServer(): Promise<SseServer> {
-  let sseResponse: http.ServerResponse | null = null
-  let sseFinished = false
-  let sseClosed = false
-  let sseWriteCount = 0
-  let sseInterval: NodeJS.Timeout | null = null
-  const openResponses: Set<http.ServerResponse> = new Set()
-  const openSockets: Set<net.Socket> = new Set()
-
-  const server = http.createServer((req, res) => {
-    if (req.url === '/') {
-      res.writeHead(200, { 'Content-Type': 'text/html' })
-      res.end(`<!doctype html>
-<html>
-  <head>
-    <meta charset="utf-8" />
-    <title>SSE Test</title>
-  </head>
-  <body>
-    <script>
-      window.__sseMessages = [];
-      window.__sseOpen = false;
-      window.__sseError = null;
-      window.startSse = function () {
-        const source = new EventSource('/sse');
-        window.__sseSource = source;
-        source.onopen = function () {
-          window.__sseOpen = true;
-        };
-        source.onmessage = function (event) {
-          window.__sseMessages.push(event.data);
-        };
-        source.onerror = function () {
-          window.__sseError = 'SSE error';
-        };
-        return true;
-      };
-      window.stopSse = function () {
-        if (window.__sseSource) {
-          window.__sseSource.close();
-        }
-      };
-    </script>
-  </body>
-</html>`)
-      return
-    }
-
-    if (req.url === '/sse') {
-      res.writeHead(200, {
-        'Content-Type': 'text/event-stream',
-        'Cache-Control': 'no-cache, no-transform',
-        Connection: 'keep-alive',
-      })
-      res.write('retry: 1000\n\n')
-      res.write('data: hello\n\n')
-      sseResponse = res
-      sseWriteCount += 1
-      openResponses.add(res)
-
-      res.on('finish', () => {
-        sseFinished = true
-      })
-      res.on('close', () => {
-        sseClosed = true
-        openResponses.delete(res)
-        if (sseInterval) {
-          clearInterval(sseInterval)
-          sseInterval = null
-        }
-      })
-
-      sseInterval = setInterval(() => {
-        res.write('data: ping\n\n')
-        sseWriteCount += 1
-      }, 200)
-      return
-    }
-
-    res.writeHead(404)
-    res.end('Not found')
-  })
-
-  server.on('connection', (socket) => {
-    openSockets.add(socket)
-    socket.on('close', () => {
-      openSockets.delete(socket)
-    })
-  })
-
-  await new Promise<void>((resolve) => {
-    server.listen(0, '127.0.0.1', () => {
-      resolve()
-    })
-  })
-
-  const address = server.address()
-  if (!address || typeof address === 'string') {
-    throw new Error('Failed to bind SSE server')
-  }
-
-  return {
-    baseUrl: `http://127.0.0.1:${address.port}`,
-    getState: () => ({
-      connected: sseResponse !== null,
-      finished: sseFinished,
-      closed: sseClosed,
-      writeCount: sseWriteCount,
-    }),
-    close: async () => {
-      for (const response of openResponses) {
-        response.destroy()
-      }
-      for (const socket of openSockets) {
-        socket.destroy()
-      }
-      if (sseInterval) {
-        clearInterval(sseInterval)
-        sseInterval = null
-      }
-      await new Promise<void>((resolve, reject) => {
-        server.close((error) => {
-          if (error) {
-            reject(error)
-            return
-          }
-          resolve()
-        })
-      })
-    },
-  }
-}
-
-export async function withTimeout<T>({
-  promise,
-  timeoutMs,
-  errorMessage,
-}: {
-  promise: Promise<T>
-  timeoutMs: number
-  errorMessage: string
-}): Promise<T> {
-  return await new Promise<T>((resolve, reject) => {
-    const timeoutId = setTimeout(() => {
-      reject(new Error(errorMessage))
-    }, timeoutMs)
-
-    promise
-      .then((value) => {
-        clearTimeout(timeoutId)
-        resolve(value)
-      })
-      .catch((error) => {
-        clearTimeout(timeoutId)
-        reject(error)
-      })
-  })
-}
-
-/** Tagged template for inline JS code strings used in MCP execute calls */
-export function js(strings: TemplateStringsArray, ...values: unknown[]): string {
-  return strings.reduce((result, str, i) => result + str + (values[i] || ''), '')
-}
-
-export function tryJsonParse(str: string) {
-  try {
-    return JSON.parse(str)
-  } catch {
-    return str
-  }
-}
-
-/**
- * Safely close a browser connected via connectOverCDP.
- *
- * Playwright's CRConnection uses async message handling (messageWrap) that can cause
- * a race condition where _onClose() runs before all pending _onMessage() handlers complete.
- * This results in "Assertion error" from crConnection.js when a CDP response arrives
- * after callbacks were cleared by dispose().
- *
- * This helper waits for the message queue to drain before closing, avoiding the race.
- *
- * @param browser - Browser instance from chromium.connectOverCDP()
- * @param drainDelayMs - Time to wait for pending messages to be processed (default: 50ms)
- */
-export async function safeCloseCDPBrowser(
-  browser: Awaited<ReturnType<typeof import('@xmorse/playwright-core').chromium.connectOverCDP>>,
-  drainDelayMs = 50,
-): Promise<void> {
-  // Wait for any queued message handlers to run
-  // This gives Playwright's messageWrap time to process pending CDP responses
-  await new Promise((r) => setTimeout(r, drainDelayMs))
-  await browser.close()
-}
-
-export type SimpleServer = {
-  baseUrl: string
-  close: () => Promise<void>
-}
-
-/** Minimal local HTTP server for tests that need cross-origin iframes or custom routes */
-export async function createSimpleServer({ routes }: { routes: Record<string, string> }): Promise<SimpleServer> {
-  const openSockets: Set<net.Socket> = new Set()
-  const server = http.createServer((req, res) => {
-    const url = req.url || '/'
-    const body = routes[url]
-    if (!body) {
-      res.writeHead(404, { 'Content-Type': 'text/plain' })
-      res.end('not found')
-      return
-    }
-    res.writeHead(200, { 'Content-Type': 'text/html' })
-    res.end(body)
-  })
-
-  server.on('connection', (socket) => {
-    openSockets.add(socket)
-    socket.on('close', () => {
-      openSockets.delete(socket)
-    })
-  })
-
-  await new Promise<void>((resolve) => {
-    server.listen(0, '127.0.0.1', () => {
-      resolve()
-    })
-  })
-
-  const address = server.address()
-  if (!address || typeof address === 'string') {
-    await new Promise<void>((resolve, reject) => {
-      server.close((error) => {
-        if (error) {
-          reject(error)
-          return
-        }
-        resolve()
-      })
-    })
-    throw new Error('Failed to start test server')
-  }
-
-  return {
-    baseUrl: `http://127.0.0.1:${address.port}`,
-    close: async () => {
-      for (const socket of openSockets) {
-        socket.destroy()
-      }
-      await new Promise<void>((resolve, reject) => {
-        server.close((error) => {
-          if (error) {
-            reject(error)
-            return
-          }
-          resolve()
-        })
-      })
-    },
-  }
-}

package/src/utils.ts

@@ -1,78 +0,0 @@
-import fs from 'node:fs'
-import path from 'node:path'
-import { fileURLToPath } from 'node:url'
-import './env-compat.js'
-import { getTabwrightUserDataDir } from './product-paths.js'
-
-// Tabwright extension IDs - used for validation and Chrome flag commands
-export const EXTENSION_IDS = [
-  'jfeammnjpkecdekppnclgkkffahnhfhe', // Production (Chrome Web Store)
-  'pebbngnfojnignonigcnkdilknapkgid', // Dev extension (stable ID from manifest key)
-]
-
-/**
- * Parse a relay host string into HTTP and WebSocket base URLs.
- * Supports both plain hostnames (appends port) and full URLs (uses as-is).
- *
- * Examples:
- *   "192.168.1.10"                        → http://192.168.1.10:19988, ws://192.168.1.10:19988
- *   "https://my-machine-tunnel.traforo.dev" → https://my-machine-tunnel.traforo.dev, wss://my-machine-tunnel.traforo.dev
- */
-export function parseRelayHost(host: string, port: number = 19988): { httpBaseUrl: string; wsBaseUrl: string } {
-  if (host.startsWith('https://') || host.startsWith('http://')) {
-    const url = new URL(host)
-    const httpBaseUrl = url.origin
-    const wsProtocol = url.protocol === 'https:' ? 'wss:' : 'ws:'
-    const wsBaseUrl = `${wsProtocol}//${url.host}`
-    return { httpBaseUrl, wsBaseUrl }
-  }
-  return {
-    httpBaseUrl: `http://${host}:${port}`,
-    wsBaseUrl: `ws://${host}:${port}`,
-  }
-}
-
-export function getCdpUrl({
-  port = 19988,
-  host = '127.0.0.1',
-  token,
-  extensionId,
-}: {
-  port?: number
-  host?: string
-  token?: string
-  extensionId?: string | null
-} = {}) {
-  const id = `${Math.random().toString(36).substring(2, 15)}_${Date.now()}`
-  const params = new URLSearchParams()
-  if (token) {
-    params.set('token', token)
-  }
-  if (extensionId) {
-    params.set('extensionId', extensionId)
-  }
-  const queryString = params.toString()
-  const suffix = queryString ? `?${queryString}` : ''
-  const { wsBaseUrl } = parseRelayHost(host, port)
-  return `${wsBaseUrl}/cdp/${id}${suffix}`
-}
-
-export function shouldAutoEnableTabwright(): boolean {
-  return (process.env.TABWRIGHT_AUTO_ENABLE || process.env.PLAYWRITER_AUTO_ENABLE)?.toLowerCase() !== 'false'
-}
-
-// Existing installations continue using their legacy data directory until users migrate it.
-const LOG_BASE_DIR = getTabwrightUserDataDir()
-export const LOG_FILE_PATH =
-  process.env.TABWRIGHT_LOG_FILE_PATH || process.env.PLAYWRITER_LOG_FILE_PATH || path.join(LOG_BASE_DIR, 'relay-server.log')
-export const LOG_CDP_FILE_PATH =
-  process.env.TABWRIGHT_CDP_LOG_FILE_PATH ||
-  process.env.PLAYWRITER_CDP_LOG_FILE_PATH ||
-  path.join(path.dirname(LOG_FILE_PATH), 'cdp.jsonl')
-
-const packageJsonPath = path.join(path.dirname(fileURLToPath(import.meta.url)), '..', 'package.json')
-export const VERSION = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8')).version as string
-
-export function sleep(ms: number): Promise<void> {
-  return new Promise((resolve) => setTimeout(resolve, ms))
-}