SassBoolean should be a class

[nex3]

Hello,

I'm trying to migrate a script from sass to sass-embedded and can't use SassBoolean anymore.

The script tries to identify the type with:

if (value instanceof SassBoolean) {

  (...)

}

At runtime, the following error is thrown:

Error: TypeError: Right-hand side of 'instanceof' is not an object

I realized SassBoolean is an interface and not a class:

embedded-host-node/lib/src/value/boolean.ts

Line 15 in 248437e

export interface SassBoolean extends Value {

Therefore, it is not present here in the compiled index.js along sassFalse and sassTrue:

var boolean_2 = require("./src/value/boolean");

Object.defineProperty(exports, "sassFalse", { enumerable: true, get: function () { return boolean_2.sassFalse; } });

Object.defineProperty(exports, "sassTrue", { enumerable: true, get: function () { return boolean_2.sassTrue; } });

Should one use sassFalse and sassTrue instead ?

As is, SassBoolean still breaks usage of sass-embedded as drop-in replacement, as mentioned by @rvock.

Originally posted by @guillaumerochelle in #337

The spec defines SassBoolean as a class, so this is indeed a bug in the Embedded Host's implementation of the Sass API.