Description

This version of @metamask/base-controller does not introduce any functional changes, but there are a few developer experience improvements:

• The controller messenger is now more type-safe, in that a messenger with an empty action/event allowlist guarantees that no actions/events are allowed to be called/emitted, and conversely, that a non-empty allowlist only allows the given actions/events
• The "selector" version of subscribe (three arguments instead of two) is now fixed so that the arguments to the selector function are correctly typed.
• It is now impossible to mutate state directly; state must only be mutated inside of a controller using the update method. This ensures that state updates are always controlled and expected.

View changelog: https://github.com/MetaMask/core/blob/%40metamask/base-controller%405.0.1/packages/base-controller/CHANGELOG.md

Related issues

Progresses #24140.

Manual testing steps

(None needed as there should be no functional changes.)

Screenshots/Recordings

(None)

Pre-merge author checklist

• I’ve followed MetaMask Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.