Conversation
…points Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: ymc9 <104139426+ymc9@users.noreply.github.com>
📝 WalkthroughWalkthroughThis PR bumps the package version from 3.8.2 to 3.8.3 across all workspace manifests (root, packages, samples, tests). Additionally, tanstack-query's client.ts guards optimisticUpdate access with optional chaining, and react.ts, vue.ts, and svelte/index.svelte.ts add re-exports of getQueryKey. ChangesVersion Bump
Tanstack Query Fixes
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
5-17: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick winPin
packageManagertopnpm@10.12.1.package.json:17still points atpnpm@10.33.0, which conflicts with the repo's{package.json,.npmrc}pnpm pinning rule and can make installs/CI use a different toolchain.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@package.json` around lines 5 - 17, The packageManager field in package.json is still pinned to the wrong pnpm version; update it from pnpm@10.33.0 to pnpm@10.12.1 so it matches the repository’s pnpm toolchain pinning. Make this change directly in package.json and keep the existing packageManager setting aligned with the repo’s pnpm versioning rule.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@package.json`:
- Around line 5-17: The packageManager field in package.json is still pinned to
the wrong pnpm version; update it from pnpm@10.33.0 to pnpm@10.12.1 so it
matches the repository’s pnpm toolchain pinning. Make this change directly in
package.json and keep the existing packageManager setting aligned with the
repo’s pnpm versioning rule.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 74e5259e-b1a2-4354-81a3-9a7d4976cb3d
📒 Files selected for processing (32)
package.jsonpackages/auth-adapters/better-auth/package.jsonpackages/cli/package.jsonpackages/clients/client-helpers/package.jsonpackages/clients/fetch-client/package.jsonpackages/clients/tanstack-query/package.jsonpackages/clients/tanstack-query/src/common/client.tspackages/clients/tanstack-query/src/react.tspackages/clients/tanstack-query/src/svelte/index.svelte.tspackages/clients/tanstack-query/src/vue.tspackages/common-helpers/package.jsonpackages/config/eslint-config/package.jsonpackages/config/tsdown-config/package.jsonpackages/config/typescript-config/package.jsonpackages/config/vitest-config/package.jsonpackages/create-zenstack/package.jsonpackages/ide/vscode/package.jsonpackages/language/package.jsonpackages/orm/package.jsonpackages/plugins/policy/package.jsonpackages/plugins/soft-delete/package.jsonpackages/schema/package.jsonpackages/sdk/package.jsonpackages/server/package.jsonpackages/testtools/package.jsonpackages/zod/package.jsonsamples/orm/package.jsonsamples/taskforge/package.jsontests/e2e/package.jsontests/regression/package.jsontests/runtimes/bun/package.jsontests/runtimes/edge-runtime/package.json
Summary by CodeRabbit
New Features
Bug Fixes
Chores