Skip to content

Comments

Clean up legacy rn_inspector entry point#236

Merged
huntie merged 1 commit intofacebook:mainfrom
huntie:remove-legacy-rn-entry-point
Feb 23, 2026
Merged

Clean up legacy rn_inspector entry point#236
huntie merged 1 commit intofacebook:mainfrom
huntie:remove-legacy-rn-entry-point

Conversation

@huntie
Copy link
Member

@huntie huntie commented Feb 10, 2026

Summary

Note

Depends on facebook/react-native#55548 landing in the same React Native release.

Removes the legacy rn_inspector.html entry point, which is no longer in use.

Changes (Claude Code summary)

Deleted:

  • front_end/entrypoints/rn_inspector/ directory (3 files: rn_inspector.ts, BUILD.gn, README.md)
  • front_end/panels/rn_welcome/rn_welcome-legacy-meta.ts

Updated (planned changes):

  • front_end/panels/rn_welcome/BUILD.gn — removed meta-legacy entrypoint target
  • front_end/panels/rn_welcome/RNWelcome.ts — removed showBetaLabel, showTechPreviewLabel options and their UIStrings
    (betaLabel, techPreviewLabel), plus the conditional badge rendering blocks
  • front_end/panels/rn_welcome/rnWelcome.css — removed .rn-welcome-title-accessory and
    .rn-welcome-title-accessory-purple rules
  • front_end/BUILD.gn — removed rn_inspector entrypoint dep and template entry
  • config/gni/devtools_grd_files.gni — removed 3 rn_inspector/legacy-meta GRD entries
  • scripts/build/generate_html_entrypoint.js — removed rn_inspector from title map
  • front_end/core/host/RNPerfMetrics.ts — changed default #entryPoint to 'rn_fusebox', simplified EntryPoint type

Updated (additional cleanup found via grep):

  • front_end/entrypoints/main/BUILD.gn — removed rn_inspector visibility
  • front_end/entrypoints/inspector_main/BUILD.gn — removed rn_inspector visibility
  • front_end/entrypoints/shell/BUILD.gn — removed rn_inspector visibility
  • front_end/core/rn_experiments/BUILD.gn — removed rn_inspector visibility
  • front_end/panels/recorder/BUILD.gn — removed rn_inspector visibility
  • front_end/core/rn_experiments/experimentsImpl.ts — updated comment
  • front_end/index.html — removed rn_inspector.html link
  • front_end/panels/rn_welcome/rn_welcome-meta.ts — removed dead showBetaLabel: false
  • scripts/eslint_rules/lib/check-license-header.js — removed rn_inspector entry
  • README.md — updated URL reference from rn_inspector to rn_fusebox

Final grep confirms zero remaining references to rn_inspector, rn_welcome-legacy-meta, showBetaLabel,
showTechPreviewLabel, or rn-welcome-title-accessory.

Test plan

npm run build

Upstreaming plan

@huntie huntie requested review from hoxyq and vzaidman February 10, 2026 17:36
@meta-cla meta-cla bot added the cla signed label Feb 10, 2026
@huntie huntie marked this pull request as ready for review February 13, 2026 17:48
@huntie huntie force-pushed the remove-legacy-rn-entry-point branch from 9285678 to 9aff395 Compare February 23, 2026 17:12
@huntie huntie merged commit 8edd9be into facebook:main Feb 23, 2026
5 checks passed
@huntie huntie deleted the remove-legacy-rn-entry-point branch February 23, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants