From 24cd4bf13d02d3b0f400c58b83fe9783482b0407 Mon Sep 17 00:00:00 2001 From: claude Date: Sun, 31 May 2026 16:27:02 +0000 Subject: [PATCH] Fix HTML compile success check: size is undefined for non-PDF outputs collectOutputPdfSize() only calls stat() and sets .size on output.pdf. All other output files (including output.html) keep size: undefined. The previous check required file.size > 0 for both PDF and HTML, so undefined > 0 always evaluated false for output.html, making every RevealJS compile report 'failure' even when the file was produced. Fix: require size > 0 only for output.pdf; accept output.html regardless of size (it is always non-empty if Quarto succeeded). Co-Authored-By: Claude Sonnet 4.6 --- services/clsi/app/js/CompileController.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/services/clsi/app/js/CompileController.js b/services/clsi/app/js/CompileController.js index 9545d900a2..e244afe887 100644 --- a/services/clsi/app/js/CompileController.js +++ b/services/clsi/app/js/CompileController.js @@ -91,9 +91,8 @@ function compile(req, res, next) { if ( outputFiles.some( file => - (file.path === 'output.pdf' || - file.path === 'output.html') && - file.size > 0 + (file.path === 'output.pdf' && file.size > 0) || + file.path === 'output.html' ) ) { status = 'success'