Merge pull request #324 from sass/merge

Merge main into feature.color-4

Author: nex3

.eslintrc

@@ -0,0 +1,14 @@
+{
+  "extends": "./node_modules/gts/",
+  "rules": {
+    "@typescript-eslint/explicit-function-return-type": [
+      "error",
+      {"allowExpressions": true}
+    ],
+    "func-style": ["error", "declaration"],
+    "prefer-const": ["error", {"destructuring": "all"}],
+    // It would be nice to sort import declaration order as well, but that's not
+    // autofixable and it's not worth the effort of handling manually.
+    "sort-imports": ["error", {"ignoreDeclarationSort": true}],
+  }
+}

.eslintrc.json

@@ -4,3 +4,7 @@ updates:
     directory: "/"
     schedule:
       interval: "weekly"
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"

.github/dependabot.yml

@@ -18,8 +18,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: 'lts/*'
           check-latest: true
@@ -46,8 +46,8 @@ jobs:
       fail-fast: false
 
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
           check-latest: true
@@ -89,10 +89,10 @@ jobs:
             node_version: lts/-2
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: dart-lang/setup-dart@v1
         with: {sdk: stable}
-      - uses: actions/setup-node@v3
+      - uses: actions/setup-node@v4
         with: {node-version: "${{ matrix.node_version }}"}
 
       - name: Check out Dart Sass
@@ -140,8 +140,8 @@ jobs:
     needs: [static_analysis, tests, sass_spec]
 
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: 'lts/*'
           check-latest: true

.github/workflows/ci.yml

@@ -1,3 +1,39 @@
+## 1.77.8
+
+* No user-visible changes.
+
+## 1.77.7
+
+* Declarations that appear after nested rules are deprecated, because the
+  semantics Sass has historically used are different from the semantics
+  specified by CSS. In the future, Sass will adopt the standard CSS semantics.
+
+  See [the Sass website](https://sass-lang.com/d/mixed-decls) for details.
+
+* **Potentially breaking bug fix:** `//` in certain places such as unknown
+  at-rule values was being preserved in the CSS output, leading to potentially
+  invalid CSS. It's now properly parsed as a silent comment and omitted from the
+  CSS output.
+
+## 1.77.6
+
+* Fix a few cases where comments and occasionally even whitespace wasn't allowed
+  between the end of Sass statements and the following semicolon.
+
+## 1.77.5
+
+* Fully trim redundant selectors generated by `@extend`.
+
+## 1.77.4
+
+### Embedded Sass
+
+* Support passing `Version` input for `fatalDeprecations` as string over
+  embedded protocol.
+
+* Fix a bug in the JS Embedded Host where `Version` could be incorrectly accepted
+  as input for `silenceDeprecations` and `futureDeprecations` in pure JS.
+
 ## 1.77.3
 
 ### Dart API

CHANGELOG.md

@@ -0,0 +1,25 @@
+#!/usr/bin/env node
+
+import * as child_process from 'child_process';
+import {compilerCommand} from '../lib/src/compiler-path';
+
+// TODO npm/cmd-shim#152 and yarnpkg/berry#6422 - If and when the package
+// managers support it, we should make this a proper shell script rather than a
+// JS wrapper.
+
+try {
+  child_process.execFileSync(
+    compilerCommand[0],
+    [...compilerCommand.slice(1), ...process.argv.slice(2)],
+    {
+      stdio: 'inherit',
+      windowsHide: true,
+    }
+  );
+} catch (error) {
+  if (error.code) {
+    throw error;
+  } else {
+    process.exitCode = error.status;
+  }
+}

bin/sass.ts

@@ -4,20 +4,29 @@
 
 import * as fs from 'fs';
 import * as p from 'path';
+import {getElfInterpreter} from './elf';
 import {isErrnoException} from './utils';
 
 /**
- * Detect if the current running node binary is linked with musl libc by
- * checking if the binary contains a string like "/.../ld-musl-$ARCH.so"
+ * Detect if the given binary is linked with musl libc by checking if
+ * the interpreter basename starts with "ld-musl-"
  */
-const isLinuxMusl = function () {
-  return fs.readFileSync(process.execPath).includes('/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 full command for the embedded compiler executable. */
 export const compilerCommand = (() => {
   const platform =
-    process.platform === 'linux' && isLinuxMusl()
+    process.platform === 'linux' && isLinuxMusl(process.execPath)
       ? 'linux-musl'
       : (process.platform as string);
 

lib/src/compiler-path.ts

@@ -9,7 +9,7 @@ import * as compilerModule from './compiler/utils';
 import {Compiler, initCompiler} from './compiler/sync';
 
 const createDispatcher = jest.spyOn(compilerModule, 'createDispatcher');
-function getIdHistory() {
+function getIdHistory(): number[] {
   return createDispatcher.mock.calls.map(([id]) => id);
 }
 

lib/src/compiler.test.ts

@@ -4,7 +4,7 @@
 
 import * as p from 'path';
 import * as supportsColor from 'supports-color';
-import {deprecations, getDeprecationIds, Deprecation} from '../deprecations';
+import {Deprecation, deprecations, getDeprecationIds} from '../deprecations';
 import {deprotofySourceSpan} from '../deprotofy-span';
 import {Dispatcher, DispatcherHandlers} from '../dispatcher';
 import {Exception} from '../exception';

lib/src/compiler/utils.ts

@@ -2,26 +2,12 @@
 // MIT-style license that can be found in the LICENSE file or at
 // https://opensource.org/licenses/MIT.
 
-import {deprecations} from './vendor/deprecations';
-import {Deprecation, DeprecationOrId} from './vendor/sass';
+import {DeprecationOrId} from './vendor/sass';
 import {Version} from './version';
 
 export {deprecations} from './vendor/deprecations';
 export {Deprecation, DeprecationOrId, DeprecationStatus} from './vendor/sass';
 
-/**
- * Returns whether the given deprecation was active in the given version.
- */
-function isActiveIn(deprecation: Deprecation, version: Version) {
-  const deprecatedIn = deprecation.deprecatedIn;
-  if (deprecation.status !== 'active' || !deprecatedIn) return false;
-  if (version.major > deprecatedIn.major) return true;
-  if (version.major < deprecatedIn.major) return false;
-  if (version.minor > deprecatedIn.minor) return true;
-  if (version.minor < deprecatedIn.minor) return false;
-  return version.patch >= deprecatedIn.patch;
-}
-
 /**
  * Converts a mixed array of deprecations, IDs, and versions to an array of IDs
  * that's ready to include in a CompileRequest.
@@ -31,9 +17,7 @@ export function getDeprecationIds(
 ): string[] {
   return arr.flatMap(item => {
     if (item instanceof Version) {
-      return Object.values(deprecations)
-        .filter(deprecation => isActiveIn(deprecation, item))
-        .map(deprecation => deprecation.id);
+      return arr.map(item => item.toString());
     } else if (typeof item === 'string') {
       return item;
     }