Relay Resolver has module resolution problem in TypeScript

[ellemedit]

As above images, generated relay artifacts import relay resolver with extension.
I think using ".js" extension or skipping extension are correct solution at this moment. So we can inference return type of RelayResolver function in TypeScript.

[alunyov]

The Relay Resolvers is still in active development, and typescript support maybe lacking in some places. Thanks for reporting!

[captbaritone]

Thanks for the report @beingbook. If you're interested in working on a fix, I suspect stripping the extension would be the right approach and the I think the code that derives this path lives here: https://github.com/facebook/relay/blob/main/compiler/crates/relay-config/src/project_config.rs#L360

cc @tbezman

[ellemedit]

oops, I'm too late. thank you for suggesting contribution! I want to work if there's next time.

[chriserickson]

This was not fixed in 14.1 - the dynamic imports in the AST omit extensions, however the type imports still have the extensions.

It appears that

relay/compiler/crates/relay-typegen/src/typescript.rs

Line 97 in 639abe1

fn write_import_module_default(&mut self, name: &str, from: &str) -> FmtResult {

and

relay/compiler/crates/relay-typegen/src/typescript.rs

Line 101 in 639abe1

fn write_import_type(&mut self, types: &[&str], from: &str) -> FmtResult {

Both also need to strip extensions, as typescript requires that extensions not be provided.

Is it possible to re-open this? I'm happy to work on a fix.

[chriserickson]

I went ahead and created a new issue:

#4056