Remove ClientParseError type and use JSON.stringify in place of serializeFetchParameter

[jerelmiller]

Replaces the call to serializeFetchParameter with JSON.stringify directly. This removes the ClientParseError type as a result in favor of throwing the json stringify error directly.

[apollo-librarian]

⚠️ Docs preview not attached to branch

The preview was not built because the PR's base branch release-4.0 is not in the list of sources.

An Apollo team member can comment one of the following commands to dictate which branch to attach the preview to:

• !docs set-base-branch version-2.6
• !docs set-base-branch main

Build ID: 1e0aa26e52ad38704b3400b8
Build Logs: View logs

[pkg-pr-new]

npm i https://pkg.pr.new/apollographql/apollo-client/@apollo/client@12825

commit: 200a4ec

[github-actions]

size-limit report 📦

Path	Size
import { ApolloClient, InMemoryCache, HttpLink } from "@apollo/client" (CJS)	42.77 KB (-0.61% 🔽)
import { ApolloClient, InMemoryCache, HttpLink } from "@apollo/client" (production) (CJS)	37.72 KB (-0.22% 🔽)
import { ApolloClient, InMemoryCache, HttpLink } from "@apollo/client"	32.76 KB (-0.2% 🔽)
import { ApolloClient, InMemoryCache, HttpLink } from "@apollo/client" (production)	26.92 KB (-0.26% 🔽)
import { ApolloProvider } from "@apollo/client/react"	5.9 KB (-0.71% 🔽)
import { ApolloProvider } from "@apollo/client/react" (production)	971 B (0%)
import { useQuery } from "@apollo/client/react"	7.21 KB (-0.42% 🔽)
import { useQuery } from "@apollo/client/react" (production)	2.23 KB (+0.05% 🔺)
import { useLazyQuery } from "@apollo/client/react"	7.1 KB (-0.44% 🔽)
import { useLazyQuery } from "@apollo/client/react" (production)	2.13 KB (0%)
import { useMutation } from "@apollo/client/react"	6.45 KB (-0.31% 🔽)
import { useMutation } from "@apollo/client/react" (production)	1.48 KB (0%)
import { useSubscription } from "@apollo/client/react"	6.78 KB (-0.45% 🔽)
import { useSubscription } from "@apollo/client/react" (production)	1.8 KB (0%)
import { useSuspenseQuery } from "@apollo/client/react"	8.58 KB (-0.32% 🔽)
import { useSuspenseQuery } from "@apollo/client/react" (production)	3.63 KB (0%)
import { useBackgroundQuery } from "@apollo/client/react"	8.33 KB (-0.41% 🔽)
import { useBackgroundQuery } from "@apollo/client/react" (production)	3.38 KB (0%)
import { useLoadableQuery } from "@apollo/client/react"	8.28 KB (-0.61% 🔽)
import { useLoadableQuery } from "@apollo/client/react" (production)	3.36 KB (0%)
import { useReadQuery } from "@apollo/client/react"	6.59 KB (-0.51% 🔽)
import { useReadQuery } from "@apollo/client/react" (production)	1.63 KB (0%)
import { useFragment } from "@apollo/client/react"	6.65 KB (-0.4% 🔽)
import { useFragment } from "@apollo/client/react" (production)	1.69 KB (0%)

[phryneas]

That's a good call. Yes, the error messages from JSON.stringify suck, but it's not worth the extra complexity of maintaining an additional error and shipping out extra bundle size.

[changeset-bot]

🦋 Changeset detected

Latest commit: ffa09b8

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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