refactor(linter): improve docs and simplify code of eslint/no-duplicate-imports

[ulrichstark]

No description provided.

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #11320 will not alter performance

_{Comparing ulrichstark:refactor-eslint-no-duplicate-imports (2dd4280) with main (9798ef1)}

Summary

✅ 38 untouched benchmarks

[ulrichstark]

@camc314 I was a bit surprised to see these improvements in the benchmark. It looks like the rule is reporting less diagnostics after my change.

I ran oxlint before and after my change on the cal.com repo and errors of eslint/no-duplicate-imports indeed decreased from 1766 to 1227. It seems like reporting for duplicated spans is now more correctly skipped.

For example, the error for the following second line was reported twice in packages/features/eventtypes/components/CheckedTeamSelect.tsx

import type { PriorityDialogCustomClassNames, WeightDialogCustomClassNames } from "./HostEditDialogs";
import { PriorityDialog, WeightDialog } from "./HostEditDialogs";

Now it's only reported once. Probably because the previous same_statement check was slightly wrong.

[camc314]

Now it's only reported once. Probably because the previous same_statement check was slightly wrong.

can you add a test for this please? (I guess i incorrectly assumeed that the existing snapshots should've caught this)