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

🦋 Changeset detected

Latest commit: 63cf607

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

Removes the deprecated livechat:removeRoom Meteor method: deletes its server implementation file, removes its import from server bootstrap, and adds a changeset declaring a major version bump for @rocket.chat/meteor.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Client
  participant DDP as Meteor DDP
  participant Server as Livechat Server
  participant Backend as Omnichannel Removal

  Note over Client,DDP: OLD (removed in this PR)
  Client->>DDP: call "livechat:removeRoom"(rid)
  DDP->>Server: invoke Meteor method
  Server->>Backend: removeOmnichannelRoom(rid)
  Backend-->>Server: removal result
  Server-->>DDP: return result
  DDP-->>Client: response

  rect `#f8f9fa`
    Note right of Client: After this PR the DDP method path is removed
  end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Check for any remaining references to livechat:removeRoom across the repo (clients, tests, packages).
• Verify that CI/tests cover removal or that replacement endpoints (if present elsewhere) are used instead.

Possibly related PRs

• chore: Adds deprecation warning on livechat:removeRoom with new endpoint replacing it #36958 — Related PR that adds a deprecation warning for livechat:removeRoom and introduces a replacement REST endpoint livechat/rooms.delete.

Suggested reviewers

• MartinSchoeler
• KevLehman
• dougfabris
• sampaiodiego

Poem

🐰 I hopped through code and said "adieu",

The old removeRoom hid from view.
A changeset notes the final drum—
Farewell method, run on, scurry some! 🥕

Pre-merge checks and finishing touches

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 (a1c8379).
⚠️ Report is 389 commits behind head on release-8.0.0.

@@                Coverage Diff                @@
##           release-8.0.0   #37391      +/-   ##
=================================================
+ Coverage          66.43%   70.92%   +4.48%     
=================================================
  Files               3276     3031     -245     
  Lines             109596   104339    -5257     
  Branches           20860    18374    -2486     
=================================================
+ Hits               72814    74006    +1192     
+ Misses             34116    28400    -5716     
+ Partials            2666     1933     -733