add node_modules to sass options includePaths and stop overwritting includePaths

[dagda1]

I finally got the bottom of making this work with my monorepo with external sass options in node_modules.

I first mentioned this in #37.

I've added node_modules to the includedPaths for sass by default.

const includePaths = [filePath, 'node_modules'];

I also encountered a bug on my travels with this code:

transformedCss = sass
  .renderSync({
    data: css,
    includePaths: [filePath],
    ...(rendererOptions.sass || {}),
  })
.css.toString();

It if you were passing in includePaths in the sass options like this:

{
  rendererOptions: {
   sass: { includePaths: ["node_modules"] }
  }
},

Then it was overwriting the original includePaths: [filePath], you can see an example here

const filePath = '/src/helpers/__tests__/fixtures';
const css = 'blah';
const rendererOptions = {
    sass: { includePaths: ["node_modules"] }
}

const result = {
    data: css,
    includePaths: [filePath],
    ...(rendererOptions.sass || {}),
}

console.log(JSON.stringify(result.includePaths));  // [node_modules] and not ['/src/helpers/__tests__/fixtures', 'node_modules']

I'm not sure if you agree with adding node_modules by default and I will remove it if needs be.

[mrmckeb]

This looks great, thanks @dagda1. A few minor suggested changes, if you can't make them, I can do it this weekend :)

[dagda1]

@mrmckeb Are these changes better?

sorry to be annoying but I am dying to get this rolled into my personal projects.

[mrmckeb]

Hi @dagda1, I'll try to do a release tonight or tomorrow - sorry for the slow reply.

You're not annoying, I'm just slow (busy few days) ;)

[mrmckeb]

Released as https://github.com/mrmckeb/typescript-plugin-css-modules/releases/tag/v2.1.0.

Thanks again!