[yowainwright]

Proposed Changes

Speeds up multi-element truncation by batching DOM writes and height reads for independent layouts, while preserving Shave’s API and using sequential processing for coupled layouts. Keeps zero runtime dependencies.

---

Read about referenced issues here. Replace words with this Pull Request's context.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	happy-dom@​20.14.5 ⏵ 21.0.4		+23	+1	-1	-30
	rolldown@​1.2.8 ⏵ 1.2.16	+5		+1	+2
	oxlint-plugin-legibility@​0.0.5
	pastoralist@​1.13.2 ⏵ 1.14.0				+2
	@​vitest/​coverage-v8@​5.0.0 ⏵ 5.0.5	+1		+1
	vitest@​5.0.0 ⏵ 5.0.5			+1
	@​types/​node@​26.5.1 ⏵ 26.6.6	+1		+1
	vite@​8.3.0 ⏵ 8.3.5	+1	+22	+1
	oxfmt@​0.67.0 ⏵ 0.75.0	+1		+1	+2
	es-check@​9.7.1 ⏵ 9.8.1	+1			-1
	oxlint@​1.82.0 ⏵ 1.90.0			+1
	codependence@​1.0.15 ⏵ 1.0.16	+2
	release-it@​21.0.2 ⏵ 21.1.2	+1			+1
	@​playwright/​test@​1.63.0 ⏵ 1.64.1

View full report

[yowainwright]

@greptile

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0614b5a) to head (8ddc169).

Additional details and impacted files

@@             Coverage Diff             @@
##             main      #567      +/-   ##
===========================================
+ Coverage   98.93%   100.00%   +1.06%     
===========================================
  Files           2         2              
  Lines          94       126      +32     
  Branches       33        43      +10     
===========================================
+ Hits           93       126      +33     
+ Partials        1         0       -1     

☔ View full report in Codecov by Harness.
📢 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The latest changes appear safe to merge and address the earlier float-wrapping issue.

Summary

Shave batches text changes and height reads for independent targets, while keeping sequential work for coupled layouts.

• Shave batches truncation work for independent elements.
• The package builds Shave with Rolldown in its existing output formats.
• CI and test setup use the updated project toolchain.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Resolve target elements] --> B{Targets pass batching checks?}
  B -->|Yes| C[Create one search per target]
  B -->|No| D[Create one sequential search]
  C --> E[Write candidates]
  D --> E
  E --> F[Read target heights]
  F --> G{More candidates needed?}
  G -->|Yes| E
  G -->|No| H[Hide remaining text and restore styles]

Loading

_{Reviews (2) · Last reviewed commit: "chore: code cov" · Reviewed by Greptile}

[yowainwright]

@greptile