From 8374c2abf13fa803233025192b8d7e87de70b087 Mon Sep 17 00:00:00 2001 From: Ruslan Lesiutin <28902667+hoxyq@users.noreply.github.com> Date: Wed, 11 Feb 2026 16:59:43 +0000 Subject: [PATCH] [DevTools] Remove experimental __IS_INTERNAL_MCP_BUILD__ flag and related code (#35755) This is unused. --- .eslintrc.js | 1 - .../react-devtools-core/webpack.backend.js | 1 - .../react-devtools-core/webpack.standalone.js | 1 - .../webpack.config.js | 6 -- .../webpack.config.frontend.js | 1 - .../react-devtools-inline/webpack.config.js | 1 - .../src/backend/fiber/renderer.js | 81 ------------------- scripts/flow/react-devtools.js | 1 - scripts/jest/devtools/setupEnv.js | 1 - 9 files changed, 94 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index 952149f30c..2cebe8698c 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -507,7 +507,6 @@ module.exports = { __IS_FIREFOX__: 'readonly', __IS_EDGE__: 'readonly', __IS_NATIVE__: 'readonly', - __IS_INTERNAL_MCP_BUILD__: 'readonly', __IS_INTERNAL_VERSION__: 'readonly', chrome: 'readonly', }, diff --git a/packages/react-devtools-core/webpack.backend.js b/packages/react-devtools-core/webpack.backend.js index c1312fc6d8..32d4fadcb5 100644 --- a/packages/react-devtools-core/webpack.backend.js +++ b/packages/react-devtools-core/webpack.backend.js @@ -72,7 +72,6 @@ module.exports = { __IS_CHROME__: false, __IS_EDGE__: false, __IS_NATIVE__: true, - __IS_INTERNAL_MCP_BUILD__: false, 'process.env.DEVTOOLS_PACKAGE': `"react-devtools-core"`, 'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`, 'process.env.GITHUB_URL': `"${GITHUB_URL}"`, diff --git a/packages/react-devtools-core/webpack.standalone.js b/packages/react-devtools-core/webpack.standalone.js index 6279ba7fb3..01437e7f0f 100644 --- a/packages/react-devtools-core/webpack.standalone.js +++ b/packages/react-devtools-core/webpack.standalone.js @@ -91,7 +91,6 @@ module.exports = { __IS_FIREFOX__: false, __IS_CHROME__: false, __IS_EDGE__: false, - __IS_INTERNAL_MCP_BUILD__: false, 'process.env.DEVTOOLS_PACKAGE': `"react-devtools-core"`, 'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`, 'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null, diff --git a/packages/react-devtools-extensions/webpack.config.js b/packages/react-devtools-extensions/webpack.config.js index 7b5acca6cc..129816e907 100644 --- a/packages/react-devtools-extensions/webpack.config.js +++ b/packages/react-devtools-extensions/webpack.config.js @@ -34,8 +34,6 @@ const IS_FIREFOX = process.env.IS_FIREFOX === 'true'; const IS_EDGE = process.env.IS_EDGE === 'true'; const IS_INTERNAL_VERSION = process.env.FEATURE_FLAG_TARGET === 'extension-fb'; -const IS_INTERNAL_MCP_BUILD = process.env.IS_INTERNAL_MCP_BUILD === 'true'; - const featureFlagTarget = process.env.FEATURE_FLAG_TARGET || 'extension-oss'; let statsFileName = `webpack-stats.${featureFlagTarget}.${__DEV__ ? 'development' : 'production'}`; @@ -48,9 +46,6 @@ if (IS_FIREFOX) { if (IS_EDGE) { statsFileName += `.edge`; } -if (IS_INTERNAL_MCP_BUILD) { - statsFileName += `.mcp`; -} statsFileName += '.json'; const babelOptions = { @@ -139,7 +134,6 @@ module.exports = { __IS_FIREFOX__: IS_FIREFOX, __IS_EDGE__: IS_EDGE, __IS_NATIVE__: false, - __IS_INTERNAL_MCP_BUILD__: IS_INTERNAL_MCP_BUILD, __IS_INTERNAL_VERSION__: IS_INTERNAL_VERSION, 'process.env.DEVTOOLS_PACKAGE': `"react-devtools-extensions"`, 'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`, diff --git a/packages/react-devtools-fusebox/webpack.config.frontend.js b/packages/react-devtools-fusebox/webpack.config.frontend.js index 0d56e81a34..a2e09f0e0a 100644 --- a/packages/react-devtools-fusebox/webpack.config.frontend.js +++ b/packages/react-devtools-fusebox/webpack.config.frontend.js @@ -85,7 +85,6 @@ module.exports = { __IS_CHROME__: false, __IS_FIREFOX__: false, __IS_EDGE__: false, - __IS_INTERNAL_MCP_BUILD__: false, 'process.env.DEVTOOLS_PACKAGE': `"react-devtools-fusebox"`, 'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`, 'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null, diff --git a/packages/react-devtools-inline/webpack.config.js b/packages/react-devtools-inline/webpack.config.js index a0300069bf..f2b5fd8a6f 100644 --- a/packages/react-devtools-inline/webpack.config.js +++ b/packages/react-devtools-inline/webpack.config.js @@ -77,7 +77,6 @@ module.exports = { __IS_FIREFOX__: false, __IS_EDGE__: false, __IS_NATIVE__: false, - __IS_INTERNAL_MCP_BUILD__: false, 'process.env.DEVTOOLS_PACKAGE': `"react-devtools-inline"`, 'process.env.DEVTOOLS_VERSION': `"${DEVTOOLS_VERSION}"`, 'process.env.EDITOR_URL': EDITOR_URL != null ? `"${EDITOR_URL}"` : null, diff --git a/packages/react-devtools-shared/src/backend/fiber/renderer.js b/packages/react-devtools-shared/src/backend/fiber/renderer.js index 7a01d6c598..266346a6bb 100644 --- a/packages/react-devtools-shared/src/backend/fiber/renderer.js +++ b/packages/react-devtools-shared/src/backend/fiber/renderer.js @@ -8871,86 +8871,6 @@ export function attach( return unresolvedSource; } - type InternalMcpFunctions = { - __internal_only_getComponentTree?: Function, - }; - - const internalMcpFunctions: InternalMcpFunctions = {}; - if (__IS_INTERNAL_MCP_BUILD__) { - // eslint-disable-next-line no-inner-declarations - function __internal_only_getComponentTree(): string { - let treeString = ''; - - function buildTreeString( - instance: DevToolsInstance, - prefix: string = '', - isLastChild: boolean = true, - ): void { - if (!instance) return; - - const name = - (instance.kind !== VIRTUAL_INSTANCE - ? getDisplayNameForFiber(instance.data) - : instance.data.name) || 'Unknown'; - - const id = instance.id !== undefined ? instance.id : 'unknown'; - - if (name !== 'createRoot()') { - treeString += - prefix + - (isLastChild ? '└── ' : '├── ') + - name + - ' (id: ' + - id + - ')\n'; - } - - const childPrefix = prefix + (isLastChild ? ' ' : '│ '); - - let childCount = 0; - let tempChild = instance.firstChild; - while (tempChild !== null) { - childCount++; - tempChild = tempChild.nextSibling; - } - - let child = instance.firstChild; - let currentChildIndex = 0; - - while (child !== null) { - currentChildIndex++; - const isLastSibling = currentChildIndex === childCount; - buildTreeString(child, childPrefix, isLastSibling); - child = child.nextSibling; - } - } - - const rootInstances: Array = []; - idToDevToolsInstanceMap.forEach(instance => { - if (instance.parent === null || instance.parent.parent === null) { - rootInstances.push(instance); - } - }); - - if (rootInstances.length > 0) { - for (let i = 0; i < rootInstances.length; i++) { - const isLast = i === rootInstances.length - 1; - buildTreeString(rootInstances[i], '', isLast); - if (!isLast) { - treeString += '\n'; - } - } - } else { - treeString = 'No component tree found.'; - } - - return treeString; - } - - internalMcpFunctions.__internal_only_getComponentTree = - __internal_only_getComponentTree; - } - return { cleanup, clearErrorsAndWarnings, @@ -8994,6 +8914,5 @@ export function attach( supportsTogglingSuspense, updateComponentFilters, getEnvironmentNames, - ...internalMcpFunctions, }; } diff --git a/scripts/flow/react-devtools.js b/scripts/flow/react-devtools.js index 09a251bbe2..4e0f2a915e 100644 --- a/scripts/flow/react-devtools.js +++ b/scripts/flow/react-devtools.js @@ -16,6 +16,5 @@ declare const __IS_FIREFOX__: boolean; declare const __IS_CHROME__: boolean; declare const __IS_EDGE__: boolean; declare const __IS_NATIVE__: boolean; -declare const __IS_INTERNAL_MCP_BUILD__: boolean; declare const chrome: any; diff --git a/scripts/jest/devtools/setupEnv.js b/scripts/jest/devtools/setupEnv.js index 32bf13e686..a797c09514 100644 --- a/scripts/jest/devtools/setupEnv.js +++ b/scripts/jest/devtools/setupEnv.js @@ -15,7 +15,6 @@ global.__IS_FIREFOX__ = false; global.__IS_CHROME__ = false; global.__IS_EDGE__ = false; global.__IS_NATIVE__ = false; -global.__IS_INTERNAL_MCP_BUILD__ = false; const ReactVersionTestingAgainst = process.env.REACT_VERSION || ReactVersion;