@jsenv/sourcemap 1.2.4 → 1.2.6

package/CHANGELOG.md

@@ -0,0 +1,3 @@
+# 1.2.3
+
+- Improve code quality

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/sourcemap",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -19,7 +19,7 @@
     "test": "node --conditions=development ./scripts/test.mjs"
   },
   "dependencies": {
-    "@jsenv/urls": "2.2.1",
+    "@jsenv/urls": "2.2.3",
     "magic-string": "0.30.5",
     "source-map-js": "1.0.2"
   }

package/scripts/test.mjs

@@ -9,8 +9,5 @@ await executeTestPlan({
       },
     },
   },
-  failFast: process.argv.includes("--workspace"),
-  logShortForCompletedExecutions: true,
-  logMergeForCompletedExecutions: process.argv.includes("--workspace"),
-  githubCheckEnabled: false,
+  githubCheck: false,
 });

package/src/error_stack_remap/detailed_message.js

@@ -1,4 +1,4 @@
-// duplicated from @jsenv/log to avoid the dependency
+// duplicated from @jsenv/humanize to avoid the dependency
 export const createDetailedMessage = (message, details = {}) => {
   let string = `${message}`;
 

package/src/magic_source.js

@@ -4,39 +4,33 @@ export const createMagicSource = (content) => {
   if (content === undefined) {
     throw new Error("content missing");
   }
-  const mutations = [];
+  const magicString = new MagicString(content);
+  let touched = false;
+
   return {
     prepend: (string) => {
-      mutations.push((magicString) => {
-        magicString.prepend(string);
-      });
+      touched = true;
+      magicString.prepend(string);
     },
     append: (string) => {
-      mutations.push((magicString) => {
-        magicString.append(string);
-      });
+      touched = true;
+      magicString.append(string);
     },
     replace: ({ start, end, replacement }) => {
-      mutations.push((magicString) => {
-        magicString.overwrite(start, end, replacement);
-      });
+      touched = true;
+      magicString.overwrite(start, end, replacement);
     },
     remove: ({ start, end }) => {
-      mutations.push((magicString) => {
-        magicString.remove(start, end);
-      });
+      touched = true;
+      magicString.remove(start, end);
     },
     toContentAndSourcemap: ({ source } = {}) => {
-      if (mutations.length === 0) {
+      if (!touched) {
         return {
           content,
           sourcemap: null,
         };
       }
-      const magicString = new MagicString(content);
-      mutations.forEach((mutation) => {
-        mutation(magicString);
-      });
       const code = magicString.toString();
       const map = magicString.generateMap({
         hires: true,