node-sass throws error on empty .scss files

[KamilLelonek]

Assertion failed: (val->IsString()), function _NanGetExternalParts, file ../node_modules/nan/nan.h, line 1725.

After brunch build or watch.

[es128]

sass/node-sass#337

for now, try downgrading to 1.8.1

[es128]

Or see if there's anything you could tweak in your config that might be leading to the error per the various suggestions in that thread.

[KamilLelonek]

This is my config:

exports.config =
  paths:
    public: 'public'

  conventions:
    assets: /(assets|vendor\/assets|font)/

  files:
    javascripts:
      joinTo:
        'javascripts/app.js': /^app/
        'javascripts/vendor.js': /^(vendor|bower_components)/

    stylesheets:
      joinTo:
        'stylesheets/app.css': /^app/
        'stylesheets/vendor.css': /^(vendor|bower_components)/

[KamilLelonek]

Problem occurs with .scss extension only.

[es128]

Yeah you're not setting any special sass plugin options there. Any chance you have any empty .scss files in your project?

You could also try providing the output of DEBUG=brunch:* brunch b

But I don't really know what's wrong, it's something upstream in node-sass/libsass. The easiest thing for you to do for the moment is pin your sass-brunch dependency to 1.8.1

[KamilLelonek]

Yeah, I have empty .scss. Downgrading works, but I thing it's not the perfect solution.

[es128]

...and deleting or adding content to your empty .scss file(s) resolves the error?

[KamilLelonek]

Yes.

[es128]

So maybe that's the perfect solution? What do you think?

[KamilLelonek]

Not really, because it's confusing to fail build with empty files.

[es128]

It appears node-sass has an unpublished fix for this type of error. You could try nudging them into releasing a new patch version.

The only thing sass-brunch might be able to control is skipping compilation when it knows the source file is empty to begin with.

[KamilLelonek]

Ok. Thanks for your help.