Exclude **/node_modules/** from .vscode/launch.json searches (#1234)

gluxon · testforstephen · web-flow · commit ac256885f9b2 · 2022-10-19T14:28:39.000+08:00
Co-authored-by: Jinbo Wang &lt;jinbwan@microsoft.com&gt;
diff --git a/src/debugCodeLensProvider.ts b/src/debugCodeLensProvider.ts
@@ -181,7 +181,9 @@ async function launchJsonExists(workspace?: vscode.Uri): Promise<boolean> {
     }
 
     const workspaceFolder = vscode.workspace.getWorkspaceFolder(workspace);
-    const results: vscode.Uri[] = await vscode.workspace.findFiles(".vscode/launch.json");
+    // Excluding "**/node_modules/**" as a common cause of excessive CPU usage.
+    // https://github.com/microsoft/vscode/issues/75314#issuecomment-503195666
+    const results: vscode.Uri[] = await vscode.workspace.findFiles(".vscode/launch.json", "**/node_modules/**");
     return !!results.find((launchJson) => vscode.workspace.getWorkspaceFolder(launchJson) === workspaceFolder);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,9 @@ async function launchJsonExists(workspace?: vscode.Uri): Promise<boolean> {`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`const workspaceFolder = vscode.workspace.getWorkspaceFolder(workspace);`
`184`		`- const results: vscode.Uri[] = await vscode.workspace.findFiles(".vscode/launch.json");`
	`184`	`+ // Excluding "/node_modules/" as a common cause of excessive CPU usage.`
	`185`	`+ // https://github.com/microsoft/vscode/issues/75314#issuecomment-503195666`
	`186`	`+ const results: vscode.Uri[] = await vscode.workspace.findFiles(".vscode/launch.json", "/node_modules/");`
`185`	`187`	`return !!results.find((launchJson) => vscode.workspace.getWorkspaceFolder(launchJson) === workspaceFolder);`
`186`	`188`	`}`
`187`	`189`