by vagrant · Mar 31 14:00 2026
Remove parseInt/toString from reduce accumulation
Delta-0.69%
Metricduration (ms)
Criterialower_is_better
LOC+1 -4
Kept Discarded Running best
Measurements
| Metric | Baseline | Candidate | Delta |
|---|
| duration (ms)How long the code takes to run. Less is better. | 10.040 ms (n=5) | 10.110 ms (n=5) | -0.69% REGRESSED |
| ops_per_sec (ops/sec)Operations completed per second. More is better. | 99.600 ops/sec | 98.960 ops/sec | -0.64% REGRESSED |
| p95 (ms)Measures p95 in ms. Less is better. | 95.000 ms | 95.000 ms | -0.00% |
Diff
11 fibs.push(value);
12 }
13
14- // Reduce with unnecessary string conversions
15- const total = fibs.reduce((acc, cur) => {
16- return parseInt(acc.toString(), 10) + parseInt(cur.toString(), 10);
17- }, 0);
14+ const total = fibs.reduce((acc, cur) => acc + cur, 0);
15
16 return total;
17 }
Environment
| Base commit | b327e2e8c0cb3dc29a75ec4f074f069663e3cc53 |
| Candidate commit | de86f1f2607db914f149e40800953f4dec837508 |
| Build | true |
| Tests | true |
| Metric ID | bbf2b1a |
| Arch | aarch64 |
| OS | Linux 6.8.0-86-generic |
| CPU | - |
| Agent | claude-code / claude-opus-4-6 |
| Files | src/index.ts |