Merge pull request #23 from p0ise/master

fix case where uglify turns file.key into identifiers when using alias
browserify · Aug 2, 2024 · c5801b4 · c5801b4
2 parents 4ce1aee + 2c4e86c
commit c5801b4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -62,7 +62,9 @@ module.exports = function (src) {
             return acc;
         }, {});
         var row = {
-            id: file.key.value,
+            id: file.key.type === 'Literal'
+                ? file.key.value
+                : file.key.name,
             source: src.slice(start, end),
             deps: deps
         };

diff --git a/test/files/uglified.js b/test/files/uglified.js
diff --git a/test/uglified.js b/test/uglified.js
@@ -8,11 +8,12 @@ var fs = require('fs');
 var src = fs.readFileSync(__dirname + '/files/uglified.js', 'utf8');
 
 test('uglified', function (t) {
-    t.plan(3);
+    t.plan(4);
     t.doesNotThrow(function() {
         var p = pack({ raw: true });
         var rows = unpack(src);
         t.equal(rows.length, 3, 'should unpack 3 rows');
+        t.ok(rows[2].hasOwnProperty('id'), 'third row should have an id property');
     }, 'should not throw');
 
     t.deepEquals(deps(src), ['./foo.js', 'gamma']);