Integrate adlrb/error-cause (#3860) into staging-38

dd-mergequeue[bot] · mormubis · web-flow · commit 18489a18751c · 2025-09-19T13:44:37.000Z
Integrated commit sha: 73dedf8 Co-authored-by: mormubis <adrian.delarosa@datadoghq.com>
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,21 @@
 
 ---
 
+## v6.21.0
+
+**Public Changes:**
+
+- ✨ [RUM-9181] Service Worker support for Logs ([#3769](https://github.com/DataDog/browser-sdk/pull/3769)) [FLAGGING] [LOGS] [RUM] [RUM-REACT] [RUM-SLIM] [WORKER]
+- ✨ [RUM-10146] implement `trackEarlyRequests` ([#3846](https://github.com/DataDog/browser-sdk/pull/3846)) [FLAGGING] [LOGS] [RUM] [RUM-REACT] [RUM-SLIM] [WORKER]
+
+**Internal Changes:**
+
+- 👷 when a request fails, display the response error ([#3853](https://github.com/DataDog/browser-sdk/pull/3853))
+- 👷 [Performance script] fix regression on size increase warnings ([#3845](https://github.com/DataDog/browser-sdk/pull/3845))
+- 🔧 configure renovate to wait 1 week before upgrading dependencies ([#3850](https://github.com/DataDog/browser-sdk/pull/3850))
+- 👷 [RUM Profiler] send view names as event attributes ([#3851](https://github.com/DataDog/browser-sdk/pull/3851)) [RUM]
+- 🐛 [PANA-4236] Fix flaky startRecorderInitTelemetry test ([#3852](https://github.com/DataDog/browser-sdk/pull/3852)) [RUM]
+
 ## v6.20.0
 
 **Public Changes:**
diff --git a/developer-extension/package.json b/developer-extension/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-sdk-developer-extension",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "private": true,
   "scripts": {
     "build": "rm -rf dist && webpack --disable-interpret --mode production",
diff --git a/lerna.json b/lerna.json
@@ -1,4 +1,4 @@
 {
   "npmClient": "yarn",
-  "version": "6.20.0"
+  "version": "6.21.0"
 }
diff --git a/packages/core/package.json b/packages/core/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-core",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/index.js",
   "module": "esm/index.js",
diff --git a/packages/core/src/domain/error/error.spec.ts b/packages/core/src/domain/error/error.spec.ts
@@ -8,7 +8,7 @@ import {
   isError,
   NO_ERROR_STACK_PRESENT_MESSAGE,
 } from './error'
-import type { RawErrorCause, ErrorWithCause } from './error.types'
+import type { ErrorWithCause, RawFlatErrorCause } from './error.types'
 import { ErrorHandling, ErrorSource, NonErrorPrefix } from './error.types'
 
 describe('computeRawError', () => {
@@ -179,7 +179,7 @@ describe('computeRawError', () => {
     expect(formatted.causes!.length).toBe(2)
     expect(formatted.stack).toContain('Error: foo: bar')
 
-    const causes = formatted.causes as RawErrorCause[]
+    const causes = formatted.causes as RawFlatErrorCause[]
 
     expect(causes[0].message).toContain(nestedError.message)
     expect(causes[0].source).toContain(ErrorSource.SOURCE)
@@ -241,7 +241,8 @@ describe('flattenErrorCauses', () => {
     error.cause = nestedError
 
     const errorCauses = flattenErrorCauses(error, ErrorSource.LOGGER)
-    expect(errorCauses?.length).toEqual(undefined)
+    expect(errorCauses?.length).toEqual(1)
+    expect(errorCauses?.[0]).toEqual(jasmine.objectContaining({ biz: 'buz', cause: jasmine.any(Error) }))
   })
 
   it('should use error to extract stack trace', () => {
@@ -250,7 +251,7 @@ describe('flattenErrorCauses', () => {
     error.cause = new Error('bar')
 
     const errorCauses = flattenErrorCauses(error, ErrorSource.LOGGER)
-    expect(errorCauses?.[0].type).toEqual('Error')
+    expect((errorCauses?.[0] as RawFlatErrorCause).type).toEqual('Error')
   })
 
   it('should only return the first 10 errors if nested chain is longer', () => {
diff --git a/packages/core/src/domain/error/error.ts b/packages/core/src/domain/error/error.ts
@@ -5,7 +5,15 @@ import { jsonStringify } from '../../tools/serialisation/jsonStringify'
 import type { StackTrace } from '../../tools/stackTrace/computeStackTrace'
 import { computeStackTrace } from '../../tools/stackTrace/computeStackTrace'
 import { toStackTraceString } from '../../tools/stackTrace/handlingStack'
-import type { ErrorSource, ErrorHandling, RawError, RawErrorCause, ErrorWithCause, NonErrorPrefix } from './error.types'
+import type {
+  ErrorSource,
+  ErrorHandling,
+  RawError,
+  RawErrorCause,
+  ErrorWithCause,
+  NonErrorPrefix,
+  RawFlatErrorCause,
+} from './error.types'
 
 export const NO_ERROR_STACK_PRESENT_MESSAGE = 'No stack, consider using an instance of Error'
 
@@ -87,17 +95,26 @@ export function isError(error: unknown): error is Error {
 }
 
 export function flattenErrorCauses(error: ErrorWithCause, parentSource: ErrorSource): RawErrorCause[] | undefined {
-  let currentError = error
   const causes: RawErrorCause[] = []
-  while (isError(currentError?.cause) && causes.length < 10) {
-    const stackTrace = computeStackTrace(currentError.cause)
-    causes.push({
-      message: currentError.cause.message,
-      source: parentSource,
-      type: stackTrace?.name,
-      stack: stackTrace && toStackTraceString(stackTrace),
-    })
-    currentError = currentError.cause
+
+  let currentCause = error.cause
+  while (currentCause !== undefined && currentCause !== null && causes.length < 10) {
+    if (isError(currentCause)) {
+      const stackTrace = computeStackTrace(currentCause)
+
+      causes.push({
+        message: currentCause.message,
+        source: parentSource,
+        type: stackTrace?.name,
+        stack: stackTrace && toStackTraceString(stackTrace),
+      } satisfies RawFlatErrorCause)
+
+      currentCause = (currentCause as ErrorWithCause).cause
+    } else {
+      causes.push(currentCause)
+
+      currentCause = undefined
+    }
   }
   return causes.length ? causes : undefined
 }
diff --git a/packages/core/src/domain/error/error.types.ts b/packages/core/src/domain/error/error.types.ts
@@ -14,7 +14,9 @@ export interface ErrorWithCause extends Omit<Error, 'cause'> {
   cause?: unknown
 }
 
-export interface RawErrorCause {
+export type RawErrorCause = unknown
+
+export interface RawFlatErrorCause {
   message: string
   source: ErrorSource
   type?: string
diff --git a/packages/flagging/package.json b/packages/flagging/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-flagging",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "private": true,
   "main": "cjs/entries/main.js",
@@ -15,10 +15,10 @@
     "replace-build-env": "node ../../scripts/build/replace-build-env.ts"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0"
   },
   "peerDependencies": {
-    "@datadog/browser-rum": "6.20.0"
+    "@datadog/browser-rum": "6.21.0"
   },
   "peerDependenciesMeta": {
     "@datadog/browser-rum": {
diff --git a/packages/logs/package.json b/packages/logs/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-logs",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/entries/main.js",
   "module": "esm/entries/main.js",
@@ -14,10 +14,10 @@
     "replace-build-env": "node ../../scripts/build/replace-build-env.ts"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0"
   },
   "peerDependencies": {
-    "@datadog/browser-rum": "6.20.0"
+    "@datadog/browser-rum": "6.21.0"
   },
   "peerDependenciesMeta": {
     "@datadog/browser-rum": {
diff --git a/packages/rum-core/package.json b/packages/rum-core/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-rum-core",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/index.js",
   "module": "esm/index.js",
@@ -13,7 +13,7 @@
     "replace-build-env": "node ../../scripts/build/replace-build-env.ts"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0"
   },
   "devDependencies": {
     "ajv": "8.17.1"
diff --git a/packages/rum-core/src/domain/error/errorCollection.spec.ts b/packages/rum-core/src/domain/error/errorCollection.spec.ts
@@ -1,6 +1,7 @@
 import type { RelativeTime, TimeStamp, ErrorWithCause } from '@datadog/browser-core'
 import { ErrorHandling, ErrorSource, NO_ERROR_STACK_PRESENT_MESSAGE } from '@datadog/browser-core'
 import { FAKE_CSP_VIOLATION_EVENT } from '@datadog/browser-core/test'
+import type { RawFlatErrorCause } from '@datadog/browser-core/src/domain/error/error.types.ts'
 import { collectAndValidateRawRumEvents } from '../../../test'
 import type { RawRumErrorEvent, RawRumEvent } from '../../rawRumEvent.types'
 import { RumEventType } from '../../rawRumEvent.types'
@@ -119,11 +120,12 @@ describe('error collection', () => {
       expect(error.message).toEqual('foo')
       expect(error.source).toEqual(ErrorSource.CUSTOM)
 
-      expect(error?.causes?.length).toEqual(2)
-      expect(error?.causes?.[0].message).toEqual('bar')
-      expect(error?.causes?.[0].source).toEqual(ErrorSource.CUSTOM)
-      expect(error?.causes?.[1].message).toEqual('biz')
-      expect(error?.causes?.[1].source).toEqual(ErrorSource.CUSTOM)
+      const causes = (error?.causes ?? []) as RawFlatErrorCause[]
+      expect(causes.length).toEqual(2)
+      expect(causes[0].message).toEqual('bar')
+      expect(causes[0].source).toEqual(ErrorSource.CUSTOM)
+      expect(causes[1].message).toEqual('biz')
+      expect(causes[1].source).toEqual(ErrorSource.CUSTOM)
     })
 
     it('should extract fingerprint from error', () => {
diff --git a/packages/rum-react/package.json b/packages/rum-react/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-rum-react",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/entries/main.js",
   "module": "esm/entries/main.js",
@@ -13,8 +13,8 @@
     "prepack": "npm run build"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0",
-    "@datadog/browser-rum-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0",
+    "@datadog/browser-rum-core": "6.21.0"
   },
   "peerDependencies": {
     "react": "18 || 19",
diff --git a/packages/rum-slim/package.json b/packages/rum-slim/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-rum-slim",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/entries/main.js",
   "module": "esm/entries/main.js",
@@ -13,11 +13,11 @@
     "build:esm": "rm -rf esm && tsc -p tsconfig.esm.json"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0",
-    "@datadog/browser-rum-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0",
+    "@datadog/browser-rum-core": "6.21.0"
   },
   "peerDependencies": {
-    "@datadog/browser-logs": "6.20.0"
+    "@datadog/browser-logs": "6.21.0"
   },
   "peerDependenciesMeta": {
     "@datadog/browser-logs": {
diff --git a/packages/rum/package.json b/packages/rum/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@datadog/browser-rum",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "cjs/entries/main.js",
   "module": "esm/entries/main.js",
@@ -14,11 +14,11 @@
     "replace-build-env": "node ../../scripts/build/replace-build-env.ts"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0",
-    "@datadog/browser-rum-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0",
+    "@datadog/browser-rum-core": "6.21.0"
   },
   "peerDependencies": {
-    "@datadog/browser-logs": "6.20.0"
+    "@datadog/browser-logs": "6.21.0"
   },
   "peerDependenciesMeta": {
     "@datadog/browser-logs": {
diff --git a/packages/worker/package.json b/packages/worker/package.json
@@ -1,14 +1,14 @@
 {
   "name": "@datadog/browser-worker",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "license": "Apache-2.0",
   "main": "bundle/worker.js",
   "scripts": {
     "pack": "yarn pack",
     "build": "rm -rf bundle && webpack --disable-interpret --mode=production"
   },
   "dependencies": {
-    "@datadog/browser-core": "6.20.0"
+    "@datadog/browser-core": "6.21.0"
   },
   "repository": {
     "type": "git",
diff --git a/performances/package.json b/performances/package.json
@@ -1,7 +1,7 @@
 {
   "private": true,
   "name": "performances",
-  "version": "6.20.0",
+  "version": "6.21.0",
   "scripts": {
     "start": "ts-node ./src/main.ts"
   },
diff --git a/test/apps/base-extension/yarn.lock b/test/apps/base-extension/yarn.lock
@@ -6,47 +6,47 @@ __metadata:
   cacheKey: 10c0
 
 "@datadog/browser-core@file:../../../packages/core/package.tgz::locator=rum-testing-extension%40workspace%3A.":
-  version: 6.20.0
-  resolution: "@datadog/browser-core@file:../../../packages/core/package.tgz#../../../packages/core/package.tgz::hash=79ce92&locator=rum-testing-extension%40workspace%3A."
-  checksum: 10c0/ec60c8fde885004d0cb0414fa7a76441c931b442f61b87d1694d49ac295497031d7e87a3e6937712606ea5391577d3577139e07ea388b08238314f41bdc02bb2
+  version: 6.21.0
+  resolution: "@datadog/browser-core@file:../../../packages/core/package.tgz#../../../packages/core/package.tgz::hash=70a5cb&locator=rum-testing-extension%40workspace%3A."
+  checksum: 10c0/d3b645a25a450006492fd3c48d96b78c7cd385bec81a9625f82faec64e7b411edb8612d698015cb12e1eea6ff9fc2dd1d6ab4754ff1be97ef2008b5b391c46b2
   languageName: node
   linkType: hard
 
 "@datadog/browser-logs@file:../../../packages/logs/package.tgz::locator=rum-testing-extension%40workspace%3A.":
-  version: 6.20.0
-  resolution: "@datadog/browser-logs@file:../../../packages/logs/package.tgz#../../../packages/logs/package.tgz::hash=2122c4&locator=rum-testing-extension%40workspace%3A."
+  version: 6.21.0
+  resolution: "@datadog/browser-logs@file:../../../packages/logs/package.tgz#../../../packages/logs/package.tgz::hash=73b714&locator=rum-testing-extension%40workspace%3A."
   dependencies:
-    "@datadog/browser-core": "npm:6.20.0"
+    "@datadog/browser-core": "npm:6.21.0"
   peerDependencies:
-    "@datadog/browser-rum": 6.20.0
+    "@datadog/browser-rum": 6.21.0
   peerDependenciesMeta:
     "@datadog/browser-rum":
       optional: true
-  checksum: 10c0/c68473b2f53e95f94b46dd45379d757e9943c45a7de9be837564ab0ff33458b79ce65997b80f07420f69ca0d735cc3bc1caf71424c3930212dc4550b105e5a3a
+  checksum: 10c0/752de81d0783998ec1cfd551674b3ffa9d07e0756033be47d06464420081d83781b0ee97a8760d073ae513580e1c28211c387df32d6604195df0f46a5c3d844b
   languageName: node
   linkType: hard
 
 "@datadog/browser-rum-core@file:../../../packages/rum-core/package.tgz::locator=rum-testing-extension%40workspace%3A.":
-  version: 6.20.0
-  resolution: "@datadog/browser-rum-core@file:../../../packages/rum-core/package.tgz#../../../packages/rum-core/package.tgz::hash=88d7a4&locator=rum-testing-extension%40workspace%3A."
+  version: 6.21.0
+  resolution: "@datadog/browser-rum-core@file:../../../packages/rum-core/package.tgz#../../../packages/rum-core/package.tgz::hash=6eb776&locator=rum-testing-extension%40workspace%3A."
   dependencies:
-    "@datadog/browser-core": "npm:6.20.0"
-  checksum: 10c0/9b3d2b4f8a823894ebc6e62c070feeeb321bd9f546ac36e6481e241ef5d0f66dc4245b84aec03e08c352ec99ad3f56862414609dc5a0d70cda0b1db0a51bae9a
+    "@datadog/browser-core": "npm:6.21.0"
+  checksum: 10c0/fb7eef26ec3f126a0b99a3bb39cc607a36731b2f48aefcaf8b44db1f9e6e833259ade0e0a50b6f8569460cd3b8eb6be0bb1fee72b681c49ab76d60a4225240c3
   languageName: node
   linkType: hard
 
 "@datadog/browser-rum@file:../../../packages/rum/package.tgz::locator=rum-testing-extension%40workspace%3A.":
-  version: 6.20.0
-  resolution: "@datadog/browser-rum@file:../../../packages/rum/package.tgz#../../../packages/rum/package.tgz::hash=8e2cb0&locator=rum-testing-extension%40workspace%3A."
+  version: 6.21.0
+  resolution: "@datadog/browser-rum@file:../../../packages/rum/package.tgz#../../../packages/rum/package.tgz::hash=837831&locator=rum-testing-extension%40workspace%3A."
   dependencies:
-    "@datadog/browser-core": "npm:6.20.0"
-    "@datadog/browser-rum-core": "npm:6.20.0"
+    "@datadog/browser-core": "npm:6.21.0"
+    "@datadog/browser-rum-core": "npm:6.21.0"
   peerDependencies:
-    "@datadog/browser-logs": 6.20.0
+    "@datadog/browser-logs": 6.21.0
   peerDependenciesMeta:
     "@datadog/browser-logs":
       optional: true
-  checksum: 10c0/33de0d690814582535c8c1dda356bc4c85f68366f2db1dd77e29a130cf907237222b2ca83a0319278968d8fe4a2a254e9c5ee36703f9def4ed8c62c39c1f8be9
+  checksum: 10c0/c3c39003e3ea18725e029a7e1daad4626d85cb089ed1a40b898338b098a328dde0fcb812751a4128e7c40a0a82a9f6499fb84c88d3b4124a71799bd6378365bb
   languageName: node
   linkType: hard
 
diff --git a/test/apps/react-router-v6-app/yarn.lock b/test/apps/react-router-v6-app/yarn.lock
diff --git a/test/apps/vanilla/yarn.lock b/test/apps/vanilla/yarn.lock
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@datadog/browser-sdk-developer-extension",`
`3`		`- "version": "6.20.0",`
	`3`	`+ "version": "6.21.0",`
`4`	`4`	`"private": true,`
`5`	`5`	`"scripts": {`
`6`	`6`	`"build": "rm -rf dist && webpack --disable-interpret --mode production",`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`	`1`	`{`
`2`	`2`	`"npmClient": "yarn",`
`3`		`- "version": "6.20.0"`
	`3`	`+ "version": "6.21.0"`
`4`	`4`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@datadog/browser-core",`
`3`		`- "version": "6.20.0",`
	`3`	`+ "version": "6.21.0",`
`4`	`4`	`"license": "Apache-2.0",`
`5`	`5`	`"main": "cjs/index.js",`
`6`	`6`	`"module": "esm/index.js",`