fix up knip reports (#420)

* fix up knip reports

* fixup

* more changes

* fixup

* update lockfiles

* change import to pure

integration-test/yarn.lock

@@ -39,27 +39,27 @@ __metadata:
   linkType: hard
 
 "@apollo/client-integration-react-router@exec:./shared/build-client-integration-react-router.cjs::locator=%40integration-test%2Froot%40workspace%3A.":
-  version: 0.11.5
+  version: 0.11.8
   resolution: "@apollo/client-integration-react-router@exec:./shared/build-client-integration-react-router.cjs#./shared/build-client-integration-react-router.cjs::hash=d34732&locator=%40integration-test%2Froot%40workspace%3A."
   dependencies:
     "@apollo/client-react-streaming": "npm:0.11.8-alpha.1"
     promiscade: "npm:^0.0.2"
   peerDependencies:
     react: ^19
     react-router: ^7.0.2
-  checksum: 10/e8d6af811d70f120e61f9c6ad3c518a892f75bbb3a84a83ba180c1ca65774660bc01313c053474ae605b0140744ce01f65cd469601180b996702027ddbe8d3c6
+  checksum: 10/33a961cd848ede8bc31907346134b7f0bfab48522271fa1be4e6deac5d624cf1d0d3c63a9def90c89b2a2894ae683195ca470f77d6fb2fdd7893823ff0c507a6
   languageName: node
   linkType: hard
 
 "@apollo/client-integration-tanstack-start@exec:./shared/build-client-integration-tanstack-start.cjs::locator=%40integration-test%2Froot%40workspace%3A.":
-  version: 0.11.5
+  version: 0.11.8
   resolution: "@apollo/client-integration-tanstack-start@exec:./shared/build-client-integration-tanstack-start.cjs#./shared/build-client-integration-tanstack-start.cjs::hash=66d731&locator=%40integration-test%2Froot%40workspace%3A."
   dependencies:
     "@apollo/client-react-streaming": "npm:0.11.5"
   peerDependencies:
     "@tanstack/react-router": "*"
     react: ^19
-  checksum: 10/925e6f086df1f3d7a3fd0e60265dce1192a7666934b2ee35f545ffdb00f04463d0734649e9320094b195aaa8b327bd8520a38dd7436051413e97564183f83fda
+  checksum: 10/8fbcea3c6b0bb00a2036c84159428822538e939ad23ae227d35c92b5df4a8704494bec7b8c7d70c3c67d8f83878e645f8444bdc64a21a899558f904b83194daa
   languageName: node
   linkType: hard
 
@@ -73,7 +73,7 @@ __metadata:
     graphql: ^16 || >=17.0.0-alpha.2
     react: ^18 || >=19.0.0-rc
     react-dom: ^18 || >=19.0.0-rc
-  checksum: 10/04dbf100bd75132b9ca2b9a00e35e31d89fb9b197339120b5469503a2e4908ec707e145e92cfcb44306b4b9db69c09fe4e10463b228530c7b3a9ac3b7b4500a2
+  checksum: 10/7eae1b9df8c5e4e37096f37f488d0779f47650b724b4cbedc534dd0c00c3f0ea99c1452cde911e20137e64c6b28c22a9bd760d9d9fb92fe26ade8e253dd607eb
   languageName: node
   linkType: hard
 
@@ -123,7 +123,7 @@ __metadata:
     "@apollo/client": ^3.10.4
     next: ^13.4.1 || ^14.0.0 || ^15.0.0-rc.0
     react: ^18 || >=19.0.0-rc
-  checksum: 10/a866cc1c43ff0358e7c4a7be793c1c73eb0f0078022cd86475fe375902378e1adfcf4ff7d672a6e365daaba812a0e7ca253af2315275146ddf283825ec1cb75e
+  checksum: 10/831e5f8f8cb0b86c4fb10bcba1c54966f60e2ff9d72bc3e6f52d0816cc0d71c495f8c2e7a3a8d1d0ab524809f2852a883a36c50de432c718a7c00ad69d160a32
   languageName: node
   linkType: hard
 

knip.config.js

@@ -5,12 +5,15 @@ const config = {
   workspaces: {
     ".": {
       ignoreBinaries: ["jq", "playwright"],
-      ignoreDependencies: [/@size-limit\/.*/, "prettier"],
+      ignoreDependencies: [/@size-limit\/.*/, "prettier", "zx"],
     },
     "packages/*": {
       entry: ["**/*.test.{ts,tsx}"],
       project: ["**/*.{ts,tsx}"],
     },
+    "packages/test-utils": {
+      entry: ["*.js"],
+    },
     "packages/client-react-streaming": {
       entry: ["**/*.test.{ts,tsx}"],
       project: ["**/*.{ts,tsx}"],
@@ -24,6 +27,7 @@ const config = {
     "scripts/**",
     "packages/client-react-streaming/api-extractor.d.ts",
     "packages/experimental-nextjs-app-support/api-extractor.d.ts",
+    "**/.yalc/**",
   ],
 };
 

packages/client-react-streaming/package.json

@@ -139,7 +139,6 @@
     "@testing-library/react": "^16.1.0",
     "@total-typescript/shoehorn": "0.1.2",
     "@tsconfig/recommended": "1.0.6",
-    "@types/eslint": "^9.6.1",
     "@types/node": "20.12.11",
     "@types/react": "^19.0.0",
     "concurrently": "8.2.2",

packages/client-react-streaming/src/DataTransportAbstraction/WrappedApolloClient.tsx

@@ -441,7 +441,7 @@ export class ApolloClientBrowserImpl extends ApolloClientClientBaseImpl {
   }
 }
 
-export class ApolloClientRSCImpl extends ApolloClientBase {
+class ApolloClientRSCImpl extends ApolloClientBase {
   constructor(options: WrappedApolloClientOptions) {
     super(options);
     this.setLink(this.link);

packages/client-react-streaming/src/transportedQueryRef.ts

@@ -32,7 +32,7 @@ import {
   type JsonString,
 } from "@apollo/client-react-streaming/stream-utils";
 
-export type RestrictedPreloadOptions = {
+type RestrictedPreloadOptions = {
   fetchPolicy?: "network-only" | "cache-and-network" | "cache-first";
   returnPartialData?: false;
   nextFetchPolicy?: undefined;
@@ -45,7 +45,7 @@ export type PreloadTransportedQueryOptions<TVariables, TData> = Omit<
 > &
   RestrictedPreloadOptions;
 
-export type TransportedQueryRefOptions = TransportedOptions &
+type TransportedQueryRefOptions = TransportedOptions &
   RestrictedPreloadOptions;
 
 /**
@@ -141,7 +141,7 @@ export function createTransportedQueryPreloader(
   };
 }
 
-export function createTransportedQueryRef<
+function createTransportedQueryRef<
   TData,
   TVariables extends OperationVariables,
 >(

packages/experimental-nextjs-app-support/package.json

@@ -121,7 +121,6 @@
     "@internal/test-utils": "workspace:^",
     "@microsoft/api-extractor": "7.43.2",
     "@tsconfig/recommended": "1.0.6",
-    "@types/eslint": "^9.6.1",
     "@types/node": "20.12.11",
     "@types/react": "^19.0.0",
     "@types/react-dom": "^19.0.0",
@@ -137,7 +136,7 @@
     "rimraf": "5.0.5",
     "tsup": "8.0.2",
     "typescript": "^5.7.3",
-    "wait-on": "^8.0.1"
+    "wait-on": "8.0.2"
   },
   "peerDependencies": {
     "@apollo/client": "^3.10.4",

packages/package.json

@@ -3,6 +3,7 @@
   "version": "0.0.0",
   "private": true,
   "devDependencies": {
+    "@types/eslint": "9.6.1",
     "@typescript-eslint/eslint-plugin": "7.5.0",
     "@typescript-eslint/parser": "7.5.0",
     "eslint": "8.57.0",

packages/react-router/package.json

@@ -54,7 +54,7 @@
     "test:browser": "NODE_OPTIONS=\"${NODE_OPTIONS:-} --conditions=browser\" yarn run test:base",
     "test:rsc": "NODE_OPTIONS=\"${NODE_OPTIONS:-} --conditions=react-server\" yarn run test:base",
     "prepack": "yarn build",
-    "prepublishOnly": "wait-on ../client-react-streaming/.published --delay 3000 --timeout 30000 && yarn pack -o attw.tgz && attw attw.tgz && rm attw.tgz && yarn run test",
+    "prepublishOnly": "wait-on ../client-react-streaming/.built --delay 3000 --timeout 60000 && yarn pack -o attw.tgz && attw attw.tgz && rm attw.tgz && yarn run test",
     "test-bundle": "yarn test-bundle:attw && yarn test-bundle:package && yarn test-bundle:publint && yarn test-bundle:shape",
     "test-bundle:attw": "attw --pack .",
     "test-bundle:package": "yarn workspace monorepo verify-package-json $PWD/package.json",
@@ -67,30 +67,21 @@
     "@apollo/client": "^3.12.6",
     "@apollo/client-react-streaming": "workspace:*",
     "@arethetypeswrong/cli": "0.15.3",
-    "@internal/test-utils": "workspace:^",
     "@microsoft/api-extractor": "7.43.2",
-    "@testing-library/dom": "^10.4.0",
-    "@testing-library/react": "^16.1.0",
-    "@total-typescript/shoehorn": "0.1.2",
     "@tsconfig/recommended": "1.0.6",
     "@types/node": "20.12.11",
     "@types/react": "^19.0.0",
     "concurrently": "8.2.2",
+    "esbuild": "0.24.2",
     "eslint": "8.57.0",
-    "glob": "10.3.12",
-    "global-jsdom": "24.0.0",
-    "globstar": "1.0.0",
     "graphql": "16.8.1",
-    "jsdom": "24.0.0",
     "publint": "0.2.7",
     "react": "^19.0.0",
     "react-router": "patch:react-router@npm%3A7.0.2#~/.yarn/patches/react-router-npm-7.0.2-b96f2bd13c.patch",
     "rimraf": "5.0.5",
-    "ts-node": "10.9.2",
     "tsup": "8.0.2",
-    "tsx": "4.7.1",
     "typescript": "5.4.5",
-    "vitest": "1.6.0"
+    "wait-on": "8.0.2"
   },
   "peerDependencies": {
     "react": "^19",

packages/react-router/src/preloader.tsx

@@ -59,7 +59,7 @@ export function createApolloLoaderHandler(
 // once that functionality has been added, all this can be removed.
 
 type EventPromiscade = Promiscade<JsonString<ReadableStreamLinkEvent>>;
-export type PromiscadedRef = Omit<TransportedQueryRef, "$__apollo_queryRef"> & {
+type PromiscadedRef = Omit<TransportedQueryRef, "$__apollo_queryRef"> & {
   $__apollo_queryRef: Omit<
     TransportedQueryRef["$__apollo_queryRef"],
     "stream"
@@ -69,7 +69,7 @@ export type PromiscadedRef = Omit<TransportedQueryRef, "$__apollo_queryRef"> & {
   };
 };
 
-export function isPromiscaded(
+function isPromiscaded(
   queryRef: TransportedQueryRef | PromiscadedRef
 ): queryRef is PromiscadedRef {
   return "promiscade" in queryRef.$__apollo_queryRef;
@@ -96,7 +96,7 @@ function replaceStreamWithPromiscade(queryRef: TransportedQueryRef) {
  * could result in poor timing and have the modified object be sent over the wire instead
  * of the one with the promiscade
  */
-export function promiscadedRefToStreamRef(
+function promiscadedRefToStreamRef(
   queryRef: PromiscadedRef
 ): TransportedQueryRef {
   const { promiscade: _, ...restRef } = queryRef.$__apollo_queryRef;
@@ -111,7 +111,7 @@ export function promiscadedRefToStreamRef(
   };
 }
 
-export const hydratedRefs = new WeakMap<PromiscadedRef, TransportedQueryRef>();
+const hydratedRefs = new WeakMap<PromiscadedRef, TransportedQueryRef>();
 /**
  * If `obj` is a Promiscaded Ref, creates a new "live" QueryRef
  * If `obj` is a Transported Ref, converts it to a "live" QueryRef

packages/tanstack-start/package.json

@@ -54,7 +54,7 @@
     "test:ssr": "NODE_OPTIONS=\"${NODE_OPTIONS:-} --conditions=node\" yarn run test:base",
     "test:browser": "NODE_OPTIONS=\"${NODE_OPTIONS:-} --conditions=browser\" yarn run test:base",
     "prepack": "yarn build",
-    "prepublishOnly": "wait-on ../client-react-streaming/.published --delay 3000 --timeout 30000 && yarn pack -o attw.tgz && attw attw.tgz && rm attw.tgz && yarn run test",
+    "prepublishOnly": "wait-on ../client-react-streaming/.built --delay 3000 --timeout 60000 && yarn pack -o attw.tgz && attw attw.tgz && rm attw.tgz && yarn run test",
     "test-bundle": "yarn test-bundle:attw && yarn test-bundle:package && yarn test-bundle:publint && yarn test-bundle:shape",
     "test-bundle:attw": "attw --pack .",
     "test-bundle:package": "yarn workspace monorepo verify-package-json $PWD/package.json",
@@ -71,27 +71,20 @@
     "@microsoft/api-extractor": "7.43.2",
     "@tanstack/react-router": "patch:@tanstack/react-router@npm%3A1.97.3#~/.yarn/patches/@tanstack-react-router-npm-1.97.3-ad16343f36.patch",
     "@tanstack/start": "patch:@tanstack/start@npm%3A1.97.3#~/.yarn/patches/@tanstack-start-npm-1.97.3-da273c2822.patch",
-    "@testing-library/dom": "^10.4.0",
-    "@testing-library/react": "^16.1.0",
-    "@total-typescript/shoehorn": "0.1.2",
     "@tsconfig/recommended": "1.0.6",
     "@types/node": "20.12.11",
     "@types/react": "^19.0.0",
     "concurrently": "8.2.2",
+    "esbuild": "0.24.2",
     "eslint": "8.57.0",
-    "glob": "10.3.12",
-    "global-jsdom": "24.0.0",
-    "globstar": "1.0.0",
     "graphql": "16.8.1",
-    "jsdom": "24.0.0",
     "publint": "0.2.7",
     "react": "^19.0.0",
+    "react-error-boundary": "5.0.0",
     "rimraf": "5.0.5",
-    "ts-node": "10.9.2",
     "tsup": "8.0.2",
-    "tsx": "4.7.1",
     "typescript": "5.4.5",
-    "vitest": "1.6.0"
+    "wait-on": "8.0.2"
   },
   "peerDependencies": {
     "@tanstack/react-router": "*",

packages/test-utils/package.json

@@ -2,9 +2,11 @@
   "name": "@internal/test-utils",
   "private": true,
   "version": "0.0.0",
-  "devDependencies": {
+  "dependencies": {
     "@testing-library/react-render-stream": "^2.0.0",
-    "expect": "^29.7.0",
+    "global-jsdom": "^26.0.0",
+    "react": "^19.0.0",
+    "react-dom": "^19.0.0",
     "typescript": "^5.7.3"
   },
   "type": "module"

packages/test-utils/react.js

@@ -37,7 +37,7 @@ export async function renderStreamEnv() {
   const jsdom = await import("global-jsdom");
   const cleanupJSDOM = jsdom.default();
   const { createRenderStream } = await import(
-    "@testing-library/react-render-stream"
+    "@testing-library/react-render-stream/pure"
   );
 
   return {