remove toggle tool. bump up timeouts while i investigate them

Change-Id: I921880b40788863151962d13dc9838c4793a840f

Author: paulirish

README.md

@@ -333,11 +333,10 @@ If you run into any issues, checkout our [troubleshooting guide](./docs/troubles
 - **Emulation** (2 tools)
   - [`emulate`](docs/tool-reference.md#emulate)
   - [`resize_page`](docs/tool-reference.md#resize_page)
-- **Performance** (4 tools)
+- **Performance** (3 tools)
   - [`performance_analyze_insight`](docs/tool-reference.md#performance_analyze_insight)
   - [`performance_start_trace`](docs/tool-reference.md#performance_start_trace)
   - [`performance_stop_trace`](docs/tool-reference.md#performance_stop_trace)
-  - [`performance_toggle_crux`](docs/tool-reference.md#performance_toggle_crux)
 - **Network** (2 tools)
   - [`get_network_request`](docs/tool-reference.md#get_network_request)
   - [`list_network_requests`](docs/tool-reference.md#list_network_requests)

docs/tool-reference.md

@@ -21,11 +21,10 @@
 - **[Emulation](#emulation)** (2 tools)
   - [`emulate`](#emulate)
   - [`resize_page`](#resize_page)
-- **[Performance](#performance)** (4 tools)
+- **[Performance](#performance)** (3 tools)
   - [`performance_analyze_insight`](#performance_analyze_insight)
   - [`performance_start_trace`](#performance_start_trace)
   - [`performance_stop_trace`](#performance_stop_trace)
-  - [`performance_toggle_crux`](#performance_toggle_crux)
 - **[Network](#network)** (2 tools)
   - [`get_network_request`](#get_network_request)
   - [`list_network_requests`](#list_network_requests)
@@ -246,16 +245,6 @@
 
 ---
 
-### `performance_toggle_crux`
-
-**Description:** Enables or disables the fetching of real-user experience data from the Chrome User Experience Report (CrUX) API during performance traces. When enabled, performance summaries will include field data (LCP, INP, CLS) for the URLs in the trace.
-
-**Parameters:**
-
-- **enabled** (boolean) **(required)**: Whether to enable or disable CrUX data fetching.
-
----
-
 ## Network
 
 ### `get_network_request`

src/McpContext.ts

@@ -60,8 +60,8 @@ interface McpContextOptions {
   experimentalIncludeAllPages?: boolean;
 }
 
-const DEFAULT_TIMEOUT = 5_000;
-const NAVIGATION_TIMEOUT = 10_000;
+const DEFAULT_TIMEOUT = 30_000;
+const NAVIGATION_TIMEOUT = 60_000;
 
 function getNetworkMultiplierFromString(condition: string | null): number {
   const puppeteerCondition =

src/tools/performance.ts

@@ -5,7 +5,7 @@
  */
 
 import {logger} from '../logger.js';
-import {DevTools, zod} from '../third_party/index.js';
+import {zod} from '../third_party/index.js';
 import type {Page} from '../third_party/index.js';
 import type {InsightName} from '../trace-processing/parse.js';
 import {
@@ -162,39 +162,6 @@ export const analyzeInsight = defineTool({
   },
 });
 
-export const toggleCrux = defineTool({
-  name: 'performance_toggle_crux',
-  description:
-    'Enables or disables the fetching of real-user experience data from the Chrome User Experience Report (CrUX) API during performance traces. When enabled, performance summaries will include field data (LCP, INP, CLS) for the URLs in the trace.',
-  annotations: {
-    category: ToolCategory.PERFORMANCE,
-    readOnlyHint: false,
-  },
-  schema: {
-    enabled: zod
-      .boolean()
-      .describe('Whether to enable or disable CrUX data fetching.'),
-  },
-  handler: async (request, response) => {
-    try {
-      const settings = DevTools.Common.Settings.Settings.instance();
-      const cruxSetting = settings.createSetting(
-        'field-data',
-        {enabled: false},
-        DevTools.Common.Settings.SettingStorageType.GLOBAL,
-      );
-      cruxSetting.set({...cruxSetting.get(), enabled: request.params.enabled});
-      response.appendResponseLine(
-        `CrUX data fetching has been ${request.params.enabled ? 'enabled' : 'disabled'}.`,
-      );
-    } catch {
-      response.appendResponseLine(
-        'Error: Could not update the CrUX setting. It might not be available in this environment.',
-      );
-    }
-  },
-});
-
 async function stopTracingAndAppendOutput(
   page: Page,
   response: Response,

tests/utils/crux.test.ts

@@ -65,9 +65,10 @@ describe('crux util', () => {
     const mockCrUXManager = {
       getSelectedScope: () => ({pageScope: 'url', deviceScope: 'ALL'}),
     };
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
     sinon
       .stub(DevTools.CrUXManager.CrUXManager, 'instance')
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       .returns(mockCrUXManager as any);
 
     const settings = DevTools.Common.Settings.Settings.instance();