fix: resolve sass-loader to avoid phantom dependencies

[ChrisCindy]

No description provided.

[odeadglaz]

Any plans of getting this merged? I also encountered a case where this solution makes more sense.

also, if you look at how docusaurus defined their webpack loaders, it's quite the same.

[rlamana]

@odeadglaz @ChrisCindy What is the case where this solution makes sense? I'm good with the change but I'd like more context on why it is needed? The PR description lacks an explanation. Thank you!

[luhc228]

@odeadglaz @ChrisCindy What is the case where this solution makes sense? I'm good with the change but I'd like more context on why it is needed? The PR description lacks an explanation. Thank you!

The @docusaurus/core and the docusaurus-plugin-sass packages are not on the same level in the node_modules.

[odeadglaz]

Thats correct, in the case of mono repo, the deps might not be present in the same level.

[luhc228]

So will this PR be merged?

[rlamana]

Looks good to me then 👍