feat(core): export signalGetFn from signal primitives (#60497)

mturco · thePunderWoman · commit a4bad8d361e8 · 2025-03-31T22:24:21.000Z
This updates `createSignal` to use `signalGetFn` to define the signal getter and exports `signalGetFn` from the shared signal primitives. PR Close #60497
diff --git a/goldens/public-api/core/primitives/signals/index.api.md b/goldens/public-api/core/primitives/signals/index.api.md
@@ -161,6 +161,9 @@ export const SIGNAL: unique symbol;
 // @public (undocumented)
 export const SIGNAL_NODE: SignalNode<unknown>;
 
+// @public (undocumented)
+export function signalGetFn<T>(node: SignalNode<T>): T;
+
 // @public (undocumented)
 export interface SignalGetter<T> extends SignalBaseGetter<T> {
     // (undocumented)
diff --git a/packages/core/primitives/signals/index.ts b/packages/core/primitives/signals/index.ts
@@ -48,6 +48,7 @@ export {
   createSignal,
   runPostSignalSetFn,
   setPostSignalSetFn,
+  signalGetFn,
   signalSetFn,
   signalUpdateFn,
 } from './src/signal';
diff --git a/packages/core/primitives/signals/src/signal.ts b/packages/core/primitives/signals/src/signal.ts
@@ -54,10 +54,7 @@ export function createSignal<T>(initialValue: T, equal?: ValueEqualityFn<T>): Si
   if (equal !== undefined) {
     node.equal = equal;
   }
-  const getter = (() => {
-    producerAccessed(node);
-    return node.value;
-  }) as SignalGetter<T>;
+  const getter = (() => signalGetFn(node)) as SignalGetter<T>;
   (getter as any)[SIGNAL] = node;
   if (typeof ngDevMode !== 'undefined' && ngDevMode) {
     const debugName = node.debugName ? ' (' + node.debugName + ')' : '';
@@ -75,9 +72,9 @@ export function setPostSignalSetFn(fn: ReactiveHookFn | null): ReactiveHookFn |
   return prev;
 }
 
-export function signalGetFn<T>(this: SignalNode<T>): T {
-  producerAccessed(this);
-  return this.value;
+export function signalGetFn<T>(node: SignalNode<T>): T {
+  producerAccessed(node);
+  return node.value;
 }
 
 export function signalSetFn<T>(node: SignalNode<T>, newValue: T) {
