Maintenance: Remove overrides from Parameters' BaseProvider

[dreamorosi]

Summary

The implementation for the upcoming Parameters utility's BaseProvider contains some method overrides as well as some aliased method in the children.

i.e. BaseProvider.ts

class BaseProvider {
  public async get(name: string, options?: SecretsGetOptionsInterface): Promise<undefined | string | Uint8Array | Record<string, unknown>>;
  public async get(name: string, options?: GetOptionsInterface): Promise<undefined | string | Uint8Array | Record<string, unknown>> {
    // ... method implementation
  }
}

i.e. SecretsProvider.ts

class SecretsProvider extends BaseProvider {
  public constructor() {
    super();
  }
  
  public async get(name: string, options?: SecretsGetOptionsInterface): Promise<undefined | string | Uint8Array | Record<string, unknown>> {
    super.get(name, options); 
  }
}

The alias in the child class (i.e. SecretsProvider) is needed so that, once build and exported, customers get the correct typings:

This however, causes type issues when the same parent class (BaseProvider) is extended by multiple child classes.

Why is this needed?

We need to remove the overrides and leave to the children classes the responsibility of giving the correct types.

Which area does this relate to?

Parameters

Solution

No response

Acknowledgment

• This request meets Lambda Powertools Tenets
• Should this be considered in other Lambda Powertools languages? i.e. Python, Java

[github-actions]

⚠️ COMMENT VISIBILITY WARNING ⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.