Restore ComposableController compatibility between different BaseControllers

src/BaseControllerV2.ts

@@ -131,6 +131,14 @@ export class BaseController<
 
   public readonly metadata: StateMetadata<S>;
 
+  /**
+   * The existence of the `subscribe` property is how the ComposableController detects whether a
+   * controller extends the old BaseController or the new one. We set it to `never` here to ensure
+   * this property is never used for new BaseController-based controllers, to ensure the
+   * ComposableController never mistakes them for an older style controller.
+   */
+  public readonly subscribe: never;
+
   /**
    * Creates a BaseController instance.
    *

src/ComposableController.ts

@@ -62,8 +62,8 @@ export class ComposableController extends BaseController<never, any> {
     this.messagingSystem = messenger;
     this.controllers.forEach((controller) => {
       const { name } = controller;
-      if (controller instanceof BaseController) {
-        controller.subscribe((state) => {
+      if ((controller as BaseController<any, any>).subscribe !== undefined) {
+        (controller as BaseController<any, any>).subscribe((state) => {
           this.update({ [name]: state });
         });
       } else if (this.messagingSystem) {