umputun / revdiff / 24231004838

88%

master: 90%

Build Type

Pull #87

github

Committed by umputun

Commit Message

fix: address Copilot review — hex validation and long-line cap

- parseHexRGB now validates each nibble explicitly; invalid hex chars
  return ok=false instead of silently parsing as black. matches the
  shiftLightness contract ("unparseable -> unchanged").
- changedTokenRanges caps line length at 500 bytes before running LCS
  to prevent O(m*n) memory blowup on pathological input (minified
  files, very long configs). lines above the cap still render with
  whole-line add/remove highlighting, just no word-level detail.

Copilot also flagged `for i := range n` as a compile error but that's
a false positive — Go 1.22+ supports integer ranges.

Pull Request Pull Request #87: feat: add intra-line word-diff highlighting

Coverage Stats

435 of 470 new or added lines in 7 files covered. (92.55%)

1 existing line in 1 file now uncovered.

5588 of 6346 relevant lines covered (88.06%)

397.38 hits per line