fix: check if the uri is a folder before trying to open it as a document

Author: daniel-lxs

src/integrations/diagnostics/index.ts

@@ -78,7 +78,9 @@ export async function diagnosticsToProblemsString(
 	const documents = new Map<vscode.Uri, vscode.TextDocument>()
 	let result = ""
 	for (const [uri, fileDiagnostics] of diagnostics) {
-		const problems = fileDiagnostics.filter((d) => severities.includes(d.severity))
+		const problems = fileDiagnostics
+			.filter((d) => severities.includes(d.severity))
+			.sort((a, b) => a.range.start.line - b.range.start.line)
 		if (problems.length > 0) {
 			result += `\n\n${path.relative(cwd, uri.fsPath).toPosix()}`
 			for (const diagnostic of problems) {
@@ -101,10 +103,19 @@ export async function diagnosticsToProblemsString(
 				}
 				const line = diagnostic.range.start.line + 1 // VSCode lines are 0-indexed
 				const source = diagnostic.source ? `${diagnostic.source} ` : ""
-				const document = documents.get(uri) || (await vscode.workspace.openTextDocument(uri))
-				documents.set(uri, document)
-				const lineContent = document.lineAt(diagnostic.range.start.line).text
-				result += `\n- [${source}${label}] ${line} | ${lineContent} : ${diagnostic.message}`
+				try {
+					const fileStat = await vscode.workspace.fs.stat(uri)
+					if (fileStat.type === vscode.FileType.File) {
+						const document = documents.get(uri) || (await vscode.workspace.openTextDocument(uri))
+						documents.set(uri, document)
+						const lineContent = document.lineAt(diagnostic.range.start.line).text
+						result += `\n- [${source}${label}] ${line} | ${lineContent} : ${diagnostic.message}`
+					} else {
+						result += `\n- [${source}${label}] 1 | (directory) : ${diagnostic.message}`
+					}
+				} catch {
+					result += `\n- [${source}${label}] ${line} | (unavailable) : ${diagnostic.message}`
+				}
 			}
 		}
 	}