refactor(migrations): gracefully proceed if reference cannot be resolved (#61426)

devversion · thePunderWoman · commit b148e272f697 · 2025-05-19T11:04:52.000Z
A runtime error can surface when TypeScript internally fails to resolve a reference that is named similar to an input, but no `.d.ts` is available for it. See example error: microsoft/TypeScript#61473 (comment). PR Close #61426
diff --git a/packages/core/schematics/migrations/signal-migration/src/passes/reference_resolution/identify_ts_references.ts b/packages/core/schematics/migrations/signal-migration/src/passes/reference_resolution/identify_ts_references.ts
@@ -48,25 +48,34 @@ export function identifyPotentialTypeScriptReference<D extends ClassFieldDescrip
 
   let target: ts.Symbol | undefined = undefined;
 
-  // Resolve binding elements to their declaration symbol.
-  // Commonly inputs are accessed via object expansion. e.g. `const {input} = this;`.
-  if (ts.isBindingElement(node.parent)) {
-    // Skip binding elements that are using spread.
-    if (node.parent.dotDotDotToken !== undefined) {
-      return;
-    }
+  try {
+    // Resolve binding elements to their declaration symbol.
+    // Commonly inputs are accessed via object expansion. e.g. `const {input} = this;`.
+    if (ts.isBindingElement(node.parent)) {
+      // Skip binding elements that are using spread.
+      if (node.parent.dotDotDotToken !== undefined) {
+        return;
+      }
 
-    const bindingInfo = resolveBindingElement(node.parent);
-    if (bindingInfo === null) {
-      // The declaration could not be resolved. Skip analyzing this.
-      return;
+      const bindingInfo = resolveBindingElement(node.parent);
+      if (bindingInfo === null) {
+        // The declaration could not be resolved. Skip analyzing this.
+        return;
+      }
+
+      const bindingType = checker.getTypeAtLocation(bindingInfo.pattern);
+      const resolved = lookupPropertyAccess(checker, bindingType, [bindingInfo.propertyName]);
+      target = resolved?.symbol;
+    } else {
+      target = checker.getSymbolAtLocation(node);
     }
+  } catch (e) {
+    console.error('Unexpected error while trying to resolve identifier reference:');
+    console.error(e);
 
-    const bindingType = checker.getTypeAtLocation(bindingInfo.pattern);
-    const resolved = lookupPropertyAccess(checker, bindingType, [bindingInfo.propertyName]);
-    target = resolved?.symbol;
-  } else {
-    target = checker.getSymbolAtLocation(node);
+    // Gracefully skip analyzing. This can happen when e.g. a reference is named similar
+    // to an input, but is dependant on `.d.ts` that is not necessarily available (clutz dts).
+    return;
   }
 
   noTargetSymbolCheck: if (target === undefined) {
