nativescript-community / nativescript-vscode-extension

A Visual Studio Code Extension for NativeScript
https://www.nativescript.org/visual-studio-code
Apache License 2.0
82 stars 32 forks source link

[Debugging] Break points in platform specific files and/or symlink files in `node_modules` are not hit with "@next" version #252

Closed VladimirAmiorkov closed 5 years ago

VladimirAmiorkov commented 5 years ago

Environment

Describe the bug

First scenario:

After installing the @next when using VS Code to either Attach to iOS or Lunch on iOS any break points placed in a plugin that is using symlink and is in _nodemodules is not hit.

Second scenario:

A different scenario that also does not hit its break points is adding a break point to a non symlink plugin in node_modules but in the platform specific file (.ios.js). In this scenarios a break point in the non platform specific file is hit.

To Reproduce

First scenario:

Using this demo app if you symlink the nativescript-ui-listview code if you place a break point in VS Code in the createNativeView in the node_modules/nativescript-ui-listview/ui-listview.ios.ts file it will not be hit after you navigate to for example "Getting Started" page. (both breakpoints in platform specific and non platform specific files are not hit)

Second scenario:

Using this demo if you place a break point in VS Code at viewDidLayoutSubviews in the node_modules/tns-core-modules/ui/page/page.ios.js. (only breakpoints in platform specific files are not hit)

Expected behavior

Break points in VS Code are hit in both platform specific files and in symlink plugins in node_modules

Sample project

joshcomley commented 5 years ago

This is marked as closed, but I have built the latest version of this plugin today and installed in VS Code, am using a shared project and the breakpoint still opens up the wrong file.

For example, in the following scenario, the breakpoint shows as hit on the correct line number but in the web version of the file:

app-home.component.ts < web version app-home.component.tns.ts < NS version

Should this fix have addressed this issue? Or did I misunderstand it?

joshcomley commented 5 years ago

Having looked at the code for the fix submitted for this, I could see the change required and made it as a pull request.