style: apply biome formatting and minor fixes to test files

- Sort imports in test files according to biome rules
- Increase timeouts in e2e tests for stability
- Add NaN check in port parsing to prevent errors

Author: MBanucu

test/e2e/pty-buffer-readraw.pw.ts

@@ -1,11 +1,10 @@
-import { test as extendedTest, expect } from './fixtures'
 import type { Page } from '@playwright/test'
-
+import type { createApiClient } from 'opencode-pty/web/shared/api-client'
+import { expect, test as extendedTest } from './fixtures'
 import {
   getSerializedContentByXtermSerializeAddon,
   waitForTerminalRegex,
 } from './xterm-test-helpers'
-import type { createApiClient } from 'opencode-pty/web/shared/api-client'
 
 async function createSession(
   api: ReturnType<typeof createApiClient>,
@@ -184,9 +183,9 @@ extendedTest.describe('PTY Buffer readRaw() Function', () => {
         args: ['-i'],
         description: 'Double Echo Test Session B',
       })
-      await gotoAndSelectSession(page, 'Double Echo Test Session B', 10000)
+      await gotoAndSelectSession(page, 'Double Echo Test Session B', 15000)
       // Debug what prompt is present before event-driven wait
-      await waitForTerminalRegex(page, /\$\s*$/)
+      await waitForTerminalRegex(page, /\$\s*$/, {}, 10000)
       await page.locator('.terminal.xterm').click()
       // Dump buffer before typing in Session B
       await page.keyboard.type('1')
@@ -197,9 +196,9 @@ extendedTest.describe('PTY Buffer readRaw() Function', () => {
         args: ['-i'],
         description: 'Double Echo Test Session C',
       })
-      await gotoAndSelectSession(page, 'Double Echo Test Session C', 10000)
+      await gotoAndSelectSession(page, 'Double Echo Test Session C', 15000)
       // Debug what prompt is present before event-driven wait
-      await waitForTerminalRegex(page, /\$\s*$/)
+      await waitForTerminalRegex(page, /\$\s*$/, {}, 10000)
       await page.locator('.terminal.xterm').click()
       // Dump buffer before typing in Session C
       await page.keyboard.type('1')

test/e2e/ui/app.pw.ts

@@ -1,5 +1,5 @@
-import { test as extendedTest, expect } from '../fixtures'
 import type { PTYSessionInfo } from '../../../src/plugin/pty/types'
+import { expect, test as extendedTest } from '../fixtures'
 
 extendedTest.describe('App Component', () => {
   extendedTest('renders the PTY Sessions title', async ({ page }) => {
@@ -9,6 +9,7 @@ extendedTest.describe('App Component', () => {
 
   extendedTest('shows connected status when WebSocket connects', async ({ page }) => {
     // Page automatically navigated to server URL by fixture
+    await page.waitForSelector('.sidebar', { timeout: 10000 })
     await expect(page.getByText('● Connected')).toBeVisible()
   })
 

test/npm-pack-integration.test.ts

@@ -1,7 +1,7 @@
-import { describe, it, expect, afterEach } from 'bun:test'
-import { mkdtempSync, rmSync, copyFileSync, existsSync, mkdirSync } from 'node:fs'
-import { join } from 'node:path'
+import { afterEach, describe, expect, it } from 'bun:test'
+import { copyFileSync, existsSync, mkdirSync, mkdtempSync, rmSync } from 'node:fs'
 import { tmpdir } from 'node:os'
+import { join } from 'node:path'
 
 // This test ensures the npm package can be packed, installed, and serves assets correctly
 
@@ -114,7 +114,9 @@ describe('npm pack integration', () => {
           await new Promise(setImmediate)
         }
         const bytes = await Bun.file(portFile).bytes()
-        const port = parseInt(new TextDecoder().decode(bytes).trim(), 10)
+        const portStr = new TextDecoder().decode(bytes).trim()
+        const port = parseInt(portStr, 10)
+        if (Number.isNaN(port)) return 0
         return port
       }