Improved Webpack-style tilde-import support for Sass

[kmark]

The current solution for dealing with sass/scss imports which are
prefixed with a tilde (~) is to strip the symbol from the import line.
This breaks down when dart-sass itself starts pulling in subsequent
imports that are also tilde-prefixed.

This change introduces a simple algorithm to climb the filesystem tree
looking for the imported module/file in a node_modules directory.

This is enabled by default since it replaces behavior which is already
forced on.

This change is covered under the existing "with a Bootstrap import"
test.

[mrmckeb]

Thanks @kmark, this looks good - thanks for the contribution.

The initial work was admittedly a quick hack to get it working, and I understand why that wouldn't scale.

Do you think it makes sense to have this as the default setting? I suspect many people are using this (from my experience on projects and from issues here). We could do that in a new major version.

[mrmckeb]

This is now available, and on by default in v3. Thanks @kmark!

https://github.com/mrmckeb/typescript-plugin-css-modules/releases/tag/v3.0.0

[kmark]

Hey @mrmckeb,

Sorry for the delay in getting back to you. Thanks for the merge! This is enabled by default which should mostly match what people currently expect. Were you wondering if we should have it opt in instead?

Thanks again,
Kevin

[kmark]

I think the only way to keep from bumping the major version would probably be to have the setting have 3 states: off, old (default), and on. I agree that bumping to 3.0 is the right call since I think most people expect it to work this way already, which should make it a trivial upgrade for them.

[mrmckeb]

No problem @kmark - as someone that often takes days to get to issues, I completely understand.

Let me know how this release goes, and if there are any issues!