fix(toggle): use -1 as index for toKey() with toggled item

[Minhir]

Summary

It's strange to call toKey for an item with different indexes. We can provide -1 instead and call it only once.

Related issue, if any:

For any code change,

• Related documentation has been updated, if needed
• Related tests have been added or updated, if needed
• Related benchmarks have been added or updated, if needed

Does this PR introduce a breaking change?

No

Bundle impact

Status	File	Size	Difference (%)
M	src/array/toggle.ts	257 1	+22 (+9%)

Footnotes

1. Function size includes the import dependencies of the function. ↩

[radashi-bot]

Benchmark Results

Name	Current	Baseline	Change
toggle ▶︎ with item that does not exist	3,444,063.7 ops/sec ±0.79%	3,853,799.84 ops/sec ±0.68% 🔗	🐢 -10.63%
toggle ▶︎ with item that does exist	3,074,539.65 ops/sec ±0.26%	3,784,823.03 ops/sec ±0.23% 🔗	🐢 -18.77%
toggle ▶︎ with item that does exist and custom matcher	3,134,063.15 ops/sec ±0.57%	2,915,861.43 ops/sec ±3.31% 🔗	🚀 +7.48%
toggle ▶︎ with item that does not exist and custom matcher	2,929,082.75 ops/sec ±1.19%	3,177,778.3 ops/sec ±3.2% 🔗	🐢 -7.83%
toggle ▶︎ with strategy prepend	3,705,687.33 ops/sec ±0.2%	3,927,617.13 ops/sec ±0.18% 🔗	🐢 -5.65%

Generally speaking, any regression ≥20% should be investigated if it wasn't to be expected.

[github-actions]

A new beta version 12.2.0-beta.6b76988 has been published to NPM. 🚀

To install:

pnpm add [email protected]

The radashi@beta tag also includes this PR.