Refactor options configuration

Author: Natallia Harshunova

README.md

@@ -616,12 +616,12 @@ The Chrome DevTools MCP server supports the following configuration option:
   - **Type:** array
 
 - **`--blocklist`**
-  Restricts network access by blocking specified URL patterns (uses URLPattern API https://urlpattern.spec.whatwg.org/). Silently detaches from targets with blocked URLs upon connection, and blocks runtime requests (including navigations and subresources).
-  - **Type:** array
+  Restricts network access by blocking specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Silently detaches from targets with blocked URLs upon connection, and blocks runtime requests (including navigations and subresources). Supports comma-separated patterns.
+  - **Type:** string
 
 - **`--allowlist`**
-  Restricts network access by allowing only specified URL patterns (uses URLPattern API https://urlpattern.spec.whatwg.org/). Requires Chrome 149+. Silently detaches from targets with unallowed URLs upon connection, and blocks runtime requests (including navigations and subresources).
-  - **Type:** array
+  Restricts network access by allowing only specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Requires Chrome 149+. Silently detaches from targets with unallowed URLs upon connection, and blocks runtime requests (including navigations and subresources). Supports comma-separated patterns.
+  - **Type:** string
 
 - **`--ignoreDefaultChromeArg`/ `--ignore-default-chrome-arg`**
   Explicitly disable default arguments for Chrome. Only applies when Chrome is launched by chrome-devtools-mcp.

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

@@ -211,16 +211,36 @@ export const cliOptions = {
       'Additional arguments for Chrome. Only applies when Chrome is launched by chrome-devtools-mcp.',
   },
   blocklist: {
-    type: 'array',
+    type: 'string',
     describe:
-      'Restricts network access by blocking specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Silently detaches from targets with blocked URLs upon connection, and blocks runtime requests (including navigations and subresources).',
+      'Restricts network access by blocking specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Silently detaches from targets with blocked URLs upon connection, and blocks runtime requests (including navigations and subresources). Supports comma-separated patterns.',
     conflicts: ['allowlist'],
+    coerce: (value: string | string[] | undefined) => {
+      if (!value) {
+        return undefined;
+      }
+      const arr = Array.isArray(value) ? value : [value];
+      return arr
+        .flatMap(val => val.split(','))
+        .map(s => s.trim())
+        .filter(Boolean);
+    },
   },
   allowlist: {
-    type: 'array',
+    type: 'string',
     describe:
-      'Restricts network access by allowing only specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Requires Chrome 149+. Silently detaches from targets with unallowed URLs upon connection, and blocks runtime requests (including navigations and subresources).',
+      'Restricts network access by allowing only specified URL patterns (uses https://urlpattern.spec.whatwg.org/). Requires Chrome 149+. Silently detaches from targets with unallowed URLs upon connection, and blocks runtime requests (including navigations and subresources). Supports comma-separated patterns.',
     conflicts: ['blocklist'],
+    coerce: (value: string | string[] | undefined) => {
+      if (!value) {
+        return undefined;
+      }
+      const arr = Array.isArray(value) ? value : [value];
+      return arr
+        .flatMap(val => val.split(','))
+        .map(s => s.trim())
+        .filter(Boolean);
+    },
   },
   ignoreDefaultChromeArg: {
     type: 'array',

tests/cli.test.ts

@@ -345,4 +345,31 @@ describe('cli args parsing', () => {
     );
     assert.strictEqual(disabledArgs.performanceCrux, false);
   });
+
+  it('parses blocklist and allowlist flags with comma separation', async () => {
+    const defaultArgs = parseArguments('1.0.0', ['node', 'main.js']);
+    assert.strictEqual(defaultArgs.blocklist, undefined);
+    assert.strictEqual(defaultArgs.allowlist, undefined);
+
+    const blocklistArgs = parseArguments(
+      '1.0.0',
+      [
+        'node',
+        'main.js',
+        '--blocklist=https://example.com/ads/*,https://tracker.com/*',
+      ],
+      {},
+    );
+    assert.deepStrictEqual(blocklistArgs.blocklist, [
+      'https://example.com/ads/*',
+      'https://tracker.com/*',
+    ]);
+
+    const allowlistArgs = parseArguments(
+      '1.0.0',
+      ['node', 'main.js', '--allowlist=https://trusted.com/*'],
+      {},
+    );
+    assert.deepStrictEqual(allowlistArgs.allowlist, ['https://trusted.com/*']);
+  });
 });