source-map - npm Package Compare versions

Comparing version 0.7.3 to 0.7.4

lib/read-wasm.js

@@ -1,18 +0,27 @@
-if (typeof fetch === "function") {
+/* Determine browser vs node environment by testing the default top level context. Solution courtesy of: https://stackoverflow.com/questions/17575790/environment-detection-node-js-or-browser */
+const isBrowserEnvironment = (function() {
+    // eslint-disable-next-line no-undef
+    return (typeof window !== "undefined") && (this === window);
+}).call();
+
+if (isBrowserEnvironment) {
   // Web version of reading a wasm file into an array buffer.
 
-  let mappingsWasmUrl = null;
+  let mappingsWasm = null;
 
   module.exports = function readWasm() {
-    if (typeof mappingsWasmUrl !== "string") {
-      throw new Error("You must provide the URL of lib/mappings.wasm by calling " +
-                      "SourceMapConsumer.initialize({ 'lib/mappings.wasm': ... }) " +
-                      "before using SourceMapConsumer");
+    if (typeof mappingsWasm === "string") {
+      return fetch(mappingsWasm)
+        .then(response => response.arrayBuffer());
     }
-
-    return fetch(mappingsWasmUrl)
-      .then(response => response.arrayBuffer());
+    if (mappingsWasm instanceof ArrayBuffer) {
+      return Promise.resolve(mappingsWasm);
+    }
+    throw new Error("You must provide the string URL or ArrayBuffer contents " +
+                    "of lib/mappings.wasm by calling " +
+                    "SourceMapConsumer.initialize({ 'lib/mappings.wasm': ... }) " +
+                    "before using SourceMapConsumer");
   };
 
-  module.exports.initialize = url => mappingsWasmUrl = url;
+  module.exports.initialize = input => mappingsWasm = input;
 } else {
@@ -19,0 +28,0 @@ // Node version of reading a wasm file into an array buffer.

lib/source-map-consumer.js

@@ -67,26 +67,9 @@ /* -*- Mode: js; js-indent-level: 2; -*- */
    */
-  static with(rawSourceMap, sourceMapUrl, f) {
-    // Note: The `acorn` version that `webpack` currently depends on doesn't
-    // support `async` functions, and the nodes that we support don't all have
-    // `.finally`. Therefore, this is written a bit more convolutedly than it
-    // should really be.
-
-    let consumer = null;
-    const promise = new SourceMapConsumer(rawSourceMap, sourceMapUrl);
-    return promise
-      .then(c => {
-        consumer = c;
-        return f(c);
-      })
-      .then(x => {
-        if (consumer) {
-          consumer.destroy();
-        }
-        return x;
-      }, e => {
-        if (consumer) {
-          consumer.destroy();
-        }
-        throw e;
-      });
+  static async with(rawSourceMap, sourceMapUrl, f) {
+    const consumer = await new SourceMapConsumer(rawSourceMap, sourceMapUrl);
+    try {
+      return await f(consumer);
+    } finally {
+      consumer.destroy();
+    }
   }
@@ -93,0 +76,0 @@

package.json

 {
   "name": "source-map",
   "description": "Generates and consumes source maps",
-  "version": "0.7.3",
+  "version": "0.7.4",
   "homepage": "https://github.com/mozilla/source-map",
@@ -78,3 +78,3 @@ "author": "Nick Fitzgerald <[e-mail]>",
   "devDependencies": {
-    "doctoc": "^0.15.0",
+    "doctoc": "^1.3.1",
     "eslint": "^4.19.1",
@@ -85,3 +85,4 @@ "live-server": "^1.2.0",
     "watch": "^1.0.2",
-    "webpack": "^3.10"
+    "webpack": "^4.9.1",
+    "webpack-cli": "^3.1"
   },
@@ -88,0 +89,0 @@ "nyc": {

README.md

@@ -211,3 +211,3 @@ # Source Map
 * `"lib/mappings.wasm"`: A `String` containing the URL of the
-  `lib/mappings.wasm` file.
+  `lib/mappings.wasm` file, or an `ArrayBuffer` with the contents of `lib/mappings.wasm`.
 
@@ -214,0 +214,0 @@ ```js

New alerts

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Minified code

Quality

This package contains minified code. This may be harmless in some cases where minified code is included in packaged libraries, however packages on npm should not minify code.

Found 1 instance in 1 package

Improved metrics

Number of medium supply chain risk alerts

3

decreased by-25%

Worsened metrics

Total package byte prevSize

226000

decreased by-27.49%

Dev dependency count

8

increased by14.29%

Number of package files

18

decreased by-5.26%

Lines of code

3157

decreased by-48.52%

Number of low quality alerts

2

increased by100%

No dependency changes