Skip to content

tinganho/transitive-deps-bug

Repository files navigation

Transitive dependency are not linked underneath node_modules

A demo of the bug that causes portability issues in rules_js:

Just run:

bazel run //:foo --sandbox_debug

And it will produce the following error:

error TS2742: The inferred type of 'Offline' cannot be named without a reference to '.aspect_rules_js/@[email protected]/node_modules/@myorg/c/qux'. This is likely not portable. A type annotation is necessary.

The reasons seems to be there is no linkage underneath node_modules(The screenshot is of the bin folder in sandbox and it seems to be missing node_modules/@myorg/c):

Screenshot of the bug

Also, to reproduce wheren't so easy. It requires at least import with file reference i.e. @org/package/file and an enum in the transitive dep, and in the source file you also need some type operation fiddling to reproduce it.

And when you remove data attribute in ts.bzl the error dissapears:

Screenshot of the bug

About

No description, website, or topics provided.

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published