fix: Improve logging of errors in the browser if a circular reference or bigint is encountered

[MarkDuckworth]

An alternative solution to get information for #8903.

As reported by the user in #8093, err.name and err.message were undefined after the fix of #8907 was applied. Based on the previous behavior it's likely that error was not actually an error object, but instead it was some object that was failing to stringify with JSON.stringify. This solution reverts getting err.name and instead attempts to work around possible JSON.stringify issues.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 0179f79

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[google-oss-bot]

Size Report ¹

Affected Products

• @firebase/firestore

  Type	Base (a4897a6)	Merge (241e0e5)	Diff
  browser	391 kB	392 kB	+236 B (+0.1%)
  module	391 kB	392 kB	+236 B (+0.1%)
  react-native	392 kB	392 kB	+236 B (+0.1%)

• @firebase/firestore-lite

  Type	Base (a4897a6)	Merge (241e0e5)	Diff
  browser	115 kB	116 kB	+262 B (+0.2%)
  module	115 kB	116 kB	+262 B (+0.2%)
  react-native	116 kB	116 kB	+262 B (+0.2%)

• bundle

  15 size changes

  Type	Base (a4897a6)	Merge (241e0e5)	Diff
  firestore (CSI Auto Indexing Disable and Delete)	287 kB	287 kB	+197 B (+0.1%)
  firestore (CSI Auto Indexing Enable)	287 kB	287 kB	+197 B (+0.1%)
  firestore (Persistence)	319 kB	319 kB	+197 B (+0.1%)
  firestore (Query Cursors)	258 kB	258 kB	+197 B (+0.1%)
  firestore (Query)	256 kB	256 kB	+197 B (+0.1%)
  firestore (Read data once)	246 kB	246 kB	+197 B (+0.1%)
  firestore (Read Write w Persistence)	339 kB	339 kB	+197 B (+0.1%)
  firestore (Realtime updates)	246 kB	246 kB	+197 B (+0.1%)
  firestore (Transaction)	224 kB	225 kB	+197 B (+0.1%)
  firestore (Write data)	226 kB	226 kB	+197 B (+0.1%)
  firestore-lite (Query Cursors)	110 kB	110 kB	+223 B (+0.2%)
  firestore-lite (Query)	106 kB	106 kB	+223 B (+0.2%)
  firestore-lite (Read data once)	81.5 kB	81.7 kB	+223 B (+0.3%)
  firestore-lite (Transaction)	107 kB	107 kB	+223 B (+0.2%)
  firestore-lite (Write data)	91.0 kB	91.2 kB	+223 B (+0.2%)

• firebase

  Type	Base (a4897a6)	Merge (241e0e5)	Diff
  firebase-compat.js	800 kB	800 kB	+186 B (+0.0%)
  firebase-firestore-compat.js	349 kB	349 kB	+186 B (+0.1%)
  firebase-firestore-lite.js	138 kB	139 kB	+242 B (+0.2%)
  firebase-firestore.js	455 kB	455 kB	+216 B (+0.0%)

Test Logs

• Base (a4897a6): https://github.com/firebase/firebase-js-sdk/actions/runs/16452995276
• Merge (241e0e5): https://github.com/firebase/firebase-js-sdk/actions/runs/16483609097

1. https://storage.googleapis.com/firebase-sdk-metric-reports/DFPq5S7og1.html

[google-oss-bot]

Size Analysis Report ¹

This report is too large (638,603 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

• Base (a4897a6): https://github.com/firebase/firebase-js-sdk/actions/runs/16452995276
• Merge (241e0e5): https://github.com/firebase/firebase-js-sdk/actions/runs/16483609097

1. https://storage.googleapis.com/firebase-sdk-metric-reports/cymZSXwZiF.html

[dconeybe]

Drive-by comment: Are you able to add unit test coverage?

[ehsannas]

+1 to adding a unit test that invokes formatJSON with object containing the two special cases. Looks good otherwise.