Fix CSP files not being included in server side search results

psteiwer · psteiwer · commit 58af642a3736 · 2025-08-20T15:01:51.000-04:00
Bug with the expected value of "projects". This was leading to projectList being undefined and throwing an error. Earlier logic to populate projectList uses if (project) {, so using if (project &amp;&amp; ... here keeps the checks consistent. isfsConfig() defaults project to "", which is why it is in this state. The following logic is used if apiVersion &gt;= 6, which Cache is not:
project: project ? project : undefined, // Needs to be undefined if project is an empty string. This means special handling is needed in this case
diff --git a/src/providers/FileSystemProvider/TextSearchProvider.ts b/src/providers/FileSystemProvider/TextSearchProvider.ts
@@ -676,7 +676,7 @@ export class TextSearchProvider implements vscode.TextSearchProvider {
                 if (token.isCancellationRequested) {
                   return;
                 }
-                if (project != undefined && file.doc.includes("/")) {
+                if (project && file.doc.includes("/")) {
                   // Check if this web app file is in the project
                   if (!projectList.includes(file.doc.slice(1))) {
                     // This web app file isn't in the project, so ignore its matches

Original file line number	Diff line number	Diff line change
`@@ -676,7 +676,7 @@ export class TextSearchProvider implements vscode.TextSearchProvider {`
`676`	`676`	`if (token.isCancellationRequested) {`
`677`	`677`	`return;`
`678`	`678`	`}`
`679`		`- if (project != undefined && file.doc.includes("/")) {`
	`679`	`+ if (project && file.doc.includes("/")) {`
`680`	`680`	`// Check if this web app file is in the project`
`681`	`681`	`if (!projectList.includes(file.doc.slice(1))) {`
`682`	`682`	`// This web app file isn't in the project, so ignore its matches`