Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

🦋 Changeset detected

Latest commit: 58799e2

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Walkthrough

The PR removes the deprecated livechat:saveAgentInfo DDP method from the Rocket.Chat Meteor server application. This includes deleting the method implementation file, removing its import from the server initialization, and documenting the removal through a changeset entry.

Changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Straightforward removal of deprecated code with clear scope and homogeneous changes
• Verify no other references to livechat:saveAgentInfo remain in the codebase
• Confirm that the replacement REST endpoint mentioned in related PRs is available for clients

Possibly related PRs

• RocketChat/Rocket.Chat#36975: Introduces deprecation logging and a REST endpoint replacement for livechat:saveAgentInfo method

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

• dougfabris
• MartinSchoeler
• sampaiodiego

Poem

🐰 A method deprecated, now sent to rest,
Its import removed, its code laid to jest,
The agent's old path fades into the night,
While newer roads lead our livechat right! ✨

Pre-merge checks and finishing touches

Learnt from: rodrigok
Repo: RocketChat/Rocket.Chat PR: 36991
File: apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts:219-221
Timestamp: 2025-09-19T15:15:04.642Z
Learning: The Federation_Matrix_homeserver_domain setting in apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts is part of the old federation system and is being deprecated/removed, so configuration issues with this setting should not be flagged for improvement.

Learnt from: rodrigok
Repo: RocketChat/Rocket.Chat PR: 36991
File: apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts:219-221
Timestamp: 2025-09-19T15:15:04.642Z
Learning: The Federation_Matrix_homeserver_domain setting in apps/meteor/server/services/federation/infrastructure/rocket-chat/adapters/Settings.ts is part of the old federation system and is being deprecated/removed, so configuration issues with this setting should not be flagged for improvement.

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 @coderabbitai help to get the list of available commands and usage tips.}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.92%. Comparing base (3c42f0d) to head (58799e2).
⚠️ Report is 389 commits behind head on release-8.0.0.

@@                Coverage Diff                @@
##           release-8.0.0   #37393      +/-   ##
=================================================
+ Coverage          66.43%   70.92%   +4.48%     
=================================================
  Files               3276     3031     -245     
  Lines             109596   104339    -5257     
  Branches           20860    18373    -2487     
=================================================
+ Hits               72814    74001    +1187     
+ Misses             34116    28405    -5711     
+ Partials            2666     1933     -733