browser-debugger-cli 0.5.1 → 0.6.1

package/dist/session/__tests__/lock.contract.test.js

@@ -0,0 +1,77 @@
+/**
+ * Lock contract tests - Test behavior, not implementation
+ *
+ * Focus: Concurrency control, stale lock detection, race conditions
+ */
+import * as fs from 'fs';
+import assert from 'node:assert';
+import { describe, it, beforeEach, afterEach } from 'node:test';
+import * as os from 'os';
+import * as path from 'path';
+import { acquireSessionLock, releaseSessionLock, acquireDaemonLock, releaseDaemonLock, } from '../../session/lock.js';
+describe('Session Lock Contract', () => {
+    let testDir;
+    beforeEach(() => {
+        testDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bdg-lock-test-'));
+        process.env['BDG_SESSION_DIR'] = testDir;
+    });
+    afterEach(() => {
+        delete process.env['BDG_SESSION_DIR'];
+        if (fs.existsSync(testDir)) {
+            fs.rmSync(testDir, { recursive: true, force: true });
+        }
+    });
+    it('prevents concurrent session locks', () => {
+        const first = acquireSessionLock();
+        const second = acquireSessionLock();
+        assert.strictEqual(first, true, 'First lock should succeed');
+        assert.strictEqual(second, false, 'Second lock should fail');
+        releaseSessionLock();
+    });
+    it('allows reacquisition after release', () => {
+        acquireSessionLock();
+        releaseSessionLock();
+        const reacquired = acquireSessionLock();
+        assert.strictEqual(reacquired, true, 'Should reacquire after release');
+        releaseSessionLock();
+    });
+    it('removes stale lock from dead process', () => {
+        const lockPath = path.join(testDir, 'session.lock');
+        fs.writeFileSync(lockPath, '99999999');
+        const acquired = acquireSessionLock();
+        assert.strictEqual(acquired, true, 'Should acquire after removing stale lock');
+        releaseSessionLock();
+    });
+    it('handles corrupt lock file', () => {
+        const lockPath = path.join(testDir, 'session.lock');
+        fs.writeFileSync(lockPath, 'not-a-number');
+        const acquired = acquireSessionLock();
+        assert.strictEqual(acquired, true, 'Should acquire after removing corrupt lock');
+        releaseSessionLock();
+    });
+    it('prevents concurrent daemon locks', () => {
+        const first = acquireDaemonLock();
+        const second = acquireDaemonLock();
+        assert.strictEqual(first, true, 'First daemon lock should succeed');
+        assert.strictEqual(second, false, 'Second daemon lock should fail');
+        releaseDaemonLock();
+    });
+    it('session and daemon locks are independent', () => {
+        const sessionLock = acquireSessionLock();
+        const daemonLock = acquireDaemonLock();
+        assert.strictEqual(sessionLock, true, 'Session lock should succeed');
+        assert.strictEqual(daemonLock, true, 'Daemon lock should succeed');
+        releaseSessionLock();
+        releaseDaemonLock();
+    });
+    it('is idempotent on release', () => {
+        acquireSessionLock();
+        releaseSessionLock();
+        releaseSessionLock();
+        releaseSessionLock();
+        const acquired = acquireSessionLock();
+        assert.strictEqual(acquired, true, 'Multiple releases should not break lock');
+        releaseSessionLock();
+    });
+});
+//# sourceMappingURL=lock.contract.test.js.map

package/dist/session/__tests__/lock.contract.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"lock.contract.test.js","sourceRoot":"","sources":["../../../src/session/__tests__/lock.contract.test.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,MAAM,MAAM,aAAa,CAAC;AACjC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAChE,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B,OAAO,EACL,kBAAkB,EAClB,kBAAkB,EAClB,iBAAiB,EACjB,iBAAiB,GAClB,MAAM,mBAAmB,CAAC;AAE3B,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,IAAI,OAAe,CAAC;IAEpB,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,gBAAgB,CAAC,CAAC,CAAC;QACnE,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,GAAG,OAAO,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,OAAO,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,CAAC;QACtC,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YAC3B,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACvD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,KAAK,GAAG,kBAAkB,EAAE,CAAC;QACnC,MAAM,MAAM,GAAG,kBAAkB,EAAE,CAAC;QAEpC,MAAM,CAAC,WAAW,CAAC,KAAK,EAAE,IAAI,EAAE,2BAA2B,CAAC,CAAC;QAC7D,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,KAAK,EAAE,yBAAyB,CAAC,CAAC;QAE7D,kBAAkB,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,kBAAkB,EAAE,CAAC;QACrB,kBAAkB,EAAE,CAAC;QAErB,MAAM,UAAU,GAAG,kBAAkB,EAAE,CAAC;QACxC,MAAM,CAAC,WAAW,CAAC,UAAU,EAAE,IAAI,EAAE,gCAAgC,CAAC,CAAC;QAEvE,kBAAkB,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,cAAc,CAAC,CAAC;QACpD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QAEvC,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAC;QACtC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,0CAA0C,CAAC,CAAC;QAE/E,kBAAkB,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,cAAc,CAAC,CAAC;QACpD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,cAAc,CAAC,CAAC;QAE3C,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAC;QACtC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,4CAA4C,CAAC,CAAC;QAEjF,kBAAkB,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;QAC1C,MAAM,KAAK,GAAG,iBAAiB,EAAE,CAAC;QAClC,MAAM,MAAM,GAAG,iBAAiB,EAAE,CAAC;QAEnC,MAAM,CAAC,WAAW,CAAC,KAAK,EAAE,IAAI,EAAE,kCAAkC,CAAC,CAAC;QACpE,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,KAAK,EAAE,gCAAgC,CAAC,CAAC;QAEpE,iBAAiB,EAAE,CAAC;IACtB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,WAAW,GAAG,kBAAkB,EAAE,CAAC;QACzC,MAAM,UAAU,GAAG,iBAAiB,EAAE,CAAC;QAEvC,MAAM,CAAC,WAAW,CAAC,WAAW,EAAE,IAAI,EAAE,6BAA6B,CAAC,CAAC;QACrE,MAAM,CAAC,WAAW,CAAC,UAAU,EAAE,IAAI,EAAE,4BAA4B,CAAC,CAAC;QAEnE,kBAAkB,EAAE,CAAC;QACrB,iBAAiB,EAAE,CAAC;IACtB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;QAClC,kBAAkB,EAAE,CAAC;QACrB,kBAAkB,EAAE,CAAC;QACrB,kBAAkB,EAAE,CAAC;QACrB,kBAAkB,EAAE,CAAC;QAErB,MAAM,QAAQ,GAAG,kBAAkB,EAAE,CAAC;QACtC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,yCAAyC,CAAC,CAAC;QAE9E,kBAAkB,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/session/__tests__/metadata.contract.test.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Metadata contract tests - Test behavior, not implementation
+ *
+ * Focus: Corruption handling, self-healing, error recovery
+ */
+export {};
+//# sourceMappingURL=metadata.contract.test.d.ts.map

package/dist/session/__tests__/metadata.contract.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"metadata.contract.test.d.ts","sourceRoot":"","sources":["../../../src/session/__tests__/metadata.contract.test.ts"],"names":[],"mappings":"AAAA;;;;GAIG"}

package/dist/session/__tests__/metadata.contract.test.js

@@ -0,0 +1,87 @@
+/**
+ * Metadata contract tests - Test behavior, not implementation
+ *
+ * Focus: Corruption handling, self-healing, error recovery
+ */
+import * as fs from 'fs';
+import assert from 'node:assert';
+import { describe, it, beforeEach, afterEach } from 'node:test';
+import * as os from 'os';
+import * as path from 'path';
+import { writeSessionMetadata, readSessionMetadata } from '../../session/metadata.js';
+describe('Metadata Contract', () => {
+    let testDir;
+    beforeEach(() => {
+        testDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bdg-metadata-test-'));
+        process.env['BDG_SESSION_DIR'] = testDir;
+    });
+    afterEach(() => {
+        delete process.env['BDG_SESSION_DIR'];
+        if (fs.existsSync(testDir)) {
+            fs.rmSync(testDir, { recursive: true, force: true });
+        }
+    });
+    it('writes and reads metadata', () => {
+        const metadata = {
+            bdgPid: process.pid,
+            chromePid: 12345,
+            startTime: Date.now(),
+            port: 9222,
+            targetId: 'target-123',
+            webSocketDebuggerUrl: 'ws://localhost:9222/devtools/page/123',
+            activeTelemetry: ['network', 'console'],
+        };
+        writeSessionMetadata(metadata);
+        const read = readSessionMetadata();
+        assert.deepStrictEqual(read, metadata, 'Should read what was written');
+    });
+    it('returns null for non-existent metadata', () => {
+        const metadata = readSessionMetadata();
+        assert.strictEqual(metadata, null, 'Should return null when file does not exist');
+    });
+    it('handles corrupt JSON', () => {
+        const metaPath = path.join(testDir, 'session.meta.json');
+        fs.writeFileSync(metaPath, 'not valid json{]');
+        const metadata = readSessionMetadata();
+        assert.strictEqual(metadata, null, 'Should return null for corrupt JSON');
+    });
+    it('self-heals corrupt metadata when requested', () => {
+        const metaPath = path.join(testDir, 'session.meta.json');
+        fs.writeFileSync(metaPath, 'not valid json{]');
+        readSessionMetadata({ selfHealOnCorruption: true });
+        assert.strictEqual(fs.existsSync(metaPath), false, 'Should remove corrupt file');
+    });
+    it('preserves corrupt file when self-heal disabled', () => {
+        const metaPath = path.join(testDir, 'session.meta.json');
+        fs.writeFileSync(metaPath, 'not valid json{]');
+        readSessionMetadata({ selfHealOnCorruption: false });
+        assert.strictEqual(fs.existsSync(metaPath), true, 'Should not remove file without self-heal');
+    });
+    it('handles empty file', () => {
+        const metaPath = path.join(testDir, 'session.meta.json');
+        fs.writeFileSync(metaPath, '');
+        const metadata = readSessionMetadata();
+        assert.strictEqual(metadata, null, 'Should handle empty file');
+    });
+    it('supports minimal metadata', () => {
+        const minimal = {
+            bdgPid: process.pid,
+            startTime: Date.now(),
+            port: 9222,
+        };
+        writeSessionMetadata(minimal);
+        const read = readSessionMetadata();
+        assert.deepStrictEqual(read, minimal, 'Should support minimal metadata');
+    });
+    it('handles optional fields', () => {
+        const withOptional = {
+            bdgPid: process.pid,
+            startTime: Date.now(),
+            port: 9222,
+        };
+        writeSessionMetadata(withOptional);
+        const read = readSessionMetadata();
+        assert.deepStrictEqual(read, withOptional, 'Should handle missing optional fields');
+    });
+});
+//# sourceMappingURL=metadata.contract.test.js.map

package/dist/session/__tests__/metadata.contract.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"metadata.contract.test.js","sourceRoot":"","sources":["../../../src/session/__tests__/metadata.contract.test.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,MAAM,MAAM,aAAa,CAAC;AACjC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAChE,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAE7B,OAAO,EAAE,oBAAoB,EAAE,mBAAmB,EAAE,MAAM,uBAAuB,CAAC;AAGlF,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,IAAI,OAAe,CAAC;IAEpB,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,oBAAoB,CAAC,CAAC,CAAC;QACvE,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,GAAG,OAAO,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,OAAO,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC,CAAC;QACtC,IAAI,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YAC3B,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACvD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,QAAQ,GAAG;YACf,MAAM,EAAE,OAAO,CAAC,GAAG;YACnB,SAAS,EAAE,KAAK;YAChB,SAAS,EAAE,IAAI,CAAC,GAAG,EAAE;YACrB,IAAI,EAAE,IAAI;YACV,QAAQ,EAAE,YAAY;YACtB,oBAAoB,EAAE,uCAAuC;YAC7D,eAAe,EAAE,CAAC,SAAS,EAAE,SAAS,CAAoB;SAC3D,CAAC;QAEF,oBAAoB,CAAC,QAAQ,CAAC,CAAC;QAC/B,MAAM,IAAI,GAAG,mBAAmB,EAAE,CAAC;QAEnC,MAAM,CAAC,eAAe,CAAC,IAAI,EAAE,QAAQ,EAAE,8BAA8B,CAAC,CAAC;IACzE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,QAAQ,GAAG,mBAAmB,EAAE,CAAC;QACvC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,6CAA6C,CAAC,CAAC;IACpF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sBAAsB,EAAE,GAAG,EAAE;QAC9B,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,kBAAkB,CAAC,CAAC;QAE/C,MAAM,QAAQ,GAAG,mBAAmB,EAAE,CAAC;QACvC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,qCAAqC,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,kBAAkB,CAAC,CAAC;QAE/C,mBAAmB,CAAC,EAAE,oBAAoB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEpD,MAAM,CAAC,WAAW,CAAC,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,EAAE,KAAK,EAAE,4BAA4B,CAAC,CAAC;IACnF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,kBAAkB,CAAC,CAAC;QAE/C,mBAAmB,CAAC,EAAE,oBAAoB,EAAE,KAAK,EAAE,CAAC,CAAC;QAErD,MAAM,CAAC,WAAW,CAAC,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,EAAE,IAAI,EAAE,0CAA0C,CAAC,CAAC;IAChG,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAC5B,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC;QACzD,EAAE,CAAC,aAAa,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAC;QAE/B,MAAM,QAAQ,GAAG,mBAAmB,EAAE,CAAC;QACvC,MAAM,CAAC,WAAW,CAAC,QAAQ,EAAE,IAAI,EAAE,0BAA0B,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACnC,MAAM,OAAO,GAAG;YACd,MAAM,EAAE,OAAO,CAAC,GAAG;YACnB,SAAS,EAAE,IAAI,CAAC,GAAG,EAAE;YACrB,IAAI,EAAE,IAAI;SACX,CAAC;QAEF,oBAAoB,CAAC,OAAO,CAAC,CAAC;QAC9B,MAAM,IAAI,GAAG,mBAAmB,EAAE,CAAC;QAEnC,MAAM,CAAC,eAAe,CAAC,IAAI,EAAE,OAAO,EAAE,iCAAiC,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACjC,MAAM,YAAY,GAAG;YACnB,MAAM,EAAE,OAAO,CAAC,GAAG;YACnB,SAAS,EAAE,IAAI,CAAC,GAAG,EAAE;YACrB,IAAI,EAAE,IAAI;SACX,CAAC;QAEF,oBAAoB,CAAC,YAAY,CAAC,CAAC;QACnC,MAAM,IAAI,GAAG,mBAAmB,EAAE,CAAC;QAEnC,MAAM,CAAC,eAAe,CAAC,IAAI,EAAE,YAAY,EAAE,uCAAuC,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/session/chrome.d.ts

@@ -43,4 +43,24 @@ export declare function readChromePid(): number | null;
  * Safe to call multiple times (idempotent).
  */
 export declare function clearChromePid(): void;
+/**
+ * Aggressively cleanup stale Chrome processes launched by bdg.
+ *
+ * Kills Chrome instances that were launched by bdg by:
+ * 1. Reading the Chrome PID from persistent cache
+ * 2. Killing that specific Chrome process using cross-platform kill logic
+ *
+ * The cache survives session cleanup, so this works even after a normal session end.
+ *
+ * Cross-platform killing:
+ * - Windows: Uses `taskkill /pid <pid> /T /F` to kill process tree
+ * - Unix/macOS: Uses `process.kill(-pid, 'SIGKILL')` to kill process group
+ *
+ * Note: We can't use chromeLauncher.killAll() because it only tracks instances
+ * created via chromeLauncher.launch(), but we use new chromeLauncher.Launcher()
+ * which doesn't register in that tracking set.
+ *
+ * @returns Number of errors encountered during cleanup
+ */
+export declare function cleanupStaleChrome(): Promise<number>;
 //# sourceMappingURL=chrome.d.ts.map

package/dist/session/chrome.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"chrome.d.ts","sourceRoot":"","sources":["../../src/session/chrome.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AASH;;;;;;;;;;;;;GAaG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI,CAItD;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,aAAa,IAAI,MAAM,GAAG,IAAI,CA+B7C;AAED;;;;GAIG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAUrC"}
+{"version":3,"file":"chrome.d.ts","sourceRoot":"","sources":["../../src/session/chrome.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAsCH;;;;;;;;;;;;;GAaG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI,CAItD;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,aAAa,IAAI,MAAM,GAAG,IAAI,CAyB7C;AAED;;;;GAIG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAQrC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,MAAM,CAAC,CAqC1D"}

package/dist/session/chrome.js

@@ -5,9 +5,36 @@
  * WHY: Enables aggressive Chrome cleanup even after session ends (for `bdg cleanup --chrome`).
  */
 import * as fs from 'fs';
+import { getErrorMessage } from '../connection/errors.js';
+import { createLogger } from '../ui/logging/index.js';
 import { AtomicFileWriter } from '../utils/atomicFile.js';
 import { getSessionFilePath, ensureSessionDir } from './paths.js';
 import { isProcessAlive } from './process.js';
+const log = createLogger('chrome');
+/**
+ * Parse Chrome PID from cache file content.
+ *
+ * @param pidStr - PID string from cache file
+ * @returns Parsed PID or null if invalid
+ */
+function parseChromePid(pidStr) {
+    const pid = parseInt(pidStr.trim(), 10);
+    return Number.isNaN(pid) ? null : pid;
+}
+/**
+ * Remove Chrome PID cache file safely.
+ *
+ * @param cachePath - Path to cache file
+ * @param reason - Reason for removal (for logging)
+ */
+function removeChromePidCache(cachePath, reason) {
+    try {
+        fs.rmSync(cachePath, { force: true });
+    }
+    catch (error) {
+        log.debug(`Failed to remove ${reason} Chrome PID cache: ${getErrorMessage(error)}`);
+    }
+}
 /**
  * Write Chrome PID to persistent cache.
  *
@@ -50,21 +77,14 @@ export function readChromePid() {
         return null;
     }
     try {
-        const pidStr = fs.readFileSync(cachePath, 'utf-8').trim();
-        const pid = parseInt(pidStr, 10);
-        if (isNaN(pid)) {
+        const pidStr = fs.readFileSync(cachePath, 'utf-8');
+        const pid = parseChromePid(pidStr);
+        if (pid === null) {
+            removeChromePidCache(cachePath, 'corrupt');
             return null;
         }
-        // Only return the PID if the process is still alive
-        // This prevents trying to kill stale PIDs
         if (!isProcessAlive(pid)) {
-            // Clean up stale cache
-            try {
-                fs.unlinkSync(cachePath);
-            }
-            catch {
-                // Ignore cleanup errors
-            }
+            removeChromePidCache(cachePath, 'stale');
             return null;
         }
         return pid;
@@ -81,12 +101,56 @@ export function readChromePid() {
 export function clearChromePid() {
     const cachePath = getSessionFilePath('CHROME_PID');
     try {
-        if (fs.existsSync(cachePath)) {
-            fs.unlinkSync(cachePath);
+        fs.rmSync(cachePath, { force: true });
+    }
+    catch (error) {
+        log.debug(`Failed to clear Chrome PID cache: ${getErrorMessage(error)}`);
+    }
+}
+/**
+ * Aggressively cleanup stale Chrome processes launched by bdg.
+ *
+ * Kills Chrome instances that were launched by bdg by:
+ * 1. Reading the Chrome PID from persistent cache
+ * 2. Killing that specific Chrome process using cross-platform kill logic
+ *
+ * The cache survives session cleanup, so this works even after a normal session end.
+ *
+ * Cross-platform killing:
+ * - Windows: Uses `taskkill /pid <pid> /T /F` to kill process tree
+ * - Unix/macOS: Uses `process.kill(-pid, 'SIGKILL')` to kill process group
+ *
+ * Note: We can't use chromeLauncher.killAll() because it only tracks instances
+ * created via chromeLauncher.launch(), but we use new chromeLauncher.Launcher()
+ * which doesn't register in that tracking set.
+ *
+ * @returns Number of errors encountered during cleanup
+ */
+export async function cleanupStaleChrome() {
+    const { cleanupChromeAttemptingMessage, cleanupChromePidNotFoundMessage, cleanupChromeKillingMessage, cleanupChromeSuccessMessage, cleanupChromeFailedMessage, cleanupChromeProcessFailedMessage, } = await import('../ui/messages/chrome.js');
+    console.error(cleanupChromeAttemptingMessage());
+    try {
+        const { killChromeProcess } = await import('./process.js');
+        const chromePid = readChromePid();
+        if (!chromePid) {
+            console.error(cleanupChromePidNotFoundMessage());
+            return 0;
+        }
+        console.error(cleanupChromeKillingMessage(chromePid));
+        try {
+            killChromeProcess(chromePid, 'SIGKILL');
+            console.error(cleanupChromeSuccessMessage());
+            clearChromePid();
+            return 0;
+        }
+        catch (killError) {
+            console.error(cleanupChromeFailedMessage(getErrorMessage(killError)));
+            return 1;
         }
     }
-    catch {
-        // Ignore errors during cleanup
+    catch (error) {
+        console.error(cleanupChromeProcessFailedMessage(getErrorMessage(error)));
+        return 1;
     }
 }
 //# sourceMappingURL=chrome.js.map

package/dist/session/chrome.js.map

@@ -1 +1 @@
-{"version":3,"file":"chrome.js","sourceRoot":"","sources":["../../src/session/chrome.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AAEzB,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAEzD,OAAO,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AAClE,OAAO,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAE9C;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,cAAc,CAAC,SAAiB;IAC9C,gBAAgB,EAAE,CAAC;IACnB,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IACnD,gBAAgB,CAAC,SAAS,CAAC,SAAS,EAAE,SAAS,CAAC,QAAQ,EAAE,CAAC,CAAC;AAC9D,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,aAAa;IAC3B,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IAEnD,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;QAC9B,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC,IAAI,EAAE,CAAC;QAC1D,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;QAEjC,IAAI,KAAK,CAAC,GAAG,CAAC,EAAE,CAAC;YACf,OAAO,IAAI,CAAC;QACd,CAAC;QAED,oDAAoD;QACpD,0CAA0C;QAC1C,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,EAAE,CAAC;YACzB,uBAAuB;YACvB,IAAI,CAAC;gBACH,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC;YAC3B,CAAC;YAAC,MAAM,CAAC;gBACP,wBAAwB;YAC1B,CAAC;YACD,OAAO,IAAI,CAAC;QACd,CAAC;QAED,OAAO,GAAG,CAAC;IACb,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,cAAc;IAC5B,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IAEnD,IAAI,CAAC;QACH,IAAI,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;YAC7B,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC;QAC3B,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,+BAA+B;IACjC,CAAC;AACH,CAAC"}
+{"version":3,"file":"chrome.js","sourceRoot":"","sources":["../../src/session/chrome.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AAEzB,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AACzD,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAC;AACrD,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAEzD,OAAO,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,YAAY,CAAC;AAClE,OAAO,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAE9C,MAAM,GAAG,GAAG,YAAY,CAAC,QAAQ,CAAC,CAAC;AAEnC;;;;;GAKG;AACH,SAAS,cAAc,CAAC,MAAc;IACpC,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;IACxC,OAAO,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC;AACxC,CAAC;AAED;;;;;GAKG;AACH,SAAS,oBAAoB,CAAC,SAAiB,EAAE,MAAc;IAC7D,IAAI,CAAC;QACH,EAAE,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACxC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,GAAG,CAAC,KAAK,CAAC,oBAAoB,MAAM,sBAAsB,eAAe,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IACtF,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,cAAc,CAAC,SAAiB;IAC9C,gBAAgB,EAAE,CAAC;IACnB,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IACnD,gBAAgB,CAAC,SAAS,CAAC,SAAS,EAAE,SAAS,CAAC,QAAQ,EAAE,CAAC,CAAC;AAC9D,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,aAAa;IAC3B,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IAEnD,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;QAC9B,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;QACnD,MAAM,GAAG,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC;QAEnC,IAAI,GAAG,KAAK,IAAI,EAAE,CAAC;YACjB,oBAAoB,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;YAC3C,OAAO,IAAI,CAAC;QACd,CAAC;QAED,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,EAAE,CAAC;YACzB,oBAAoB,CAAC,SAAS,EAAE,OAAO,CAAC,CAAC;YACzC,OAAO,IAAI,CAAC;QACd,CAAC;QAED,OAAO,GAAG,CAAC;IACb,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,cAAc;IAC5B,MAAM,SAAS,GAAG,kBAAkB,CAAC,YAAY,CAAC,CAAC;IAEnD,IAAI,CAAC;QACH,EAAE,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACxC,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,GAAG,CAAC,KAAK,CAAC,qCAAqC,eAAe,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IAC3E,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB;IACtC,MAAM,EACJ,8BAA8B,EAC9B,+BAA+B,EAC/B,2BAA2B,EAC3B,2BAA2B,EAC3B,0BAA0B,EAC1B,iCAAiC,GAClC,GAAG,MAAM,MAAM,CAAC,yBAAyB,CAAC,CAAC;IAE5C,OAAO,CAAC,KAAK,CAAC,8BAA8B,EAAE,CAAC,CAAC;IAEhD,IAAI,CAAC;QACH,MAAM,EAAE,iBAAiB,EAAE,GAAG,MAAM,MAAM,CAAC,cAAc,CAAC,CAAC;QAE3D,MAAM,SAAS,GAAG,aAAa,EAAE,CAAC;QAElC,IAAI,CAAC,SAAS,EAAE,CAAC;YACf,OAAO,CAAC,KAAK,CAAC,+BAA+B,EAAE,CAAC,CAAC;YACjD,OAAO,CAAC,CAAC;QACX,CAAC;QAED,OAAO,CAAC,KAAK,CAAC,2BAA2B,CAAC,SAAS,CAAC,CAAC,CAAC;QAEtD,IAAI,CAAC;YACH,iBAAiB,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;YACxC,OAAO,CAAC,KAAK,CAAC,2BAA2B,EAAE,CAAC,CAAC;YAC7C,cAAc,EAAE,CAAC;YACjB,OAAO,CAAC,CAAC;QACX,CAAC;QAAC,OAAO,SAAS,EAAE,CAAC;YACnB,OAAO,CAAC,KAAK,CAAC,0BAA0B,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;YACtE,OAAO,CAAC,CAAC;QACX,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,CAAC,KAAK,CAAC,iCAAiC,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QACzE,OAAO,CAAC,CAAC;IACX,CAAC;AACH,CAAC"}

package/dist/session/cleanup.d.ts

@@ -28,10 +28,23 @@ export declare function cleanupStaleSession(): boolean;
 /**
  * Cleanup all session files after a session ends.
  *
- * Removes PID, lock, and metadata files.
+ * Removes session-specific files while preserving chrome-profile directory
+ * which contains user preferences, cookies, and cached data.
+ *
+ * Files removed:
+ * - session.pid (worker PID)
+ * - session.lock (session lock)
+ * - session.meta.json (session metadata)
+ * - daemon.pid, daemon.sock, daemon.lock (daemon files)
+ *
+ * Files preserved:
+ * - session.json (output file - user needs to read it)
+ * - chrome.pid (for emergency cleanup - auto-removed if Chrome is dead)
+ * - chrome-profile/ directory (cookies, preferences, cache)
+ *
  * Safe to call multiple times (idempotent).
  *
- * WHY: Ensures clean slate for next session, prevents stale file accumulation.
+ * WHY: Ensures clean slate for next session while preserving user preferences.
  */
 export declare function cleanupSession(): void;
 /**

package/dist/session/cleanup.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cleanup.d.ts","sourceRoot":"","sources":["../../src/session/cleanup.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AA4DH;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,mBAAmB,IAAI,OAAO,CAgF7C;AAED;;;;;;;GAOG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAYrC;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,IAAI,OAAO,CA0D/C"}
+{"version":3,"file":"cleanup.d.ts","sourceRoot":"","sources":["../../src/session/cleanup.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AA6CH;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,mBAAmB,IAAI,OAAO,CA0F7C;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAyCrC;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,IAAI,OAAO,CAoD/C"}