fix(signals): run rxMethod outside of reactive context (#4224)

Co-authored-by: markostanimirovic <markostanimirovic95@gmail.com>

Author: rainerhahnekamp

modules/signals/rxjs-interop/spec/rx-method.spec.ts

@@ -218,4 +218,17 @@ describe('rxMethod', () => {
       /NG0203: rxMethod\(\) can only be used within an injection context/
     );
   });
+
+  it('allows signal updates', () => {
+    const counter = signal(1);
+    const increment = TestBed.runInInjectionContext(() =>
+      rxMethod<number>(tap((n) => counter.update((value) => value + n)))
+    );
+
+    const num = signal(3);
+    increment(num);
+
+    TestBed.flushEffects();
+    expect(counter()).toBe(4);
+  });
 });

modules/signals/rxjs-interop/src/rx-method.ts

@@ -6,6 +6,7 @@ import {
   Injector,
   isSignal,
   Signal,
+  untracked,
 } from '@angular/core';
 import { isObservable, noop, Observable, Subject, Unsubscribable } from 'rxjs';
 
@@ -31,7 +32,13 @@ export function rxMethod<Input>(
 
   const rxMethodFn = (input: RxMethodInput<Input>) => {
     if (isSignal(input)) {
-      const watcher = effect(() => source$.next(input()), { injector });
+      const watcher = effect(
+        () => {
+          const value = input();
+          untracked(() => source$.next(value));
+        },
+        { injector }
+      );
       const instanceSub = { unsubscribe: () => watcher.destroy() };
       sourceSub.add(instanceSub);