The CLS metrics report does not follow options.metricOptions.reportAllChanges

[ys-zhifu]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Self-hosted/on-premise

Which package are you using?

@sentry/vue

SDK Version

7.16.0

Framework Version

vue 3.2.33

Link to Sentry event

No response

Steps to Reproduce

1. sentry init
2. Check the reported transaction data and find that the CLS metrics is missing
3. After reading the code, I found that only when the page is hidden can I report
4. This reportAllChanges seems meaningless

The following is the code for reporting CLS

Expected Result

CLS metrics data reporting

Actual Result

CLS metrics data not reporting

[AbhiPrasad]

Hey! This is intentional behaviour, as we generally don't want to users to report intermediate values as they are less accurate. LCP was put in here while we were testing improving this generally. @k-fish should we remove the reportAllChanges option?

[k-fish]

@AbhiPrasad yeah, now that idleTimeout is fixed and there is some measure of control over when a transaction ends users shouldn't be seeing as missing as many LCP. Users that were using it may need to bump idleTimeout if they find their counts of LCP measurements count down, though.

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[timfish]

@k-fish should we be removing the reportAllChanges option?

[AbhiPrasad]

I think we can remove it @timfish

[k-fish]

Yep, let's remove it.