ryanhefner / react-fathom

91%

main: 93%

push

circleci

Implement codebase review recommendations

- Extract shared buildTrackingUrl utility to src/utils.ts, eliminating
  duplicate URL-building logic across React Router, Gatsby, TanStack
  Router, and Next.js adapters
- Make debug event counter instance-scoped via useRef instead of
  module-level variable to prevent ID collisions across providers
- Replace unconditional console.log on provider mount with production
  debug mode warning
- Add siteId validation: reject empty strings with a specific warning
- Add console.warn when debug mode is accidentally enabled in production
- Add comprehensive tests for all new behaviors (shared URL utility,
  siteId validation, production debug warning, counter isolation)

All 310 tests pass.

https://claude.ai/code/session_01NURi27sBQFDWM585Fna9yK

343 of 398 branches covered (86.18%)

Branch coverage included in aggregate %.

26 of 26 new or added lines in 6 files covered. (100.0%)

557 of 596 relevant lines covered (93.46%)

26.91 hits per line