domflax 0.1.1 → 0.1.2

package/README.md

@@ -27,7 +27,7 @@ It rewrites only the **static shape** of your markup. Dynamic class lists (`clas
 - **Flattening is conservative.** A wrapper is removed only when removal is *provably* render-neutral — it establishes no layout context and has no style to reproduce on its child. It never drops a style it can't reproduce, and never touches a wrapper a CSS selector depends on (`.list > .item h3`).
 - domflax runs as a **purely static** source transform. It never launches a browser, so builds stay fast and deterministic.
 
-> **Status: v0.1.1.** Works end-to-end on real `.jsx`/`.tsx` — in component-return position **and inside `.map()` / expressions (list rows)** — via Vite, Next.js (webpack), and the CLI, with Tailwind and custom-CSS providers. 22 patterns. Wrappers that establish a layout context (e.g. `flex`/`grid` centering) are **conservatively preserved** — proving those render-identical needs context a static pass can't see; recovering them safely is on the Roadmap. APIs may change before 1.0.
+> **Status: v0.1.2.** Works end-to-end on real `.jsx`/`.tsx` — in component-return position **and inside `.map()` / expressions (list rows)** — via Vite, Next.js (webpack), and the CLI, with Tailwind and custom-CSS providers. 22 patterns. Wrappers that establish a layout context (e.g. `flex`/`grid` centering) are **conservatively preserved** — proving those render-identical needs context a static pass can't see; recovering them safely is on the Roadmap. APIs may change before 1.0.
 
 ## Install
 

package/dist/cli.cjs

@@ -5636,24 +5636,26 @@ function createTransform(options) {
     }
   };
 }
-function builtinPatternNames() {
-  return builtinPatterns.map((p2) => p2.name);
-}
 
 // ../cli/src/walk.ts
 init_cjs_shims();
 var fs = __toESM(require("fs"), 1);
 var path4 = __toESM(require("path"), 1);
-var SUPPORTED_EXTS = [".jsx", ".tsx", ".html", ".htm"];
+var SUPPORTED_EXTS = [".jsx", ".tsx"];
+var HTML_EXTS = [".html", ".htm"];
 var SKIP_DIRS = /* @__PURE__ */ new Set(["node_modules", ".git", "domflax-out"]);
 function isSupported(file) {
   const lower = file.toLowerCase();
   return SUPPORTED_EXTS.some((ext) => lower.endsWith(ext));
 }
+function isHtml(file) {
+  const lower = file.toLowerCase();
+  return HTML_EXTS.some((ext) => lower.endsWith(ext));
+}
 function hasGlobMagic(p2) {
   return /[*?[\]{}]/.test(p2);
 }
-function walkDir(dir, out) {
+function walkDir(dir, out, counts) {
   let entries;
   try {
     entries = fs.readdirSync(dir, { withFileTypes: true });
@@ -5664,9 +5666,10 @@ function walkDir(dir, out) {
     const full = path4.join(dir, entry.name);
     if (entry.isDirectory()) {
       if (SKIP_DIRS.has(entry.name)) continue;
-      walkDir(full, out);
-    } else if (entry.isFile() && isSupported(entry.name)) {
-      out.push(full);
+      walkDir(full, out, counts);
+    } else if (entry.isFile()) {
+      if (isSupported(entry.name)) out.push(full);
+      else if (isHtml(entry.name)) counts.html += 1;
     }
   }
 }
@@ -5677,6 +5680,7 @@ function globSyncMaybe() {
 function discoverInputs(paths) {
   const files = [];
   const warnings = [];
+  const counts = { html: 0 };
   const seen = /* @__PURE__ */ new Set();
   const push = (f) => {
     const abs = path4.resolve(f);
@@ -5700,11 +5704,13 @@ function discoverInputs(paths) {
       stat = null;
     }
     if (stat?.isDirectory()) {
-      walkDir(path4.resolve(p2), files);
+      walkDir(path4.resolve(p2), files, counts);
       continue;
     }
     if (stat?.isFile()) {
-      push(p2);
+      if (isSupported(p2)) push(p2);
+      else if (isHtml(p2)) counts.html += 1;
+      else warnings.push(`unsupported file type, skipped: ${p2}`);
       continue;
     }
     if (hasGlobMagic(p2)) {
@@ -5713,9 +5719,11 @@ function discoverInputs(paths) {
         warnings.push(`glob not supported on this Node version, skipped: ${p2}`);
         continue;
       }
-      const matches = glob(p2).filter(isSupported);
-      if (matches.length === 0) warnings.push(`no files matched: ${p2}`);
-      for (const m2 of matches) push(m2);
+      const matched = glob(p2);
+      const supported = matched.filter(isSupported);
+      counts.html += matched.filter(isHtml).length;
+      if (supported.length === 0) warnings.push(`no .jsx/.tsx files matched: ${p2}`);
+      for (const m2 of supported) push(m2);
       continue;
     }
     warnings.push(`no such file or directory: ${p2}`);
@@ -5729,6 +5737,11 @@ function discoverInputs(paths) {
       deduped.push(abs);
     }
   }
+  if (deduped.length === 0 && counts.html > 0) {
+    warnings.push(
+      `found ${counts.html} .html file${counts.html === 1 ? "" : "s"} but HTML optimization isn't supported yet (domflax currently optimizes .jsx/.tsx source; HTML is on the roadmap: https://github.com/Krishnesh-Mishra/domflax#roadmap).`
+    );
+  }
   return { files: deduped, inputRoot, warnings };
 }
 
@@ -6329,15 +6342,17 @@ var SKIP_DIRS2 = /* @__PURE__ */ new Set([
 ]);
 var COMMON_INPUT_DIRS = ["src", "app", "components", "pages", "lib", "ui", "public"];
 var CSS_FILE_CAP = 200;
+var DEFAULT_CSS_DEPTH = 10;
 function toRelative(root, abs) {
   const rel = path5.relative(root, abs);
   return rel.split(path5.sep).join("/");
 }
-function detectCssFiles(root) {
-  const found = [];
+function detectCssFiles(root, scanRoots = [], maxDepth = DEFAULT_CSS_DEPTH) {
+  const base = path5.resolve(root);
+  const found = /* @__PURE__ */ new Map();
   let capped = false;
-  const walk = (dir) => {
-    if (capped) return;
+  const walk = (dir, depth) => {
+    if (capped || depth > maxDepth) return;
     let entries;
     try {
       entries = fs2.readdirSync(dir, { withFileTypes: true });
@@ -6348,23 +6363,31 @@ function detectCssFiles(root) {
       const full = path5.join(dir, entry.name);
       if (entry.isDirectory()) {
         if (SKIP_DIRS2.has(entry.name)) continue;
-        walk(full);
+        walk(full, depth + 1);
         if (capped) return;
       } else if (entry.isFile() && entry.name.toLowerCase().endsWith(".css")) {
-        found.push(toRelative(root, full));
-        if (found.length >= CSS_FILE_CAP) {
-          capped = true;
-          return;
+        const abs = path5.resolve(full);
+        if (!found.has(abs)) {
+          found.set(abs, toRelative(base, abs));
+          if (found.size >= CSS_FILE_CAP) {
+            capped = true;
+            return;
+          }
         }
       }
     }
   };
-  walk(path5.resolve(root));
-  found.sort((a, b3) => a.localeCompare(b3));
+  walk(base, 0);
+  for (const r2 of scanRoots) {
+    if (capped) break;
+    const abs = path5.resolve(r2);
+    if (abs !== base) walk(abs, 0);
+  }
+  const list = [...found.values()].sort((a, b3) => a.localeCompare(b3));
   if (capped) {
     console.error(`domflax: more than ${CSS_FILE_CAP} CSS files found; showing the first ${CSS_FILE_CAP}.`);
   }
-  return found;
+  return list;
 }
 function detectInputDirs(root) {
   const resolved = path5.resolve(root);
@@ -6446,15 +6469,6 @@ async function runWizard(base) {
   } else if (outputMode === "overwrite") {
     dangerouslyOverwriteSource = true;
   }
-  const allPasses = builtinPatternNames();
-  const passSelection = await fe({
-    message: "Which optimization passes should run?",
-    options: allPasses.map((name) => ({ value: name, label: name })),
-    initialValues: [...allPasses],
-    required: true
-  });
-  if (cancelled(passSelection)) return done();
-  const passes = passSelection;
   const provider = await ve({
     message: "How should class names resolve to styles?",
     options: [
@@ -6467,7 +6481,7 @@ async function runWizard(base) {
   if (cancelled(provider)) return done();
   let css = base.css;
   if (provider === "custom") {
-    const detectedCss = detectCssFiles(root);
+    const detectedCss = detectCssFiles(root, [inputPath]);
     if (detectedCss.length > 0) {
       const picked = await fe({
         message: "Which CSS files should resolve your classes? (all detected files are pre-selected)",
@@ -6495,7 +6509,7 @@ async function runWizard(base) {
     css,
     dryRun,
     dangerouslyOverwriteSource,
-    passes: passes.length === allPasses.length ? null : passes,
+    passes: null,
     safety: base.safety ?? DEFAULT_SAFETY
   };
   function done() {
@@ -6525,7 +6539,7 @@ async function execute(options) {
   const { files, inputRoot, warnings } = discoverInputs(options.paths);
   for (const w2 of warnings) console.error(`domflax: ${w2}`);
   if (files.length === 0) {
-    console.error("domflax: no .jsx/.tsx/.html files found for the given paths");
+    console.error("domflax: no .jsx/.tsx files found for the given paths");
     return { exitCode: 1 };
   }
   const projectRoot = options.projectRoot ?? process.cwd();
@@ -6572,8 +6586,20 @@ async function execute(options) {
       failures += 1;
     }
   }
+  if (options.dryRun) {
+    console.log("\ndomflax: dry run \u2014 no files were written.");
+  } else if (totals.changed === 0) {
+    console.log(
+      `
+domflax: processed ${totals.files} file${totals.files === 1 ? "" : "s"} \u2014 nothing to optimize (0 changed).`
+    );
+  } else {
+    console.log(
+      `
+domflax: optimized ${totals.changed} of ${totals.files} file${totals.files === 1 ? "" : "s"} (${totals.nodesRemoved} nodes removed, ${totals.classesSaved} classes saved, ${totals.bytesSaved} bytes saved).`
+    );
+  }
   if (options.report) printReport(totals);
-  if (options.dryRun) console.log("\ndomflax: dry run \u2014 no files were written.");
   return { exitCode: failures > 0 ? 1 : 0 };
 }
 async function main(argv = process.argv.slice(2)) {