Skip to content

perf: remove redundant seen-Set deduplication in merge_entry_ranges#95

Merged
bartveneman merged 2 commits intomainfrom
perf/rm-redundant-checks
Mar 17, 2026
Merged

perf: remove redundant seen-Set deduplication in merge_entry_ranges#95
bartveneman merged 2 commits intomainfrom
perf/rm-redundant-checks

Conversation

@bartveneman
Copy link
Member

merge_ranges already eliminates exact duplicates via sort-and-merge, so the seen Set and its string-key allocations were unnecessary.

@bartveneman bartveneman merged commit 4c9ceaf into main Mar 17, 2026
6 of 7 checks passed
@bartveneman bartveneman deleted the perf/rm-redundant-checks branch March 17, 2026 21:08
@codecov-commenter
Copy link

codecov-commenter commented Mar 17, 2026

Bundle Report

Changes will decrease total bundle size by 163 bytes (-0.63%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
index.js-esm 11.8kB -163 bytes (-1.36%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: index.js-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.js -163 bytes 10.45kB -1.54%

Files in index.js:

  • ./src/lib/decuplicate.ts → Total Size: 937 bytes

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.52%. Comparing base (12bd9d8) to head (30df4f0).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   96.54%   96.52%   -0.03%     
==========================================
  Files          14       14              
  Lines        1014     1008       -6     
  Branches      135      133       -2     
==========================================
- Hits          979      973       -6     
  Misses         34       34              
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants