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

🦋 Changeset detected

Latest commit: df41d77

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

This PR removes the deprecated sendFileLivechatMessage method from the Rocket.Chat livechat server by deleting its Meteor.methods wrapper and DDP interface declaration. The underlying exported function remains intact. A changeset documents the major version bump and changelog entry.

Changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Single cohesive change targeting one deprecated method
• Minimal file scope with straightforward deletions
• Verify no existing callers directly invoke this as a Meteor method
• Confirm function remains properly exported for any internal callers

Possibly related PRs

• chore: adds deprecation warning on sendFileLivechatMessage #37235: Added deprecation warning to the sendFileLivechatMessage Meteor method wrapper—this PR completes the removal cycle by deleting the deprecated DDP method interface entirely.

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

• ggazzo
• aleksandernsilva

Poem

🐰 Away with the old, deprecated and worn,
The sendFile method bids farewell at dawn,
Its wrapper removed with a tidy clean sweep,
While exports stay cozy, their promises to keep!
—A rabbit's goodbye to legacy code ✨

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 68.44%. Comparing base (750af59) to head (df41d77).
⚠️ Report is 3 commits behind head on release-8.0.0.

@@                Coverage Diff                @@
##           release-8.0.0   #37463      +/-   ##
=================================================
- Coverage          68.44%   68.44%   -0.01%     
=================================================
  Files               2877     2877              
  Lines             101867   101934      +67     
  Branches           17812    17728      -84     
=================================================
+ Hits               69727    69764      +37     
- Misses             30493    30522      +29     
- Partials            1647     1648       +1