From 84b4c4e2a76ef438c8c7d32ce7958f85c621a862 Mon Sep 17 00:00:00 2001
From: Matthieu Riegler <kyro38@gmail.com>
Date: Tue, 24 Oct 2023 15:57:28 +0200
Subject: [PATCH] refactor(core): remove signal mutate implementation (#52348)

It's not used anymore.

PR Close #52348
---
 goldens/public-api/core/primitives/signals/index.md | 3 ---
 packages/core/primitives/signals/index.ts           | 2 +-
 packages/core/primitives/signals/src/signal.ts      | 9 ---------
 packages/core/src/render3/reactivity/signal.ts      | 2 +-
 4 files changed, 2 insertions(+), 14 deletions(-)

diff --git a/goldens/public-api/core/primitives/signals/index.md b/goldens/public-api/core/primitives/signals/index.md
index 1bc7a7bcb9430..5fe48ba12c547 100644
--- a/goldens/public-api/core/primitives/signals/index.md
+++ b/goldens/public-api/core/primitives/signals/index.md
@@ -104,9 +104,6 @@ export interface SignalGetter<T> extends SignalBaseGetter<T> {
     readonly [SIGNAL]: SignalNode<T>;
 }
 
-// @public (undocumented)
-export function signalMutateFn<T>(node: SignalNode<T>, mutator: (value: T) => void): void;
-
 // @public (undocumented)
 export interface SignalNode<T> extends ReactiveNode {
     // (undocumented)
diff --git a/packages/core/primitives/signals/index.ts b/packages/core/primitives/signals/index.ts
index 1d7917ec12581..b12a60d179aef 100644
--- a/packages/core/primitives/signals/index.ts
+++ b/packages/core/primitives/signals/index.ts
@@ -10,6 +10,6 @@ export {createComputed} from './src/computed';
 export {defaultEquals, ValueEqualityFn} from './src/equality';
 export {setThrowInvalidWriteToSignalError} from './src/errors';
 export {consumerAfterComputation, consumerBeforeComputation, consumerDestroy, consumerPollProducersForChange, getActiveConsumer, isInNotificationPhase, isReactive, producerAccessed, producerNotifyConsumers, producerUpdatesAllowed, producerUpdateValueVersion, Reactive, REACTIVE_NODE, ReactiveNode, setActiveConsumer, SIGNAL} from './src/graph';
-export {createSignal, setPostSignalSetFn, SIGNAL_NODE, SignalGetter, signalMutateFn, SignalNode, signalSetFn, signalUpdateFn} from './src/signal';
+export {createSignal, setPostSignalSetFn, SIGNAL_NODE, SignalGetter, SignalNode, signalSetFn, signalUpdateFn} from './src/signal';
 export {createWatch, Watch, WatchCleanupFn, WatchCleanupRegisterFn} from './src/watch';
 export {setAlternateWeakRefImpl} from './src/weak_ref';
diff --git a/packages/core/primitives/signals/src/signal.ts b/packages/core/primitives/signals/src/signal.ts
index 08c5f6577e447..e30c3e6ce01a8 100644
--- a/packages/core/primitives/signals/src/signal.ts
+++ b/packages/core/primitives/signals/src/signal.ts
@@ -79,15 +79,6 @@ export function signalUpdateFn<T>(node: SignalNode<T>, updater: (value: T) => T)
   signalSetFn(node, updater(node.value));
 }
 
-export function signalMutateFn<T>(node: SignalNode<T>, mutator: (value: T) => void): void {
-  if (!producerUpdatesAllowed()) {
-    throwInvalidWriteToSignalError();
-  }
-  // Mutate bypasses equality checks as it's by definition changing the value.
-  mutator(node.value);
-  signalValueChanged(node);
-}
-
 // Note: Using an IIFE here to ensure that the spread assignment is not considered
 // a side-effect, ending up preserving `COMPUTED_NODE` and `REACTIVE_NODE`.
 // TODO: remove when https://github.com/evanw/esbuild/issues/3392 is resolved.
diff --git a/packages/core/src/render3/reactivity/signal.ts b/packages/core/src/render3/reactivity/signal.ts
index 8353afb1e344d..1d4f43df32515 100644
--- a/packages/core/src/render3/reactivity/signal.ts
+++ b/packages/core/src/render3/reactivity/signal.ts
@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {createSignal, SIGNAL, SignalGetter, signalMutateFn, SignalNode, signalSetFn, signalUpdateFn} from '@angular/core/primitives/signals';
+import {createSignal, SIGNAL, SignalGetter, SignalNode, signalSetFn, signalUpdateFn} from '@angular/core/primitives/signals';
 
 import {Signal, ValueEqualityFn} from './api';