Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

breaking: Remove cli-debugger-ui package #2583

Merged

Conversation

huntie
Copy link
Collaborator

@huntie huntie commented Jan 27, 2025

Summary:

Important

Breaking change: This is intended to target main / the future 18.x release of CLI (aligned with React Native 0.79).

Following our intent/announcement to remove Remote JS Debugging from React Native in 0.78, this PR removes the corresponding cli-debugger-ui frontend from the Community CLI.

Also deletes related middleware/WebSocket support code in cli-server-api:

  • /launch-js-devtools
  • debuggerProxyEndpoint

Test Plan:

✅ CI should pass

Checklist

  • Documentation is up to date to reflect these changes.
  • Follows commit message convention described in CONTRIBUTING.md

@github-actions github-actions bot added the docs Documentation change label Jan 27, 2025
@huntie huntie force-pushed the remove-remote-debugging-1 branch from a793852 to 1afce36 Compare January 27, 2025 17:18
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@szymonrybczak szymonrybczak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments, overall looks good!

packages/cli-server-api/package.json Outdated Show resolved Hide resolved
packages/cli/package.json Outdated Show resolved Hide resolved
packages/cli/package.json Outdated Show resolved Hide resolved
@huntie
Copy link
Collaborator Author

huntie commented Jan 27, 2025

Very good spots @szymonrybczak — sorry I rebased my branch from my fork's main, rather than the up-to-date one!

@huntie
Copy link
Collaborator Author

huntie commented Jan 27, 2025

For safety, latest push is a recreation of the commit after rebasing and re-deleting/searching for references.

@huntie huntie force-pushed the remove-remote-debugging-1 branch from 1afce36 to 5efa968 Compare January 27, 2025 17:35
Copy link
Collaborator

@szymonrybczak szymonrybczak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now, thanks!

@szymonrybczak
Copy link
Collaborator

@huntie there's a conflict in packages/cli-server-api/index.ts file - can you take a look?

@huntie huntie force-pushed the remove-remote-debugging-1 branch from 60d96aa to 57ba392 Compare January 28, 2025 10:19
@huntie
Copy link
Collaborator Author

huntie commented Jan 28, 2025

@szymonrybczak Resolved.

@huntie huntie merged commit 9113869 into react-native-community:main Jan 28, 2025
8 checks passed
@huntie huntie deleted the remove-remote-debugging-1 branch January 28, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants