Fix tinytex install silently ignoring extraction failures

Check the return value of unzip() during TinyTeX installation. When
extraction fails (e.g., tar xf on a .tar.xz without xz-utils), the
installer now throws a clear error instead of proceeding to move
non-existent files and producing a confusing lstat/NotFound error.

For .tar.xz archives, the error message includes a hint to install
xz-utils. Also add xz-utils to .deb Recommends and xz to .rpm
Recommends since TinyTeX now ships .tar.xz as the preferred format.

Fixes #14304

Author: cderv

.claude/rules/testing/typescript-tests.md

@@ -104,8 +104,11 @@ const markdown = asMappedString("");
 const markdownWithContent = asMappedString("# Title\nSome content");
 ```
 
-**Mock ProjectContext:**
-```typescript
-import { createMockProjectContext } from "./utils.ts";  // tests/unit/project/utils.ts
-const project = createMockProjectContext();  // Creates temp dir + FileInformationCacheMap
-```
+**Mock Contexts:**
+
+Several subsystems use context interfaces passed to functions. For unit tests, create `createMock*()` helpers with no-op stubs. Key pattern: async callbacks (like `withSpinner`) should just `await op()` so errors propagate normally. Check existing test files for helpers before writing new ones.
+
+| Context | Interface | Existing helpers |
+|---------|-----------|-----------------|
+| `ProjectContext` | `src/project/types.ts` | `tests/unit/project/utils.ts` → `createMockProjectContext()` |
+| `InstallContext` | `src/tools/types.ts` | `tests/unit/tools/chrome-headless-shell.test.ts` → `createMockContext()` |

package/src/linux/installer.ts

@@ -92,11 +92,15 @@ async function createNfpmConfig(
   // Format-specific configuration
   if (format === 'deb') {
     config.overrides.deb = {
-      recommends: ["unzip"],
+      recommends: ["unzip", "xz-utils"],
     };
     // Add Debian-specific metadata
     config.section = "user/text";
     config.priority = "optional";
+  } else if (format === 'rpm') {
+    config.overrides.rpm = {
+      recommends: ["xz"],
+    };
   }
   return config;
 }

src/tools/impl/tinytex.ts

@@ -190,7 +190,15 @@ async function install(
       await context.withSpinner(
         { message: `Unzipping ${basename(pkgInfo.filePath)}` },
         async () => {
-          await unzip(pkgInfo.filePath);
+          const result = await unzip(pkgInfo.filePath);
+          if (!result.success) {
+            const hint = pkgInfo.filePath.endsWith(".tar.xz")
+              ? "\nOn Linux, you may need to install xz-utils (e.g., apt install xz-utils)."
+              : "";
+            throw new Error(
+              `Failed to extract ${basename(pkgInfo.filePath)}.${hint}\n${result.stderr}`,
+            );
+          }
         },
       );
 

tests/unit/tools/tinytex.test.ts

@@ -5,10 +5,14 @@
  */
 
 import { unitTest } from "../../test.ts";
-import { assert, assertEquals } from "testing/asserts";
-import { tinyTexPkgName } from "../../../src/tools/impl/tinytex.ts";
+import { assert, assertEquals, assertRejects } from "testing/asserts";
+import {
+  tinyTexInstallable,
+  tinyTexPkgName,
+} from "../../../src/tools/impl/tinytex.ts";
 import { getLatestRelease } from "../../../src/tools/github.ts";
-import { GitHubRelease } from "../../../src/tools/types.ts";
+import { GitHubRelease, InstallContext, PackageInfo } from "../../../src/tools/types.ts";
+import { join } from "../../../src/deno_ral/path.ts";
 
 // ---- Pure logic tests for tinyTexPkgName ----
 
@@ -158,3 +162,81 @@ unitTest(
     assertAssetExists(candidates, assetNames, "Windows");
   },
 );
+
+// ---- Extraction failure tests ----
+
+function createMockContext(workingDir: string): InstallContext {
+  return {
+    workingDir,
+    info: (_msg: string) => {},
+    withSpinner: async (_options, op) => {
+      await op();
+    },
+    error: (_msg: string) => {},
+    confirm: async (_msg: string) => true,
+    download: async (_name: string, _url: string, _target: string) => {},
+    props: {},
+    flags: {},
+  };
+}
+
+unitTest(
+  "install - throws on extraction failure for corrupt archive",
+  async () => {
+    const workingDir = Deno.makeTempDirSync({ prefix: "quarto-tinytex-test" });
+    try {
+      // Create a corrupt .tar.xz file that tar cannot extract
+      const badArchive = join(
+        workingDir,
+        "TinyTeX-linux-x86_64-v2026.04.tar.xz",
+      );
+      Deno.writeTextFileSync(badArchive, "this is not a valid archive");
+
+      const pkgInfo: PackageInfo = {
+        filePath: badArchive,
+        version: "v2026.04",
+      };
+      const context = createMockContext(workingDir);
+
+      await assertRejects(
+        () => tinyTexInstallable.install(pkgInfo, context),
+        Error,
+        "Failed to extract",
+      );
+    } finally {
+      Deno.removeSync(workingDir, { recursive: true });
+    }
+  },
+);
+
+unitTest(
+  "install - extraction failure for .tar.xz includes xz-utils hint",
+  async () => {
+    const workingDir = Deno.makeTempDirSync({ prefix: "quarto-tinytex-test" });
+    try {
+      const badArchive = join(
+        workingDir,
+        "TinyTeX-linux-x86_64-v2026.04.tar.xz",
+      );
+      Deno.writeTextFileSync(badArchive, "this is not a valid archive");
+
+      const pkgInfo: PackageInfo = {
+        filePath: badArchive,
+        version: "v2026.04",
+      };
+      const context = createMockContext(workingDir);
+
+      try {
+        await tinyTexInstallable.install(pkgInfo, context);
+        throw new Error("Expected install to throw");
+      } catch (e) {
+        assert(
+          e instanceof Error && e.message.includes("xz-utils"),
+          `Error message should mention xz-utils, got: ${e}`,
+        );
+      }
+    } finally {
+      Deno.removeSync(workingDir, { recursive: true });
+    }
+  },
+);