Ensure unique aliases

[mackjmr]

Description

This PR ensures that unique aliases are used for modules with same suffix.

Link to tracking issue

Fixes #12162

Testing

Documentation

[songy23]

does this need a changelog?

[mackjmr]

@songy23 added in latest commit

[jackgopack4]

what is expected behavior in the following case:
import github.com/org/module
import github.com/org2/module
import github.com/org/module2

it seems we will still have collision in this case.

might need to implement a counter for the transformed names, too, if we go with this approach.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.85%. Comparing base (30b8d53) to head (ab9cd40).
Report is 90 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #12201   +/-   ##
=======================================
  Coverage   91.84%   91.85%           
=======================================
  Files         465      465           
  Lines       25325    25339   +14     
=======================================
+ Hits        23261    23275   +14     
  Misses       1675     1675           
  Partials      389      389           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mackjmr]

good catch @jackgopack4, addressed in latest commit

[jackgopack4]

looks good to me, thanks for addressing

[songy23]

https://github.com/open-telemetry/opentelemetry-collector/actions/runs/13013886944/job/36298551353?pr=12201 please fix linter

[mackjmr]

Are we good to merge this in ?