Make file size more compact

Author: caub

src/lib/helpers.ts

@@ -8,22 +8,21 @@ export function getFileContent(file: Buffer | string): string {
   return buffer.toString();
 }
 
-const BYTE_SIZES = ['B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB'];
+const BYTE_SIZES = ['B', 'K', 'M', 'G', 'TB', 'PB', 'EB', 'ZB', 'YB'];
 // Bytes
 const SIZE_BASE = 1024;
 
 /**
  * Format number of bytes as string
- * Source @see https://stackoverflow.com/a/18650828/388951
  */
-export function formatBytes(bytes: number, decimals = 2): string {
-  if (bytes === 0) return `0 ${BYTE_SIZES[0]}`;
+export function formatBytes(bytes: number, maximumSignificantDigits = 3): string {
+  if (bytes === 0) return `0${BYTE_SIZES[0]}`;
 
   const exponent = Math.floor(Math.log(bytes) / Math.log(SIZE_BASE));
   const value = bytes / Math.pow(SIZE_BASE, exponent);
 
   // `parseFloat` removes trailing zero
-  return `${parseFloat(value.toFixed(decimals))} ${BYTE_SIZES[exponent]}`;
+  return `${new Intl.NumberFormat('default', { maximumSignificantDigits }).format(value)}${BYTE_SIZES[exponent]}`;
 }
 
 export function formatPercent(value: number, total: number, fractionDigits?: number): string {

tests/unit/helpers.test.ts

@@ -14,10 +14,10 @@ import type { MappingRange } from '../../src/lib/types';
 describe('helpers', () => {
   describe('formatBytes', () => {
     const tests: { bytes: number; expected: string }[] = [
-      { bytes: 0, expected: '0 B' },
-      { bytes: 1024, expected: '1 KB' },
-      { bytes: 27291, expected: '26.65 KB' },
-      { bytes: 6232294, expected: '5.94 MB' },
+      { bytes: 0, expected: '0B' },
+      { bytes: 1024, expected: '1K' },
+      { bytes: 27291, expected: '26.7K' },
+      { bytes: 6232294, expected: '5.94M' },
     ];
 
     tests.forEach(({ bytes, expected }) => {
@@ -81,43 +81,43 @@ describe('helpers', () => {
       position: [number, number];
       expected: boolean;
     }[] = [
-      {
-        name: 'should handle CR LF',
-        string: 'function calculate(a, b) {\r\n  return a + b;\r\n}\r\n',
-        position: [2, 15],
-        expected: true,
-      },
-      {
-        name: 'should handle LF',
-        string: 'function calculate(a, b) {\n  return a + b;\n}\n',
-        position: [2, 15],
-        expected: true,
-      },
-      {
-        name: 'should handle string w/o EOLs',
-        string: 'Math.random().toString(36).substring(2);',
-        position: [1, 40],
-        expected: false,
-      },
-      {
-        name: 'should handle out of range line',
-        string: 'const result = await process();\nconsole.log(result)',
-        position: [3, 10],
-        expected: false,
-      },
-      {
-        name: 'should handle out of range column',
-        string: 'const result = await process();\nconsole.log(result)\n',
-        position: [2, 23],
-        expected: false,
-      },
-      {
-        name: 'should handle string with unicode',
-        string: 'A 🦊 went a hunting in the 🌲.\r\nThe 🦊 wasted no ⏳ in talking.\r\n',
-        position: [2, 30],
-        expected: true,
-      },
-    ];
+        {
+          name: 'should handle CR LF',
+          string: 'function calculate(a, b) {\r\n  return a + b;\r\n}\r\n',
+          position: [2, 15],
+          expected: true,
+        },
+        {
+          name: 'should handle LF',
+          string: 'function calculate(a, b) {\n  return a + b;\n}\n',
+          position: [2, 15],
+          expected: true,
+        },
+        {
+          name: 'should handle string w/o EOLs',
+          string: 'Math.random().toString(36).substring(2);',
+          position: [1, 40],
+          expected: false,
+        },
+        {
+          name: 'should handle out of range line',
+          string: 'const result = await process();\nconsole.log(result)',
+          position: [3, 10],
+          expected: false,
+        },
+        {
+          name: 'should handle out of range column',
+          string: 'const result = await process();\nconsole.log(result)\n',
+          position: [2, 23],
+          expected: false,
+        },
+        {
+          name: 'should handle string with unicode',
+          string: 'A 🦊 went a hunting in the 🌲.\r\nThe 🦊 wasted no ⏳ in talking.\r\n',
+          position: [2, 30],
+          expected: true,
+        },
+      ];
 
     tests.forEach(({ name, string, position, expected }) => {
       it(name, () => {