(angular) Hidden source maps flag breaks linking

[smeubank]

Description

Issue when setting hidden: "true"

https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#use-hidden-source-map

Example repo:
https://github.com/indra-uolles/test-sentry-angular

Solution

Confirm if only impacting angular

There is a work around in GH discussion: min.js in your extension then the algorithm in Sentry will make it work, but this approach didn;t work

update docs, to include the work around has the "official solution" unless there is a fix

[Lms24]

First observation with an Angular 15 test app: Can't reproduce this with the Sentry webpack plugin 1.x (pre-unplugin). Uploading hidden sourcemaps works ootb w/o additional configuration.

[Lms24]

Tried reproducing with Angular 15 test app + Sentry CLI v2 -> not reproducible. Hidden source maps work.

Reproduced so far only with the provided test app. Maybe source maps were emitted differently in Angular 12?

[Lms24]

Found it! It's because of an unnecessary command in the upload script provided in the repro:

sentry-cli releases files "$version" upload docs

is unnecessary because

sentry-cli releases files "$version" upload-sourcemaps docs

already uploads js and map files. No need to upload them separately. My suspicion is that because the js files were uploaded before, some rewriting or matching rules weren't applied in the same way. When removing the upload command, source maps are correctly displayed.

I don't think there's something we need to do here in the SDK. Perhaps a docs change to point this out?

related: getsentry/sentry-cli#1483

[Lms24]

Closing because IMO there's no action required on our end. Hidden source maps work ootb with the webpack plugin and with Sentry-CLI as long as all files are uploaded in one command (i.e. js files and source maps are not separately uploaded). Checked the docs and we never tell users to upload them separately.

@smeubank should we follow up with the customer?