6.1: Module resolution with baseUrl

[Quadriphobs1]

Describe the bug
Upgrading to 6.1 failed to compile Module not found: Error: Can't resolve 'components/**/**' when using baseUrl with typescript, also fails using alias with typescript. It obviously

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
I think storybook should use the project supplied tsconfig/jsconfig and maybe merge alongside any overrides, if its possible to probably do merging at this step info => Removing existing JavaScript and TypeScript rules. instead of removing, this would potentially make it more flexible to any environment

[Quadriphobs1]

A quick fix is to use tsconfig-paths-webpack-plugin to extend the tsconfig in the main.js. But it would be better if probably from the CLI a user can option to specify a typescript config or jsconfig file which I believe webpack would use internally.

const { TsconfigPathsPlugin } = require('tsconfig-paths-webpack-plugin')

module.exports = {
  webpackFinal: async (config) => {
    config.resolve.plugins = [new TsconfigPathsPlugin({ extensions: config.resolve.extensions })]
    return config
  },
 ...
};

[shilman]

@Quadriphobs1 what version are you upgrading from?

[Quadriphobs1]

@shilman ^6.0.28

[shilman]

How were you able to get it working in 6.0.28?

[Quadriphobs1]

It works with 6.0.28... I am not sure why... But it does work...

[stale]

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks!

[bodinsamuel]

Thanks for the solution, adding TsconfigPathsPlugin worked for me (version 6.1.15)

[ghost]

I had to use TsConfigPathsPlugin to use Storybook with my app. So did this fella in a different issue. Here's the place in the docs that needs to be updated with this workaround: https://storybook.js.org/docs/react/configure/webpack

[shilman]

@balibebas any interest in submitting a PR with the update?

[ghost]

@balibebas any interest in submitting a PR with the update?

Can do. But I'd like to answer this question before laying down a suggestion in the docs. #14087 (comment)

[shilman]

@balibebas sorry, I didn't see a question ... what am I missing? 😄

[ghost]

@balibebas sorry, I didn't see a question ... what am I missing? 😄

Seeing as how I linked to your own comment I can see how that could be confusing. I've updated the link above to link to the related issue with the question. Whoops! :D

[ghost]

@shilman subbed pull #14371 with the pattern you left in #14087. thanks for cleaning up the code 👍🏼