Merge pull request #982 from lnewson/lnewson-diff-perf

fix: stat diff summary performance
steveukx · Mar 17, 2024 · 175ae77 · 175ae77
2 parents 994f2e3 + 8a3118d
commit 175ae77
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/six-cherries-grab.md b/.changeset/six-cherries-grab.md
@@ -0,0 +1,5 @@
+---
+"simple-git": patch
+---
+
+Fixed a performance issue when parsing stat diff summaries
diff --git a/simple-git/src/lib/parsers/parse-diff-summary.ts b/simple-git/src/lib/parsers/parse-diff-summary.ts
@@ -6,7 +6,7 @@ import { asNumber, LineParser, orVoid, parseStringResponse } from '../utils';
 
 const statParser = [
    new LineParser<DiffResult>(
-      /(.+)\s+\|\s+(\d+)(\s+[+\-]+)?$/,
+      /^(.+)\s+\|\s+(\d+)(\s+[+\-]+)?$/,
       (result, [file, changes, alterations = '']) => {
          result.files.push({
             file: file.trim(),
@@ -18,7 +18,7 @@ const statParser = [
       }
    ),
    new LineParser<DiffResult>(
-      /(.+) \|\s+Bin ([0-9.]+) -> ([0-9.]+) ([a-z]+)/,
+      /^(.+) \|\s+Bin ([0-9.]+) -> ([0-9.]+) ([a-z]+)/,
       (result, [file, before, after]) => {
          result.files.push({
             file: file.trim(),