fix: useController() hook triggers rerenders if it is used without de…

…pendencies.

packages/rx-effects-react/src/useController.test.ts

@@ -27,6 +27,25 @@ describe('useController()', () => {
     expect(destroy).toBeCalledTimes(1);
   });
 
+  it('should not recreate the controller with empty dependencies after rerendering', () => {
+    const destroy = jest.fn();
+
+    const createController = () => ({ destroy });
+
+    const { result, rerender, unmount } = renderHook(() =>
+      useController(createController),
+    );
+
+    const controller1 = result.current;
+    rerender();
+    const controller2 = result.current;
+
+    expect(controller1 === controller2).toBe(true);
+
+    unmount();
+    expect(destroy).toBeCalledTimes(1);
+  });
+
   it('should recreate the controller if a dependency is changed', () => {
     const destroy = jest.fn();
 

packages/rx-effects-react/src/useController.ts

@@ -4,6 +4,8 @@
 import { useEffect, useMemo } from 'react';
 import { Controller } from 'rx-effects';
 
+const EMPTY_DEPENDENCIES: unknown[] = [];
+
 /**
  * Creates an ad-hoc controller by the factory and destroys it on unmounting a
  * component.
@@ -16,7 +18,7 @@ import { Controller } from 'rx-effects';
  */
 export function useController<T extends Controller<Record<string, any>>>(
   factory: () => T,
-  dependencies?: unknown[],
+  dependencies: unknown[] = EMPTY_DEPENDENCIES,
 ): T {
   const controller = useMemo(factory, dependencies);
   useEffect(() => controller.destroy, [controller]);