feat: support allowedUrlPattern & blockedUrlPattern Options by nattallius · Pull Request #2037 · ChromeDevTools/chrome-devtools-mcp

nattallius · 2026-05-11T19:10:49Z

No description provided.

OrKoN · 2026-05-18T12:36:01Z

+      },
+      {
+        blockedUrlPattern: [server.getRoute('/blocked.html')],
+        executablePath: process.env.CHROME_M149_EXECUTABLE_PATH,


block list should not need M149, right?

No, it also fails if runs against stable

OrKoN · 2026-05-18T12:36:17Z

+      },
+      {
+        blockedUrlPattern: [server.getRoute('/blocked.html')],
+        executablePath: process.env.CHROME_M149_EXECUTABLE_PATH,


blocklist should not need M149?

Lightning00Blade

LGTM with some small comment and questions.

Lightning00Blade · 2026-06-02T09:05:03Z

  });
 }

+const chromePath = _installChrome('149.0.7827.14');


Do we need this this is already stable https://chromestatus.com/roadmap
And that is the version pinned by Puppeteer.

Lightning00Blade · 2026-06-02T09:18:02Z

-      await page.emulateNetworkConditions(networkCondition);
-      newSettings.networkConditions = options.networkConditions;
+    // Skip network emulation if blocklist/allowlist is configured, as it is rejected by Puppeteer.
+    if (!this.#options.hasNetworkBlockOrAllowlist) {


Should we throw if we try to set emulation? Reason is I try to emulate network conditions and this just silently ignores it, an LLM most likely will be confused.

Lightning00Blade · 2026-06-02T09:20:40Z

+            await fetch(url);
+            return 'SUCCESS';
+          } catch (err) {
+            return err instanceof Error ? err.message : String(err);


Nit: We can just return true or false as the message may change.

Lightning00Blade · 2026-06-02T09:21:07Z

+
+    it(
+      'blocks URLs not in allowlist',
+      {skip: 'Requires Chrome 149 or greater'},


Is this also still needed?

Lightning00Blade · 2026-06-02T09:22:45Z

+      [
+        'node',
+        'main.js',
+        '--blocked-url-pattern=https://a.com/*',


Can you double check if '--blocked-url-pattern', '<pattern-1>', <pattern-2> works? I think that is also valid CLI args.

Lightning00Blade · 2026-06-02T09:24:12Z

+              function: `async () => {
+                try {
+                  await fetch("${blockedUrl}");
+                  return 'SUCCESS';


Same here. we can use true/false.

Lightning00Blade · 2026-06-02T09:26:04Z

+        );
+      },
+      {
+        blockedUrlPattern: [server.getRoute('/blocked.html')],


This was confusingly at the end we should move it to the start, some where easier to find.

Lightning00Blade · 2026-06-02T09:26:45Z

+      },
+      {
+        allowedUrlPattern: [server.getRoute('/allowed.html')],
+        executablePath: process.env.CHROME_M149_EXECUTABLE_PATH,


Do we need this?

nattallius requested a review from OrKoN May 11, 2026 19:10

nattallius force-pushed the add-blocklist-and-allowlist-options branch from b5eca0f to 788efce Compare May 11, 2026 19:16

OrKoN reviewed May 12, 2026

View reviewed changes

Comment thread src/bin/chrome-devtools-mcp-cli-options.ts Outdated

OrKoN reviewed May 12, 2026

View reviewed changes

Comment thread src/bin/chrome-devtools-mcp-cli-options.ts Outdated

OrKoN reviewed May 12, 2026

View reviewed changes

Comment thread src/bin/chrome-devtools-mcp-cli-options.ts

nattallius force-pushed the add-blocklist-and-allowlist-options branch from f537e11 to 998d4be Compare May 18, 2026 11:04

OrKoN reviewed May 18, 2026

View reviewed changes

Comment thread tests/security_policies.test.ts Outdated

OrKoN reviewed May 18, 2026

View reviewed changes

nattallius force-pushed the add-blocklist-and-allowlist-options branch from 703ef62 to eab8536 Compare May 18, 2026 12:36

OrKoN reviewed May 18, 2026

View reviewed changes

nattallius changed the title ~~feat: Implement url allowlist and blocklist~~ feat: Support allowedUrlPattern & blockedUrlPattern Options May 18, 2026

OrKoN force-pushed the add-blocklist-and-allowlist-options branch from b90f3a7 to 57c0f95 Compare May 19, 2026 09:41

nattallius changed the title ~~feat: Support allowedUrlPattern & blockedUrlPattern Options~~ feat: support allowedUrlPattern & blockedUrlPattern Options May 19, 2026

Natallia Harshunova and others added 11 commits June 2, 2026 07:28

Implement url allowlist and blocklist

df09ac6

update docs

790b708

Update documentation

6b21527

Refactor options configuration

e17cf83

Refactor tests

82fc71a

Rename cli arguments to allowedUrlPattern and blockedUrlPattern

688644c

prevent netwrok emulation if there is allowlist or blocklist

1bc3ce2

file and description rename

c6392c1

Run tests only on 149 version

2250c76

Prevent devtools from override netwrok conditions

e6a761e

chore: fix network

6f1034f

nattallius force-pushed the add-blocklist-and-allowlist-options branch from 57c0f95 to 6f1034f Compare June 2, 2026 07:33

Lightning00Blade approved these changes Jun 2, 2026

View reviewed changes

Conversation

nattallius commented May 11, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Lightning00Blade left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants