chore(meteor-methods)!: remove deprecated canAccessRoom method

[blackmamba1231]

Proposed changes

This PR removes the deprecated canAccessRoom method that was scheduled for removal after v5.0 but is still present in v7.4.0. The method was only enabled when the environment variable ALLOW_CANACCESSROOM_METHOD was set to "yes" or "true".

Changes made:

• Removed the canAccessRoom.ts file
• Updated the index.ts file to remove the import for the canAccessRoom method

Issue(s)

Fixes #35432

Steps to test or reproduce

1. Verify that the canAccessRoom method is no longer available
2. Ensure that applications using the async alternative canAccessRoomAsync continue to work correctly
3. No functionality should be affected as this method was already deprecated and only available when explicitly enabled via environment variable

Further comments

This change is part of the ongoing effort to clean up deprecated code in the Rocket.Chat codebase. The method was marked for removal after v5.0 and we're now at v7.4.0, so it's time to complete the deprecation process.

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is missing the 'stat: QA assured' label
• This PR is targeting the wrong base branch. It should target 8.0.0, but it targets 7.5.0

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

🦋 Changeset detected

Latest commit: 8c9d28f

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

This PR includes changesets to release 35 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/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/ui-contexts	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/stream-hub-service	Patch
@rocket.chat/license	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/ui-avatar	Major
@rocket.chat/ui-client	Major
@rocket.chat/ui-video-conf	Major
@rocket.chat/ui-voip	Major
@rocket.chat/web-ui-registration	Major
@rocket.chat/instance-status	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

[Gustrb]

Even though this was already scheduled for removal I don't see why removing it in a minor version. We should probably wait for 8.0 so we can remove it without the concern of introducing breaking changes

[blackmamba1231]

I appreciate your perspective on following semantic versioning principles. You make a valid point about avoiding breaking changes in minor versions.

While the method was marked for removal after v5.0 and is currently only enabled via an environment variable, I understand the concern about potential disruption to users who might still be relying on it.

I'm happy to adjust the PR to target v8.0 instead. Would you like me to update the PR description to reflect this timing, or would you prefer to handle this differently? I'll follow the team's guidance on the appropriate approach for handling this deprecated functionality.