chore(maintenance): migrate logger utility to biome

[daschaa]

Summary

Changes

Adds biome to the logger package. Closes #2798

Issue number: 2798

---

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]

Thank you for working on this!

I've just left one comment and then it's good to merge :)

[dreamorosi]

@daschaa - Could you also take a look at the SonarCloud finding here?

I'll check your other two comments in the meanwhile - need to clone the repo to answer definitively.

[dreamorosi]

When you can, could you also pls leave a comment under the linked issue (& all others you're working or want to pick up) so that I can assign them to you? Thanks!

[daschaa]

@dreamorosi I'll check the sonarcloud issue and comment on the issues i'd like to pick :) Thank you for your support ❤️ Really awesome opportunity for me to learn about biome and to (hopefully) help you a bit out here :)

[dreamorosi]

That's great - yes, like I said in the other issue any help is more than welcome!

It seems that I made a mistake and that I was too hasty in my review. I saw a couple more instances where the comment block at the top of the test files got shifted down due to import sorting from Biome (the same is happening in other PRs I've worked).

I think it'd be helpful if you could go through all the packages/logger/tests/**/*.test.ts files and make sure that comments like this are at the very top:

/**
 * Test logger basic features
 *
 * @group e2e/logger/logEventEnvVarSetting
 */

Thanks!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

[daschaa]

@dreamorosi Thanks for the review!

• I have put the grouping comment to the top of each test file (btw I did not know about this feature in jest, really nice)
• I have removed the suppression of the variable override and introduced the local variable like you suggested
• I have fixed the SonarCloud finding by changing from a traditional for loop to a for...of loop

[dreamorosi]

Thank you for addressing all the feedback and for helping improve the project!

[github-actions]

@aws-powertools/lambda-typescript No related issues found. Please ensure 'pending-release' label is applied before releasing.