Use unknown rather than any for BaseController state

.eslintrc.js

@@ -25,6 +25,9 @@ module.exports = {
     'no-shadow': 'off',
     '@typescript-eslint/no-shadow': 'error',
     '@typescript-eslint/indent': 'off',
+    // disabled due to incompatibility with Record<string, unknown>
+    // See https://github.com/Microsoft/TypeScript/issues/15300#issuecomment-702872440
+    '@typescript-eslint/consistent-type-definitions': 'off',
 
     // TODO re-enable most of these rules
     '@typescript-eslint/no-non-null-assertion': 'off',

src/BaseControllerV2.test.ts

@@ -3,9 +3,9 @@ import * as sinon from 'sinon';
 
 import { BaseController } from './BaseControllerV2';
 
-interface MockControllerState {
+type MockControllerState = {
   count: number;
-}
+};
 
 class MockController extends BaseController<MockControllerState> {
   update(callback: (state: Draft<MockControllerState>) => void | MockControllerState) {

src/BaseControllerV2.ts

@@ -14,7 +14,7 @@ export type Listener<T> = (state: T, patches: Patch[]) => void;
 /**
  * Controller class that provides state management and subscriptions
  */
-export class BaseController<S extends Record<string, any>> {
+export class BaseController<S extends Record<string, unknown>> {
   private internalState: S;
 
   private internalListeners: Set<Listener<S>> = new Set();