changing category in mcp-cli-options

Author: nroscino

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

@@ -237,9 +237,10 @@ export const cliOptions = {
     describe:
       'Set to true to include tools related to extensions. Note: This feature is currently only supported with a pipe connection. autoConnect, browserUrl, and wsEndpoint are not supported with this feature until 149 will be released.',
   },
-  categoryInPageTools: {
+  categoryExperimentalInPage: {
     type: 'boolean',
     hidden: true,
+    default: false,
     describe:
       'Set to true to enable tools exposed by the inspected page itself',
   },

src/telemetry/flag_usage_metrics.json

@@ -192,11 +192,13 @@
   },
   {
     "name": "category_in_page_tools",
-    "flagType": "boolean"
+    "flagType": "boolean",
+    "isDeprecated": true
   },
   {
     "name": "category_in_page_tools_present",
-    "flagType": "boolean"
+    "flagType": "boolean",
+    "isDeprecated": true
   },
   {
     "name": "clearcut_endpoint_present",
@@ -265,5 +267,13 @@
   {
     "name": "via_cli_present",
     "flagType": "boolean"
+  },
+  {
+    "name": "category_experimental_in_page_present",
+    "flagType": "boolean"
+  },
+  {
+    "name": "category_experimental_in_page",
+    "flagType": "boolean"
   }
 ]

tests/McpResponse.test.ts

@@ -1111,7 +1111,7 @@ describe('inPage tools', () => {
         t.assert.snapshot?.(JSON.stringify(structuredContent, null, 2));
       },
       undefined,
-      {categoryInPageTools: true} as ParsedArguments,
+      {categoryExperimentalInPage: true} as ParsedArguments,
     );
   });
 
@@ -1162,14 +1162,14 @@ describe('inPage tools', () => {
         );
       },
       undefined,
-      {categoryInPageTools: true} as ParsedArguments,
+      {categoryExperimentalInPage: true} as ParsedArguments,
     );
   }
 
   it('includes in-page tools in list_pages response', async () => {
     await testIncludesInPageTools(async (response, context) => {
       const listPagesDef = listPages({
-        categoryInPageTools: true,
+        categoryExperimentalInPage: true,
       } as ParsedArguments);
       await listPagesDef.handler({params: {}}, response, context);
     }, 'list_pages');

tests/tools/inPage.test.ts

@@ -72,7 +72,7 @@ describe('inPage', () => {
           });
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -102,7 +102,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -131,7 +131,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -150,7 +150,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
   });
@@ -215,7 +215,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -302,7 +302,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -347,7 +347,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -486,7 +486,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -535,7 +535,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -590,7 +590,7 @@ describe('inPage', () => {
           );
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -647,7 +647,7 @@ describe('inPage', () => {
           stub.restore();
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -713,7 +713,7 @@ describe('inPage', () => {
           stubSnapshot.restore();
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
 
@@ -771,7 +771,7 @@ describe('inPage', () => {
           stubSnapshot.restore();
         },
         undefined,
-        {categoryInPageTools: true} as ParsedArguments,
+        {categoryExperimentalInPage: true} as ParsedArguments,
       );
     });
   });