lazyslides 0.3.0 → 0.3.2

package/index.js

@@ -112,6 +112,14 @@ export default function lazyslides(eleventyConfig, options = {}) {
         d2Cache.set(source, `<div class="d2-error">${safeMsg}</div>`);
       }
     }
+
+    // Terminate the D2 worker thread so the Node process can exit cleanly.
+    // The D2 class spawns a Worker for WASM but never exposes a cleanup method.
+    if (d2Instance?.worker) {
+      await d2Instance.worker.terminate();
+      d2Instance = null;
+      d2Available = null;
+    }
   });
 
   // ---------------------------------------------------------------

package/lib/export-pdf.js

@@ -1,4 +1,4 @@
-import { execSync, spawn } from "node:child_process";
+import { spawn } from "node:child_process";
 import fs from "node:fs";
 import path from "node:path";
 import http from "node:http";
@@ -6,14 +6,18 @@ import http from "node:http";
 const PORT = 4100;
 const BASE_URL = `http://127.0.0.1:${PORT}`;
 const OUTPUT_DIR = "_pdfs";
-const TIMEOUT_SECONDS = 30;
+const SERVER_TIMEOUT_SECONDS = 60;
+const BUILD_IDLE_TIMEOUT_MS = 15000;
+const BUILD_HARD_TIMEOUT_MS = 180000;
 
 let serverProcess = null;
 
 function cleanup() {
   if (serverProcess) {
     console.log("\n  Stopping server...");
-    serverProcess.kill();
+    try {
+      serverProcess.kill("SIGTERM");
+    } catch {}
     serverProcess = null;
   }
 }
@@ -29,9 +33,9 @@ function waitForServer() {
       });
       req.on("error", () => {
         elapsed++;
-        if (elapsed >= TIMEOUT_SECONDS) {
+        if (elapsed >= SERVER_TIMEOUT_SECONDS) {
           clearInterval(interval);
-          reject(new Error(`Server failed to start within ${TIMEOUT_SECONDS}s`));
+          reject(new Error(`Server failed to start within ${SERVER_TIMEOUT_SECONDS}s`));
         }
       });
       req.end();
@@ -39,29 +43,107 @@ function waitForServer() {
   });
 }
 
-function runDecktape(name) {
-  const url = `${BASE_URL}/presentations/${name}/?pdf`;
-  const output = path.join(OUTPUT_DIR, `${name}.pdf`);
+/**
+ * Run `pnpm run build`, but tolerate the eleventy child not exiting cleanly
+ * (known issue: d2 WASM / chokidar handles can keep the event loop alive after
+ * "Wrote N files"). We detect idle stdout and force-kill.
+ */
+function runBuild(cwd) {
+  return new Promise((resolve, reject) => {
+    // Pipe stdio so we can observe activity and reset the idle timer.
+    const child = spawn("pnpm", ["run", "build"], {
+      cwd,
+      stdio: ["ignore", "pipe", "pipe"],
+    });
+
+    let idleTimer = null;
+    let hardTimer = null;
+    let resolved = false;
+
+    const done = (err) => {
+      if (resolved) return;
+      resolved = true;
+      clearTimeout(idleTimer);
+      clearTimeout(hardTimer);
+      try { child.kill("SIGKILL"); } catch {}
+      err ? reject(err) : resolve();
+    };
+
+    const bumpIdle = () => {
+      clearTimeout(idleTimer);
+      idleTimer = setTimeout(() => {
+        console.log(`\n  Build child idle for ${BUILD_IDLE_TIMEOUT_MS / 1000}s — assuming done and force-exiting.`);
+        done();
+      }, BUILD_IDLE_TIMEOUT_MS);
+    };
+
+    child.stdout.on("data", (chunk) => {
+      process.stdout.write(chunk);
+      bumpIdle();
+    });
+    child.stderr.on("data", (chunk) => {
+      process.stderr.write(chunk);
+      bumpIdle();
+    });
+
+    bumpIdle();
+
+    hardTimer = setTimeout(() => {
+      done(new Error(`Build exceeded hard timeout ${BUILD_HARD_TIMEOUT_MS / 1000}s`));
+    }, BUILD_HARD_TIMEOUT_MS);
+
+    child.on("exit", (code) => {
+      if (code === 0 || code === null) done();
+      else done(new Error(`Build failed with exit code ${code}`));
+    });
+    child.on("error", (err) => done(err));
+  });
+}
 
-  console.log(`  → Exporting: ${name}`);
-  try {
-    execSync(
-      `npx decktape reveal --size 1920x1080 --pause 1000 --load-pause 2000 "${url}" "${output}"`,
-      { stdio: ["ignore", "pipe", "pipe"] }
+function runDecktape(name, cwd) {
+  return new Promise((resolve) => {
+    const url = `${BASE_URL}/presentations/${name}/?pdf`;
+    const output = path.join(OUTPUT_DIR, `${name}.pdf`);
+
+    console.log(`  → Exporting: ${name}`);
+    console.log(`    URL:    ${url}`);
+    console.log(`    Output: ${output}\n`);
+
+    const child = spawn(
+      "npx",
+      [
+        "decktape",
+        "reveal",
+        "--size", "1920x1080",
+        "--pause", "1000",
+        "--load-pause", "2000",
+        url,
+        output,
+      ],
+      { cwd, stdio: ["ignore", "inherit", "inherit"] }
     );
-    console.log(`  \u2713 ${output}`);
-    return true;
-  } catch {
-    console.log(`  \u2717 Failed: ${name}`);
-    return false;
-  }
+
+    child.on("exit", (code) => {
+      if (code === 0) {
+        console.log(`\n  \u2713 ${output}`);
+        resolve(true);
+      } else {
+        console.log(`\n  \u2717 Failed: ${name} (decktape exit code ${code})`);
+        resolve(false);
+      }
+    });
+    child.on("error", (err) => {
+      console.log(`\n  \u2717 Failed: ${name} (${err.message})`);
+      resolve(false);
+    });
+  });
 }
 
-function getAvailablePresentations(cwd) {
+function getAvailablePresentations(cwd, { includeHidden = false } = {}) {
   const dir = path.join(cwd, "presentations");
   return fs
     .readdirSync(dir, { withFileTypes: true })
-    .filter((d) => d.isDirectory() && d.name !== "_template")
+    .filter((d) => d.isDirectory() && (includeHidden || !d.name.startsWith("_")))
     .map((d) => d.name);
 }
 
@@ -75,9 +157,10 @@ export async function run(opts = {}) {
   const cwd = opts.cwd || process.cwd();
   const requestedName = opts.name;
   const available = getAvailablePresentations(cwd);
+  const allForLookup = getAvailablePresentations(cwd, { includeHidden: true });
 
   if (requestedName) {
-    if (!available.includes(requestedName)) {
+    if (!allForLookup.includes(requestedName)) {
       console.error(`Presentation not found: ${requestedName}`);
       console.error("   Available presentations:");
       for (const name of available) {
@@ -89,16 +172,25 @@ export async function run(opts = {}) {
 
   const presentations = requestedName ? [requestedName] : available;
 
-  // Build the site
+  // Build the site (tolerate lingering child handles)
   console.log("Building site...");
-  execSync("pnpm run build", { stdio: "inherit", cwd });
+  try {
+    await runBuild(cwd);
+  } catch (err) {
+    console.error(`Build failed: ${err.message}`);
+    process.exit(1);
+  }
 
   // Start server
   console.log(`\nStarting server on port ${PORT}...`);
   serverProcess = spawn("npx", ["eleventy", "--serve", "--port", String(PORT)], {
-    stdio: "ignore",
+    stdio: ["ignore", "pipe", "pipe"],
     cwd,
   });
+  serverProcess.stdout.on("data", () => {}); // drain
+  serverProcess.stderr.on("data", (chunk) => {
+    process.stderr.write(`[server] ${chunk}`);
+  });
 
   process.on("exit", cleanup);
   process.on("SIGINT", () => { cleanup(); process.exit(1); });
@@ -106,7 +198,13 @@ export async function run(opts = {}) {
 
   // Wait for server
   console.log("Waiting for server...");
-  await waitForServer();
+  try {
+    await waitForServer();
+  } catch (err) {
+    console.error(`\n${err.message}`);
+    cleanup();
+    process.exit(1);
+  }
   console.log("Server ready");
 
   // Create output directory
@@ -118,7 +216,7 @@ export async function run(opts = {}) {
   console.log(`\nExporting ${presentations.length} presentation(s) to PDF...\n`);
 
   for (const name of presentations) {
-    if (runDecktape(name)) {
+    if (await runDecktape(name, cwd)) {
       success++;
     } else {
       failure++;
@@ -131,7 +229,6 @@ export async function run(opts = {}) {
   // Summary
   console.log("\u2501".repeat(30));
   console.log(`Exported: ${success}  Failed: ${failure}`);
-  if (failure > 0) {
-    process.exit(1);
-  }
+  // Force exit to ensure all child processes (server, npx wrappers) are cleaned up.
+  process.exit(failure > 0 ? 1 : 0);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lazyslides",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "Slide decks with native agentic AI integration",
   "license": "MIT",
   "author": "Chris Tietz",
@@ -50,6 +50,7 @@
     "@11ty/eleventy": "^3.0.0"
   },
   "dependencies": {
+    "decktape": "^3.14.0",
     "js-yaml": "^4.1.0"
   },
   "optionalDependencies": {
@@ -59,7 +60,6 @@
     "@11ty/eleventy": "^3.0.0",
     "@tailwindcss/cli": "4.1.18",
     "concurrently": "^9.0.0",
-    "decktape": "^3.14.0",
     "tailwindcss": "4.1.18",
     "vitest": "^4.1.2"
   },