Update from version 3.2.0 to latest 4.1.0 breaks build. Undefined scss variables

Hello @pmowrer ,

for my current project we updated the node-sass-json-importer from version 3.2.0 to version 4.1.0 but that causes the build to break with error message of undefined variables. The variables it complains about are normal scss variables within scss files which are not part of any json file (see screenshot). Any idea?

The project setup is build with grunt. Using node-sass und grunt-sass. The relevant part in grunt config is the following:

const jsonImporter = require('node-sass-json-importer');

module.exports = {
    css : {
        options: {
            importer: jsonImporter,
            style: 'nested',
            precision: 8,
            trace: true,
            lineNumbers: true,
            sourceMap: true,
            sourceComments: true
        },
        files : [{
            expand: true,
            cwd: '<%= package.sourceFolder %>/scss/',
            src: ['**/*.scss'],
            dest: '<%= package.buildFolder %>/css/',
            ext: '.css'
        }]
    }
}

It would be unfortunate circumstances if we would be forced to stick with an old version of node-sass-json-importer unable to update.

Best regards, Annick

[pmowrer]

Closed by #83

[Tylerian]

IMHO this is an issue with #74 and that commit should be reverted in order to fix it.

The reasoning behind this is that sass either expects an object literal with a contentskey in order to load that contents, or an object literal with a filekey to load that file as though it had been imported directly (as seen in docs). But not an object literal containing both file and contents.

@pmowrer would you accept a pull request reverting that behavior?