[pylint] Exempt required imports from PLR0402

[danparizher]

Summary

Fixes #20380

The fix changes the includes_import function in I002 to recognize equivalent imports based on bound name and qualified name rather than exact syntax matching.

This allows the function to correctly identify that from concurrent import futures satisfies the requirement for import concurrent.futures as futures.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[dylwil3]

Hmmm... I'm a bit conflicted here about whether I002 or PLR0402 should take precedence to resolve the infinite loop. Generally it feels like the more "customized" or "manually entered" choice should take precedence. Since a user enters required imports exactly as they'd like them to appear, whereas PLR0402 is a "general" rule, I'm tempted to say that it's PLR0402 that should be modified here.

Would it be possible to have PLR0402 skip if it's triggering on a required import instead? I would also be open to adding either a warning or a debug message explaining why the rule is being skipped in this case. (Probably a debug message since we wouldn't have a good way to override the warning).

What do you think?

[ntBre]

I agree with Dylan, that seems closer to how we've handled the other I002 conflicts recently (#20327, #19413). Possibly we can even reuse the helper method you added in #19413 :)

[dylwil3]

Thanks! One nit