Skip to content

Conversation

@sampaiodiego
Copy link
Member

@sampaiodiego sampaiodiego commented Nov 3, 2025

Summary by CodeRabbit

  • Chores
    • Updated federation-sdk to version 0.3.0
    • Updated dependencies in core and room packages for security and compatibility improvements

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 3, 2025

Walkthrough

This PR updates dependency and package versions across three packages: downgrades pino in packages/core from ^9.11.0 to ^8.21.0, bumps packages/federation-sdk version from 0.2.0 to 0.3.0, and upgrades zod in packages/room from ^3.22.4 to ^3.24.1.

Changes

Cohort / File(s) Summary
Package version bump
packages/federation-sdk/package.json
Version bumped from 0.2.0 to 0.3.0
Dependency updates
packages/core/package.json, packages/room/package.json
Pino downgraded from ^9.11.0 to ^8.21.0 in core; Zod upgraded from ^3.22.4 to ^3.24.1 in room

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Pino downgrade in packages/core: Verify compatibility implications of downgrading from 9.11.0 to 8.21.0—downgrade operations warrant attention to ensure no breaking changes are introduced.

Possibly related PRs

Suggested reviewers

  • rodrigok

Poem

🐰 Version bumps and deps take flight,
Zod climbs higher, pino slides right,
Federated dreams now version three,
Lock those packages carefully!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The pull request title mentions only the federation-sdk package bump to 0.3.0, but the changeset also includes downgrades to pino and updates to zod in other packages. Update the title to reflect all significant changes, such as 'Bump federation-sdk to 0.3.0 and update dependencies' or provide a more comprehensive description of the multi-package changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bump-0.3.0

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between fc10b0c and f8f7ba7.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • packages/core/package.json (1 hunks)
  • packages/federation-sdk/package.json (1 hunks)
  • packages/room/package.json (1 hunks)
🔇 Additional comments (3)
packages/core/package.json (1)

22-22: Verify rationale for pino downgrade.

Downgrading pino from ^9.11.0 to ^8.21.0 (major version down) is unusual. This suggests either a compatibility issue with v9 or a breaking change that affects the codebase. Ensure this is intentional and document the reason for the downgrade.

packages/room/package.json (1)

21-21: Zod upgrade is safe.

The zod upgrade from ^3.22.4 to ^3.24.1 is a minor version bump within the caret range and should maintain backward compatibility.

packages/federation-sdk/package.json (1)

3-3: Verify federation-sdk version bump aligns with code changes.

The version is being bumped from 0.2.0 to 0.3.0 (minor release). Ensure that the actual code changes in the federation-sdk justify this minor version bump according to semantic versioning principles. Review the CHANGELOG or commit history to confirm.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.46%. Comparing base (fc10b0c) to head (f8f7ba7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   60.46%   60.46%           
=======================================
  Files          67       67           
  Lines        6672     6672           
=======================================
  Hits         4034     4034           
  Misses       2638     2638           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ggazzo ggazzo merged commit 0a479b1 into main Nov 4, 2025
3 checks passed
@ggazzo ggazzo deleted the bump-0.3.0 branch November 4, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants