False negative ``reimported`` when using an import alias

[ddangu525]

Current problem

I think it would be nice if Pylint could check if there is same import name from different packages.
For example, in the code below, import order is fine, but the 'Path' imports overlap, causing errors somewhere.

from pathlib import Path
from FastAPI import Path

Desired solution

overlapping imports found. Path

Additional context

No response

[Pierre-Sassoulas]

Thank you for your proposition, it makes sense.

[DudeNr33]

The given example already triggers the message reimported: Reimport 'Path' (imported line 8).
What is not covered yet are aliased imports, e.g.:

from pathlib import Path
from some_other_lib import CustomPath as Path

or

from pathlib import Path
import FastAPI.Path as Path

Reading the help for reimport, I am unsure if this is check actually is the right place for it:
"Used when a module is reimported multiple times."
The docstring for the method which handles the check also implies that this check is not intended for checking if the same name is imported multiple times, but if the import is necessary:

def _check_reimport(self, node, basename=None, level=None):
        """check if the import is necessary (i.e. not already done)"""

@Pierre-Sassoulas what is your opinion on this?
Is the fact that reimport triggers on the first example an actual false-positive and we should add a new check (was thinking about shadowed-import)?
Or should we adjust the help/docstring of reimport to state that it handles the case where the same names are imported multiple times?