Address code review feedback

- Fix setupBidi leak: defer state.bidi.set() until all listeners are wired up
- Fix close_current_window missing state.bidi.delete() cleanup
- Add after hook to Session Isolation test for reliable cleanup
- Use node:assert/strict for consistency with other test files
- Add bidi.test.mjs and bidi.html to AGENTS.md file tree

Author: angiejones

AGENTS.md

@@ -37,12 +37,14 @@ mcp-selenium/
     ├── browser.test.mjs   ← start_browser, close_session, take_screenshot, multi-session
     ├── navigation.test.mjs ← navigate, all 6 locator strategies
     ├── interactions.test.mjs ← click, send_keys, get_element_text, hover, double_click, right_click, press_key, drag_and_drop, upload_file
+    ├── bidi.test.mjs      ← BiDi enablement, console/error/network capture, session isolation
     └── fixtures/           ← HTML files loaded via file:// URLs
         ├── locators.html
         ├── interactions.html
         ├── mouse-actions.html
         ├── drag-drop.html
-        └── upload.html
+        ├── upload.html
+        └── bidi.html
 ```
 
 ### Key Files in Detail

src/lib/server.js

@@ -67,7 +67,6 @@ const newBidiState = () => ({
 
 async function setupBidi(driver, sessionId) {
     const bidi = newBidiState();
-    state.bidi.set(sessionId, bidi);
 
     const logInspector = await LogInspector(driver);
     await logInspector.onConsoleEntry((entry) => {
@@ -106,6 +105,7 @@ async function setupBidi(driver, sessionId) {
     });
 
     bidi.available = true;
+    state.bidi.set(sessionId, bidi);
 }
 
 function registerBidiTool(name, description, logKey, emptyMessage, unavailableMessage) {
@@ -787,6 +787,7 @@ server.tool(
                 console.error(`Error quitting driver for session ${sessionId}:`, quitError);
             }
             state.drivers.delete(sessionId);
+            state.bidi.delete(sessionId);
             state.currentSession = null;
             return {
                 content: [{ type: 'text', text: 'Last window closed. Session ended.' }]

test/bidi.test.mjs

@@ -1,5 +1,5 @@
 import { describe, it, after, before } from 'node:test';
-import assert from 'node:assert';
+import assert from 'node:assert/strict';
 import { McpClient, getResponseText, fixture } from './mcp-client.mjs';
 
 describe('BiDi Diagnostic Tools', () => {
@@ -136,6 +136,10 @@ describe('BiDi Diagnostic Tools', () => {
     });
 
     describe('Session Isolation', () => {
+        after(async () => {
+            try { await client.callTool('close_session', {}); } catch (_) {}
+        });
+
         it('should reset BiDi logs when starting a new session', async () => {
             await client.callTool('start_browser', {
                 browser: 'chrome',