fix: System messages are counted as agents' first responses in livechat rooms

[matheusbsilva137]

Proposed changes (including videos or screenshots)

• Stop considering system messages (such as "agent joined room") as agents' first responses in livechat rooms. This caused the "average first response time" and "best first respone time" metrics to be much lower than expected.

Issue(s)

Steps to test or reproduce

Add a new agent to the woskspace, then create a livechat conversation and make sure to serve it to the new agent. Then, go to Omnichannel > Analytics and set "Productivity" as the analytics type.

Current behavior: the "Best first response time" and "Average of first response time" metrics will always be lower than the actual time that the agent took to respond to the livechat room (eg possibly 1s). System messages (such as "agent joined room") are being interpreted as first responses, which causes these metrics to go much lower;
Expected behavior: RC should only consider a first response in a room when the agent sends a text message. System messages should be disconsidered in the "Best first response time" and "Average of first response time" metrics.

Further comments

CORE-562

[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: f7151c7

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

This PR includes changesets to release 32 packages

Name	Type
@rocket.chat/meteor	Patch
@rocket.chat/core-typings	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/fuselage-ui-kit	Patch
@rocket.chat/gazzodown	Patch
@rocket.chat/livechat	Patch
@rocket.chat/model-typings	Patch
@rocket.chat/rest-typings	Patch
@rocket.chat/ui-contexts	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/license	Patch
@rocket.chat/omnichannel-services	Patch
@rocket.chat/pdf-worker	Patch
@rocket.chat/presence	Patch
rocketchat-services	Patch
@rocket.chat/uikit-playground	Patch
@rocket.chat/models	Patch
@rocket.chat/ui-avatar	Patch
@rocket.chat/ui-client	Patch
@rocket.chat/ui-video-conf	Patch
@rocket.chat/web-ui-registration	Patch
@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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.43%. Comparing base (eb5e60e) to head (f7151c7).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #32846   +/-   ##
========================================
  Coverage    59.43%   59.43%           
========================================
  Files         2547     2547           
  Lines        63265    63265           
  Branches     14237    14250   +13     
========================================
  Hits         37604    37604           
  Misses       22941    22941           
  Partials      2720     2720           

Flag	Coverage Δ
unit	76.04% <ø> (ø)

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

[MarcosSpessatto]

There are some failing tests... Can you check those?

[matheusbsilva137]

Converting this PR to draft since tests will only pass after #33022 gets merged

Changes addressed