chore!: remove deprecated livechat:sendTranscript method

[juliajforesti]

ARCH-1862

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Summary by CodeRabbit

• Breaking Changes
  • Removed the livechat:sendTranscript method from Livechat. Users relying on this feature to send chat transcripts should identify alternative solutions before upgrading to this major version.

[dionisio-bot]

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

[changeset-bot]

🦋 Changeset detected

Latest commit: 8d790c7

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

This PR includes changesets to release 40 packages

Name	Type
@rocket.chat/meteor	Major
@rocket.chat/core-typings	Major
@rocket.chat/rest-typings	Major
@rocket.chat/uikit-playground	Patch
@rocket.chat/api-client	Patch
@rocket.chat/apps	Patch
@rocket.chat/core-services	Patch
@rocket.chat/cron	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/freeswitch	Patch
@rocket.chat/fuselage-ui-kit	Major
@rocket.chat/gazzodown	Major
@rocket.chat/http-router	Patch
@rocket.chat/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/ui-avatar	Major
@rocket.chat/ui-client	Major
@rocket.chat/ui-contexts	Major
@rocket.chat/web-ui-registration	Major
@rocket.chat/account-service	Patch
@rocket.chat/authorization-service	Patch
@rocket.chat/ddp-streamer	Patch
@rocket.chat/omnichannel-transcript	Patch
@rocket.chat/presence-service	Patch
@rocket.chat/queue-worker	Patch
@rocket.chat/federation-matrix	Patch
@rocket.chat/license	Patch
@rocket.chat/media-calls	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/presence	Patch
rocketchat-services	Patch
@rocket.chat/models	Patch
@rocket.chat/network-broker	Patch
@rocket.chat/omni-core-ee	Patch
@rocket.chat/mock-providers	Patch
@rocket.chat/ui-video-conf	Major
@rocket.chat/ui-voip	Major
@rocket.chat/instance-status	Patch
@rocket.chat/omni-core	Patch

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

[coderabbitai]

Walkthrough

A deprecated Livechat server-side Meteor method livechat:sendTranscript is removed alongside its implementation, rate limiter configuration, import references, and associated test suite, marking a major version bump for the module.

Changes

Cohort / File(s)	Summary
Changeset Documentation .changeset/strong-bats-swim.md	New changeset documenting major version bump for @rocket.chat/meteor and removal of deprecated livechat:sendTranscript method
Server Module Cleanup apps/meteor/app/livechat/server/index.ts	Removed import of ./methods/sendTranscript module
Method Implementation Removal apps/meteor/app/livechat/server/methods/sendTranscript.ts	Deleted entire server-side method implementation including Meteor.methods registration, input validation, permission checks, user/room document retrieval, MAC limit verification, rate limiter configuration, and DDP type augmentation
Test Suite Removal apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts	Removed livechat:sendTranscript test suite block with permission and scenario checks; removed unused imports (methodCall, parseMethodResponse)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• All changes are straightforward removals of deprecated functionality with no behavioral complexity or new logic to evaluate
• Homogeneous nature of changes (all deletions) across a small set of files minimizes cognitive load

Possibly related PRs

• chore!: removes livechat:removeAllClosedRooms deprecated method #36931: Removes deprecated Livechat server-side Meteor methods by deleting method implementation files and their imports
• chore: Adds deprecation warning on livechat:sendTranscript #36986: Related transcript flow modernization that adds deprecation warnings and switches clients to REST /v1/livechat/transcript endpoint
• chore!: removes livechat:getRoutingConfig deprecated method #36924: Removes other deprecated Livechat server methods using the same pattern (method file deletion, DDP augmentation removal, import cleanup)

Suggested labels

stat: ready to merge, stat: QA assured

Suggested reviewers

• ggazzo
• dougfabris

Poem

🐰 A method once lived, now fades away,
Deprecated and ready for its final day,
Tests removed, imports gone with grace,
The sendTranscript finds its resting place. ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: removal of the deprecated livechat:sendTranscript method, which aligns with all file changes in the changeset.
Linked Issues check	✅ Passed	The PR removes the livechat:sendTranscript method implementation, associated tests, and imports as required by ARCH-1862.
Out of Scope Changes check	✅ Passed	All changes are directly related to removing the livechat:sendTranscript method and its dependencies; no unrelated modifications detected.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/remove-sendTranscript

---

📜 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 589f2a1 and 8d790c7.

📒 Files selected for processing (4)

• .changeset/strong-bats-swim.md (1 hunks)
• apps/meteor/app/livechat/server/index.ts (0 hunks)
• apps/meteor/app/livechat/server/methods/sendTranscript.ts (0 hunks)
• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts (2 hunks)

💤 Files with no reviewable changes (2)

• apps/meteor/app/livechat/server/index.ts
• apps/meteor/app/livechat/server/methods/sendTranscript.ts

🧰 Additional context used

🧠 Learnings (12)

📓 Common learnings

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.

📚 Learning: 2025-09-19T15:15:04.642Z

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.

Applied to files:

• .changeset/strong-bats-swim.md

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.{ts,tsx,js,jsx} : Avoid code comments in the implementation

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Use descriptive test names that clearly communicate expected behavior

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Ensure tests run reliably in parallel without shared state conflicts

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Maintain test isolation between test cases

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Group related tests in the same file

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Ensure a clean state for each test execution

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.{ts,tsx,js,jsx} : Follow DRY by extracting reusable logic into helper functions or page objects

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.{ts,tsx,js,jsx} : Write concise, technical TypeScript/JavaScript with accurate typing

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/page-objects/**/*.ts : Utilize and place Page Object implementations under apps/meteor/tests/e2e/page-objects/

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

📚 Learning: 2025-09-16T22:08:51.490Z

Learnt from: CR
Repo: RocketChat/Rocket.Chat PR: 0
File: .cursor/rules/playwright.mdc:0-0
Timestamp: 2025-09-16T22:08:51.490Z
Learning: Applies to apps/meteor/tests/e2e/**/*.spec.ts : Use test.beforeAll() and test.afterAll() for setup and teardown

Applied to files:

• apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: CodeQL-Build
• GitHub Check: CodeQL-Build

🔇 Additional comments (4)

apps/meteor/tests/end-to-end/api/livechat/00-rooms.ts (3)

22-22: LGTM: Import cleanup aligned with test removal.

The removal of methodCall from the imports is appropriate since the tests that used this utility were removed along with the deprecated livechat:sendTranscript method.

---

55-55: LGTM: Import cleanup properly maintained.

The removal of parseMethodResponse is correct since it was only needed for the removed livechat:sendTranscript tests. The sleep utility is retained and continues to be used throughout the test suite.

---

22-55: Removal verification complete—no further action needed.

The deprecated method livechat:sendTranscript has been fully removed with no remaining references in the codebase. The parseMethodResponse utility is not used elsewhere in the livechat tests. The import cleanups are correct and the code is in a stable state.

.changeset/strong-bats-swim.md (1)

1-5: Changeset correctly documents the breaking change removal.

The livechat:sendTranscript method was deprecated in v7.11.0 and scheduled for removal in v8.0.0, with migration to the REST endpoint POST /api/v1/livechat/transcript. The changeset properly marks this as a major version bump, and the deprecation was communicated through official channels with documented migration guidance available. No further action needed.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.94%. Comparing base (589f2a1) to head (8d790c7).
⚠️ Report is 2 commits behind head on release-8.0.0.

Additional details and impacted files

@@              Coverage Diff               @@
##           release-8.0.0   #37396   +/-   ##
==============================================
  Coverage          70.93%   70.94%           
==============================================
  Files               3035     3035           
  Lines             104621   104621           
  Branches           18424    18422    -2     
==============================================
+ Hits               74217    74221    +4     
+ Misses             28460    28455    -5     
- Partials            1944     1945    +1     

Flag	Coverage Δ
e2e	58.05% <ø> (-0.01%)	⬇️
unit	72.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 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.