fix: pass categoryExtensions to the target filter when connecting

src/BrowserManager.ts

@@ -336,12 +336,13 @@ export class BrowserManager {
       userDataDir,
       blockedUrlPattern: blocklist,
       allowedUrlPattern: allowlist,
+      categoryExtensions: enableExtensions,
     } = this.#serverArgs;
     // Important: only pass channel, if autoConnect is true.
     const channel = isAutoConnect ? this.#serverArgs.channel : undefined;
 
     const connectOptions: Parameters<typeof puppeteer.connect>[0] = {
-      targetFilter: BrowserManager.makeTargetFilter(),
+      targetFilter: BrowserManager.makeTargetFilter(enableExtensions),
       defaultViewport: null,
       handleDevToolsAsPage: true,
       blocklist,

tests/browser.test.ts

@@ -149,6 +149,28 @@ describe('browser', () => {
       sinon.assert.notCalled(pptrBrowser.close);
     });
 
+    for (const categoryExtensions of [false, true]) {
+      it(`${categoryExtensions ? 'keeps' : 'drops'} extension targets when connecting with categoryExtensions=${categoryExtensions}`, async () => {
+        const connectStub = sinon
+          .stub(puppeteer, 'connect')
+          .resolves(createMockPuppeteerBrowser());
+
+        const args = createMockParsedArguments({
+          browserUrl: 'http://127.0.0.1:9222',
+          categoryExtensions,
+        });
+        await new BrowserManager(args).ensureBrowser();
+
+        const {targetFilter} = connectStub.firstCall.args[0]!;
+        assert.ok(targetFilter);
+        const target = {url: () => 'chrome-extension://abcdefgh/popup.html'};
+        assert.strictEqual(
+          targetFilter(target as Parameters<typeof targetFilter>[0]),
+          categoryExtensions,
+        );
+      });
+    }
+
     it('deduplicates concurrent ensureBrowser() calls while launch is in-flight', async () => {
       const pptrBrowser = createMockPuppeteerBrowser();
       const {promise, resolve} = Promise.withResolvers<Browser>();