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

[VS Code]Validate the bundle loading for vnext #2876

Closed
kmelmon opened this issue Aug 2, 2019 · 4 comments
Closed

[VS Code]Validate the bundle loading for vnext #2876

kmelmon opened this issue Aug 2, 2019 · 4 comments

Comments

@kmelmon
Copy link
Contributor

kmelmon commented Aug 2, 2019

Currently, when using the VS Code debugging extension for react-native, starting a debugging session is not working. This issue is tracking one of the issues in that experience.

The issue here is that the VS Code extension was written against 'current' which makes some assumptions about how current works. It has a "pre warm bundle cache" step which downloads the bundle in parallel with doing a build, which makes it faster to start a debug session. However this pre-warming step tries to load a bundle using the platform string of "windows". This won't work on vnext because vnext is doing some things differently than current. The vs code extension is looking for index.bundle?platform=windows, instead of app.bundle?platform=uwp

So there are 2 problems here:

  1. The platform string is "uwp" instead of "windows"
  2. The bundle is app.bundle instead of index.bundle

However it's not clear when we'll get to part 1, so we may need to change the VS Code extension in the short term.

For the bundle name part of this issue, we need to decide how to best fix this. We may want to change the CLI to work similar to how it does on current, or we may decide to keep this as is, and instead change the VS Code extension to look for app.bundle.

Also see this bug on the VS Code side of things:
microsoft/vscode-react-native#1007

@ghost ghost added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Aug 2, 2019
@harinikmsft harinikmsft removed the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Aug 7, 2019
@harinikmsft harinikmsft added this to the vNext Milestone 3 milestone Aug 7, 2019
@harinikmsft harinikmsft changed the title [VS Code] Fix bundle loading for vnext [VS Code]Validate the bundle loading for vnext Aug 7, 2019
@chrisglein
Copy link
Member

For the bundle name part of this issue, we need to decide how to best fix this. We may want to change the CLI to work similar to how it does on current, or we may decide to keep this as is, and instead change the VS Code extension to look for app.bundle.

The recommendation here seems to be to change the CLI and go with the flow of whatever the other platforms use as a naming pattern.

@chrisglein
Copy link
Member

@kmelmon We did the rename. So... what now?

@kmelmon
Copy link
Contributor Author

kmelmon commented Aug 29, 2019

@chrisglein we still need to do #2901, I just assigned to @savatia

@savatia
Copy link
Member

savatia commented Sep 10, 2019

Done here: #3099

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants