[composable-controller] Subscribe to `stateChange` events of V1 controllers with messenger, type fixes#3964

Merged

MajorLift merged 12 commits intomainfrom

240222-composable-controller-type-fixes

Mar 7, 2024

Commits on Mar 7, 2024

If a controller is V1 but it has a messaging system, subscribe to its `stateChange` event
MajorLift
committed
Remove `@metamask/utils` as a dependency
MajorLift
committed
Remove package-level exports for `BaseControllerV{1,2}Instance`, `ControllerInstance`, as these are unsuited for general use
MajorLift
committed
Remove noisy TODOs
MajorLift
committed
Rename `BaseControllerV2Instance` as `BaseControllerInstance` for consistency with `isBaseController` type guard
MajorLift
committed
For controller that could be V1 or V2, state type constraint is `Record<string, unknown>`
MajorLift
committed
Use `Object.assign` in `update` callbacks
MajorLift
committed
Use narrowest supertype of `BaseController` in type predicate for `isBaseController`
MajorLift
committed
Improve jsdoc for controller instance types
MajorLift
committed
Remove unnecessary or inaccurate generic arguments for `reduce`
MajorLift
committed
Adjust conditional branches to prevent redundant state updates in v1 controllers with messaging system

MajorLift
and
mcmire
committed
Merge branch 'main' into 240222-composable-controller-type-fixes
MajorLift
authored