chore: unify condition names with experimental convention

Author: nroscino

scripts/generate-cli.ts

@@ -11,7 +11,7 @@ import {Client} from '@modelcontextprotocol/sdk/client/index.js';
 import {StdioClientTransport} from '@modelcontextprotocol/sdk/client/stdio.js';
 
 import {parseArguments} from '../build/src/bin/chrome-devtools-mcp-cli-options.js';
-import {CONDITION_TO_FLAG, buildFlag} from '../build/src/index.js';
+import {buildFlag} from '../build/src/index.js';
 import {
   labels,
   ToolCategory,
@@ -174,11 +174,7 @@ async function generateCli() {
 
     const conditions = toolNameToConditions.get(tool.name) || [];
     for (const condition of conditions) {
-      const flag =
-        CONDITION_TO_FLAG[condition as keyof typeof CONDITION_TO_FLAG];
-      if (flag) {
-        requiredFlags.push(`--${flag}=true`);
-      }
+      requiredFlags.push(`--${condition}=true`);
     }
 
     if (requiredFlags.length > 0) {

src/index.ts

@@ -34,14 +34,6 @@ import {pageIdSchema} from './tools/ToolDefinition.js';
 import {createTools} from './tools/tools.js';
 import {VERSION} from './version.js';
 
-export const CONDITION_TO_FLAG: Record<string, string> = {
-  computerVision: 'experimentalVision',
-  experimentalMemory: 'experimentalMemory',
-  experimentalInteropTools: 'experimentalInteropTools',
-  screencast: 'experimentalScreencast',
-  experimentalWebmcp: 'experimentalWebmcp',
-};
-
 export function buildFlag(category: ToolCategory) {
   return category === ToolCategory.IN_PAGE
     ? 'categoryInPageTools'
@@ -88,9 +80,8 @@ function getConditionStatus(
   condition: string,
   serverArgs: ReturnType<typeof parseArguments>,
 ): {conditionFlag?: string; disabled: boolean} {
-  const experimentalFlag = CONDITION_TO_FLAG[condition];
-  if (experimentalFlag && !serverArgs[experimentalFlag]) {
-    return {conditionFlag: experimentalFlag, disabled: true};
+  if (condition && !serverArgs[condition]) {
+    return {conditionFlag: condition, disabled: true};
   }
 
   return {disabled: false};

src/tools/inPage.ts

@@ -48,7 +48,7 @@ export const listInPageTools = definePageTool({
   annotations: {
     category: ToolCategory.IN_PAGE,
     readOnlyHint: true,
-    conditions: ['inPageTools'],
+    conditions: ['experimentalInPageTools'],
   },
   schema: {},
   handler: async (_request, response, _context) => {
@@ -62,7 +62,7 @@ export const executeInPageTool = definePageTool({
   annotations: {
     category: ToolCategory.IN_PAGE,
     readOnlyHint: false,
-    conditions: ['inPageTools'],
+    conditions: ['experimentalInPageTools'],
   },
   schema: {
     toolName: zod.string().describe('The name of the tool to execute'),

src/tools/input.ts

@@ -89,7 +89,7 @@ export const clickAt = definePageTool({
   annotations: {
     category: ToolCategory.INPUT,
     readOnlyHint: false,
-    conditions: ['computerVision'],
+    conditions: ['experimentalVision'],
   },
   schema: {
     x: zod.number().describe('The x coordinate'),

src/tools/screencast.ts

@@ -29,7 +29,7 @@ export const startScreencast = definePageTool(args => ({
     category: ToolCategory.DEBUGGING,
     readOnlyHint: false,
 
-    conditions: ['screencast'],
+    conditions: ['experimentalScreencast'],
   },
   schema: {
     filePath: zod
@@ -99,7 +99,7 @@ export const stopScreencast = definePageTool({
   annotations: {
     category: ToolCategory.DEBUGGING,
     readOnlyHint: false,
-    conditions: ['screencast'],
+    conditions: ['experimentalScreencast'],
   },
   schema: {},
   handler: async (_request, response, context) => {

tests/index.test.ts

@@ -114,7 +114,7 @@ describe('e2e', () => {
         );
         assert.ok(listInPageTools);
       },
-      ['--category-in-page-tools'],
+      ['--category-in-page-tools', '--experimental-in-page-tools'],
     );
   });