perf(linter): remove unnecessary clone of owned String in drain loop

[Boshen]

Summary

• After drain(), rule_name is already an owned String that can be moved directly
• Remove the unnecessary .clone() call

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR removes an unnecessary String::clone() when creating a DisabledRule::Single after draining disable_start_map, leveraging the fact that drain() yields owned String keys that can be moved directly. This is a small performance/cleanup change in the linter’s disable-directive interval construction logic.

Changes:

• Remove a redundant .clone() of rule_name in the post-drain() loop when building lone eslint-disable rule_name intervals.

---

You can also share your feedback on Copilot code review. Take the survey.

[codspeed-hq]

Merging this PR will not alter performance

✅ 4 untouched benchmarks
⏩ 52 skipped benchmarks¹

---

_{Comparing perf/linter-disable-directives-remove-clone (43df6bf) with main (e62524d)}

Footnotes

1. 52 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[Boshen]

Merge activity

• Mar 15, 1:50 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Mar 15, 1:50 AM UTC: Boshen added this pull request to the Graphite merge queue.
• Mar 15, 2:12 AM UTC: Merged by the Graphite merge queue.