uglify-js-minify-css-allfiles 2.8.0 → 2.8.2

package/dist/module.js

@@ -98,7 +98,6 @@ async function processPattern(pattern, content, fileExt, filePath, logger, hashM
       }
     }
 
-    await Promise.all(promises.filter((p) => p instanceof Promise));
   }
 
   return { content: newContent, modified };
@@ -252,7 +251,7 @@ async function processFile(filePath, logger, options) {
 
       if (options.useJsMap) {
         const fileName = path.basename(filePath);
-        const mapFilePath = filePath.replace('.js', '.js.map');
+        const mapFilePath = filePath.replace(/\.js$/, '.js.map');
 
         await writeFile(filePath, result.code + `\n//# sourceMappingURL=${fileName}.map`, logger);
         await writeFile(mapFilePath, result.map, logger);
@@ -269,7 +268,7 @@ async function processFile(filePath, logger, options) {
 
       if (options.useCssMap && output.sourceMap) {
         const fileName = path.basename(filePath);
-        const mapFilePath = filePath.replace('.css', '.css.map');
+        const mapFilePath = filePath.replace(/\.css$/, '.css.map');
 
         // Add source map reference to the CSS file
         const sourceMapComment = `\n/*# sourceMappingURL=${fileName}.map */`;

package/dist/plugins/append-polyfill-plugin.js

@@ -2,7 +2,8 @@
  * Babel plugin to transform Element.append() calls to be compatible with older browsers.
  * This plugin converts:
  * - element.append("text") to element.appendChild(document.createTextNode("text"))
- * - element.append(node) to element.appendChild(node)
+ * - element.append(3) to element.appendChild(document.createTextNode(3))
+ * - element.append(node) to element.appendChild(node instanceof Node ? node : document.createTextNode(String(node)))
  * - element.append(node1, "text", node2) to a sequence of appendChild calls
  *
  * @param {Object} param0 - Babel plugin parameters
@@ -10,6 +11,34 @@
  * @returns {Object} Babel plugin object
  */
 export default function ({ types: t }) {
+  function createTextNodeExpression(arg) {
+    return t.callExpression(t.memberExpression(t.identifier('document'), t.identifier('createTextNode')), [arg]);
+  }
+
+  function isLiteral(node) {
+    return (
+      t.isStringLiteral(node) ||
+      t.isNumericLiteral(node) ||
+      t.isBooleanLiteral(node) ||
+      t.isNullLiteral(node) ||
+      t.isTemplateLiteral(node)
+    );
+  }
+
+  function toAppendChildArg(arg) {
+    // Known primitives: always wrap with createTextNode
+    if (isLiteral(arg)) {
+      return createTextNodeExpression(arg);
+    }
+    // Unknown values (variables, expressions): runtime check
+    // If it's a Node, use directly; otherwise wrap with createTextNode
+    return t.conditionalExpression(
+      t.binaryExpression('instanceof', t.cloneNode(arg), t.identifier('Node')),
+      t.cloneNode(arg),
+      createTextNodeExpression(t.cloneNode(arg)),
+    );
+  }
+
   return {
     name: 'append-polyfill-transform',
     visitor: {
@@ -28,39 +57,22 @@ export default function ({ types: t }) {
 
         // Handle single argument case
         if (args.length === 1) {
-          const arg = args[0];
-
-          // String literal case: element.append("text")
-          if (t.isStringLiteral(arg)) {
-            path.replaceWith(
-              t.callExpression(t.memberExpression(t.cloneNode(objectNode), t.identifier('appendChild')), [
-                t.callExpression(t.memberExpression(t.identifier('document'), t.identifier('createTextNode')), [arg]),
-              ]),
-            );
-          } else {
-            // Node case: element.append(node)
-            callee.get('property').replaceWith(t.identifier('appendChild'));
-          }
+          path.replaceWith(
+            t.callExpression(t.memberExpression(t.cloneNode(objectNode), t.identifier('appendChild')), [
+              toAppendChildArg(args[0]),
+            ]),
+          );
           return;
         }
 
         // Handle multiple arguments: element.append(node1, "text", node2)
-        const statements = args.map((arg) => {
-          // String literal case
-          if (t.isStringLiteral(arg)) {
-            return t.callExpression(t.memberExpression(t.cloneNode(objectNode), t.identifier('appendChild')), [
-              t.callExpression(t.memberExpression(t.identifier('document'), t.identifier('createTextNode')), [arg]),
-            ]);
-          } else {
-            // Node case
-            return t.callExpression(t.memberExpression(t.cloneNode(objectNode), t.identifier('appendChild')), [arg]);
-          }
-        });
+        const statements = args.map((arg) =>
+          t.callExpression(t.memberExpression(t.cloneNode(objectNode), t.identifier('appendChild')), [
+            toAppendChildArg(arg),
+          ]),
+        );
 
-        if (statements.length > 0) {
-          // Combine multiple statements into a sequence expression
-          path.replaceWith(t.sequenceExpression(statements));
-        }
+        path.replaceWith(t.sequenceExpression(statements));
       },
     },
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "uglify-js-minify-css-allfiles",
-  "version": "2.8.0",
+  "version": "2.8.2",
   "description": "you will be able to minify all files as same file names which is js or css",
   "main": "minify.js",
   "type": "module",