portapack 0.3.0 → 0.3.2

package/dist/cli/cli-entry.cjs

@@ -55,7 +55,11 @@ function parseRecursiveValue(val) {
 }
 function parseOptions(argv = process.argv) {
   const program = new import_commander.Command();
-  program.name("portapack").version("0.0.0").description("\u{1F4E6} Bundle HTML and its dependencies into a portable file").argument("[input]", "Input HTML file or URL").option("-o, --output <file>", "Output file path").option("-m, --minify", "Enable all minification (HTML, CSS, JS)").option("--no-minify", "Disable all minification").option("--no-minify-html", "Disable HTML minification").option("--no-minify-css", "Disable CSS minification").option("--no-minify-js", "Disable JavaScript minification").option("-e, --embed-assets", "Embed assets as data URIs").option("--no-embed-assets", "Keep asset links relative/absolute").option("-r, --recursive [depth]", "Recursively crawl site (optional depth)", parseRecursiveValue).option("--max-depth <n>", "Set max depth for recursive crawl (alias for -r <n>)", parseInt).option("-b, --base-url <url>", "Base URL for resolving relative links").option("-d, --dry-run", "Run without writing output file").option("-v, --verbose", "Enable verbose (debug) logging").addOption(new import_commander.Option("--log-level <level>", "Set logging level").choices(logLevels));
+  program.name("portapack").version("0.0.0").description("\u{1F4E6} Bundle HTML and its dependencies into a portable file").argument("[input]", "Input HTML file or URL").option("-o, --output <file>", "Output file path").option("-m, --minify", "Enable all minification (HTML, CSS, JS)").option("--no-minify", "Disable all minification").option("--no-minify-html", "Disable HTML minification").option("--no-minify-css", "Disable CSS minification").option("--no-minify-js", "Disable JavaScript minification").option("-e, --embed-assets", "Embed assets as data URIs").option("--no-embed-assets", "Keep asset links relative/absolute").option(
+    "-r, --recursive [depth]",
+    "Recursively crawl site (optional depth)",
+    parseRecursiveValue
+  ).option("--max-depth <n>", "Set max depth for recursive crawl (alias for -r <n>)", parseInt).option("-b, --base-url <url>", "Base URL for resolving relative links").option("-d, --dry-run", "Run without writing output file").option("-v, --verbose", "Enable verbose (debug) logging").addOption(new import_commander.Option("--log-level <level>", "Set logging level").choices(logLevels));
   program.parse(argv);
   const opts = program.opts();
   const inputArg = program.args.length > 0 ? program.args[0] : void 0;
@@ -246,7 +250,9 @@ var init_logger = __esm({
           case "none":
             return new _Logger(0 /* NONE */);
           default:
-            console.warn(`[Logger] Invalid log level name "${levelName}". Defaulting to ${LogLevel[defaultLevel]}.`);
+            console.warn(
+              `[Logger] Invalid log level name "${levelName}". Defaulting to ${LogLevel[defaultLevel]}.`
+            );
             return new _Logger(defaultLevel);
         }
       }
@@ -326,7 +332,6 @@ function isUtf8DecodingLossy(originalBuffer, decodedString) {
   }
 }
 function determineBaseUrl(inputPathOrUrl, logger) {
-  console.log(`[DEBUG determineBaseUrl] Input: "${inputPathOrUrl}"`);
   logger?.debug(`Determining base URL for input: ${inputPathOrUrl}`);
   if (!inputPathOrUrl) {
     logger?.warn("Cannot determine base URL: inputPathOrUrl is empty or invalid.");
@@ -340,11 +345,11 @@ function determineBaseUrl(inputPathOrUrl, logger) {
       url.hash = "";
       const baseUrl = url.href;
       logger?.debug(`Determined remote base URL: ${baseUrl}`);
-      console.log(`[DEBUG determineBaseUrl] Determined Remote URL: "${baseUrl}"`);
       return baseUrl;
     } else if (inputPathOrUrl.includes("://") && !inputPathOrUrl.startsWith("file:")) {
-      logger?.warn(`Input "${inputPathOrUrl}" looks like a URL but uses an unsupported protocol. Cannot determine base URL.`);
-      console.log(`[DEBUG determineBaseUrl] Unsupported protocol.`);
+      logger?.warn(
+        `Input "${inputPathOrUrl}" looks like a URL but uses an unsupported protocol. Cannot determine base URL.`
+      );
       return void 0;
     } else {
       let resourcePath;
@@ -360,9 +365,7 @@ function determineBaseUrl(inputPathOrUrl, logger) {
           isInputLikelyDirectory = false;
         }
       }
-      console.log(`[DEBUG determineBaseUrl] resourcePath: "${resourcePath}", isInputLikelyDirectory: ${isInputLikelyDirectory}`);
       const baseDirPath = isInputLikelyDirectory ? resourcePath : import_path2.default.dirname(resourcePath);
-      console.log(`[DEBUG determineBaseUrl] Calculated baseDirPath: "${baseDirPath}"`);
       let normalizedPathForURL = baseDirPath.replace(/\\/g, "/");
       if (/^[A-Z]:\//i.test(normalizedPathForURL) && !normalizedPathForURL.startsWith("/")) {
         normalizedPathForURL = "/" + normalizedPathForURL;
@@ -372,14 +375,16 @@ function determineBaseUrl(inputPathOrUrl, logger) {
       }
       const fileUrl = new import_url.URL("file://" + normalizedPathForURL);
       const fileUrlString = fileUrl.href;
-      logger?.debug(`Determined base URL: ${fileUrlString} (from: ${inputPathOrUrl}, resolved base dir: ${baseDirPath})`);
-      console.log(`[DEBUG determineBaseUrl] Determined File URL: "${fileUrlString}"`);
+      logger?.debug(
+        `Determined base URL: ${fileUrlString} (from: ${inputPathOrUrl}, resolved base dir: ${baseDirPath})`
+      );
       return fileUrlString;
     }
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
-    console.error(`[DEBUG determineBaseUrl] Error determining base URL: ${message}`);
-    logger?.error(`\u{1F480} Failed to determine base URL for "${inputPathOrUrl}": ${message}${error instanceof Error && error.stack ? ` - Stack: ${error.stack}` : ""}`);
+    logger?.error(
+      `\u{1F480} Failed to determine base URL for "${inputPathOrUrl}": ${message}${error instanceof Error && error.stack ? ` - Stack: ${error.stack}` : ""}`
+    );
     return void 0;
   }
 }
@@ -394,7 +399,9 @@ function resolveAssetUrl(assetUrl, baseContextUrl, logger) {
       const base = new import_url.URL(baseContextUrl);
       resolvableUrl = base.protocol + resolvableUrl;
     } catch (e) {
-      logger?.warn(`Could not extract protocol from base "${baseContextUrl}" for protocol-relative URL "${trimmedUrl}". Skipping.`);
+      logger?.warn(
+        `Could not extract protocol from base "${baseContextUrl}" for protocol-relative URL "${trimmedUrl}". Skipping.`
+      );
       return null;
     }
   }
@@ -408,95 +415,94 @@ function resolveAssetUrl(assetUrl, baseContextUrl, logger) {
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     if (!/^[a-z]+:/i.test(resolvableUrl) && !resolvableUrl.startsWith("/") && !baseContextUrl) {
-      logger?.warn(`Cannot resolve relative URL "${resolvableUrl}" - Base context URL was not provided or determined.`);
+      logger?.warn(
+        `Cannot resolve relative URL "${resolvableUrl}" - Base context URL was not provided or determined.`
+      );
     } else {
-      logger?.warn(`\u26A0\uFE0F Failed to parse/resolve URL "${resolvableUrl}" ${baseContextUrl ? 'against base "' + baseContextUrl + '"' : "(no base provided)"}: ${message}`);
+      logger?.warn(
+        `\u26A0\uFE0F Failed to parse/resolve URL "${resolvableUrl}" ${baseContextUrl ? 'against base "' + baseContextUrl + '"' : "(no base provided)"}: ${message}`
+      );
     }
     return null;
   }
 }
 function resolveCssRelativeUrl(relativeUrl, cssBaseContextUrl, logger) {
-  console.log(`[DEBUG resolveCssRelativeUrl] Input: relative="${relativeUrl}", base="${cssBaseContextUrl}"`);
   if (!relativeUrl || relativeUrl.startsWith("data:") || relativeUrl.startsWith("#")) {
     return null;
   }
   try {
     const resolvedUrl = new import_url.URL(relativeUrl, cssBaseContextUrl);
-    console.log(`[DEBUG resolveCssRelativeUrl] Resolved URL object href: "${resolvedUrl.href}"`);
     return resolvedUrl.href;
   } catch (error) {
     logger?.warn(
       `Failed to resolve CSS URL: "${relativeUrl}" relative to "${cssBaseContextUrl}": ${String(error)}`
     );
-    console.error(`[DEBUG resolveCssRelativeUrl] Error resolving: ${String(error)}`);
     return null;
   }
 }
 async function fetchAsset(resolvedUrl, logger, timeout = 1e4) {
-  console.log(`[DEBUG fetchAsset] Attempting fetch for URL: ${resolvedUrl.href}`);
   logger?.debug(`Attempting to fetch asset: ${resolvedUrl.href}`);
   const protocol = resolvedUrl.protocol;
   try {
     if (protocol === "http:" || protocol === "https:") {
       const response = await axiosNs.default.get(resolvedUrl.href, {
         responseType: "arraybuffer",
+        // Fetch as binary data
         timeout
+        // Apply network timeout
       });
-      logger?.debug(`Workspaceed remote asset ${resolvedUrl.href} (Status: ${response.status}, Type: ${response.headers["content-type"] || "N/A"}, Size: ${response.data?.byteLength ?? 0} bytes)`);
-      console.log(`[DEBUG fetchAsset] HTTP fetch SUCCESS for: ${resolvedUrl.href}, Status: ${response.status}`);
+      logger?.debug(
+        `Workspaceed remote asset ${resolvedUrl.href} (Status: ${response.status}, Type: ${response.headers["content-type"] || "N/A"}, Size: ${response.data?.byteLength ?? 0} bytes)`
+      );
       return Buffer.from(response.data);
     } else if (protocol === "file:") {
       let filePath;
       try {
         filePath = (0, import_url.fileURLToPath)(resolvedUrl);
       } catch (e) {
-        console.error(`[DEBUG fetchAsset] fileURLToPath FAILED for: ${resolvedUrl.href}`, e);
-        logger?.error(`Could not convert file URL to path: ${resolvedUrl.href}. Error: ${e.message}`);
+        logger?.error(
+          `Could not convert file URL to path: ${resolvedUrl.href}. Error: ${e.message}`
+        );
         return null;
       }
       const normalizedForLog = import_path2.default.normalize(filePath);
-      console.log(`[DEBUG fetchAsset] Attempting readFile with path: "${normalizedForLog}" (Original from URL: "${filePath}")`);
       const data = await (0, import_promises.readFile)(filePath);
-      console.log(`[DEBUG fetchAsset] readFile call SUCCEEDED for path: "${normalizedForLog}". Data length: ${data?.byteLength}`);
       logger?.debug(`Read local file ${filePath} (${data.byteLength} bytes)`);
       return data;
     } else {
-      console.log(`[DEBUG fetchAsset] Unsupported protocol: ${protocol}`);
       logger?.warn(`Unsupported protocol "${protocol}" in URL: ${resolvedUrl.href}`);
       return null;
     }
   } catch (error) {
     const failedId = protocol === "file:" ? import_path2.default.normalize((0, import_url.fileURLToPath)(resolvedUrl)) : resolvedUrl.href;
-    console.error(`[DEBUG fetchAsset] fetch/read FAILED for: "${failedId}". Error:`, error);
-    if ((protocol === "http:" || protocol === "https:") && axiosNs.isAxiosError(error)) {
-      const status = error.response?.status ?? "N/A";
-      const statusText = error.response?.statusText ?? "Error";
-      const code = error.code ?? "N/A";
-      const message = error.message;
-      const logMessage = `\u26A0\uFE0F Failed to fetch remote asset ${resolvedUrl.href}: Status ${status} - ${statusText}. Code: ${code}, Message: ${message}`;
+    if ((protocol === "http:" || protocol === "https:") && error?.isAxiosError === true) {
+      const axiosError = error;
+      const status = axiosError.response?.status ?? "N/A";
+      const code = axiosError.code ?? "N/A";
+      const logMessage = `\u26A0\uFE0F Failed to fetch remote asset ${resolvedUrl.href}: ${axiosError.message} (Code: ${code})`;
       logger?.warn(logMessage);
-    }
-    if (error instanceof Error && error.code === "ENOENT") {
+    } else if (protocol === "file:" && error instanceof Error) {
       let failedPath = resolvedUrl.href;
       try {
         failedPath = (0, import_url.fileURLToPath)(resolvedUrl);
       } catch {
       }
       failedPath = import_path2.default.normalize(failedPath);
-      if (error instanceof Error && error.code === "ENOENT") {
+      if (error.code === "ENOENT") {
         logger?.warn(`\u26A0\uFE0F File not found (ENOENT) for asset: ${failedPath}.`);
-      } else if (error instanceof Error && error.code === "EACCES") {
+      } else if (error.code === "EACCES") {
         logger?.warn(`\u26A0\uFE0F Permission denied (EACCES) reading asset: ${failedPath}.`);
-        logger?.warn(`\u26A0\uFE0F Failed to read local asset ${failedPath}: ${error.message}`);
-      } else if (error instanceof Error) {
-        logger?.warn(`\u26A0\uFE0F Failed to read local asset ${failedPath}: ${error.message}`);
       } else {
-        logger?.warn(`\u26A0\uFE0F An unknown error occurred while reading local asset ${failedPath}: ${String(error)}`);
+        logger?.warn(`\u26A0\uFE0F Failed to read local asset ${failedPath}: ${error.message}`);
       }
     } else if (error instanceof Error) {
-      logger?.warn(`\u26A0\uFE0F An unexpected error occurred processing asset ${resolvedUrl.href}: ${error.message}`);
+      logger?.warn(
+        `\u26A0\uFE0F An unexpected error occurred processing asset ${resolvedUrl.href}: ${error.message}`
+      );
     } else {
-      logger?.warn(`\u26A0\uFE0F An unknown and unexpected error occurred processing asset ${resolvedUrl.href}: ${String(error)}`);
+      logger?.warn(
+        `\u26A0\uFE0F An unknown and unexpected error occurred processing asset ${resolvedUrl.href}: ${String(error)}`
+      );
     }
     return null;
   }
@@ -507,7 +513,8 @@ function extractUrlsFromCSS(cssContent, cssBaseContextUrl, logger) {
   const urlRegex = /url\(\s*(['"]?)(.*?)\1\s*\)/gi;
   const importRegex = /@import\s+(?:url\(\s*(['"]?)(.*?)\1\s*\)|(['"])(.*?)\3)\s*;/gi;
   const processFoundUrl = (rawUrl, ruleType) => {
-    if (!rawUrl || rawUrl.trim() === "" || rawUrl.startsWith("data:")) return;
+    if (!rawUrl || rawUrl.trim() === "" || rawUrl.startsWith("data:") || rawUrl.startsWith("#"))
+      return;
     const resolvedUrl = resolveCssRelativeUrl(rawUrl, cssBaseContextUrl, logger);
     if (resolvedUrl && !processedInThisParse.has(resolvedUrl)) {
       processedInThisParse.add(resolvedUrl);
@@ -515,11 +522,13 @@ function extractUrlsFromCSS(cssContent, cssBaseContextUrl, logger) {
       newlyDiscovered.push({
         type: assetType,
         url: resolvedUrl,
-        // The resolved absolute URL string
+        // Store the resolved absolute URL string
         content: void 0
         // Content will be fetched later if needed
       });
-      logger?.debug(`Discovered nested ${assetType} asset (${ruleType}) in CSS ${cssBaseContextUrl}: ${resolvedUrl}`);
+      logger?.debug(
+        `Discovered nested ${assetType} asset (${ruleType}) in CSS ${cssBaseContextUrl}: ${resolvedUrl}`
+      );
     }
   };
   let match;
@@ -533,14 +542,20 @@ function extractUrlsFromCSS(cssContent, cssBaseContextUrl, logger) {
   return newlyDiscovered;
 }
 async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger) {
-  logger?.info(`\u{1F680} Starting asset extraction! Embed: ${embedAssets}. Input: ${inputPathOrUrl || "(HTML content only)"}`);
+  logger?.info(
+    `\u{1F680} Starting asset extraction! Embed: ${embedAssets}. Input: ${inputPathOrUrl || "(HTML content only)"}`
+  );
   const initialAssets = parsed.assets || [];
   const finalAssetsMap = /* @__PURE__ */ new Map();
   let assetsToProcess = [];
   const processedOrQueuedUrls = /* @__PURE__ */ new Set();
   const htmlBaseContextUrl = determineBaseUrl(inputPathOrUrl || "", logger);
-  if (!htmlBaseContextUrl && initialAssets.some((a) => !/^[a-z]+:/i.test(a.url) && !a.url.startsWith("data:") && !a.url.startsWith("#") && !a.url.startsWith("/"))) {
-    logger?.warn("\u{1F6A8} No valid base path/URL determined for the HTML source! Resolution of relative asset paths from HTML may fail.");
+  if (!htmlBaseContextUrl && initialAssets.some(
+    (a) => !/^[a-z]+:/i.test(a.url) && !a.url.startsWith("data:") && !a.url.startsWith("#") && !a.url.startsWith("/")
+  )) {
+    logger?.warn(
+      "\u{1F6A8} No valid base path/URL determined for the HTML source! Resolution of relative asset paths from HTML may fail."
+    );
   } else if (htmlBaseContextUrl) {
     logger?.debug(`Using HTML base context URL: ${htmlBaseContextUrl}`);
   }
@@ -552,7 +567,7 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
       continue;
     }
     const urlToQueue = resolvedUrlObj.href;
-    if (!urlToQueue.startsWith("data:") && !processedOrQueuedUrls.has(urlToQueue)) {
+    if (!processedOrQueuedUrls.has(urlToQueue)) {
       processedOrQueuedUrls.add(urlToQueue);
       const { assetType: guessedType } = guessMimeType(urlToQueue);
       const initialType = asset.type ?? guessedType;
@@ -561,10 +576,9 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
         // Use the resolved URL
         type: initialType,
         content: void 0
+        // Content is initially undefined
       });
       logger?.debug(` -> Queued initial asset: ${urlToQueue} (Original raw: ${asset.url})`);
-    } else if (urlToQueue.startsWith("data:")) {
-      logger?.debug(` -> Skipping data URI: ${urlToQueue.substring(0, 50)}...`);
     } else {
       logger?.debug(` -> Skipping already processed/queued initial asset: ${urlToQueue}`);
     }
@@ -573,9 +587,13 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
   while (assetsToProcess.length > 0) {
     iterationCount++;
     if (iterationCount > MAX_ASSET_EXTRACTION_ITERATIONS) {
-      logger?.error(`\u{1F6D1} Asset extraction loop limit hit (${MAX_ASSET_EXTRACTION_ITERATIONS})! Aborting.`);
+      logger?.error(
+        `\u{1F6D1} Asset extraction loop limit hit (${MAX_ASSET_EXTRACTION_ITERATIONS})! Aborting.`
+      );
       const remainingUrls = assetsToProcess.map((a) => a.url).slice(0, 10).join(", ");
-      logger?.error(`Remaining queue sample (${assetsToProcess.length} items): ${remainingUrls}...`);
+      logger?.error(
+        `Remaining queue sample (${assetsToProcess.length} items): ${remainingUrls}...`
+      );
       assetsToProcess.forEach((asset) => {
         if (!finalAssetsMap.has(asset.url)) {
           finalAssetsMap.set(asset.url, { ...asset, content: void 0 });
@@ -601,7 +619,9 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
         try {
           assetUrlObj = new import_url.URL(asset.url);
         } catch (urlError) {
-          logger?.warn(`Cannot create URL object for "${asset.url}", skipping fetch. Error: ${urlError instanceof Error ? urlError.message : String(urlError)}`);
+          logger?.warn(
+            `Cannot create URL object for "${asset.url}", skipping fetch. Error: ${urlError instanceof Error ? urlError.message : String(urlError)}`
+          );
           finalAssetsMap.set(asset.url, { ...asset, content: void 0 });
           continue;
         }
@@ -637,7 +657,9 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
               cssContentForParsing = textContent;
             }
           } else {
-            logger?.warn(`Could not decode ${asset.type} asset ${asset.url} as valid UTF-8 text.${embedAssets ? " Falling back to base64 data URI." : ""}`);
+            logger?.warn(
+              `Could not decode ${asset.type} asset ${asset.url} as valid UTF-8 text.${embedAssets ? " Falling back to base64 data URI." : ""}`
+            );
             cssContentForParsing = void 0;
             if (embedAssets) {
               finalContent = `data:${effectiveMime};base64,${assetContentBuffer.toString("base64")}`;
@@ -658,14 +680,18 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
             try {
               const attemptedTextContent = assetContentBuffer.toString("utf-8");
               if (isUtf8DecodingLossy(assetContentBuffer, attemptedTextContent)) {
-                logger?.warn(`Couldn't embed unclassified asset ${asset.url} as text due to invalid UTF-8 sequences. Falling back to base64 (octet-stream).`);
+                logger?.warn(
+                  `Couldn't embed unclassified asset ${asset.url} as text due to invalid UTF-8 sequences. Falling back to base64 (octet-stream).`
+                );
                 finalContent = `data:application/octet-stream;base64,${assetContentBuffer.toString("base64")}`;
               } else {
                 finalContent = attemptedTextContent;
                 logger?.debug(`Successfully embedded unclassified asset ${asset.url} as text.`);
               }
             } catch (decodeError) {
-              logger?.warn(`Error during text decoding for unclassified asset ${asset.url}: ${decodeError instanceof Error ? decodeError.message : String(decodeError)}. Falling back to base64.`);
+              logger?.warn(
+                `Error during text decoding for unclassified asset ${asset.url}: ${decodeError instanceof Error ? decodeError.message : String(decodeError)}. Falling back to base64.`
+              );
               finalContent = `data:application/octet-stream;base64,${assetContentBuffer.toString("base64")}`;
             }
           } else {
@@ -679,34 +705,44 @@ async function extractAssets(parsed, embedAssets = true, inputPathOrUrl, logger)
       finalAssetsMap.set(asset.url, { ...asset, url: asset.url, content: finalContent });
       if (asset.type === "css" && cssContentForParsing) {
         const cssBaseContextUrl = determineBaseUrl(asset.url, logger);
-        logger?.debug(`CSS base context for resolving nested assets within ${asset.url}: ${cssBaseContextUrl}`);
+        logger?.debug(
+          `CSS base context for resolving nested assets within ${asset.url}: ${cssBaseContextUrl}`
+        );
         if (cssBaseContextUrl) {
           const newlyDiscoveredAssets = extractUrlsFromCSS(
             cssContentForParsing,
             cssBaseContextUrl,
-            // Use CSS file's base URL
+            // Use the CSS file's own URL as the base
             logger
           );
           if (newlyDiscoveredAssets.length > 0) {
-            logger?.debug(`Discovered ${newlyDiscoveredAssets.length} nested assets in CSS ${asset.url}. Checking against queue...`);
+            logger?.debug(
+              `Discovered ${newlyDiscoveredAssets.length} nested assets in CSS ${asset.url}. Checking against queue...`
+            );
             for (const newAsset of newlyDiscoveredAssets) {
               if (!processedOrQueuedUrls.has(newAsset.url)) {
                 processedOrQueuedUrls.add(newAsset.url);
                 assetsToProcess.push(newAsset);
                 logger?.debug(` -> Queued new nested asset: ${newAsset.url}`);
               } else {
-                logger?.debug(` -> Skipping already processed/queued nested asset: ${newAsset.url}`);
+                logger?.debug(
+                  ` -> Skipping already processed/queued nested asset: ${newAsset.url}`
+                );
               }
             }
           }
         } else {
-          logger?.warn(`Could not determine base URL context for CSS file ${asset.url}. Cannot resolve nested relative paths within it.`);
+          logger?.warn(
+            `Could not determine base URL context for CSS file ${asset.url}. Cannot resolve nested relative paths within it.`
+          );
         }
       }
     }
   }
-  const finalIterationCount = iterationCount > MAX_ASSET_EXTRACTION_ITERATIONS ? "MAX+" : iterationCount;
-  logger?.info(`\u2705 Asset extraction COMPLETE! Found ${finalAssetsMap.size} unique assets in ${finalIterationCount} iterations.`);
+  const finalIterationCount = iterationCount > MAX_ASSET_EXTRACTION_ITERATIONS ? `${MAX_ASSET_EXTRACTION_ITERATIONS}+ (limit hit)` : iterationCount;
+  logger?.info(
+    `\u2705 Asset extraction COMPLETE! Found ${finalAssetsMap.size} unique assets in ${finalIterationCount} iterations.`
+  );
   return {
     htmlContent: parsed.htmlContent,
     assets: Array.from(finalAssetsMap.values())
@@ -745,7 +781,7 @@ async function minifyAssets(parsed, options = {}, logger) {
   logger?.debug(`Minification flags: ${JSON.stringify(minifyFlags)}`);
   const minifiedAssets = await Promise.all(
     currentAssets.map(async (asset) => {
-      let processedAsset = { ...asset };
+      const processedAsset = { ...asset };
       if (typeof processedAsset.content !== "string" || processedAsset.content.length === 0) {
         return processedAsset;
       }
@@ -760,13 +796,17 @@ async function minifyAssets(parsed, options = {}, logger) {
             logger?.warn(`\u26A0\uFE0F CleanCSS failed for ${assetIdentifier}: ${result.errors.join(", ")}`);
           } else {
             if (result.warnings && result.warnings.length > 0) {
-              logger?.debug(`CleanCSS warnings for ${assetIdentifier}: ${result.warnings.join(", ")}`);
+              logger?.debug(
+                `CleanCSS warnings for ${assetIdentifier}: ${result.warnings.join(", ")}`
+              );
             }
             if (result.styles) {
               newContent = result.styles;
               logger?.debug(`CSS minified successfully: ${assetIdentifier}`);
             } else {
-              logger?.warn(`\u26A0\uFE0F CleanCSS produced no styles but reported no errors for ${assetIdentifier}. Keeping original.`);
+              logger?.warn(
+                `\u26A0\uFE0F CleanCSS produced no styles but reported no errors for ${assetIdentifier}. Keeping original.`
+              );
             }
           }
         }
@@ -779,15 +819,21 @@ async function minifyAssets(parsed, options = {}, logger) {
           } else {
             const terserError = result.error;
             if (terserError) {
-              logger?.warn(`\u26A0\uFE0F Terser failed for ${assetIdentifier}: ${terserError.message || terserError}`);
+              logger?.warn(
+                `\u26A0\uFE0F Terser failed for ${assetIdentifier}: ${terserError.message || terserError}`
+              );
             } else {
-              logger?.warn(`\u26A0\uFE0F Terser produced no code but reported no errors for ${assetIdentifier}. Keeping original.`);
+              logger?.warn(
+                `\u26A0\uFE0F Terser produced no code but reported no errors for ${assetIdentifier}. Keeping original.`
+              );
             }
           }
         }
       } catch (err) {
         const errorMessage = err instanceof Error ? err.message : String(err);
-        logger?.warn(`\u26A0\uFE0F Failed to minify asset ${assetIdentifier} (${processedAsset.type}): ${errorMessage}`);
+        logger?.warn(
+          `\u26A0\uFE0F Failed to minify asset ${assetIdentifier} (${processedAsset.type}): ${errorMessage}`
+        );
       }
       processedAsset.content = newContent;
       return processedAsset;
@@ -950,7 +996,9 @@ function inlineAssets($, assets, logger) {
       logger?.debug(`Inlining image via ${srcAttr}: ${asset.url}`);
       element.attr(srcAttr, asset.content);
     } else if (src) {
-      logger?.warn(`Could not inline image via ${srcAttr}: ${src}. Content missing or not a data URI.`);
+      logger?.warn(
+        `Could not inline image via ${srcAttr}: ${src}. Content missing or not a data URI.`
+      );
     }
   });
   $("img[srcset], source[srcset]").each((_, el) => {
@@ -1084,7 +1132,9 @@ function bundleMultiPageHTML(pages, logger) {
     } else if (!baseSlug) {
       if (isRootIndex) {
         baseSlug = "index";
-        logger?.debug(`URL "${page.url}" sanitized to empty string, using "index" as it is a root index.`);
+        logger?.debug(
+          `URL "${page.url}" sanitized to empty string, using "index" as it is a root index.`
+        );
       } else {
         baseSlug = "page";
         logger?.debug(`URL "${page.url}" sanitized to empty string, using fallback slug "page".`);
@@ -1092,14 +1142,18 @@ function bundleMultiPageHTML(pages, logger) {
     }
     if (!baseSlug) {
       baseSlug = `page-${pageCounterForFallback++}`;
-      logger?.warn(`Could not determine a valid base slug for "${page.url}", using generated fallback "${baseSlug}".`);
+      logger?.warn(
+        `Could not determine a valid base slug for "${page.url}", using generated fallback "${baseSlug}".`
+      );
     }
     let slug = baseSlug;
     let collisionCounter = 1;
     const originalBaseSlugForLog = baseSlug;
     while (usedSlugs.has(slug)) {
       const newSlug = `${originalBaseSlugForLog}-${collisionCounter++}`;
-      logger?.warn(`Slug collision detected for "${page.url}" (intended slug: '${originalBaseSlugForLog}'). Using "${newSlug}" instead.`);
+      logger?.warn(
+        `Slug collision detected for "${page.url}" (intended slug: '${originalBaseSlugForLog}'). Using "${newSlug}" instead.`
+      );
       slug = newSlug;
     }
     usedSlugs.add(slug);
@@ -1109,7 +1163,8 @@ function bundleMultiPageHTML(pages, logger) {
     }
   }
   const defaultPageSlug = usedSlugs.has("index") ? "index" : firstValidSlug || "page";
-  let output = `<!DOCTYPE html>
+  const output = `
+<!DOCTYPE html>
 <html lang="en">
 <head>
     <meta charset="UTF-8">
@@ -1125,74 +1180,74 @@ function bundleMultiPageHTML(pages, logger) {
     </style>
 </head>
 <body>
-    <nav id="main-nav">
-        ${validPages.map((p) => {
+<nav id="main-nav">
+    ${validPages.map((p) => {
     const slug = slugMap.get(p.url);
     const label = slug;
     return `<a href="#${slug}" data-page="${slug}">${label}</a>`;
   }).join("\n        ")}
-    </nav>
-    <div id="page-container"></div>
-    ${validPages.map((p) => {
+</nav>
+<div id="page-container"></div>
+${validPages.map((p) => {
     const slug = slugMap.get(p.url);
     return `<template id="page-${slug}">${p.html}</template>`;
   }).join("\n    ")}
-    <script id="router-script">
-        document.addEventListener('DOMContentLoaded', function() {
-            const pageContainer = document.getElementById('page-container');
-            const navLinks = document.querySelectorAll('#main-nav a');
+<script id="router-script">
+    document.addEventListener('DOMContentLoaded', function() {
+        const pageContainer = document.getElementById('page-container');
+        const navLinks = document.querySelectorAll('#main-nav a');
 
-            function navigateTo(slug) {
-                const template = document.getElementById('page-' + slug);
-                if (!template || !pageContainer) {
-                    console.warn('Navigation failed: Template or container not found for slug:', slug);
-                    // Maybe try navigating to default page? Or just clear container?
-                    if (pageContainer) pageContainer.innerHTML = '<p>Page not found.</p>';
-                    return;
-                }
-                // Clear previous content and append new content
-                pageContainer.innerHTML = ''; // Clear reliably
-                pageContainer.appendChild(template.content.cloneNode(true));
+        function navigateTo(slug) {
+            const template = document.getElementById('page-' + slug);
+            if (!template || !pageContainer) {
+                console.warn('Navigation failed: Template or container not found for slug:', slug);
+                // Maybe try navigating to default page? Or just clear container?
+                if (pageContainer) pageContainer.innerHTML = '<p>Page not found.</p>';
+                return;
+            }
+            // Clear previous content and append new content
+            pageContainer.innerHTML = ''; // Clear reliably
+            pageContainer.appendChild(template.content.cloneNode(true));
 
-                // Update active link styling
-                navLinks.forEach(link => {
-                    link.classList.toggle('active', link.getAttribute('data-page') === slug);
-                });
+            // Update active link styling
+            navLinks.forEach(link => {
+                link.classList.toggle('active', link.getAttribute('data-page') === slug);
+            });
 
-                // Update URL hash without triggering hashchange if already correct
-                if (window.location.hash.substring(1) !== slug) {
-                    // Use pushState for cleaner history
-                    history.pushState({ slug: slug }, '', '#' + slug);
-                }
+            // Update URL hash without triggering hashchange if already correct
+            if (window.location.hash.substring(1) !== slug) {
+                // Use pushState for cleaner history
+                history.pushState({ slug: slug }, '', '#' + slug);
             }
+        }
 
-            // Handle back/forward navigation
-            window.addEventListener('popstate', (event) => {
-                let slug = window.location.hash.substring(1);
-                // If popstate event has state use it, otherwise fallback to hash or default
-                if (event && event.state && event.state.slug) { // Check event exists
-                    slug = event.state.slug;
-                }
-                // Ensure the target page exists before navigating, fallback to default slug
-                const targetSlug = document.getElementById('page-' + slug) ? slug : '${defaultPageSlug}';
-                navigateTo(targetSlug);
-            });
+        // Handle back/forward navigation
+        window.addEventListener('popstate', (event) => {
+            let slug = window.location.hash.substring(1);
+            // If popstate event has state use it, otherwise fallback to hash or default
+            if (event && event.state && event.state.slug) { // Check event exists
+                slug = event.state.slug;
+            }
+            // Ensure the target page exists before navigating, fallback to default slug
+            const targetSlug = document.getElementById('page-' + slug) ? slug : '${defaultPageSlug}';
+            navigateTo(targetSlug);
+        });
 
-            // Handle direct link clicks
-            navLinks.forEach(link => {
-                link.addEventListener('click', function(e) {
-                    e.preventDefault();
-                    const slug = this.getAttribute('data-page');
-                    if (slug) navigateTo(slug);
-                });
+        // Handle direct link clicks
+        navLinks.forEach(link => {
+            link.addEventListener('click', function(e) {
+                e.preventDefault();
+                const slug = this.getAttribute('data-page');
+                if (slug) navigateTo(slug);
             });
-
-            // Initial page load
-            const initialHash = window.location.hash.substring(1);
-            const initialSlug = document.getElementById('page-' + initialHash) ? initialHash : '${defaultPageSlug}';
-            navigateTo(initialSlug);
         });
-    </script>
+
+        // Initial page load
+        const initialHash = window.location.hash.substring(1);
+        const initialSlug = document.getElementById('page-' + initialHash) ? initialHash : '${defaultPageSlug}';
+        navigateTo(initialSlug);
+    });
+</script>
 </body>
 </html>`;
   logger?.info(`Multi-page bundle generated. Size: ${Buffer.byteLength(output, "utf-8")} bytes.`);
@@ -1258,7 +1313,9 @@ async function fetchAndPackWebPage(url, logger, timeout = DEFAULT_PAGE_TIMEOUT,
       throw pageError;
     }
   } catch (launchError) {
-    logger?.error(`Critical error during browser launch or page setup for ${url}: ${launchError.message}`);
+    logger?.error(
+      `Critical error during browser launch or page setup for ${url}: ${launchError.message}`
+    );
     if (browser) {
       try {
         await browser.close();
@@ -1271,7 +1328,9 @@ async function fetchAndPackWebPage(url, logger, timeout = DEFAULT_PAGE_TIMEOUT,
     throw launchError;
   } finally {
     if (browser) {
-      logger?.warn(`Closing browser in final cleanup for ${url}. This might indicate an unusual error path.`);
+      logger?.warn(
+        `Closing browser in final cleanup for ${url}. This might indicate an unusual error path.`
+      );
       try {
         await browser.close();
       } catch (closeErr) {
@@ -1388,21 +1447,26 @@ async function crawlWebsite(startUrl, options) {
 }
 async function recursivelyBundleSite(startUrl, outputFile, maxDepth = 1, loggerInstance) {
   const logger = loggerInstance || new Logger();
-  logger.info(`Starting recursive site bundle for ${startUrl} to ${outputFile} (maxDepth: ${maxDepth})`);
+  logger.info(
+    `Starting recursive site bundle for ${startUrl} to ${outputFile} (maxDepth: ${maxDepth})`
+  );
   try {
     const crawlOptions = {
       maxDepth,
       logger
-      /* Add other options like timeout, userAgent if needed */
     };
     const pages = await crawlWebsite(startUrl, crawlOptions);
     if (pages.length === 0) {
-      logger.warn("Crawl completed but found 0 pages. Output file may be empty or reflect an empty bundle.");
+      logger.warn(
+        "Crawl completed but found 0 pages. Output file may be empty or reflect an empty bundle."
+      );
     } else {
       logger.info(`Crawl successful, found ${pages.length} pages. Starting bundling.`);
     }
     const bundledHtml = bundleMultiPageHTML(pages, logger);
-    logger.info(`Bundling complete. Output size: ${Buffer.byteLength(bundledHtml, "utf-8")} bytes.`);
+    logger.info(
+      `Bundling complete. Output size: ${Buffer.byteLength(bundledHtml, "utf-8")} bytes.`
+    );
     logger.info(`Writing bundled HTML to ${outputFile}`);
     await fs2.writeFile(outputFile, bundledHtml, "utf-8");
     logger.info(`Successfully wrote bundled output to ${outputFile}`);