Possible richer ConnectResponse type in '@redux-devtools/extension'

[dai-shi]

Can ConnectResponse in '@redux-devtools/extension'

redux-devtools/packages/redux-devtools-extension/src/index.ts

Lines 187 to 190 in b82de74

	interface ConnectResponse {
	init: (state: unknown) => void;
	send: (action: Action<unknown>, state: unknown) => void;
	}

be compatible with or imported from

redux-devtools/extension/src/app/api/index.ts

Lines 522 to 536 in b82de74

	export interface ConnectResponse {
	init: <S, A extends Action<unknown>>(
	state: S,
	liftedData?: LiftedState<S, A, unknown>
	) => void;
	subscribe: <S, A extends Action<unknown>>(
	listener: (message: ListenerMessage<S, A>) => void
	) => (() => void) | undefined;
	unsubscribe: () => void;
	send: <S, A extends Action<unknown>>(
	action: A,
	state: LiftedState<S, A, unknown>
	) => void;
	error: (payload: string) => void;
	}

which seems more precise?

[Methuselah96]

Yes, good catch. Ideally we would share all the types, but I'm not sure how straightforward that will be. Feel free to open a PR to update them and/or share them. I'll try to get to around to it myself when I get a chance if you don't have time.