regression: move AppsPermissions definition back to @rocket.chat/apps-engine

Merged
D-
5mo ago
+168-171

Proposed changes (including videos or screenshots)

PR #40343 moved the AppsPermissions map from @rocket.chat/apps-engine to the internal "apps" package - this makes sense semantically, as the server should control what permissions are available.

However, the @rocket.chat/apps-compiler package uses the permission list from the @rocket.chat/apps-engine to validate the app's defined permissions during the packaging process, which means removing the list from the apps-engine would make this validation impossible.

So we're bringing the permissions list back to the apps-engine.

Issue(s)

ARCH-2129

Steps to test or reproduce

Further comments

This is related to https://github.com/RocketChat/Rocket.Chat.Apps-compiler/pull/71

Summary by CodeRabbit

  • New Features

    • Implemented a centralized application permissions management system with predefined permission sets to ensure consistent security policies and support backward compatibility with existing applications.

  • Refactor

    • Consolidated duplicate permission definitions across the platform by centralizing them, reducing code duplication and enhancing long-term maintainability.

Review Change Stack