Fix getTypeFromJSDocValueReference

When using `{import('./b').FOO}` which is defined as a string literal,
`valueType` doesn't have a `symbol`.  Leave it for the fallback value
for now.

This was exposed in 8223c07.

Fixes microsoft#34869.

Author: elibarzilay

src/compiler/checker.ts

@@ -10831,7 +10831,8 @@ namespace ts {
                 }
                 const isDelayedMergeClass = symbol !== valueType.symbol && getMergedSymbol(symbol) === valueType.symbol;
                 const isImportTypeWithQualifier = node.kind === SyntaxKind.ImportType && (node as ImportTypeNode).qualifier;
-                if (isRequireAlias || isImportTypeWithQualifier || isDelayedMergeClass) {
+                // valueType might not have a symbol, eg, {import('./b').STRING_LITERAL}
+                if (valueType.symbol && (isRequireAlias || isImportTypeWithQualifier || isDelayedMergeClass)) {
                     typeType = getTypeReferenceType(node, valueType.symbol);
                 }
             }

tests/baselines/reference/jsdocImportTypeReferenceToStringLiteral.symbols

@@ -0,0 +1,9 @@
+=== tests/cases/conformance/jsdoc/b.js ===
+export const FOO = "foo";
+>FOO : Symbol(FOO, Decl(b.js, 0, 12))
+
+=== tests/cases/conformance/jsdoc/a.js ===
+/** @type {import('./b').FOO} */
+let x;
+>x : Symbol(x, Decl(a.js, 1, 3))
+

tests/baselines/reference/jsdocImportTypeReferenceToStringLiteral.types

@@ -0,0 +1,10 @@
+=== tests/cases/conformance/jsdoc/b.js ===
+export const FOO = "foo";
+>FOO : "foo"
+>"foo" : "foo"
+
+=== tests/cases/conformance/jsdoc/a.js ===
+/** @type {import('./b').FOO} */
+let x;
+>x : "foo"
+

tests/cases/conformance/jsdoc/jsdocImportTypeReferenceToStringLiteral.ts

@@ -0,0 +1,9 @@
+// @noEmit: true
+// @allowJs: true
+// @checkJs: true
+// @Filename: b.js
+export const FOO = "foo";
+
+// @Filename: a.js
+/** @type {import('./b').FOO} */
+let x;