Build output includes .scss files from src directory

[francislavoie]

As I reported here #782 (comment) builds (including --watch) output .scss files that are in my src/ to dist/. I expect dist/ to only have the .js files for my build (I don't have a public/ dir to copy for my build). I'm using 2.8.0-pre.4.

2.7.5 did not have this behaviour, I think it started with 2.7.7.

[drwpow]

Thanks for raising. I understand and agree this isn’t expected behavior from the standpoint of your project.

From Snowpack’s end, this is technically “expected behavior” until we release an official Sass plugin (“expected behavior” in quotes because it’s not ideal, but it is consistent with our plan of having plugins do more and Snowpack do less custom behavior management). Reason is: our Sass recipe uses the run script, which currently doesn’t have a way to clean up file extensions. Further, you can’t exclude: these files either, because then Snowpack would just ignore processing them altogether. Currently the only way to mark certain file extensions as not needed for production build is with the plugin extension mapping.

We currently have a Sass community plugin which should prevent this behavior, but I haven’t tested this myself. Just a note: it uses node-sass, which is a different Sass implementation from the Dart Sass recipe featured in our docs, and may behave differently for consumers of the Dart Sass CLI.

2.7.5 did not have this behaviour, I think it started with 2.7.7.

That sounds about right. Even though we overhauled the build pipeline in 2.7.0, that initial release had a bug for non-web-friendly filetypes (.scss), so I think Sass files were being omitted by accident until that bug was fixed. Now it’s working “as expected” (again in quotes; this isn’t the plan long-term)

[francislavoie]

Further, you can’t exclude: these files either, because then Snowpack would just ignore processing them altogether.

Actually, that is what I want. I just want to exclude them altogether. I couldn't find a relevant config option to do this in the docs. How would I exclude certain file extensions from the build process?

Edit: Oh - the top level exclude, I see. I was looking in buildOptions.

Adding "exclude": ["**/node_modules/**/*", "**/__tests__/*", "**/*.@(spec|test).@(js|mjs)", "**/*.scss"], did the trick.

Thanks for pointing out the sass plugin, I may look into that later on. For now, we do the sass build separately from snowpack so excluding altogether is what I want.

[drwpow]

Edit: Oh - the top level exclude, I see. I was looking in buildOptions.

Oh yup! That’s the one. exclude is passed straight to glob’s ignore option, so it operates the same way.

🤔 I had originally been thinking that excluding Sass altogether would break HMR in the dev server, but as long as Sass is watching & outputting CSS, I guess it wouldn’t matter! Perhaps we should recommend excluding Sass for now in the docs. But I’d still like to have an official Sass plugin someday so that people don’t have to fuss with the run-script-plugin + config options.