@jsenv/core 41.2.10 → 41.2.12

package/dist/build/build.js

@@ -2499,7 +2499,7 @@ const createUrlInfoTransformer = ({
       urlInfo.isInline ||
       (sources &&
         sources.some((source) => !source || !source.startsWith("file:")));
-    if (sources && sources.length > 1) {
+    if (sources && sources.length > 0) {
       sourcemap.sources = sources.map(
         (source) => new URL(source, urlInfo.originalUrl).href,
       );
@@ -2508,6 +2508,11 @@ const createUrlInfoTransformer = ({
       }
       return sourcemap;
     }
+    // No source info at all (e.g. an empty sourcemap): only then fall back
+    // to describing this url as its own source. A single source is real
+    // information (it can point at a different file entirely, e.g. a
+    // bundled chunk whose only literal content comes from another module)
+    // and must not be discarded in favor of this assumption.
     sourcemap.sources = [urlInfo.originalUrl];
     sourcemap.sourcesContent = [urlInfo.originalContent];
     if (!wantSourcesContent) {

package/dist/build/jsenv_core_packages.js

@@ -10435,9 +10435,14 @@ const renderTable = (
         const cell = row[x];
         const { value, format } = cell;
 
-        if (format !== "size" && isFinite(value) && value !== "") {
-          if (value % 1 === 0) {
-            const { integer } = tokenizeInteger(Math.abs(value));
+        const numericValue =
+          typeof value === "string"
+            ? Number(value.replace(/_/g, ""))
+            : value;
+        if (format !== "size" && isFinite(numericValue) && value !== "") {
+          const isNegative = numericValue < 0;
+          if (numericValue % 1 === 0) {
+            const { integer } = tokenizeInteger(Math.abs(numericValue));
             const integerFormatted = groupDigits(integer);
             const integerWidth = measureTextWidth(integerFormatted);
             const largestIntegerInColumn =
@@ -10456,11 +10461,18 @@ const renderTable = (
               if (floatWidth) {
                 integerText += " ".repeat(floatWidth);
               }
+              if (isNegative) {
+                const firstDigitIdx = integerText.search(/\S/);
+                integerText =
+                  firstDigitIdx > 0
+                    ? `${integerText.slice(0, firstDigitIdx - 1)}-${integerText.slice(firstDigitIdx)}`
+                    : `-${integerText}`;
+              }
               cell.updateValue(integerText);
             });
           } else {
             const { integer, decimalSeparator, decimal } = tokenizeFloat(
-              Math.abs(value),
+              Math.abs(numericValue),
             );
             const integerFormatted = groupDigits(integer);
             const integerWidth = measureTextWidth(integerFormatted);
@@ -10484,6 +10496,13 @@ const renderTable = (
                 const padding = floatColumnWidth - floatWidth;
                 floatText += " ".repeat(padding - 1);
               }
+              if (isNegative) {
+                const firstDigitIdx = floatText.search(/\S/);
+                floatText =
+                  firstDigitIdx > 0
+                    ? `${floatText.slice(0, firstDigitIdx - 1)}-${floatText.slice(firstDigitIdx)}`
+                    : `-${floatText}`;
+              }
               cell.updateValue(floatText);
             });
           }

package/dist/start_dev_server/start_dev_server.js

@@ -8322,7 +8322,7 @@ const createUrlInfoTransformer = ({
       urlInfo.isInline ||
       (sources &&
         sources.some((source) => !source || !source.startsWith("file:")));
-    if (sources && sources.length > 1) {
+    if (sources && sources.length > 0) {
       sourcemap.sources = sources.map(
         (source) => new URL(source, urlInfo.originalUrl).href,
       );
@@ -8331,6 +8331,11 @@ const createUrlInfoTransformer = ({
       }
       return sourcemap;
     }
+    // No source info at all (e.g. an empty sourcemap): only then fall back
+    // to describing this url as its own source. A single source is real
+    // information (it can point at a different file entirely, e.g. a
+    // bundled chunk whose only literal content comes from another module)
+    // and must not be discarded in favor of this assumption.
     sourcemap.sources = [urlInfo.originalUrl];
     sourcemap.sourcesContent = [urlInfo.originalContent];
     if (!wantSourcesContent) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.2.10",
+  "version": "41.2.12",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {
@@ -73,13 +73,13 @@
   },
   "dependencies": {
     "@jsenv/ast": "6.8.3",
-    "@jsenv/js-module-fallback": "1.4.33",
-    "@jsenv/plugin-bundling": "2.10.13",
+    "@jsenv/js-module-fallback": "1.4.34",
+    "@jsenv/plugin-bundling": "2.10.14",
     "@jsenv/plugin-minification": "1.7.4",
-    "@jsenv/plugin-supervisor": "1.8.4",
-    "@jsenv/plugin-transpilation": "1.5.75",
-    "@jsenv/server": "17.3.0",
-    "@jsenv/sourcemap": "1.3.19",
+    "@jsenv/plugin-supervisor": "1.8.5",
+    "@jsenv/plugin-transpilation": "1.5.76",
+    "@jsenv/server": "17.3.1",
+    "@jsenv/sourcemap": "1.4.0",
     "react-table": "7.8.0"
   },
   "devDependencies": {

package/src/kitchen/url_graph/url_info_transformations.js

@@ -59,7 +59,7 @@ export const createUrlInfoTransformer = ({
       urlInfo.isInline ||
       (sources &&
         sources.some((source) => !source || !source.startsWith("file:")));
-    if (sources && sources.length > 1) {
+    if (sources && sources.length > 0) {
       sourcemap.sources = sources.map(
         (source) => new URL(source, urlInfo.originalUrl).href,
       );
@@ -68,6 +68,11 @@ export const createUrlInfoTransformer = ({
       }
       return sourcemap;
     }
+    // No source info at all (e.g. an empty sourcemap): only then fall back
+    // to describing this url as its own source. A single source is real
+    // information (it can point at a different file entirely, e.g. a
+    // bundled chunk whose only literal content comes from another module)
+    // and must not be discarded in favor of this assumption.
     sourcemap.sources = [urlInfo.originalUrl];
     sourcemap.sourcesContent = [urlInfo.originalContent];
     if (!wantSourcesContent) {