@oclif/table 0.2.0 → 0.2.2

package/lib/table.js

@@ -37,7 +37,7 @@ function determineConfiguredWidth(providedWidth, columns = process.stdout.column
  * This allows us to use the minimum width required to display the table if the configured width is too small.
  */
 function determineWidthToUse(columns, configuredWidth) {
-    const tableWidth = columns.map((c) => c.width).reduce((a, b) => a + b) + columns.length + 1;
+    const tableWidth = columns.map((c) => c.width).reduce((a, b) => a + b, 0) + columns.length + 1;
     return tableWidth < configuredWidth ? configuredWidth : tableWidth;
 }
 function determineTruncatePosition(overflow) {
@@ -60,6 +60,9 @@ export function formatTextWithMargins({ horizontalAlignment, overflow, padding,
     function calculateMargins(spaces) {
         let marginLeft;
         let marginRight;
+        if (spaces <= 0 || Number.isNaN(spaces)) {
+            return { marginLeft: 0, marginRight: 0 };
+        }
         if (horizontalAlignment === 'left') {
             marginLeft = padding;
             marginRight = spaces - marginLeft;
@@ -72,7 +75,11 @@ export function formatTextWithMargins({ horizontalAlignment, overflow, padding,
             marginRight = padding;
             marginLeft = spaces - marginRight;
         }
-        return { marginLeft, marginRight };
+        return {
+            // Ensure that the margin is never negative
+            marginLeft: Math.max(0, marginLeft),
+            marginRight: Math.max(0, marginRight),
+        };
     }
     // Some terminals don't play nicely with zero-width characters, so we replace them with spaces.
     // https://github.com/sindresorhus/terminal-link/issues/18
@@ -87,7 +94,11 @@ export function formatTextWithMargins({ horizontalAlignment, overflow, padding,
         };
     }
     if (overflow === 'wrap') {
-        const wrappedText = wrapAnsi(valueWithNoZeroWidthChars, spaceForText, { hard: true, trim: true, wordWrap: true });
+        const wrappedText = wrapAnsi(valueWithNoZeroWidthChars, spaceForText, {
+            hard: true,
+            trim: true,
+            wordWrap: true,
+        }).replace(/^\n/, ''); // remove leading newline (wrapAnsi adds it to emojis)
         const { marginLeft, marginRight } = calculateMargins(width - determineWidthOfWrappedText(stripAnsi(wrappedText)));
         const lines = wrappedText.split('\n').map((line, idx) => {
             const { marginLeft: lineSpecificLeftMargin, marginRight: lineSpecificRightMargin } = calculateMargins(width - stripAnsi(line).length);
@@ -306,7 +317,7 @@ const createStdout = () => {
         const stripped = stripAnsi(f);
         return stripped !== '' && stripped !== '\n';
     })
-        .at(-1);
+        .at(-1) ?? '';
     return stdout;
 };
 class Output {

package/lib/utils.js

@@ -67,7 +67,7 @@ export function getColumns(config, headings) {
         };
     });
     const numberOfBorders = widths.length + 1;
-    const calculateTableWidth = (widths) => widths.map((w) => w.width + w.padding * 2).reduce((a, b) => a + b) + numberOfBorders;
+    const calculateTableWidth = (widths) => widths.map((w) => w.width + w.padding * 2).reduce((a, b) => a + b, 0) + numberOfBorders;
     // If the table is too wide, reduce the width of the largest column as little as possible to fit the table.
     // At most, it will reduce the width to the length of the column's header plus padding.
     // If the table is still too wide, it will reduce the width of the next largest column and so on

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@oclif/table",
   "description": "Display table in terminal",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "author": "Salesforce",
   "bugs": "https://github.com/oclif/table/issues",
   "dependencies": {