fix(v7/cdn): Stop using Object.assign to be ES5 compatible

[mydea]

This adds a lint rule to avoid usage of Object.assign (we already have similar rules for other things that are not polyfilled). We can just use object spread instead, which is polyfilled.

Fixes #16970

Replacing #17053 as that stalled on CI somehow...

[cursor]

Bug: Optional Parameter Handling Regression

The new object spread syntax ...opts introduces a TypeError when opts is undefined. As opts is an optional parameter (opts?: PerformanceObserverInit), this causes a runtime error when the function is called without the opts argument, unlike the previous opts || {} which safely defaulted to an empty object.

packages/tracing-internal/src/browser/web-vitals/lib/observe.ts#L51-L56

sentry-javascript/packages/tracing-internal/src/browser/web-vitals/lib/observe.ts

Lines 51 to 56 in ccf3153

	});
	po.observe({
	type,
	buffered: true,
	...opts,
	} as PerformanceObserverInit);

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎