Hide diagnostics from virtual documents

apps/vscode/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Fixed an issue where attribute values containing '='s could be truncated in some scenarios (<https://github.com/quarto-dev/quarto/pull/814>).
 - Fixed an issue where a loading spinner for qmd previews wasn't dismissed on preview errors (<https://github.com/quarto-dev/quarto/pull/823>)
+- Diagnostics are no longer reported for internal temporary virtual document files (<https://github.com/quarto-dev/quarto/pull/832>).
 
 ## 1.124.0 (Release on 2025-08-20)
 

apps/vscode/package.json

@@ -1045,6 +1045,13 @@
           "default": true,
           "markdownDescription": "Use reticulate to execute Python cells within Knitr engine documents."
         },
+        "quarto.vdoc.hideDiagnostics": {
+          "order": 26,
+          "scope": "window",
+          "type": "boolean",
+          "default": true,
+          "markdownDescription": "Hide diagnostics (problems) reported for internal virtual document files. These are temporary files used to provide language features in code blocks."
+        },
         "quarto.mathjax.scale": {
           "order": 15,
           "scope": "window",

apps/vscode/src/lsp/client.ts

@@ -23,7 +23,8 @@ import {
   LocationLink,
   Definition,
   LogOutputChannel,
-  Uri
+  Uri,
+  Diagnostic
 } from "vscode";
 import {
   LanguageClient,
@@ -46,13 +47,15 @@ import {
   ProvideHoverSignature,
   ProvideSignatureHelpSignature,
   State,
+  HandleDiagnosticsSignature
 } from "vscode-languageclient";
 import { MarkdownEngine } from "../markdown/engine";
 import {
   adjustedPosition,
   unadjustedRange,
   virtualDoc,
   withVirtualDocUri,
+  isVirtualDoc,
 } from "../vdoc/vdoc";
 import { activateVirtualDocEmbeddedContent } from "../vdoc/vdoc-content";
 import { vdocCompletions } from "../vdoc/vdoc-completion";
@@ -112,6 +115,9 @@ export async function activateLsp(
   if (config.get("cells.signatureHelp.enabled", true)) {
     middleware.provideSignatureHelp = embeddedSignatureHelpProvider(engine);
   }
+  if (config.get("vdoc.hideDiagnostics", true)) {
+    middleware.handleDiagnostics = createDiagnosticFilter();
+  }
   extensionHost().registerStatementRangeProvider(engine);
   extensionHost().registerHelpTopicProvider(engine);
 
@@ -338,3 +344,21 @@ function isWithinYamlComment(doc: TextDocument, pos: Position) {
   const line = doc.lineAt(pos.line).text;
   return !!line.match(/^\s*#\s*\| /);
 }
+
+/**
+ * Creates a diagnostic handler middleware that filters out diagnostics from virtual documents
+ *
+ * @returns A handler function for the middleware
+ */
+export function createDiagnosticFilter() {
+  return (uri: Uri, diagnostics: Diagnostic[], next: HandleDiagnosticsSignature) => {
+    // If this is not a virtual document, pass through all diagnostics
+    if (!isVirtualDoc(uri)) {
+      next(uri, diagnostics);
+      return;
+    }
+
+    // For virtual documents, filter out all diagnostics
+    next(uri, []);
+  };
+}

apps/vscode/src/test/diagnosticFiltering.test.ts

@@ -0,0 +1,53 @@
+import * as vscode from "vscode";
+import * as assert from "assert";
+import { createDiagnosticFilter } from "../lsp/client";
+
+suite("Diagnostic Filtering", function () {
+
+  test("Diagnostic filter removes diagnostics for virtual documents", async function () {
+    // Create mocks
+    const virtualDocUri = vscode.Uri.file("/tmp/.vdoc.12345678-1234-1234-1234-123456789abc.py");
+    const regularDocUri = vscode.Uri.file("/tmp/regular-file.py");
+
+    // Create some test diagnostics
+    const testDiagnostics = [
+      new vscode.Diagnostic(
+        new vscode.Range(0, 0, 0, 10),
+        "Test diagnostic message",
+        vscode.DiagnosticSeverity.Error
+      )
+    ];
+
+    // Create a mock diagnostics handler function to verify behavior
+    let capturedUri: vscode.Uri | undefined;
+    let capturedDiagnostics: vscode.Diagnostic[] | undefined;
+
+    const mockHandler = (uri: vscode.Uri, diagnostics: vscode.Diagnostic[]) => {
+      capturedUri = uri;
+      capturedDiagnostics = diagnostics;
+    };
+
+    // Create the filter function
+    const diagnosticFilter = createDiagnosticFilter();
+
+    // Test with a virtual document
+    diagnosticFilter(virtualDocUri, testDiagnostics, mockHandler);
+
+    // Verify diagnostics were filtered (empty array)
+    assert.strictEqual(capturedUri, virtualDocUri, "URI should be passed through");
+    assert.strictEqual(capturedDiagnostics!.length, 0, "Diagnostics should be empty for virtual documents");
+
+    // Reset captured values
+    capturedUri = undefined;
+    capturedDiagnostics = undefined;
+
+    // Test with a regular document
+    diagnosticFilter(regularDocUri, testDiagnostics, mockHandler);
+
+    // Verify diagnostics were not filtered
+    assert.strictEqual(capturedUri, regularDocUri, "URI should be passed through");
+    assert.strictEqual(capturedDiagnostics!.length, testDiagnostics.length, "Diagnostics should not be filtered for regular documents");
+    assert.deepStrictEqual(capturedDiagnostics!, testDiagnostics, "Original diagnostics should be passed through unchanged");
+  });
+
+});

apps/vscode/src/vdoc/vdoc.ts

@@ -15,6 +15,7 @@
 
 import { Position, TextDocument, Uri, Range } from "vscode";
 import { Token, isExecutableLanguageBlock, languageBlockAtPosition, languageNameFromBlock } from "quarto-core";
+import * as path from "path";
 
 import { isQuartoDoc } from "../core/doc";
 import { MarkdownEngine } from "../markdown/engine";
@@ -53,6 +54,17 @@ export async function virtualDoc(
   }
 }
 
+export function isVirtualDoc(uri: Uri): boolean {
+  // Check for tempfile virtual docs
+  if (uri.scheme === "file") {
+    const filename = path.basename(uri.fsPath);
+    // Virtual docs have a specific filename pattern .vdoc.[uuid].[extension]
+    return filename.startsWith(".vdoc.") && filename.split(".").length > 3;
+  }
+
+  return false;
+}
+
 export function virtualDocForBlock(document: TextDocument, block: Token, language: EmbeddedLanguage) {
   const lines = linesForLanguage(document, language);
   fillLinesFromBlock(lines, document, block);