chore(maintenance): migrate metrics utility to biome

[daschaa]

Summary

Changes

Closes #2799

Issue number: closes #2799

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[dreamorosi]

No related issues found. Please ensure there is an open issue related to this change to avoid significant delays or closure.

Sorry about this @daschaa - we need to improve our automation (I'm working on that on the side) to better detect linked issues.

In the meantime, as a workaround - if you want to avoid the bot nagging you - you could edit/write the PR body with the issue number like here (I edited your comment to fix it) 😁

[daschaa]

@dreamorosi I think with this regex the automation might work (at least i get good matches):

\b\*{0,2}[iI]ssue [nN]umber\*{0,2}:\*{0,2}\s*(?<closingWord>closes?|closed|fix|fixes?|fixed|resolves?|resolved)*?\s*#*(?<issue>\d+)\b

1. I added a * to the closing word
2. I added a * to the hashtag before the issue number

[dreamorosi]

Thank you for the work on this.

I have left only one other comment, but seems we are going in the right direction.

We'll have to do something about all the // biome-ignore lint/complexity/useLiteralKeys but since it'll likely require a refactor of either the tests or the utility we will address this separately (same for Logger).

[dreamorosi]

@daschaa - I just saw there are two Sonar findings.

They are not areas you've touched, but the way the tool works is that it runs on files that were edited as part of the PR, and scans them. So that's why they're coming up now.

[daschaa]

@daschaa - I just saw there are two Sonar findings.

I'll check them :) Sorry that I did not catch them earlier. I guess the checkmark icon in the sonar report just let's my brain ignore the report. 😄

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[dreamorosi]

Thank you so much for the work on this PR!

We're going to merge this for now, despite the addition of lint/complexity/useLiteralKeys in the unit tests. We have another conversation ongoing in #2821 to make changes that we hope will allow us to remove most/all of the suppressions (thank you for opening the issue 😉).