apply suggestions

* fix handling of single digit range

Author: ktrz

src/extractRangeInfo.ts

@@ -3,7 +3,7 @@ export function extractRangeInfo(range: string | undefined): { prefix: string; v
         return undefined;
     }
 
-    const matches = range.match(/(?<prefix>(\+-|±)\s*)(?<value>\d.+)/);
+    const matches = range.match(/(?<prefix>(\+-|±)\s*)(?<value>\d.*)/);
 
     if (!matches || !matches.groups) {
         return undefined;

src/normalizeBenchmarkResult.ts

@@ -4,19 +4,19 @@ import { extractRangeInfo } from './extractRangeInfo';
 
 export function normalizeBenchmarkResult(
     prevBenchResult: BenchmarkResult | undefined | null,
-    currenBenchResult: BenchmarkResult,
+    currentBenchResult: BenchmarkResult,
 ): BenchmarkResult {
     if (!prevBenchResult) {
-        return currenBenchResult;
+        return currentBenchResult;
     }
 
     const prevUnit = prevBenchResult.unit;
-    const currentUnit = currenBenchResult.unit;
-    const currentRange = currenBenchResult.range;
+    const currentUnit = currentBenchResult.unit;
+    const currentRange = currentBenchResult.range;
     const currentRangeInfo = extractRangeInfo(currentRange);
 
-    const normalizedValue = normalizeValueByUnit(prevUnit, currentUnit, currenBenchResult.value);
-    const normalizedUnit = currenBenchResult.value !== normalizedValue ? prevUnit : currentUnit;
+    const normalizedValue = normalizeValueByUnit(prevUnit, currentUnit, currentBenchResult.value);
+    const normalizedUnit = currentBenchResult.value !== normalizedValue ? prevUnit : currentUnit;
     const normalizedRangeInfo = currentRangeInfo
         ? {
               prefix: currentRangeInfo.prefix,
@@ -25,7 +25,7 @@ export function normalizeBenchmarkResult(
         : undefined;
 
     return {
-        ...currenBenchResult,
+        ...currentBenchResult,
         value: normalizedValue,
         unit: normalizedUnit,
         range: normalizedRangeInfo ? `${normalizedRangeInfo.prefix}${normalizedRangeInfo.value}` : currentRange,

test/extractRangeInfo.spec.ts

@@ -17,6 +17,19 @@ describe('extractRangeInfo', () => {
         expect(extractRangeInfo('+- 20')).toEqual({ value: 20, prefix: '+- ' });
     });
 
+    it('should extract single-digit integer', () => {
+        expect(extractRangeInfo('±2')).toEqual({ value: 2, prefix: '±' });
+    });
+    it('should extract decimal and preserve prefix space', () => {
+        expect(extractRangeInfo('± 0.5')).toEqual({ value: 0.5, prefix: '± ' });
+    });
+    it('should extract scientific notation', () => {
+        expect(extractRangeInfo('+-1e-3')).toEqual({ value: 1e-3, prefix: '+-' });
+    });
+    it('should tolerate surrounding whitespace', () => {
+        expect(extractRangeInfo('  ±20  ')).toEqual({ value: 20, prefix: '±' });
+    });
+
     it('should NOT extract range with unknown prefix', () => {
         expect(extractRangeInfo('unknown prefix 20')).toBeUndefined();
     });