Terminate Quarto preview when terminal is closed externally

apps/vscode/CHANGELOG.md

@@ -5,6 +5,7 @@
 - Added support for Positron's statement execution feature that reports the approximate line number of the parse error (<https://github.com/quarto-dev/quarto/pull/919>).
 - Fixed a bug where `Quarto: Format Cell` would notify you that no formatter was available for code cells that were already formatted (<https://github.com/quarto-dev/quarto/pull/933>).
 - No longer claim `.typ` files. Typst syntax highlighting in Quarto documents is unaffected, but standalone Typst files are now left to dedicated extensions like Tinymist (<https://github.com/quarto-dev/quarto/pull/943>).
+- Fixed a bug where closing the Quarto Preview terminal via the trash icon did not clean up intermediate `.quarto_ipynb` files (<https://github.com/quarto-dev/quarto/pull/947>).
 
 ## 1.130.0 (Release on 2026-02-18)
 

apps/vscode/src/providers/preview/preview.ts

@@ -282,6 +282,18 @@ class PreviewManager {
       this.outputSink_,
       this.renderToken_
     );
+
+    // When the user closes the preview terminal via the trash icon (or any
+    // external means), send the HTTP termination request so Quarto can clean
+    // up temp files (e.g. _ipynb files created during Python preview).
+    context.subscriptions.push(
+      vscode.window.onDidCloseTerminal(async (closedTerminal) => {
+        if (closedTerminal === this.terminal_) {
+          this.terminal_ = undefined;
+          await this.previewTerminateRequest();
+        }
+      })
+    );
   }
 
   dispose() {
@@ -416,6 +428,9 @@ class PreviewManager {
   }
 
   private async killPreview() {
+    // Clear this.terminal_ before disposing so the onDidCloseTerminal listener
+    // does not send a duplicate termination request.
+    this.terminal_ = undefined;
     await killTerminal(kPreviewWindowTitle, async () => await this.previewTerminateRequest());
     this.progressDismiss();
     this.progressCancellationToken_ = undefined;