fix: removing default to false for categoryExtensions to fix tests

nroscino · nroscino · commit 6fb8295c1bf5 · 2026-03-18T16:57:17.000+01:00
diff --git a/src/bin/chrome-devtools-mcp-cli-options.ts b/src/bin/chrome-devtools-mcp-cli-options.ts
@@ -211,7 +211,6 @@ export const cliOptions = {
   },
   categoryExtensions: {
     type: 'boolean',
-    default: false,
     hidden: true,
     conflicts: ['browserUrl', 'autoConnect', 'wsEndpoint'],
     describe:
diff --git a/src/index.ts b/src/index.ts
@@ -129,7 +129,7 @@ export async function createMcpServer(
     }
     if (
       tool.annotations.category === ToolCategory.EXTENSIONS &&
-      serverArgs.categoryExtensions === false
+      !serverArgs.categoryExtensions
     ) {
       return;
     }
diff --git a/tests/cli.test.ts b/tests/cli.test.ts
@@ -15,8 +15,6 @@ describe('cli args parsing', () => {
     categoryEmulation: true,
     'category-performance': true,
     categoryPerformance: true,
-    'category-extensions': false,
-    categoryExtensions: false,
     'category-network': true,
     categoryNetwork: true,
     'auto-connect': undefined,

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ export async function createMcpServer(`
`129`	`129`	`}`
`130`	`130`	`if (`
`131`	`131`	`tool.annotations.category === ToolCategory.EXTENSIONS &&`
`132`		`- serverArgs.categoryExtensions === false`
	`132`	`+ !serverArgs.categoryExtensions`
`133`	`133`	`) {`
`134`	`134`	`return;`
`135`	`135`	`}`