[bug] JSX support in `preserve` mode w/ custom `jsxImportSource` causing `TS2875`

[tmm1]

I have a tsconfig.json approximately like:

"compilerOptions": {
        "module": "es2022",
        "jsx": "preserve",
        "jsxImportSource": "external/solid",
        "baseUrl": ".",
        "paths": {
                "external/*": ["./external/*"]
        },
        "target": "es2022",

error TS2875: This JSX tag requires the module path 'external/solid/jsx-runtime' to exist, but none could be found. Make sure you have types for the appropriate package installed.

even though this file exists:

❯ cat external/solid/jsx-runtime.d.ts
export * from "./solid/jsx";

❯ ls -l external/solid/solid/jsx.d.ts
-rw-r--r--  1 tmm1  staff  73590 Apr  4 17:34 external/solid/solid/jsx.d.ts

Is this due to compilerOptions.paths?

[jakebailey]

baseUrl is not supported: #474

[tmm1]

Thanks. I removed baseUrl, and also tried to add paths['*'] = ['*'], but the behavior remains the same.

[jakebailey]

Ah, well I don't think jsxImportSource has been implemented yet.

[tmm1]

The error comes from:

typescript-go/internal/checker/jsx.go

Lines 1249 to 1259 in 34dd103

	runtimeImportSpecifier := getJSXRuntimeImport(getJSXImplicitImportBase(c.compilerOptions, file), c.compilerOptions)
	if runtimeImportSpecifier == "" {
	return nil
	}
	errorMessage := diagnostics.This_JSX_tag_requires_the_module_path_0_to_exist_but_none_could_be_found_Make_sure_you_have_types_for_the_appropriate_package_installed
	specifier := c.getJSXRuntimeImportSpecifier(file, runtimeImportSpecifier)
	mod := c.resolveExternalModule(core.OrElse(specifier, location), runtimeImportSpecifier, errorMessage, location, false)
	var result *ast.Symbol
	if mod != nil && mod != c.unknownSymbol {
	result = c.getMergedSymbol(c.resolveSymbol(mod))
	}

where getJSXImplicitImportBase does plumb the value of jsximportsource through. So the issue is likely in resolveExternalModule here?

https://github.com/microsoft/typescript-go/blob/34dd1030316c6f25c5af8fdf2bef671fa0413a8d/internal/checker/checker.go#L14208-L14209

	// !!! The following only implements simple module resolution
	sourceFile := c.program.GetResolvedModule(ast.GetSourceFileOfNode(location), moduleReference)

[tmm1]

There are some references to "the synthesized JSX runtime import" which I think may be missing on the Corsa side.

Atleast when I look at the program's resolvedModules, there is no entry for jsx-runtime which causes the failure.

---

Imports synthesized here:

https://github.com/microsoft/TypeScript/blob/0693cc72e6ddb6c7468a14e05888efa2f43ac7b0/src/compiler/program.ts#L3512-L3516

            const jsxImport = getJSXRuntimeImport(getJSXImplicitImportBase(options, file), options);
            if (jsxImport) {
                // synthesize `import "base/jsx-runtime"` declaration
                (imports ||= []).push(createSyntheticImport(jsxImport, file));
            }

and unimplemented here:

typescript-go/internal/parser/references.go

Lines 11 to 12 in ce957e2

	func collectExternalModuleReferences(file *ast.SourceFile) {
	// !!!

[tmm1]

A little stuck because collectExternalModuleReferences is in internal/parser but the original version needs access to compiler options.

Also createSyntheticImport needs to be ported which is probably best done by someone familiar with the parser internals. A *ast.NodeFactory reference will be required for that function.

cc @ahejlsberg