feat(uikit): add room to some other interaction contexts

[Copilot]

• Fix modal UIKit interactions to send rid to server (ActionManager approach)
• Add UIKit room context test app
  • Test app with slash command supporting three surfaces: modal, ctx (contextual bar), and message
  • Implements executeBlockActionHandler, executeViewSubmitHandler, executeViewClosedHandler
  • Logs room, user, triggerId, actionId, and container type from each handler to enable verification
• Add Playwright e2e tests for UIKit interaction data (uikit-interactions.spec.ts)
  • Add getAppLogs helper and update installLocalTestPackage to return app data in insert-apps.ts
  • Test executeBlockActionHandler from message, contextual bar, and modal surfaces
  • Test executeViewSubmitHandler from modal and contextual bar surfaces
  • Test executeViewClosedHandler from modal and contextual bar surfaces
  • Each test validates room, user, triggerId, actionId, and container data via app logs
• Document uikit-room-test_0.0.1.zip in app-packages/README.md with full TypeScript source

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

[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 missing the required milestone or project

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

🦋 Changeset detected

Latest commit: 4c13525

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

This PR includes changesets to release 42 packages

Name	Type
@rocket.chat/ui-kit	Minor
@rocket.chat/meteor	Minor
@rocket.chat/core-services	Patch
@rocket.chat/core-typings	Minor
@rocket.chat/fuselage-ui-kit	Major
@rocket.chat/livechat	Patch
@rocket.chat/rest-typings	Minor
@rocket.chat/ui-voip	Major
rocketchat-services	Patch
@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/abac	Patch
@rocket.chat/federation-matrix	Patch
@rocket.chat/network-broker	Patch
@rocket.chat/omni-core-ee	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/presence	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/api-client	Patch
@rocket.chat/apps	Patch
@rocket.chat/cron	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/gazzodown	Major
@rocket.chat/http-router	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/license	Patch
@rocket.chat/media-calls	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/models	Patch
@rocket.chat/mock-providers	Patch
@rocket.chat/ui-video-conf	Major
@rocket.chat/instance-status	Patch
@rocket.chat/omni-core	Patch
@rocket.chat/server-fetch	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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.60%. Comparing base (1dbfce4) to head (4c13525).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #39719      +/-   ##
===========================================
- Coverage    70.61%   70.60%   -0.02%     
===========================================
  Files         3256     3256              
  Lines       115778   115777       -1     
  Branches     21042    21059      +17     
===========================================
- Hits         81756    81743      -13     
- Misses       31965    31976      +11     
- Partials      2057     2058       +1     

Flag	Coverage Δ
unit	71.12% <ø> (-0.02%)	⬇️

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.

[d-gubert]

@copilot This solution doesn't work because the UiKitModal component is not rendered inside the component tree that provides the RoomContext; so it will always be undefined.

We need to find a way for the ActionManager to access the rid from a context, and pass it over on line 260

[Copilot]

@copilot This solution doesn't work because the UiKitModal component is not rendered inside the component tree that provides the RoomContext; so it will always be undefined.

We need to find a way for...

You're right. I've fixed this by tracking rid in the ActionManager itself:

1. Changed appIdByTriggerId to triggerIdInfo to store both appId and rid per trigger
2. When emitInteraction is called with an interaction that has rid, it's stored with the trigger
3. When the server responds with modal.open, the rid is passed to openModal and then to UiKitModal as a prop
4. Added getRidByViewId method to support modal-to-modal rid inheritance

Commit: 23cdcc8

[Copilot]

@copilot fix the issue on CI https://github.com/RocketChat/Rocket.Chat/actions/runs/23447872808/job/68219474625?pr=39719

Fixed in fbf30d1. The CI failure was a Prettier lint error in uikit-room-context.ts at line 44 — two &&-chained conditions were split across lines but Prettier required them on a single line.

[d-gubert]

@copilot the test at apps/meteor/tests/end-to-end/apps/uikit-room-context.ts is not applicable - manually sending interactions defeats the purpose for the test. We'll actually need an e2e, playwright test.

I've included a helper function for installing apps in apps/meteor/tests/e2e/fixtures/insert-apps.ts, and created the scaffold for the playwright test here apps/meteor/tests/e2e/apps/uikit-interactions.spec.ts

You have to complete each test case with their appropriate validation. Include any extra test case if you deem necessary.

You may need to add helpers to apps/meteor/tests/e2e while working on the tests, for instance to fetch apps' logs. Follow patterns in apps/meteor/tests/e2e for that.

The app you've built does not provide all the functionality needed for all test cases. Review the app and add things accordingly. Refer back to my example app here for reference.

[Copilot]

@copilot the test at apps/meteor/tests/end-to-end/apps/uikit-room-context.ts is not applicable - manually sending interactions defeats the purpose for the test. We'll actually need an ...

Implemented in a9d6682 and b8fa0f8.

Changes made:

• Enhanced test app (uikit-room-test_0.0.1.zip): Now supports all three surfaces via /open-uikit-room-test-modal <modal|ctx|message>, each handler logs room, user, triggerId, actionId, and container type
• Added helpers in insert-apps.ts: installLocalTestPackage now returns { app: { id } }, added getAppLogs(appId) for fetching app logs
• Completed all 7 Playwright tests validating correct data in executeBlockActionHandler (message, contextual bar, modal), executeViewSubmitHandler (modal, contextual bar), and executeViewClosedHandler (modal, contextual bar)
• Removed old API-based uikit-room-context.ts test
• Updated README with the new app source code

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://api.github.com/repos/RocketChat/Rocket.Chat/issues/comments/4111646611
  • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[d-gubert]

@copilot https://github.com/RocketChat/Rocket.Chat/actions/runs/23468073147/job/68285933076?pr=39719 CI issue involving your changes

This one is unrelated, don't worry about it

[Copilot]

@copilot https://github.com/RocketChat/Rocket.Chat/actions/runs/23468073147/job/68285933076?pr=39719 CI issue involving your changes

[This one](https://github.com/RocketChat/Rocke...

Fixed in 56e9106.

The failure was caused by a race condition: when openSurfaceView(CONTEXTUAL_BAR) is called via the Apps Engine, Rocket.Chat performs a client-side URL navigation (to /channel/general/app/{viewId}). When locator.waitFor() was called while this navigation was still in progress, Playwright threw "Target page, context or browser has been closed".

The fix adds await page.waitForURL(/\/app\//) immediately after the slash command dispatch for all three contextual bar tests. This waits for the navigation to complete before any locator operations begin.

[d-gubert]

closing in favor of #39858