From 197a94c7e0e5cfbe3e69a6bffdc35a5125075bf0 Mon Sep 17 00:00:00 2001
From: Kristiyan Kostadinov <crisbeto@abv.bg>
Date: Sat, 15 Mar 2025 09:24:29 +0100
Subject: [PATCH] fix(core): include input name in error message

Includes either the `debugName` or alias of an input in the error message about a value not being available.

Fixes #60199.
---
 .../core/src/authoring/input/input_signal.ts  | 10 ++++---
 .../core/test/authoring/input_signal_spec.ts  | 28 +++++++++++++++++++
 2 files changed, 34 insertions(+), 4 deletions(-)

diff --git a/packages/core/src/authoring/input/input_signal.ts b/packages/core/src/authoring/input/input_signal.ts
index 379d36e99804..578c43b3dc73 100644
--- a/packages/core/src/authoring/input/input_signal.ts
+++ b/packages/core/src/authoring/input/input_signal.ts
@@ -127,10 +127,12 @@ export function createInputSignal<T, TransformT>(
     producerAccessed(node);
 
     if (node.value === REQUIRED_UNSET_VALUE) {
-      throw new RuntimeError(
-        RuntimeErrorCode.REQUIRED_INPUT_NO_VALUE,
-        ngDevMode && 'Input is required but no value is available yet.',
-      );
+      let message: string | null = null;
+      if (ngDevMode) {
+        const name = options?.debugName ?? options?.alias;
+        message = `Input${name ? ` "${name}"` : ''} is required but no value is available yet.`;
+      }
+      throw new RuntimeError(RuntimeErrorCode.REQUIRED_INPUT_NO_VALUE, message);
     }
 
     return node.value;
diff --git a/packages/core/test/authoring/input_signal_spec.ts b/packages/core/test/authoring/input_signal_spec.ts
index b98d3dddcc4d..b5371f39e8e5 100644
--- a/packages/core/test/authoring/input_signal_spec.ts
+++ b/packages/core/test/authoring/input_signal_spec.ts
@@ -78,6 +78,34 @@ describe('input signal', () => {
     });
   });
 
+  it('should include debugName in required inputs error message, if available', () => {
+    TestBed.runInInjectionContext(() => {
+      const signal = input.required({debugName: 'mySignal'});
+      const node = signal[SIGNAL];
+
+      expect(() => signal()).toThrowError(
+        /Input "mySignal" is required but no value is available yet\./,
+      );
+
+      node.applyValueToInputSignal(node, 1);
+      expect(signal()).toBe(1);
+    });
+  });
+
+  it('should include alias in required inputs error message, if available', () => {
+    TestBed.runInInjectionContext(() => {
+      const signal = input.required({alias: 'alias'});
+      const node = signal[SIGNAL];
+
+      expect(() => signal()).toThrowError(
+        /Input "alias" is required but no value is available yet\./,
+      );
+
+      node.applyValueToInputSignal(node, 1);
+      expect(signal()).toBe(1);
+    });
+  });
+
   it('should throw if a `computed` depends on an uninitialized required input', () => {
     TestBed.runInInjectionContext(() => {
       const signal = input.required<number>();