Merge #3175

3175: vscode: skip node internals while debugging r=matklad a=Veetaha

Before:
![without-skip-files](https://user-images.githubusercontent.com/36276403/74610221-de9aff00-50f9-11ea-81bb-73dce8ca7acc.gif)
After:
![with-skip-files](https://user-images.githubusercontent.com/36276403/74610222-e22e8600-50f9-11ea-980b-e8d098ee82b4.gif)


Co-authored-by: Veetaha <[email protected]>

1 files changed, 16 insertions, 10 deletions

diff --git a/.vscode/launch.json b/.vscode/launch.json
index 33fc3bdea..b1bd98d4a 100644
--- a/.vscode/launch.json
+++ b/.vscode/launch.json
@@ -4,8 +4,7 @@
   // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
 
   // NOTE: --disable-extensions
-  // Disable all installed extensions for increas performance of playground 
-  // Disable all installed extensions to increase performance of the debug instance 
+  // Disable all installed extensions to increase performance of the debug instance
   // and prevent potential conflicts with other installed extensions.
 
   "version": "0.2.0",
@@ -23,10 +22,13 @@
       "outFiles": [
         "${workspaceFolder}/editors/code/out/**/*.js"
       ],
-      "preLaunchTask": "Build Extension"
+      "preLaunchTask": "Build Extension",
+      "skipFiles": [
+        "<node_internals>/**/*.js"
+      ]
     },
     {
-      // Used for testing theextension with a local build of the LSP server (in `target/debug`).
+      // Used for testing the extension with a local build of the LSP server (in `target/debug`).
       "name": "Run Extension (Dev Server)",
       "type": "extensionHost",
       "request": "launch",
@@ -39,17 +41,21 @@
         "${workspaceFolder}/editors/code/out/**/*.js"
       ],
       "preLaunchTask": "Build Extension",
+      "skipFiles": [
+        "<node_internals>/**/*.js"
+      ],
       "env": {
         "__RA_LSP_SERVER_DEBUG": "${workspaceFolder}/target/debug/ra_lsp_server"
       }
     },
     {
-      // Attaching LLDB by Pid to lsp server
-      // NOTE: it requires root permission, also for better debugging
-      // if you want more than just assembler representation for ex: 
-      // walk in file, you need debugging symbols.
-      // Project can build with it, for set - chane inside
-      // ./Cargo.toml field debug 
+      // Used to attach LLDB to a running LSP server.
+      // NOTE: Might require root permissions. For this run:
+      //
+      // `echo 0 | sudo tee /proc/sys/kernel/yama/ptrace_scope`
+      //
+      // Don't forget to set `debug = 2` in `Cargo.toml` before building the server
+
       "name": "Attach To Server",
       "type": "lldb",
       "request": "attach",