Delete printFn from options sent to zprint

* Fixes #2630
BetterThanTomorrow · Sep 22, 2024 · 82e03f4 · 82e03f4
1 parent 423dfc7
commit 82e03f4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@ Changes to Calva.
 ## [Unreleased]
 
 - [Synchronize the file extensions for Calva and Calva Spritz](https://github.com/BetterThanTomorrow/calva/issues/2629)
+- [Terminal output pretty printing fails when using `printerFn` pretty print option](https://github.com/BetterThanTomorrow/calva/issues/2630)
 
 ## [2.0.473] - 2024-09-21
 

diff --git a/src/results-output/output.ts b/src/results-output/output.ts
@@ -221,6 +221,7 @@ function appendClojure(
     }
   } else if (destination === 'terminal') {
     const printerOptions = { ...printer.prettyPrintingOptions(), 'color?': true };
+    delete printerOptions.printFn; // Zprint croaks on options it doesn't understand
     const prettyMessage = printer.prettyPrint(message, printerOptions)?.value || message;
     // TODO: Figure if it's worth a setting to opt-in on an ns info line
     getOutputPTY().write(`${didLastTerminateLine ? '' : '\n'}${nsInfoLine(destination, options)}`);