Support pure JS sass executable

.github/workflows/ci.yml

@@ -117,17 +117,7 @@ jobs:
         working-directory: sass-spec
 
       - name: Compile
-        run: |
-          npm run compile
-          if [[ "$RUNNER_OS" == "Windows" ]]; then
-            # Avoid copying the entire Dart Sass build directory on Windows,
-            # since it may contain symlinks that cp will choke on.
-            mkdir -p dist/lib/src/vendor/dart-sass/
-            cp {`pwd`/,dist/}lib/src/vendor/dart-sass/sass.bat
-            cp {`pwd`/,dist/}lib/src/vendor/dart-sass/sass.snapshot
-          else
-            ln -s {`pwd`/,dist/}lib/src/vendor/dart-sass
-          fi
+        run: npm run compile
 
       - name: Run tests
         run: npm run js-api-spec -- --sassPackage .. --sassSassRepo ../language

bin/sass.ts

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 
 import * as child_process from 'child_process';
+import * as path from 'path';
 import {compilerCommand} from '../lib/src/compiler-path';
 
 // TODO npm/cmd-shim#152 and yarnpkg/berry#6422 - If and when the package
@@ -12,6 +13,10 @@ try {
     compilerCommand[0],
     [...compilerCommand.slice(1), ...process.argv.slice(2)],
     {
+      // Node blocks launching .bat and .cmd without a shell due to CVE-2024-27980
+      shell: ['.bat', '.cmd'].includes(
+        path.extname(compilerCommand[0]).toLowerCase(),
+      ),
       stdio: 'inherit',
       windowsHide: true,
     },

lib/src/compiler-module.ts

@@ -0,0 +1,34 @@
+// Copyright 2025 Google LLC. Use of this source code is governed by an
+// MIT-style license that can be found in the LICENSE file or at
+// https://opensource.org/licenses/MIT.
+
+import * as p from 'path';
+import {getElfInterpreter} from './elf';
+
+/**
+ * Detect if the given binary is linked with musl libc by checking if
+ * the interpreter basename starts with "ld-musl-"
+ */
+function isLinuxMusl(path: string): boolean {
+  try {
+    const interpreter = getElfInterpreter(path);
+    return p.basename(interpreter).startsWith('ld-musl-');
+  } catch (error) {
+    console.warn(
+      `Warning: Failed to detect linux-musl, fallback to linux-gnu: ${error.message}`,
+    );
+    return false;
+  }
+}
+
+/** The module name for the embedded compiler executable. */
+export const compilerModule = (() => {
+  const platform =
+    process.platform === 'linux' && isLinuxMusl(process.execPath)
+      ? 'linux-musl'
+      : (process.platform as string);
+
+  const arch = process.arch;
+
+  return `sass-embedded-${platform}-${arch}`;
+})();

lib/src/compiler-path.ts

@@ -2,78 +2,52 @@
 // MIT-style license that can be found in the LICENSE file or at
 // https://opensource.org/licenses/MIT.
 
-import * as fs from 'fs';
 import * as p from 'path';
-import {getElfInterpreter} from './elf';
-import {isErrnoException} from './utils';
-
-/**
- * Detect if the given binary is linked with musl libc by checking if
- * the interpreter basename starts with "ld-musl-"
- */
-function isLinuxMusl(path: string): boolean {
-  try {
-    const interpreter = getElfInterpreter(path);
-    return p.basename(interpreter).startsWith('ld-musl-');
-  } catch (error) {
-    console.warn(
-      `Warning: Failed to detect linux-musl, fallback to linux-gnu: ${error.message}`,
-    );
-    return false;
-  }
-}
+import {compilerModule} from './compiler-module';
 
 /** The full command for the embedded compiler executable. */
 export const compilerCommand = (() => {
-  const platform =
-    process.platform === 'linux' && isLinuxMusl(process.execPath)
-      ? 'linux-musl'
-      : (process.platform as string);
-
-  const arch = process.arch;
-
-  // find for development
-  for (const path of ['vendor', '../../../lib/src/vendor']) {
-    const executable = p.resolve(
-      __dirname,
-      path,
-      `dart-sass/sass${platform === 'win32' ? '.bat' : ''}`,
-    );
-
-    if (fs.existsSync(executable)) return [executable];
-  }
-
   try {
     return [
       require.resolve(
-        `sass-embedded-${platform}-${arch}/dart-sass/src/dart` +
-          (platform === 'win32' ? '.exe' : ''),
-      ),
-      require.resolve(
-        `sass-embedded-${platform}-${arch}/dart-sass/src/sass.snapshot`,
+        `${compilerModule}/dart-sass/src/dart` +
+          (process.platform === 'win32' ? '.exe' : ''),
       ),
+      require.resolve(`${compilerModule}/dart-sass/src/sass.snapshot`),
     ];
-  } catch (ignored) {
-    // ignored
+  } catch (e) {
+    if (e.code !== 'MODULE_NOT_FOUND') {
+      throw e;
+    }
   }
 
   try {
     return [
       require.resolve(
-        `sass-embedded-${platform}-${arch}/dart-sass/sass` +
-          (platform === 'win32' ? '.bat' : ''),
+        `${compilerModule}/dart-sass/sass` +
+          (process.platform === 'win32' ? '.bat' : ''),
       ),
     ];
-  } catch (e: unknown) {
-    if (!(isErrnoException(e) && e.code === 'MODULE_NOT_FOUND')) {
+  } catch (e) {
+    if (e.code !== 'MODULE_NOT_FOUND') {
+      throw e;
+    }
+  }
+
+  try {
+    return [
+      process.execPath,
+      p.join(p.dirname(require.resolve('sass')), 'sass.js'),
+    ];
+  } catch (e) {
+    if (e.code !== 'MODULE_NOT_FOUND') {
       throw e;
     }
   }
 
   throw new Error(
     "Embedded Dart Sass couldn't find the embedded compiler executable. " +
-      'Please make sure the optional dependency ' +
-      `sass-embedded-${platform}-${arch} is installed in ` +
-      'node_modules.',
+      `Please make sure the optional dependency ${compilerModule} or sass is ` +
+      'installed in node_modules.',
   );
 })();

npm/all-unknown/README.md

@@ -0,0 +1,3 @@
+# `sass-embedded-all-unknown`
+
+This is the **all-unknown** binary for [`sass-embedded`](https://www.npmjs.com/package/sass-embedded)

npm/all-unknown/package.json

@@ -0,0 +1,17 @@
+{
+  "name": "sass-embedded-all-unknown",
+  "version": "1.89.2",
+  "description": "The all-unknown binary for sass-embedded",
+  "repository": "sass/embedded-host-node",
+  "author": "Google Inc.",
+  "license": "MIT",
+  "cpu": [
+    "!arm",
+    "!arm64",
+    "!riscv64",
+    "!x64"
+  ],
+  "dependencies": {
+    "sass": "1.89.2"
+  }
+}

npm/unknown-all/README.md

@@ -0,0 +1,3 @@
+# `sass-embedded-unknown-all`
+
+This is the **unknown-all** binary for [`sass-embedded`](https://www.npmjs.com/package/sass-embedded)

npm/unknown-all/package.json

@@ -0,0 +1,17 @@
+{
+  "name": "sass-embedded-unknown-all",
+  "version": "1.89.2",
+  "description": "The unknown-all binary for sass-embedded",
+  "repository": "sass/embedded-host-node",
+  "author": "Google Inc.",
+  "license": "MIT",
+  "os": [
+    "!android",
+    "!darwin",
+    "!linux",
+    "!win32"
+  ],
+  "dependencies": {
+    "sass": "1.89.2"
+  }
+}

package.json

@@ -38,6 +38,7 @@
     "test": "jest"
   },
   "optionalDependencies": {
+    "sass-embedded-all-unknown": "1.89.2",
     "sass-embedded-android-arm": "1.89.2",
     "sass-embedded-android-arm64": "1.89.2",
     "sass-embedded-android-riscv64": "1.89.2",
@@ -46,12 +47,13 @@
     "sass-embedded-darwin-x64": "1.89.2",
     "sass-embedded-linux-arm": "1.89.2",
     "sass-embedded-linux-arm64": "1.89.2",
-    "sass-embedded-linux-riscv64": "1.89.2",
-    "sass-embedded-linux-x64": "1.89.2",
     "sass-embedded-linux-musl-arm": "1.89.2",
     "sass-embedded-linux-musl-arm64": "1.89.2",
     "sass-embedded-linux-musl-riscv64": "1.89.2",
     "sass-embedded-linux-musl-x64": "1.89.2",
+    "sass-embedded-linux-riscv64": "1.89.2",
+    "sass-embedded-linux-x64": "1.89.2",
+    "sass-embedded-unknown-all": "1.89.2",
     "sass-embedded-win32-arm64": "1.89.2",
     "sass-embedded-win32-x64": "1.89.2"
   },

tool/get-embedded-compiler.ts

@@ -2,33 +2,45 @@
 // MIT-style license that can be found in the LICENSE file or at
 // https://opensource.org/licenses/MIT.
 
+import {promises as fs} from 'fs';
 import * as p from 'path';
 import * as shell from 'shelljs';
 
+import {compilerModule} from '../lib/src/compiler-module';
 import * as utils from './utils';
 
 /**
  * Downloads and builds the Embedded Dart Sass compiler.
  *
  * Can check out and build the source from a Git `ref` or build from the source
  * at `path`. By default, checks out the latest revision from GitHub.
+ *
+ * The embedded compiler will be built as dart snapshot by default, or pure node
+ * js if the `js` option is `true`.
  */
 export async function getEmbeddedCompiler(
-  outPath: string,
-  options?: {ref: string} | {path: string},
+  options?:
+    | {
+        ref?: string;
+        js?: boolean;
+      }
+    | {
+        path: string;
+        js?: boolean;
+      },
 ): Promise<void> {
   const repo = 'dart-sass';
 
   let source: string;
-  if (!options || 'ref' in options) {
+  if (options !== undefined && 'path' in options) {
+    source = options.path;
+  } else {
     utils.fetchRepo({
       repo,
       outPath: 'build',
       ref: options?.ref ?? 'main',
     });
     source = p.join('build', repo);
-  } else {
-    source = options.path;
   }
 
   // Make sure the compiler sees the same version of the language repo that the
@@ -41,21 +53,44 @@ export async function getEmbeddedCompiler(
     await utils.link(languageInHost, languageInCompiler);
   }
 
-  buildDartSassEmbedded(source);
-  await utils.link(p.join(source, 'build'), p.join(outPath, repo));
+  const js = options?.js ?? false;
+  buildDartSassEmbedded(source, js);
+
+  const jsModulePath = p.resolve('node_modules/sass');
+  const dartModulePath = p.resolve(p.join('node_modules', compilerModule));
+  if (js) {
+    await fs.rm(dartModulePath, {force: true, recursive: true});
+    await utils.link(p.join(source, 'build/npm'), jsModulePath);
+  } else {
+    await fs.rm(jsModulePath, {force: true, recursive: true});
+    await utils.link(p.join(source, 'build'), p.join(dartModulePath, repo));
+  }
 }
 
 // Builds the Embedded Dart Sass executable from the source at `repoPath`.
-function buildDartSassEmbedded(repoPath: string): void {
+function buildDartSassEmbedded(repoPath: string, js: boolean): void {
   console.log("Downloading Dart Sass's dependencies.");
   shell.exec('dart pub upgrade', {
     cwd: repoPath,
     silent: true,
   });
 
-  console.log('Building the Dart Sass executable.');
-  shell.exec('dart run grinder protobuf pkg-standalone-dev', {
-    cwd: repoPath,
-    env: {...process.env, UPDATE_SASS_PROTOCOL: 'false'},
-  });
+  if (js) {
+    shell.exec('npm install', {
+      cwd: repoPath,
+      silent: true,
+    });
+
+    console.log('Building the Dart Sass npm package.');
+    shell.exec('dart run grinder protobuf pkg-npm-dev', {
+      cwd: repoPath,
+      env: {...process.env, UPDATE_SASS_PROTOCOL: 'false'},
+    });
+  } else {
+    console.log('Building the Dart Sass executable.');
+    shell.exec('dart run grinder protobuf pkg-standalone-dev', {
+      cwd: repoPath,
+      env: {...process.env, UPDATE_SASS_PROTOCOL: 'false'},
+    });
+  }
 }

tool/init.ts

@@ -18,6 +18,10 @@ const argv = yargs(process.argv.slice(2))
     type: 'string',
     description: 'Build the Embedded Dart Sass binary from this Git ref.',
   })
+  .option('compiler-js', {
+    type: 'boolean',
+    description: 'Build the Embedded Dart Sass with dart2js.',
+  })
   .option('skip-compiler', {
     type: 'boolean',
     description: "Don't Embedded Dart Sass at all.",
@@ -55,15 +59,19 @@ void (async () => {
 
     if (!argv['skip-compiler']) {
       if (argv['compiler-ref']) {
-        await getEmbeddedCompiler(outPath, {
+        await getEmbeddedCompiler({
           ref: argv['compiler-ref'],
+          js: argv['compiler-js'],
         });
       } else if (argv['compiler-path']) {
-        await getEmbeddedCompiler(outPath, {
+        await getEmbeddedCompiler({
           path: argv['compiler-path'],
+          js: argv['compiler-js'],
         });
       } else {
-        await getEmbeddedCompiler(outPath);
+        await getEmbeddedCompiler({
+          js: argv['compiler-js'],
+        });
       }
     }