[rate-limit-controller] Fix and improve typing, remove `any` usage#3963

Merged

MajorLift merged 16 commits into

mainfrom

240222-rate-limit-controller-fix-any

Mar 7, 2024

Commits on Mar 7, 2024

Rename types to align with conventions followed by other controllers
MajorLift
committed
Define and apply types `RateLimitedApiMap`, `RateLimitedRequests`, add jsdoc
MajorLift
committed
Fix wildcard export by enumerating package-level exports
MajorLift
committed
Fix three counts of `any` usage
MajorLift
committed
Fix typing: `args` should be queried with the specific `type` passed in, not all possible api types stored in state. `keyof RateLimitedApis` is only the constraint for the `type` param, not its type.
MajorLift
committed
Narrow type of `acc`, `key` in this reduce operation
MajorLift
committed
Apply suggestions for jsdoc

MajorLift
and
mcmire
committed
Prefer `getKnownPropertyNames` over index signature assertion
MajorLift
committed
Access state value from outside of `update` callback
MajorLift
committed
Add `@metamask/utils` as dependency
MajorLift
committed
Do not return `Object.assign` in `update` callback, so it's clear that state is being mutated
MajorLift
committed
Merge branch 'main' into 240222-rate-limit-controller-fix-any
MajorLift
authored
Merge branch 'main' into 240222-rate-limit-controller-fix-any
MajorLift
authored
Merge branch 'main' into 240222-rate-limit-controller-fix-any
MajorLift
authored
Merge branch 'main' into 240222-rate-limit-controller-fix-any
MajorLift
authored
Merge branch 'main' into 240222-rate-limit-controller-fix-any
MajorLift
authored