Add test for output-dir: ../dirname pattern

Add test case for when output-dir uses parent traversal to reference
the project directory (e.g., project in "quarto-proj" with
output-dir: "../quarto-proj"). This edge case was raised in PR review.

The fix using resolve() correctly handles this pattern since
resolve(dir, "../dirname") equals resolve(dir) when dirname is
the current directory name.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: cderv

tests/docs/project/output-dir-dot/_quarto.yml

@@ -1,3 +1,3 @@
 project:
-  type: default
+  type: website
   output-dir: .

tests/docs/project/output-dir-parent-ref/.gitignore

@@ -0,0 +1,2 @@
+/.quarto/
+**/*.quarto_ipynb

tests/docs/project/output-dir-parent-ref/_quarto.yml

@@ -0,0 +1,3 @@
+project:
+  type: website
+  output-dir: ../output-dir-parent-ref

tests/docs/project/output-dir-parent-ref/marker.txt

@@ -0,0 +1,2 @@
+This file should NOT be deleted when rendering with output-dir: ../output-dir-parent-ref
+This pattern references the project directory via parent traversal.

tests/docs/project/output-dir-parent-ref/test.qmd

@@ -0,0 +1,8 @@
+---
+title: "Output Dir Parent Ref Test"
+format: html
+---
+
+# Test Document
+
+This document tests that `output-dir: ../output-dir-parent-ref` (referencing the project directory via parent traversal) does not delete the project directory.

tests/docs/project/output-dir-safety/_quarto.yml

@@ -1,3 +1,3 @@
 project:
-  type: default
+  type: website
   output-dir: ./

tests/smoke/project/project-output-dir-safety.test.ts

@@ -12,10 +12,30 @@ import { existsSync } from "../../../src/deno_ral/fs.ts";
 import { testQuartoCmd } from "../../test.ts";
 import { fileExists, noErrors } from "../../verify.ts";
 
+// Helper to clean up website output files from a directory
+// Used when output-dir points to the project directory itself
+async function cleanWebsiteOutput(dir: string) {
+  const filesToRemove = ["index.html", "test.html", "search.json"];
+  const dirsToRemove = ["site_libs", ".quarto"];
+
+  for (const file of filesToRemove) {
+    const path = join(dir, file);
+    if (existsSync(path)) {
+      await Deno.remove(path);
+    }
+  }
+  for (const subdir of dirsToRemove) {
+    const path = join(dir, subdir);
+    if (existsSync(path)) {
+      await Deno.remove(path, { recursive: true });
+    }
+  }
+}
+
 // Helper to create output-dir safety tests
 function testOutputDirSafety(
   name: string,
-  outputDir: string | null, // null means output is in project dir
+  outputDir: string | null, // null means output is in project dir (website type)
 ) {
   const testDir = docs(`project/output-dir-${name}`);
   const dir = join(Deno.cwd(), testDir);
@@ -37,17 +57,14 @@ function testOutputDirSafety(
           if (existsSync(outputPath)) {
             await Deno.remove(outputPath, { recursive: true });
           }
-        } else {
-          // In-place case - just remove the html file
-          const htmlFile = join(dir, "test.html");
-          if (existsSync(htmlFile)) {
-            await Deno.remove(htmlFile);
+          // Clean up .quarto directory
+          const quartoDir = join(dir, ".quarto");
+          if (existsSync(quartoDir)) {
+            await Deno.remove(quartoDir, { recursive: true });
           }
-        }
-        // Clean up .quarto directory
-        const quartoDir = join(dir, ".quarto");
-        if (existsSync(quartoDir)) {
-          await Deno.remove(quartoDir, { recursive: true });
+        } else {
+          // In-place website case - clean up website output files only
+          await cleanWebsiteOutput(dir);
         }
       },
     },
@@ -62,3 +79,8 @@ testOutputDirSafety("dot", null);
 
 // Test 3: output-dir: _output (normal subdirectory case)
 testOutputDirSafety("subdir", "_output");
+
+// Test 4: output-dir: ../dirname (parent traversal back to project dir)
+// This tests the case where output-dir references the project directory
+// via parent traversal (e.g., project in "quarto-proj", output-dir: "../quarto-proj")
+testOutputDirSafety("parent-ref", null);

tests/unit/path.test.ts

@@ -176,6 +176,16 @@ unitTest("path - output-dir equivalence with resolve()", async () => {
     );
   }
 
+  // Parent traversal back to project dir should also be equivalent
+  // e.g., project in "quarto-proj", output-dir: "../quarto-proj"
+  const dirName = testDir.split(/[/\\]/).pop()!;
+  const parentRef = `../${dirName}`;
+  const resolvedParentRef = resolve(testDir, parentRef);
+  assert(
+    resolvedParentRef === resolve(testDir),
+    `output-dir "${parentRef}" should resolve to project dir, got ${resolvedParentRef} vs ${resolve(testDir)}`,
+  );
+
   // Actual subdirectories should NOT be equivalent
   const subdir = "output";
   const resolvedSubdir = resolve(testDir, subdir);