[chore] Remove use of alias types in service/telemetry

[bogdandrutu]

This PR moves internal definitions that were aliased publicly to the public package, and makes sure that hides any other type so that the API surface is the same.

[codecov]

Codecov Report

Attention: Patch coverage is 49.18033% with 31 lines in your changes missing coverage. Please review.

Project coverage is 91.81%. Comparing base (1edd311) to head (61ddbe6).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
service/telemetry/factory.go	15.38%	22 Missing ⚠️
service/telemetry/factory_impl.go	74.28%	6 Missing and 3 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #11182   +/-   ##
=======================================
  Coverage   91.81%   91.81%           
=======================================
  Files         422      422           
  Lines       20002    20002           
=======================================
  Hits        18365    18365           
  Misses       1262     1262           
  Partials      375      375           

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

[codeboten]

I was looking at the original PR, @mx-psi was there a reason to have the internal aliases?

[mx-psi]

I believe the internal aliasing stuff may have been needed as part of the refactor but not now. In any case I believe this leaves the API surface the same so if we were to need this we can always revisit this

[codeboten]

Thanks for confirming!