fix: First response time livechat metrics are associated with the last agent who served the room

[matheusbsilva137]

Proposed changes (including videos or screenshots)

• Fixed rooms' "first response time" metric being attached to the last agent who served a room instead of the first one -- this leads to incorrect values for the "Best first response time" and "Average first response time" metrics for each agent in case rooms are forwarded among agents/departments.

Issue(s)

Steps to test or reproduce

Make sure to grant the transfer-livechat-guest permission to agents or to a manager.
Create a new agent and respond a livechat room with it, then transfer the room to another one. The "first response time" aggregation metrics are available under Omnichannel > Analytics > Productivity (or by using the livechat/analytics/agent-overview endpoint), and we need to make sure that both metrics related to the room that has been transferred are associated to the first agent who responded to the room (and not to the last one who served it).

Further comments

CORE-669

[dionisio-bot]

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

[changeset-bot]

🦋 Changeset detected

Latest commit: 5fae225

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	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/apps	Patch
@rocket.chat/models	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/stream-hub-service	Patch
@rocket.chat/omnichannel-services	Patch
rocketchat-services	Patch
@rocket.chat/core-services	Patch
@rocket.chat/cron	Patch
@rocket.chat/instance-status	Patch
@rocket.chat/presence	Patch
@rocket.chat/network-broker	Patch
@rocket.chat/core-typings	Patch
@rocket.chat/rest-typings	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/api-client	Patch
@rocket.chat/ddp-client	Patch
@rocket.chat/freeswitch	Patch
@rocket.chat/fuselage-ui-kit	Patch
@rocket.chat/gazzodown	Patch
@rocket.chat/livechat	Patch
@rocket.chat/ui-contexts	Patch
@rocket.chat/license	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/ui-avatar	Patch
@rocket.chat/ui-client	Patch
@rocket.chat/ui-video-conf	Patch
@rocket.chat/ui-voip	Patch
@rocket.chat/web-ui-registration	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

[github-actions]

PR Preview Action v1.4.8
🚀 Deployed preview to https://RocketChat.github.io/Rocket.Chat/pr-preview/pr-34156/
on branch gh-pages at 2024-12-16 11:32 UTC

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.79%. Comparing base (b32c629) to head (3c10350).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #34156   +/-   ##
========================================
  Coverage    75.79%   75.79%           
========================================
  Files          511      511           
  Lines        22132    22132           
  Branches      5394     5394           
========================================
  Hits         16775    16775           
  Misses        4709     4709           
  Partials       648      648           

Flag	Coverage Δ
unit	75.79% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[KevLehman]

Can u add unit tests cases for your fix?
if possible, similar tests to the API tests ones 👀