es-module-shims 1.7.1 → 1.7.3

package/README.md

@@ -29,7 +29,7 @@ Because we are still using the native module loader the edge cases work out comp
 Include ES Module Shims with a `async` attribute on the script, then include an import map and module scripts normally:
 
 ```html
-<script async src="https://ga.jspm.io/npm:es-module-shims@1.7.1/dist/es-module-shims.js"></script>
+<script async src="https://ga.jspm.io/npm:es-module-shims@1.7.3/dist/es-module-shims.js"></script>
 
 <!-- https://generator.jspm.io/#U2NhYGBkDM0rySzJSU1hKEpNTC5xMLTQM9Az0C1K1jMAAKFS5w0gAA -->
 <script type="importmap">

package/dist/es-module-shims.debug.js

@@ -1,4 +1,4 @@
-/* ES Module Shims DEBUG BUILD 1.7.1 */
+/* ES Module Shims DEBUG BUILD 1.7.3 */
 (function () {
 
   const hasWindow = typeof window !== 'undefined';
@@ -334,14 +334,14 @@
       iframe.style.display = 'none';
       iframe.setAttribute('nonce', nonce);
       function cb ({ data }) {
-        // failed feature detection (security policy) -> revert to default assumptions
-        if (Array.isArray(data)) {
-          supportsImportMaps = data[0];
-          supportsImportMeta = data[1];
-          supportsCssAssertions = data[2];
-          supportsJsonAssertions = data[3];
+        const isFeatureDetectionMessage = Array.isArray(data) && data[0] === 'esms';
+        if (!isFeatureDetectionMessage) {
+          return;
         }
-        else console.info(`es-module-shims: feature detection failure, using defaults`);
+        supportsImportMaps = data[1];
+        supportsImportMeta = data[2];
+        supportsCssAssertions = data[3];
+        supportsJsonAssertions = data[4];
         resolve();
         document.head.removeChild(iframe);
         window.removeEventListener('message', cb, false);
@@ -350,7 +350,7 @@
 
       const importMapTest = `<script nonce=${nonce || ''}>b=(s,type='text/javascript')=>URL.createObjectURL(new Blob([s],{type}));document.head.appendChild(Object.assign(document.createElement('script'),{type:'importmap',nonce:"${nonce}",innerText:\`{"imports":{"x":"\${b('')}"}}\`}));Promise.all([${
       supportsImportMaps ? 'true,true' : `'x',b('${importMetaCheck}')`}, ${cssModulesEnabled ? `b('${cssModulesCheck}'.replace('x',b('','text/css')))` : 'false'}, ${
-      jsonModulesEnabled ? `b('${jsonModulesCheck}'.replace('x',b('{}','text/json')))` : 'false'}].map(x =>typeof x==='string'?import(x).then(x =>!!x,()=>false):x)).then(a=>parent.postMessage(a,'*'))<${''}/script>`;
+      jsonModulesEnabled ? `b('${jsonModulesCheck}'.replace('x',b('{}','text/json')))` : 'false'}].map(x =>typeof x==='string'?import(x).then(x =>!!x,()=>false):x)).then(a=>parent.postMessage(['esms'].concat(a),'*'))<${''}/script>`;
 
       // Safari will call onload eagerly on head injection, but we don't want the Wechat
       // path to trigger before setting srcdoc, therefore we track the timing
@@ -603,88 +603,109 @@
     // edge doesnt execute sibling in order, so we fix this up by ensuring all previous executions are explicit dependencies
     let resolvedSource = edge && lastLoad ? `import '${lastLoad}';` : '';
 
-    if (!imports.length) {
-      resolvedSource += source;
-    }
-    else {
-      // once all deps have loaded we can inline the dependency resolution blobs
-      // and define this blob
-      let lastIndex = 0, depIndex = 0, dynamicImportEndStack = [];
-      function pushStringTo (originalIndex) {
-        while (dynamicImportEndStack[dynamicImportEndStack.length - 1] < originalIndex) {
-          const dynamicImportEnd = dynamicImportEndStack.pop();
-          resolvedSource += `${source.slice(lastIndex, dynamicImportEnd)}, ${urlJsString(load.r)}`;
-          lastIndex = dynamicImportEnd;
-        }
-        resolvedSource += source.slice(lastIndex, originalIndex);
-        lastIndex = originalIndex;
+    // once all deps have loaded we can inline the dependency resolution blobs
+    // and define this blob
+    let lastIndex = 0, depIndex = 0, dynamicImportEndStack = [];
+    function pushStringTo (originalIndex) {
+      while (dynamicImportEndStack[dynamicImportEndStack.length - 1] < originalIndex) {
+        const dynamicImportEnd = dynamicImportEndStack.pop();
+        resolvedSource += `${source.slice(lastIndex, dynamicImportEnd)}, ${urlJsString(load.r)}`;
+        lastIndex = dynamicImportEnd;
       }
-      for (const { s: start, ss: statementStart, se: statementEnd, d: dynamicImportIndex } of imports) {
-        // dependency source replacements
-        if (dynamicImportIndex === -1) {
-          let depLoad = load.d[depIndex++], blobUrl = depLoad.b, cycleShell = !blobUrl;
-          if (cycleShell) {
-            // circular shell creation
-            if (!(blobUrl = depLoad.s)) {
-              blobUrl = depLoad.s = createBlob(`export function u$_(m){${
-              depLoad.a[1].map(({ s, e }, i) => {
-                const q = depLoad.S[s] === '"' || depLoad.S[s] === "'";
-                return `e$_${i}=m${q ? `[` : '.'}${depLoad.S.slice(s, e)}${q ? `]` : ''}`;
-              }).join(',')
-            }}${
-              depLoad.a[1].length ? `let ${depLoad.a[1].map((_, i) => `e$_${i}`).join(',')};` : ''
-            }export {${
-              depLoad.a[1].map(({ s, e }, i) => `e$_${i} as ${depLoad.S.slice(s, e)}`).join(',')
-            }}\n//# sourceURL=${depLoad.r}?cycle`);
-            }
-          }
-
-          pushStringTo(start - 1);
-          resolvedSource += `/*${source.slice(start - 1, statementEnd)}*/${urlJsString(blobUrl)}`;
+      resolvedSource += source.slice(lastIndex, originalIndex);
+      lastIndex = originalIndex;
+    }
 
-          // circular shell execution
-          if (!cycleShell && depLoad.s) {
-            resolvedSource += `;import*as m$_${depIndex} from'${depLoad.b}';import{u$_ as u$_${depIndex}}from'${depLoad.s}';u$_${depIndex}(m$_${depIndex})`;
-            depLoad.s = undefined;
+    for (const { s: start, ss: statementStart, se: statementEnd, d: dynamicImportIndex } of imports) {
+      // dependency source replacements
+      if (dynamicImportIndex === -1) {
+        let depLoad = load.d[depIndex++], blobUrl = depLoad.b, cycleShell = !blobUrl;
+        if (cycleShell) {
+          // circular shell creation
+          if (!(blobUrl = depLoad.s)) {
+            blobUrl = depLoad.s = createBlob(`export function u$_(m){${
+            depLoad.a[1].map(({ s, e }, i) => {
+              const q = depLoad.S[s] === '"' || depLoad.S[s] === "'";
+              return `e$_${i}=m${q ? `[` : '.'}${depLoad.S.slice(s, e)}${q ? `]` : ''}`;
+            }).join(',')
+          }}${
+            depLoad.a[1].length ? `let ${depLoad.a[1].map((_, i) => `e$_${i}`).join(',')};` : ''
+          }export {${
+            depLoad.a[1].map(({ s, e }, i) => `e$_${i} as ${depLoad.S.slice(s, e)}`).join(',')
+          }}\n//# sourceURL=${depLoad.r}?cycle`);
           }
-          lastIndex = statementEnd;
-        }
-        // import.meta
-        else if (dynamicImportIndex === -2) {
-          load.m = { url: load.r, resolve: metaResolve };
-          metaHook(load.m, load.u);
-          pushStringTo(start);
-          resolvedSource += `importShim._r[${urlJsString(load.u)}].m`;
-          lastIndex = statementEnd;
         }
-        // dynamic import
-        else {
-          pushStringTo(statementStart + 6);
-          resolvedSource += `Shim(`;
-          dynamicImportEndStack.push(statementEnd - 1);
-          lastIndex = start;
+
+        pushStringTo(start - 1);
+        resolvedSource += `/*${source.slice(start - 1, statementEnd)}*/${urlJsString(blobUrl)}`;
+
+        // circular shell execution
+        if (!cycleShell && depLoad.s) {
+          resolvedSource += `;import*as m$_${depIndex} from'${depLoad.b}';import{u$_ as u$_${depIndex}}from'${depLoad.s}';u$_${depIndex}(m$_${depIndex})`;
+          depLoad.s = undefined;
         }
+        lastIndex = statementEnd;
+      }
+      // import.meta
+      else if (dynamicImportIndex === -2) {
+        load.m = { url: load.r, resolve: metaResolve };
+        metaHook(load.m, load.u);
+        pushStringTo(start);
+        resolvedSource += `importShim._r[${urlJsString(load.u)}].m`;
+        lastIndex = statementEnd;
+      }
+      // dynamic import
+      else {
+        pushStringTo(statementStart + 6);
+        resolvedSource += `Shim(`;
+        dynamicImportEndStack.push(statementEnd - 1);
+        lastIndex = start;
       }
+    }
 
-      // support progressive cycle binding updates (try statement avoids tdz errors)
-      if (load.s)
-        resolvedSource += `\n;import{u$_}from'${load.s}';try{u$_({${exports.filter(e => e.ln).map(({ s, e, ln }) => `${source.slice(s, e)}:${ln}`).join(',')}})}catch(_){};\n`;
+    // support progressive cycle binding updates (try statement avoids tdz errors)
+    if (load.s)
+      resolvedSource += `\n;import{u$_}from'${load.s}';try{u$_({${exports.filter(e => e.ln).map(({ s, e, ln }) => `${source.slice(s, e)}:${ln}`).join(',')}})}catch(_){};\n`;
+
+    function pushSourceURL (commentPrefix, commentStart) {
+      const urlStart = commentStart + commentPrefix.length;
+      const commentEnd = source.indexOf('\n', urlStart);
+      const urlEnd = commentEnd !== -1 ? commentEnd : source.length;
+      pushStringTo(urlStart);
+      resolvedSource += new URL(source.slice(urlStart, urlEnd), load.r).href;
+      lastIndex = urlEnd;
+    }
+
+    let sourceURLCommentStart = source.lastIndexOf(sourceURLCommentPrefix);
+    let sourceMapURLCommentStart = source.lastIndexOf(sourceMapURLCommentPrefix);
+
+    // ignore sourceMap comments before already spliced code
+    if (sourceURLCommentStart < lastIndex) sourceURLCommentStart = -1;
+    if (sourceMapURLCommentStart < lastIndex) sourceMapURLCommentStart = -1;
 
-      pushStringTo(source.length);
+    // sourceURL first / only
+    if (sourceURLCommentStart !== -1 && (sourceMapURLCommentStart === -1 || sourceMapURLCommentStart > sourceURLCommentStart)) {
+      pushSourceURL(sourceURLCommentPrefix, sourceURLCommentStart);
     }
+    // sourceMappingURL
+    if (sourceMapURLCommentStart !== -1) {
+      pushSourceURL(sourceMapURLCommentPrefix, sourceMapURLCommentStart);
+      // sourceURL last
+      if (sourceURLCommentStart !== -1 && (sourceURLCommentStart > sourceMapURLCommentStart))
+        pushSourceURL(sourceURLCommentPrefix, sourceURLCommentStart);
+    }
+
+    pushStringTo(source.length);
 
-    let hasSourceURL = false;
-    resolvedSource = resolvedSource.replace(sourceMapURLRegEx, (match, isMapping, url) => (hasSourceURL = !isMapping, match.replace(url, () => new URL(url, load.r))));
-    if (!hasSourceURL)
-      resolvedSource += '\n//# sourceURL=' + load.r;
+    if (sourceURLCommentStart === -1)
+      resolvedSource += sourceURLCommentPrefix + load.r;
 
     load.b = lastLoad = createBlob(resolvedSource);
     load.S = undefined;
   }
 
-  // ; and // trailer support added for Ruby on Rails 7 source maps compatibility
-  // https://github.com/guybedford/es-module-shims/issues/228
-  const sourceMapURLRegEx = /\n\/\/# source(Mapping)?URL=([^\n]+)\s*((;|\/\/[^#][^\n]*)\s*)*$/;
+  const sourceURLCommentPrefix = '\n//# sourceURL=';
+  const sourceMapURLCommentPrefix = '\n//# sourceMappingURL=';
 
   const jsContentType = /^(text|application)\/(x-)?javascript(;|$)/;
   const jsonContentType = /^(text|application)\/json(;|$)/;

package/dist/es-module-shims.js

@@ -1,4 +1,4 @@
-/* ES Module Shims 1.7.1 */
+/* ES Module Shims 1.7.3 */
 (function () {
 
   const hasWindow = typeof window !== 'undefined';
@@ -333,13 +333,14 @@
       iframe.style.display = 'none';
       iframe.setAttribute('nonce', nonce);
       function cb ({ data }) {
-        // failed feature detection (security policy) -> revert to default assumptions
-        if (Array.isArray(data)) {
-          supportsImportMaps = data[0];
-          supportsImportMeta = data[1];
-          supportsCssAssertions = data[2];
-          supportsJsonAssertions = data[3];
+        const isFeatureDetectionMessage = Array.isArray(data) && data[0] === 'esms';
+        if (!isFeatureDetectionMessage) {
+          return;
         }
+        supportsImportMaps = data[1];
+        supportsImportMeta = data[2];
+        supportsCssAssertions = data[3];
+        supportsJsonAssertions = data[4];
         resolve();
         document.head.removeChild(iframe);
         window.removeEventListener('message', cb, false);
@@ -348,7 +349,7 @@
 
       const importMapTest = `<script nonce=${nonce || ''}>b=(s,type='text/javascript')=>URL.createObjectURL(new Blob([s],{type}));document.head.appendChild(Object.assign(document.createElement('script'),{type:'importmap',nonce:"${nonce}",innerText:\`{"imports":{"x":"\${b('')}"}}\`}));Promise.all([${
       supportsImportMaps ? 'true,true' : `'x',b('${importMetaCheck}')`}, ${cssModulesEnabled ? `b('${cssModulesCheck}'.replace('x',b('','text/css')))` : 'false'}, ${
-      jsonModulesEnabled ? `b('${jsonModulesCheck}'.replace('x',b('{}','text/json')))` : 'false'}].map(x =>typeof x==='string'?import(x).then(x =>!!x,()=>false):x)).then(a=>parent.postMessage(a,'*'))<${''}/script>`;
+      jsonModulesEnabled ? `b('${jsonModulesCheck}'.replace('x',b('{}','text/json')))` : 'false'}].map(x =>typeof x==='string'?import(x).then(x =>!!x,()=>false):x)).then(a=>parent.postMessage(['esms'].concat(a),'*'))<${''}/script>`;
 
       // Safari will call onload eagerly on head injection, but we don't want the Wechat
       // path to trigger before setting srcdoc, therefore we track the timing
@@ -595,88 +596,109 @@
     // edge doesnt execute sibling in order, so we fix this up by ensuring all previous executions are explicit dependencies
     let resolvedSource = edge && lastLoad ? `import '${lastLoad}';` : '';
 
-    if (!imports.length) {
-      resolvedSource += source;
-    }
-    else {
-      // once all deps have loaded we can inline the dependency resolution blobs
-      // and define this blob
-      let lastIndex = 0, depIndex = 0, dynamicImportEndStack = [];
-      function pushStringTo (originalIndex) {
-        while (dynamicImportEndStack[dynamicImportEndStack.length - 1] < originalIndex) {
-          const dynamicImportEnd = dynamicImportEndStack.pop();
-          resolvedSource += `${source.slice(lastIndex, dynamicImportEnd)}, ${urlJsString(load.r)}`;
-          lastIndex = dynamicImportEnd;
-        }
-        resolvedSource += source.slice(lastIndex, originalIndex);
-        lastIndex = originalIndex;
+    // once all deps have loaded we can inline the dependency resolution blobs
+    // and define this blob
+    let lastIndex = 0, depIndex = 0, dynamicImportEndStack = [];
+    function pushStringTo (originalIndex) {
+      while (dynamicImportEndStack[dynamicImportEndStack.length - 1] < originalIndex) {
+        const dynamicImportEnd = dynamicImportEndStack.pop();
+        resolvedSource += `${source.slice(lastIndex, dynamicImportEnd)}, ${urlJsString(load.r)}`;
+        lastIndex = dynamicImportEnd;
       }
-      for (const { s: start, ss: statementStart, se: statementEnd, d: dynamicImportIndex } of imports) {
-        // dependency source replacements
-        if (dynamicImportIndex === -1) {
-          let depLoad = load.d[depIndex++], blobUrl = depLoad.b, cycleShell = !blobUrl;
-          if (cycleShell) {
-            // circular shell creation
-            if (!(blobUrl = depLoad.s)) {
-              blobUrl = depLoad.s = createBlob(`export function u$_(m){${
-              depLoad.a[1].map(({ s, e }, i) => {
-                const q = depLoad.S[s] === '"' || depLoad.S[s] === "'";
-                return `e$_${i}=m${q ? `[` : '.'}${depLoad.S.slice(s, e)}${q ? `]` : ''}`;
-              }).join(',')
-            }}${
-              depLoad.a[1].length ? `let ${depLoad.a[1].map((_, i) => `e$_${i}`).join(',')};` : ''
-            }export {${
-              depLoad.a[1].map(({ s, e }, i) => `e$_${i} as ${depLoad.S.slice(s, e)}`).join(',')
-            }}\n//# sourceURL=${depLoad.r}?cycle`);
-            }
-          }
-
-          pushStringTo(start - 1);
-          resolvedSource += `/*${source.slice(start - 1, statementEnd)}*/${urlJsString(blobUrl)}`;
+      resolvedSource += source.slice(lastIndex, originalIndex);
+      lastIndex = originalIndex;
+    }
 
-          // circular shell execution
-          if (!cycleShell && depLoad.s) {
-            resolvedSource += `;import*as m$_${depIndex} from'${depLoad.b}';import{u$_ as u$_${depIndex}}from'${depLoad.s}';u$_${depIndex}(m$_${depIndex})`;
-            depLoad.s = undefined;
+    for (const { s: start, ss: statementStart, se: statementEnd, d: dynamicImportIndex } of imports) {
+      // dependency source replacements
+      if (dynamicImportIndex === -1) {
+        let depLoad = load.d[depIndex++], blobUrl = depLoad.b, cycleShell = !blobUrl;
+        if (cycleShell) {
+          // circular shell creation
+          if (!(blobUrl = depLoad.s)) {
+            blobUrl = depLoad.s = createBlob(`export function u$_(m){${
+            depLoad.a[1].map(({ s, e }, i) => {
+              const q = depLoad.S[s] === '"' || depLoad.S[s] === "'";
+              return `e$_${i}=m${q ? `[` : '.'}${depLoad.S.slice(s, e)}${q ? `]` : ''}`;
+            }).join(',')
+          }}${
+            depLoad.a[1].length ? `let ${depLoad.a[1].map((_, i) => `e$_${i}`).join(',')};` : ''
+          }export {${
+            depLoad.a[1].map(({ s, e }, i) => `e$_${i} as ${depLoad.S.slice(s, e)}`).join(',')
+          }}\n//# sourceURL=${depLoad.r}?cycle`);
           }
-          lastIndex = statementEnd;
-        }
-        // import.meta
-        else if (dynamicImportIndex === -2) {
-          load.m = { url: load.r, resolve: metaResolve };
-          metaHook(load.m, load.u);
-          pushStringTo(start);
-          resolvedSource += `importShim._r[${urlJsString(load.u)}].m`;
-          lastIndex = statementEnd;
         }
-        // dynamic import
-        else {
-          pushStringTo(statementStart + 6);
-          resolvedSource += `Shim(`;
-          dynamicImportEndStack.push(statementEnd - 1);
-          lastIndex = start;
+
+        pushStringTo(start - 1);
+        resolvedSource += `/*${source.slice(start - 1, statementEnd)}*/${urlJsString(blobUrl)}`;
+
+        // circular shell execution
+        if (!cycleShell && depLoad.s) {
+          resolvedSource += `;import*as m$_${depIndex} from'${depLoad.b}';import{u$_ as u$_${depIndex}}from'${depLoad.s}';u$_${depIndex}(m$_${depIndex})`;
+          depLoad.s = undefined;
         }
+        lastIndex = statementEnd;
+      }
+      // import.meta
+      else if (dynamicImportIndex === -2) {
+        load.m = { url: load.r, resolve: metaResolve };
+        metaHook(load.m, load.u);
+        pushStringTo(start);
+        resolvedSource += `importShim._r[${urlJsString(load.u)}].m`;
+        lastIndex = statementEnd;
+      }
+      // dynamic import
+      else {
+        pushStringTo(statementStart + 6);
+        resolvedSource += `Shim(`;
+        dynamicImportEndStack.push(statementEnd - 1);
+        lastIndex = start;
       }
+    }
 
-      // support progressive cycle binding updates (try statement avoids tdz errors)
-      if (load.s)
-        resolvedSource += `\n;import{u$_}from'${load.s}';try{u$_({${exports.filter(e => e.ln).map(({ s, e, ln }) => `${source.slice(s, e)}:${ln}`).join(',')}})}catch(_){};\n`;
+    // support progressive cycle binding updates (try statement avoids tdz errors)
+    if (load.s)
+      resolvedSource += `\n;import{u$_}from'${load.s}';try{u$_({${exports.filter(e => e.ln).map(({ s, e, ln }) => `${source.slice(s, e)}:${ln}`).join(',')}})}catch(_){};\n`;
+
+    function pushSourceURL (commentPrefix, commentStart) {
+      const urlStart = commentStart + commentPrefix.length;
+      const commentEnd = source.indexOf('\n', urlStart);
+      const urlEnd = commentEnd !== -1 ? commentEnd : source.length;
+      pushStringTo(urlStart);
+      resolvedSource += new URL(source.slice(urlStart, urlEnd), load.r).href;
+      lastIndex = urlEnd;
+    }
+
+    let sourceURLCommentStart = source.lastIndexOf(sourceURLCommentPrefix);
+    let sourceMapURLCommentStart = source.lastIndexOf(sourceMapURLCommentPrefix);
+
+    // ignore sourceMap comments before already spliced code
+    if (sourceURLCommentStart < lastIndex) sourceURLCommentStart = -1;
+    if (sourceMapURLCommentStart < lastIndex) sourceMapURLCommentStart = -1;
 
-      pushStringTo(source.length);
+    // sourceURL first / only
+    if (sourceURLCommentStart !== -1 && (sourceMapURLCommentStart === -1 || sourceMapURLCommentStart > sourceURLCommentStart)) {
+      pushSourceURL(sourceURLCommentPrefix, sourceURLCommentStart);
     }
+    // sourceMappingURL
+    if (sourceMapURLCommentStart !== -1) {
+      pushSourceURL(sourceMapURLCommentPrefix, sourceMapURLCommentStart);
+      // sourceURL last
+      if (sourceURLCommentStart !== -1 && (sourceURLCommentStart > sourceMapURLCommentStart))
+        pushSourceURL(sourceURLCommentPrefix, sourceURLCommentStart);
+    }
+
+    pushStringTo(source.length);
 
-    let hasSourceURL = false;
-    resolvedSource = resolvedSource.replace(sourceMapURLRegEx, (match, isMapping, url) => (hasSourceURL = !isMapping, match.replace(url, () => new URL(url, load.r))));
-    if (!hasSourceURL)
-      resolvedSource += '\n//# sourceURL=' + load.r;
+    if (sourceURLCommentStart === -1)
+      resolvedSource += sourceURLCommentPrefix + load.r;
 
     load.b = lastLoad = createBlob(resolvedSource);
     load.S = undefined;
   }
 
-  // ; and // trailer support added for Ruby on Rails 7 source maps compatibility
-  // https://github.com/guybedford/es-module-shims/issues/228
-  const sourceMapURLRegEx = /\n\/\/# source(Mapping)?URL=([^\n]+)\s*((;|\/\/[^#][^\n]*)\s*)*$/;
+  const sourceURLCommentPrefix = '\n//# sourceURL=';
+  const sourceMapURLCommentPrefix = '\n//# sourceMappingURL=';
 
   const jsContentType = /^(text|application)\/(x-)?javascript(;|$)/;
   const jsonContentType = /^(text|application)\/json(;|$)/;