Feature request: define more specific types for decorators

[dreamorosi]

Description of the feature request

As discussed internally, we need to define more specific types for the method decorator that currently uses the any type:

// TODO: Revisit type below & make it more specific
// eslint-disable-next-line  @typescript-eslint/no-explicit-any
type MethodDecorator = (target: any, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<any>) => any;

Related issues, RFCs

#107

[dreamorosi]

This issue requires further investigation and depending on the outcome, the current type might end up staying.

The new potential type should be able to accept any sync or async function. If this is not possible, we should document the reasoning here and remove the #TODO from the code.

If anyone is interested in picking this up, please leave a comment here, then discuss your findings and proposed changes before opening a PR.

[dVp007]

I would like to contribute and open a PR.

[dreamorosi]

Hi @dVp007 thank you for offering to help on this issue.

Before starting to work on a PR, could you please write a short proposal of the implementation/changes in this issue so that we can discuss it and agree on the changes?

Once we are aligned on the plan, you can open a PR.

[dreamorosi]

Closing this in favor of #1204

[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.