Summary

This pull request addresses and resolves all 9 SonarQube code quality issues identified in the packages/parameters package, as detailed in issue #4256. These improvements enhance maintainability, enforce immutability where intended, optimize imports, and clean up test code for clarity and reliability.

Changes

1. Immutability with readonly Members

• AppConfigProvider.ts
  • Marked application and environment as readonly (lines 192, 193), making their immutability explicit.
• BaseProvider.test.ts
  • Set the private class field #name as readonly (line 27).
• tinyLogger.ts
  • Marked the console property as readonly (line 16).

2. Import Optimization

• DynamoDBProvider.ts
  • Consolidated duplicate imports from @aws-sdk/client-dynamodb into a single import statement (lines 5, 11).

3. Union Type Improvement

• BaseProvider.ts
  • Refined the return type of get method to remove unnecessary undefined from the union (line 95), improving type safety and API clarity.

4. Test Cleanup

• SSMProvider.ts

  • Removed two unnecessary type assertions (lines 1015, 1235) to improve test code readability and reliability.

• SSMProvider.test.ts

  • Updated response field type from GetParametersCommandOutput to Partial (lines 824,923)

Issue number: closes #4256

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.