decentraland / comms-gatekeeper / 24415762924

85%

push

github

feat: presentation bot token and presenter management endpoints (#249)

* feat: add presentation bot token endpoint and local preview support

* feat: add presenter management endpoints (promote/demote/list)

Add REST endpoints for scene admins to manage presenters in cast rooms:
- POST /cast/presenter to promote a participant to presenter role
- DELETE /cast/presenter to demote a presenter back to watcher
- GET /cast/presenter to list current presenters in a room

Authorization checks ensure only scene admins can manage presenters.
Also sets canUpdateOwnMetadata=false for watchers to prevent metadata
impersonation.

* docs: add presentation-bot-token and presenter endpoints to OpenAPI spec

Document the new /cast/presentation-bot-token and /cast/presenter
endpoints with their request/response schemas.

* test: add integration tests for presenter handler and comms-scene presenter metadata

Add integration tests for:
- presenter-handler: GET/POST/DELETE with auth and error cases
- comms-scene-handler: presenter metadata for admins, non-admins, and
  fail-open on admin check failure

Also only pass skipAdminCheck to generateStreamLink when in preview
mode, keeping the call signature clean for production requests.

* refactor: introduce cast domain errors and remove handler try/catch

Replace UnauthorizedError usage in cast component with domain-specific
errors (InvalidStreamingKeyError, ExpiredStreamingKeyError, etc.).
Error-handler middleware now catches these directly, removing manual
try/catch from cast handlers. Also switches validatePresenterAdmin to
use sceneManager.isSceneOwnerOrAdmin for broader permission checks,
sets canUpdateOwnMetadata: false on streamer and bot tokens, and adds
local preview bypass for presenter admin validation.

* refactor: move ALLOW_LOCAL_PREVIEW check into livekit.isLocalPreview

The config flag is now checked inside the LiveKit component, removing
duplicate config reads from handlers. Also splits generateStreamLink
in... (continued)

1018 of 1337 branches covered (76.14%)

Branch coverage included in aggregate %.

214 of 242 new or added lines in 16 files covered. (88.43%)

2845 of 3285 relevant lines covered (86.61%)

41.77 hits per line