[Antoliny0919]

Change

Replace Array.from with toArray.
Also, if the value is already an array, it would be more efficient to use it as is rather than applying Array.from.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
es-toolkit	Ready	Preview, Comment	Jul 26, 2026 7:15am

[Antoliny0919]

Suggested change

	for (let i = 1; i < arrays.length; i++) {
	for (let i = 1; i < arrays.length && result.length > 0; i++) {

What do you think about adding a result.length > 0 check?
Once there's nothing left to compare, we can skip the rest of the loop.
This is more efficient for that edge case, while in the general case it adds only a single extra check, which doesn't seem like it would have any meaningful performance impact.

[raon0211]

We would like to keep the implementation simple and straightforward if possible, so I would like to stick with the current approach.

[raon0211]

Could you add a benchmark to check whether this doesn't hurt the original performance? You can run it with a command like yarn bench intersection.bench.ts

[Antoliny0919]

Could you add a benchmark to check whether this doesn't hurt the original performance? You can run it with a command like yarn bench intersection.bench.ts

Before

After