refactor(module-source): More compact boilerplate

endojs · Sep 5, 2024 · e04e72d · e04e72d
1 parent 1c71b66
commit e04e72d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 13 deletions.
diff --git a/packages/module-source/src/transform-analyze.js b/packages/module-source/src/transform-analyze.js
@@ -71,7 +71,7 @@ const makeCreateStaticRecord = transformSource =>
       .map(
         src =>
           `[${js(src)}, [${Object.entries(isrc[src])
-            .map(([exp, upds]) => `[${js(exp)}, [${upds.join(',')}]]`)
+            .map(([exp, upds]) => `[${js(exp)},[${upds.join(',')}]]`)
             .join(',')}]]`,
       )
       .join(',')}]);`;
@@ -80,9 +80,7 @@ const makeCreateStaticRecord = transformSource =>
         let src = '';
         if (cvname) {
           // It's a function assigned to, so set its name property.
-          src = `Object.defineProperty(${cvname}, 'name', {value: ${js(
-            vname,
-          )}});`;
+          src = `Object.defineProperty(${cvname},'name',{value:${js(vname)}});`;
         }
         const hDeclId = isOnce ? h.HIDDEN_ONCE : h.HIDDEN_LIVE;
         src += `${hDeclId}.${vname}(${cvname || ''});`;
@@ -99,14 +97,9 @@ const makeCreateStaticRecord = transformSource =>
     // well.
     // Relies on the evaluator to ensure these functions are strict.
     let functorSource = `\
-({ \
-  imports: ${h.HIDDEN_IMPORTS}, \
-  liveVar: ${h.HIDDEN_LIVE}, \
-  onceVar: ${h.HIDDEN_ONCE}, \
-  importMeta: ${h.HIDDEN_META}, \
-}) => (function () { 'use strict'; \
-  ${preamble} \
-  ${scriptSource}
+({imports:${h.HIDDEN_IMPORTS},liveVar:${h.HIDDEN_LIVE},onceVar:${h.HIDDEN_ONCE},importMeta:${h.HIDDEN_META}})=>(function(){'use strict';\
+${preamble}\
+${scriptSource}
 })()
 `;
 

diff --git a/packages/module-source/test/fixtures/format-preserved.txt b/packages/module-source/test/fixtures/format-preserved.txt
@@ -1,4 +1,4 @@
-({   imports: $h͏_imports,   liveVar: $h͏_live,   onceVar: $h͏_once,   importMeta: $h͏____meta, }) => (function () { 'use strict';   $h͏_imports([]);Object.defineProperty(createBinop, 'name', {value: "createBinop"});$h͏_once.createBinop(createBinop);   // deliberately offset
+({imports:$h͏_imports,liveVar:$h͏_live,onceVar:$h͏_once,importMeta:$h͏____meta})=>(function(){'use strict';$h͏_imports([]);Object.defineProperty(createBinop,'name',{value:"createBinop"});$h͏_once.createBinop(createBinop);// deliberately offset
 function TokenType() {}
 const beforeExpr = 0;