test(client-react): give the package a test harness and pin realtime hook behavior (#4682)

`packages/client-react` shipped 8 public hooks with `build` and `typecheck` as
its only scripts and zero test files. `tsc --noEmit` is structurally blind to
what actually breaks in a hook: a dependency array is a value, not a type, so a
missing entry, a missing cleanup and a callback that never fires all typecheck
perfectly. #4678 was exactly that shape — `useAutoRefresh` ignored predicate
writes, the case that dirties a list hardest, and no type noticed.

Adds the workspace's first DOM test environment (jsdom + @testing-library/react,
`environment: 'jsdom'`; every other package runs `node`) and 17 tests over the
realtime hooks covering the three things the type checker cannot see:

- re-subscription driven by the dependency array — changing `object` opens a
  subscription on the new name and releases the old one, an unrelated re-render
  churns nothing, and the hooks key on the primitive `options?.recordId` /
  `options?.packageId` rather than the options object's identity, so an
  equal-but-new object stays a no-op;
- release on unmount, including `useAutoRefresh`, which holds two subscriptions;
- delivery — events reach state and callbacks, and `useAutoRefresh` refetches on
  the per-record *and* the bulk stream (the #4678 regression pin).

Every assertion was verified by sabotage rather than assumed: dropping the
`object` dep (1 failure), deleting a cleanup (3) and reverting `useAutoRefresh`
to the single-stream version (3) each turn the suite red; reverting turns it
green again.

CI needs no wiring — `Test Core` partitions by package off `turbo ls`, and the
new `test` script puts client-react on shard 2 (verified with
scripts/partition-test-shards.mjs).

No runtime code changed.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LYnZrTwXbrctB8E8HpJAPT

Author: claude

.changeset/client-react-test-harness.md

@@ -0,0 +1,35 @@
+---
+"@objectstack/client-react": patch
+---
+
+test(client-react): give the package a test harness and pin the realtime hooks' behavior (#4682)
+
+`packages/client-react` shipped 8 public hooks with `build` and `typecheck` as
+its only scripts and not a single test file. `tsc --noEmit` cannot see any of
+what actually breaks in a hook: a dependency array is a value, not a type, so a
+missing entry, a missing cleanup, and a callback that never fires all typecheck
+perfectly. #4678 was precisely that shape — `useAutoRefresh` ignored predicate
+writes, the one case that dirties a list hardest, and no type noticed.
+
+Adds the workspace's first DOM test environment (`jsdom` + `@testing-library/
+react`, `environment: 'jsdom'` — every other package runs `node`) and 17 tests
+over the realtime hooks, covering the three things the type checker is blind to:
+
+- **Re-subscription on dependency change** — changing `object` opens a
+  subscription on the new name and releases the old one; a re-render that
+  changes nothing must not churn. Also pins that the hooks key on the primitive
+  `options?.recordId` / `options?.packageId` rather than on the options object's
+  identity, so an equal-but-new object stays a no-op.
+- **Release on unmount** — every subscription hook unsubscribes, including
+  `useAutoRefresh`, which holds two.
+- **Delivery** — events reach state and callbacks, and `useAutoRefresh`
+  refetches on the per-record *and* the bulk stream (the #4678 regression pin).
+
+Each assertion was verified by sabotage: dropping the `object` dep, deleting a
+cleanup, and reverting `useAutoRefresh` to the single-stream version each turn
+the suite red, and only reverting turns it green again.
+
+The package is picked up by CI's `Test Core` shards automatically — they
+partition by package off `turbo ls`, so a `test` script is all that was needed.
+
+No runtime code changed.

packages/client-react/package.json

@@ -14,6 +14,8 @@
   },
   "scripts": {
     "build": "tsup src/index.tsx --config ../../tsup.config.ts",
+    "test": "vitest run",
+    "test:watch": "vitest",
     "typecheck": "tsc --noEmit"
   },
   "peerDependencies": {
@@ -25,8 +27,14 @@
     "@objectstack/spec": "workspace:*"
   },
   "devDependencies": {
+    "@testing-library/react": "^16.3.2",
     "@types/react": "^19.2.17",
-    "typescript": "^6.0.3"
+    "@types/react-dom": "^19.2.3",
+    "jsdom": "^30.0.1",
+    "react": "^19.2.8",
+    "react-dom": "^19.2.8",
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.10"
   },
   "keywords": [
     "objectstack",