SharedMappings ignores multiple tsconfig.json paths

[glebmachine]

By design in tsconfig.json you able to set multilple paths like so:

"@wololo/auth": [
  "dist/@wololo/auth", 
  "libs/auth/src/index"
]

This declaration works by following idea: "use compiled library, if not - compile from source"

But, here as you can see you just pick first path:

module-federation-plugin/packages/mf/src/utils/shared-mappings.ts

Line 33 in c60bf4f

const libPath = path.normalize(path.join(rootPath, mappings[key][0]));

[manfredsteyer]

Yes, this is correct. Nx mono repo are the role model here. They just define one path.

[glebmachine]

This is bad(

In my project i'm compile monorepo libraries and use artefacts for faster build/serve in development mode
Multiple pass allow me to enumerate all paths available for my libraries (first - compiled path, secondary - source path as shown in case below)

And compiler choose proper source automatically

[glebmachine]

"Nx mono repo are the role model here. They just define one path."
I think they define only one path because of "why not")

But they are doesn't restrict user to do that, so the role model here is tsconfig specification, not nx "autogenerated tsconfig files"

Either they should rework it to single string, instead of an string array

[oggythorpe1610]

Having been working on a project myself, I agree this is an issue. I had a definition in my tsconfig.json file for my shared library below two other 'dist/ ...' entries. in the paths:{} section for the two libraries I had created. Everything broke saying it couldn't find the modules despite having added the path of 'projects//src/public-api.ts' to the end of the list. Only moving them to the very first entry fixed things and everything worked as it should. Either this needs to be documented to add your project library path reference as the first entry above any 'dist/ ...' or the code needs to account for multiple entries correctly.

Obviously 'my-lib-name' is replaced by the library you have yourself here :)

[shankaryadavy9]

Hi @manfredsteyer
Is there any update on this..?. Having support for multiple entries will definitely help everyone.
I even explained my scenario on this PR(Let me know if there is another way to achieve it.)
#52 (comment)