Remove unused source-map-explorer package

[anshgoyalevil]

Which problem is this PR solving?

• This PR removes the source-map-explorer package because:
  • It is not being used anywhere inside the project
  • According to its npm desc., it needs to be installed globally, so anyways, not needed inside the project directly.

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully
  • for jaeger: make lint test
  • for jaeger-ui: yarn lint and yarn test

[yurishkuro]

what are we doing for source maps if we're not using this module?

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (fe0c087) 96.01% compared to head (c7d6456) 96.01%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1671   +/-   ##
=======================================
  Coverage   96.01%   96.01%           
=======================================
  Files         241      241           
  Lines        7558     7558           
  Branches     1984     1984           
=======================================
  Hits         7257     7257           
  Misses        301      301           

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

[anshgoyalevil]

I don't think this library is being used anywhere. Where do we have source-maps inside the repository?

[yurishkuro]

I don't know, but I am able to see some source code in the Inspector:

[yurishkuro]

but I can also see the same after applying your PR, so probably fine to merge. It's possible source maps are provided by webpack or vite directly.

[anshgoyalevil]

Yes. I can see a webpack config for that which uses source-map. (It's different from source-map-explorer)

output: {
      path: join(__dirname, 'build'),
      publicPath: '/',
      filename: 'assets/[name].js',
    },

Safe to merge, thus.