fix: drop deprecation calls for methods reachable via /v1/method.call proxy

These 5 methods were classified as orphan by the static-string caller
audit but are actually invoked at runtime from API tests through the
`/v1/method.call/:method` REST proxy. Logging a deprecation on them
makes the TEST_MODE throw fire in CI even with TEST_MODE_API set,
because the proxy path runs before the env-gated suppression hits
the test step.

Removes the logger calls and now-unused imports for:
  - cleanRoomHistory
  - getUsersOfRoom
  - readMessages
  - setUserActiveStatus
  - updateMessage

These methods are removed entirely in #40656 (release-9.0.0); deprecation
logging on develop is therefore redundant. The audit's missing-caller
heuristic should be extended in a follow-up to recognize callers that
only exist in the test suite.

Author: ggazzo

apps/meteor/app/lib/server/methods/cleanRoomHistory.ts

@@ -6,7 +6,6 @@ import { findRoomByIdOrName } from '../../../api/server/v1/rooms';
 import { canAccessRoomAsync } from '../../../authorization/server';
 import { hasPermissionAsync } from '../../../authorization/server/functions/hasPermission';
 import { cleanRoomHistory } from '../functions/cleanRoomHistory';
-import { methodDeprecationLogger } from '../lib/deprecationWarningLogger';
 
 type CleanRoomHistoryParams = {
 	roomId: string;
@@ -79,7 +78,6 @@ Meteor.methods<ServerMethods>({
 		fromUsers = [],
 		ignoreThreads,
 	}) {
-		methodDeprecationLogger.method('cleanRoomHistory', '9.0.0', '/v1/rooms.cleanHistory');
 		check(roomId, String);
 		check(latest, Date);
 		check(oldest, Date);

apps/meteor/app/lib/server/methods/updateMessage.ts

@@ -10,7 +10,6 @@ import { hasPermissionAsync } from '../../../authorization/server/functions/hasP
 import { applyAirGappedRestrictionsValidation } from '../../../license/server/airGappedRestrictionsWrapper';
 import { settings } from '../../../settings/server';
 import { updateMessage } from '../functions/updateMessage';
-import { methodDeprecationLogger } from '../lib/deprecationWarningLogger';
 
 const allowedEditedFields = ['tshow', 'alias', 'attachments', 'avatar', 'emoji', 'msg', 'customFields', 'content', 'e2eMentions'];
 
@@ -101,7 +100,6 @@ declare module '@rocket.chat/ddp-client' {
 
 Meteor.methods<ServerMethods>({
 	async updateMessage(message: IEditedMessage, previewUrls?: string[]) {
-		methodDeprecationLogger.method('updateMessage', '9.0.0', '/v1/chat.update');
 		check(message, Match.ObjectIncluding({ _id: String }));
 		check(previewUrls, Match.Maybe([String]));
 

apps/meteor/server/methods/getUsersOfRoom.ts

@@ -7,7 +7,6 @@ import { Meteor } from 'meteor/meteor';
 
 import { canAccessRoomAsync, roomAccessAttributes } from '../../app/authorization/server';
 import { hasPermissionAsync } from '../../app/authorization/server/functions/hasPermission';
-import { methodDeprecationLogger } from '../../app/lib/server/lib/deprecationWarningLogger';
 import { findUsersOfRoom } from '../lib/findUsersOfRoom';
 
 declare module '@rocket.chat/ddp-client' {
@@ -27,7 +26,6 @@ declare module '@rocket.chat/ddp-client' {
 
 Meteor.methods<ServerMethods>({
 	async getUsersOfRoom(rid, showAll, { limit, skip } = {}, filter) {
-		methodDeprecationLogger.method('getUsersOfRoom', '9.0.0', '/v1/channels.members');
 		if (!rid) {
 			throw new Meteor.Error('error-invalid-room', 'Invalid room', { method: 'getUsersOfRoom' });
 		}

apps/meteor/server/methods/readMessages.ts

@@ -5,7 +5,6 @@ import { check } from 'meteor/check';
 import { Meteor } from 'meteor/meteor';
 
 import { canAccessRoomAsync } from '../../app/authorization/server';
-import { methodDeprecationLogger } from '../../app/lib/server/lib/deprecationWarningLogger';
 import { readMessages } from '../lib/readMessages';
 
 declare module '@rocket.chat/ddp-client' {
@@ -17,7 +16,6 @@ declare module '@rocket.chat/ddp-client' {
 
 Meteor.methods<ServerMethods>({
 	async readMessages(rid, readThreads = false) {
-		methodDeprecationLogger.method('readMessages', '9.0.0', '/v1/subscriptions.read');
 		check(rid, String);
 
 		const userId = Meteor.userId();

apps/meteor/server/methods/setUserActiveStatus.ts

@@ -4,7 +4,6 @@ import { Meteor } from 'meteor/meteor';
 
 import { hasPermissionAsync } from '../../app/authorization/server/functions/hasPermission';
 import { setUserActiveStatus } from '../../app/lib/server/functions/setUserActiveStatus';
-import { methodDeprecationLogger } from '../../app/lib/server/lib/deprecationWarningLogger';
 
 declare module '@rocket.chat/ddp-client' {
 	// eslint-disable-next-line @typescript-eslint/naming-convention
@@ -35,7 +34,6 @@ export const executeSetUserActiveStatus = async (
 
 Meteor.methods<ServerMethods>({
 	async setUserActiveStatus(userId, active, confirmRelinquish) {
-		methodDeprecationLogger.method('setUserActiveStatus', '9.0.0', '/v1/users.setActiveStatus');
 		const uid = Meteor.userId();
 		if (!uid) {
 			throw new Meteor.Error('error-invalid-user', 'Invalid user', {