Skip to content

Commit 0016d82

Browse files
author
Noam Kfir
committed
Only show stack trace links for js workspace packages excluding node_modules
1 parent 91cccdd commit 0016d82

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/services/languages/javascript/packageReader.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,14 @@ type PackagesMap = Map<string, vscode.Uri>;
77
export class JSPackageReader {
88
private packagesMap: PackagesMap = new Map<string, vscode.Uri>();
99

10-
public async findAllPackagesInWorkspace() {
11-
const packages = await vscode.workspace.findFiles('**/package.json');
10+
private async findPackagesInWorkspace(includeNodeModules = true) {
11+
const exclude = includeNodeModules ? undefined : '**/node_modules/**';
12+
const packages = await vscode.workspace.findFiles('**/package.json', exclude);
1213
return packages;
1314
}
1415

1516
public async findPackage(uri: vscode.Uri): Promise<vscode.Uri | undefined> {
16-
const packages = await this.findAllPackagesInWorkspace();
17+
const packages = await this.findPackagesInWorkspace();
1718
const packageFile = packages.find(f => uri.fsPath.startsWith(dirname(f.fsPath)));
1819
if (!packageFile) {
1920
Logger.warn(`Could not resolve package file for '${uri.path}'`);
@@ -38,7 +39,7 @@ export class JSPackageReader {
3839
this.packagesMap.clear();
3940
}
4041
if(this.packagesMap.size === 0) {
41-
const packages = await this.findAllPackagesInWorkspace();
42+
const packages = await this.findPackagesInWorkspace(false);
4243
for await (const packageFile of packages) {
4344
try {
4445
const packageName = await this.getPackageName(packageFile);

0 commit comments

Comments
 (0)