Fix babel hyphenation warning detection with straight quotes (#10291)

Updated regex to accept both backtick-quote (`Spanish') and straight-quote
('Spanish') formats for detecting missing hyphenation packages.

Added unit tests and modified smoke-all tests to skip package removal on CI.

Author: cderv

news/changelog-1.9.md

@@ -37,6 +37,7 @@ All changes included in 1.9:
 
 ### `pdf`
 
+- ([#10291](https://github.com/quarto-dev/quarto-cli/issues/10291)): Fix detection of babel hyphenation warnings with straight-quote format instead of backtick-quote format.
 - ([#13633](https://github.com/quarto-dev/quarto-cli/issues/13633)): Fix detection and auto-installation of babel language packages from newer error format that doesn't explicitly mention `.ldf` filename.
 - ([rstudio/tinytex-releases#49](https://github.com/rstudio/tinytex-releases/issues/49)): Fix detection of LuaTeX-ja missing file errors by matching both "File" and "file" in error messages.
 

src/command/render/latexmk/parse-error.ts

@@ -133,7 +133,7 @@ export function findMissingHyphenationFiles(logText: string) {
   const babelWarningRegex = /^Package babel Warning:/m;
   const hasWarning = logText.match(babelWarningRegex);
   if (hasWarning) {
-    const languageRegex = /^\(babel\).* language `(\S+)'.*$/m;
+    const languageRegex = /^\(babel\).* language [`'](\S+)[`'].*$/m;
     const languageMatch = logText.match(languageRegex);
     if (languageMatch) {
       return filterLang(languageMatch[1]);

tests/docs/smoke-all/2024/08/30/10291/latex-hyphen-lang-es-no-install.qmd

@@ -3,14 +3,17 @@ format: pdf
 lang: es
 latex-auto-install: false
 _quarto:
+  tests-on-ci: false
   tests:
     pdf:
-      noErrors: true 
+      shouldError: true 
       printsMessage:
-        - WARN
-        - 'missing hyphenation.*hyphen-spanish'
+        level: INFO
+        regex: "Unknown option 'spanish'"
 ---
 
+** This test is to be run manually as we don't want to uninstall and reinstall package in CI**
+
 ```{r}
 #| include: false
 

tests/docs/smoke-all/2024/08/30/10291/latex-hyphen-lang-es.qmd

@@ -6,14 +6,21 @@ _quarto:
     pdf: null
 ---
 
+**Only remove package on CI**
 ```{r}
+#| eval: !expr isFALSE(as.logical(Sys.getenv("CI", "false")))
 #| include: false
+#| messages: "NA"
 
 # Remove the hyphen package for spanish
 if (tinytex::check_installed("hyphen-spanish")) {
   message("Removing 'hyphen-spanish' package for the render")
   tinytex::tlmgr_remove("hyphen-spanish")
 }
+if (tinytex::check_installed("babel-spanish")) {
+  message("Removing 'babel-spanish' package for the render")
+  tinytex::tlmgr_remove("babel-spanish")
+}
 ```
 
 # Hola !

tests/docs/smoke-all/2025/11/04/13633.qmd

@@ -8,7 +8,9 @@ _quarto:
 ---
 
 ```{r}
+#| eval: !expr isFALSE(as.logical(Sys.getenv("CI", "false")))
 #| include: false
+#| messages: "NA"
 
 # Remove babel-english package to test auto-installation
 if (tinytex::check_installed("babel-english")) {

tests/unit/latexmk/parse-error.test.ts

@@ -5,7 +5,7 @@
 *
 */
 
-import { findMissingFontsAndPackages } from "../../../src/command/render/latexmk/parse-error.ts"
+import { findMissingFontsAndPackages, findMissingHyphenationFiles } from "../../../src/command/render/latexmk/parse-error.ts"
 import { unitTest } from "../../test.ts";
 import { assert } from "testing/asserts";
 
@@ -43,6 +43,9 @@ unitTest("Detect missing files with `findMissingFontsAndPackages`", async () =>
     (babel)                If it’s the main language, try adding \`provide=*'
     (babel)                to the babel package options.`, "ngerman.ldf")
   assertFound("! Package babel Error: Unknown option 'english'.", "english.ldf");
+  assertFound(`! Package babel Error: Unknown option 'ngerman'.
+(babel)                Suggested actions:
+(babel)                * Make sure you haven't misspelled it`, "ngerman.ldf");
   assertFound("!pdfTeX error: pdflatex (file 8r.enc): cannot open encoding file for reading", "8r.enc");
   assertFound("! CTeX fontset `fandol' is unavailable in current mode", "fandol");
   assertFound("! CTeX fontset 'fandol' is unavailable in current mode", "fandol");
@@ -71,4 +74,61 @@ unitTest("Detect missing files with `findMissingFontsAndPackages`", async () =>
   assertFound("No file LGRcmr.fd. ! LaTeX Error: This NFSS system isn't set up properly.", "lgrcmr.fd");
 },{
   cwd: () => "unit/latexmk/"
+})
+
+unitTest("Detect missing hyphenation with babel warnings", async () => {
+  // Test backtick-quote format (old format)
+  const logWithBacktick = `Package babel Warning: No hyphenation patterns were preloaded for
+(babel)                the language \`Spanish' into the format.
+(babel)                Please, configure your TeX system to add them and
+(babel)                rebuild the format. Now I will use the patterns
+(babel)                preloaded for \\language=0 instead on input line 51.`;
+  assert(
+    findMissingHyphenationFiles(logWithBacktick) === "hyphen-spanish",
+    "Should detect hyphen-spanish from backtick-quote format"
+  );
+
+  // Test straight-quote format (new format - the bug we're fixing)
+  const logWithStraightQuotes = `Package babel Warning: No hyphenation patterns were preloaded for
+(babel)                the language 'Spanish' into the format.
+(babel)                Please, configure your TeX system to add them and
+(babel)                rebuild the format. Now I will use the patterns
+(babel)                preloaded for \\language=0 instead on input line 51.`;
+  assert(
+    findMissingHyphenationFiles(logWithStraightQuotes) === "hyphen-spanish",
+    "Should detect hyphen-spanish from straight-quote format"
+  );
+
+  // Test ngerman special case (should return hyphen-german, not hyphen-ngerman)
+  const logGerman = `Package babel Warning: No hyphenation patterns were preloaded for
+(babel)                the language 'ngerman' into the format.`;
+  assert(
+    findMissingHyphenationFiles(logGerman) === "hyphen-german",
+    "Should map ngerman to hyphen-german"
+  );
+
+  // Test Chinese - no hyphen package exists
+  const logChinese = `Package babel Warning: No hyphenation patterns were preloaded for
+(babel)                the language 'chinese' into the format.`;
+  assert(
+    findMissingHyphenationFiles(logChinese) === undefined,
+    "Should return undefined for Chinese (no hyphen package)"
+  );
+
+  // Test alternative Info pattern (issue #10291)
+  const logInfoChinese = `Package babel Info: Hyphen rules for 'chinese-hans' set to \\l@nil
+(babel)             (\\language10). Reported on input line 143.`;
+  assert(
+    findMissingHyphenationFiles(logInfoChinese) === undefined,
+    "Should return undefined for chinese-hans via Info pattern"
+  );
+
+  // Test no warning present
+  const logNoWarning = "Some other log text without babel warnings";
+  assert(
+    findMissingHyphenationFiles(logNoWarning) === undefined,
+    "Should return undefined when no babel warning present"
+  );
+}, {
+  cwd: () => "unit/latexmk/"
 })