chore(deps): update effect beta and Firebase dependencies#52
Conversation
📝 WalkthroughWalkthroughDependency versions are updated across the workspace catalog, root package manifest, backend example manifest, and app dependency documentation. ChangesDependency version bumps
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
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 |
|
View your CI Pipeline Execution ↗ for commit e025e47
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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.
Inline comments:
In `@example/app/package.json`:
- Around line 14-16: The dependency versions in the package manifest are out of
alignment because tailwind-merge is still set to the newer range instead of the
required one. Update the tailwind-merge entry in the same dependencies block
where class-variance-authority and clsx are listed so it is pinned to ^2.5.5,
keeping the rest of the package versions unchanged.
In `@example/backend/package.json`:
- Around line 15-17: The firebase-admin dependency range in package.json is
inconsistent with the root package version, so align the example/backend
declaration with the same ^13.7.0 range used elsewhere. Update the
firebase-admin entry in this package manifest only, keeping the rest of the
dependencies unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 7fd963c9-ecbf-498f-b3d1-94a441523609
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (8)
example/app/package.jsonexample/backend/package.jsonexample/shared/package.jsonpackage.jsonpackages/admin/package.jsonpackages/client/package.jsonpackages/effect-firebase/package.jsonpackages/mock/package.json
Rebased onto next's pnpm catalog setup. Version bumps now live in the catalog block of pnpm-workspace.yaml; non-catalogued deps updated inline. Catalog: - effect / @effect/platform-browser / @effect/vitest: beta.90 -> beta.94 - firebase: ^12.9.0 -> ^12.16.0 - firebase-functions: ^7.0.0 -> ^7.2.5 - firebase-admin: ^13.6.0 -> ^13.7.0 Inline (not catalogued): - @google-cloud/functions-framework: ^5.0.x -> ^5.0.5 (root + example/backend) - firebase-tools: ^15.5.1 -> ^15.22.4 Also corrects a stale tailwind-merge version in example/app/AGENTS.md (^2.5.5 -> ^3.4.0) to match the actual dependency. Held back majors that would break the current stack: firebase-admin v14 (firebase-functions@7 peer tops out at ^13), nx 23, vite 8, typescript 7, tailwindcss 4. Build, tests and lint all pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HXYrJ9rH3gxu8YBdehmSC6
580280c to
e025e47
Compare
Summary
Routine dependency refresh on the
nexttrack, rebased onto next's pnpm catalog setup. Centralised versions are bumped in thecatalog:block ofpnpm-workspace.yaml; the two non-catalogued deps are bumped inline. Deliberately holds back major bumps that would break the current stack.Updated — catalog (
pnpm-workspace.yaml)effect/@effect/platform-browser/@effect/vitestfirebasefirebase-functionsfirebase-adminUpdated — inline (not catalogued)
@google-cloud/functions-framework(root + example/backend)firebase-tools(root)Docs
tailwind-mergeversion inexample/app/AGENTS.md(^2.5.5→^3.4.0) so it matches the actual dependency.Held back (intentionally)
firebase-functions@7.2.5still declares its peer range as^11 || ^12 || ^13, so adopting admin v14 would violate the peer contract.nx22→23,vite7→8,typescript5.9→7.0,tailwindcss3→4,@types/node20→26.Verification
pnpm build— all projects build ✅pnpm test— all tests pass ✅pnpm lint— passes (pre-existing unrelated warnings only) ✅No source migration was required for the effect beta jump.
🤖 Generated with Claude Code