Merge remote-tracking branch 'origin/main' into merge-2

Author: nex3

.github/workflows/ci.yml

@@ -19,6 +19,26 @@ jobs:
       actions: read # Forwarded
     uses: ./.github/workflows/test.yml
 
+  dependabot_merge:
+    name: Auto-merge Dependabot PR
+    permissions:
+      contents: write # Needed to merge PR
+      pull-requests: write # Needed to merge PR
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.user.login == 'dependabot[bot]' && github.repository == 'sass/dart-sass'
+
+    steps:
+      - id: metadata
+        uses: dependabot/fetch-metadata@25dd0e34f4fe68f24cc83900b1fe3fe149efef98 # v3.1.0
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+
+      - if: steps.metadata.outputs.update-type != 'version-update:semver-major'
+        run: gh pr merge --auto --merge "$PR_URL"
+        env:
+          PR_URL: ${{ github.event.pull_request.html_url }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
   double_check:
     name: Double-check
     runs-on: ubuntu-latest

.github/workflows/release.yml

@@ -340,7 +340,11 @@ jobs:
           GIT_AUTHOR_EMAIL: sass.bot.beep.boop@gmail.com
           GIT_COMMITTER_NAME: Sass Bot
           GIT_COMMITTER_EMAIL: sass.bot.beep.boop@gmail.com
-      - run: git tag "$VERSION"
+      - run: git tag -a "$VERSION" -m "$VERSION"
         env:
+          GIT_AUTHOR_NAME: Sass Bot
+          GIT_AUTHOR_EMAIL: sass.bot.beep.boop@gmail.com
+          GIT_COMMITTER_NAME: Sass Bot
+          GIT_COMMITTER_EMAIL: sass.bot.beep.boop@gmail.com
           VERSION: ${{ steps.version.outputs.version }}
-      - run: git push
+      - run: git push --follow-tags

.github/workflows/test.yml

@@ -222,7 +222,7 @@ jobs:
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
         with: {persist-credentials: false}
-      - uses: browser-actions/setup-chrome@2e1d749697dd1612b833dba4a722266286fbefcd # v2
+      - uses: browser-actions/setup-chrome@48ad923757ca74d66703209fe939badbdf80f2f4 # v2
       - uses: ./.github/util/initialize
         with:
           dart-sdk: ${{ matrix.dart_channel }}
@@ -326,7 +326,7 @@ jobs:
     steps:
     - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
       with: {persist-credentials: false}
-    - uses: browser-actions/setup-chrome@2e1d749697dd1612b833dba4a722266286fbefcd # v2
+    - uses: browser-actions/setup-chrome@48ad923757ca74d66703209fe939badbdf80f2f4 # v2
     - uses: ./.github/util/initialize
       with:
         dart-sdk: ${{ matrix.dart_channel }}

CHANGELOG.md

@@ -173,6 +173,39 @@
   spaces. You can pass the `space` parameter to explicitly convert to a new
   space.
 
+## 1.104.1-dev
+
+* Fix a bug in which an invalid selector at the end of an indented syntax
+  stylesheet would cause the parser to crash instead of emitting a useful error
+  message.
+
+### Command Line Interface
+
+* Many-to-many compilations no longer compile any files that appear in the
+  output directory if the output directory is also within the source directory.
+  This fixes a bug where `--watch` mode could enter an infinite loop recompiling
+  the same CSS file over and over.
+
+## 1.104.0
+
+* **Potentially breaking compatibility fix:** Colors now convert the special
+  values NaN and negative zero, as well as infinity and negative infinity for
+  polar-hue channels, to 0 as per the CSS spec.
+
+* The special value negative zero is now serialized as `-0` instead of `0` for
+  greater compatibility when using it in CSS calculations.
+
+## 1.103.1
+
+* No user-visible changes.
+
+## 1.103.0
+
+* **Potentially breaking compatibility fix:** Colors now preserve "analogous
+  sets" of missing channels during conversions, per the CSS spec. For example,
+  `color.to-space(lch(50% none none), lab)` now returns `lab(50% none none)`
+  instead of `lab(50% 0 0)`.
+
 ## 1.102.0
 
 * Use the 2.4 gamma transfer function for rec2020, as specified by the latest

lib/src/ast/sass/argument_list.dart

@@ -5,7 +5,6 @@
 import 'package:sass/src/utils.dart';
 import 'package:source_span/source_span.dart';
 
-import '../../value/list.dart';
 import '../../util/map.dart';
 import 'expression.dart';
 import 'expression/list.dart';

lib/src/ast/sass/expression/binary_operation.dart

@@ -14,16 +14,16 @@ import 'list.dart';
 /// A binary operator, as in `1 + 2` or `$this and $other`.
 ///
 /// {@category AST}
-final class BinaryOperationExpression extends Expression {
+final class BinaryOperationExpression(
   /// The operator being invoked.
-  final BinaryOperator operator;
+  final BinaryOperator operator,
 
   /// The left-hand operand.
-  final Expression left;
+  final Expression left,
 
   /// The right-hand operand.
-  final Expression right;
-
+  final Expression right,
+) extends Expression {
   @override
   FileSpan get span {
     // Avoid creating a bunch of intermediate spans for multiple binary
@@ -51,9 +51,6 @@ final class BinaryOperationExpression extends Expression {
             .span(left.span.end.offset, right.span.start.offset)
             .trim()
       : span;
-
-  new(this.operator, this.left, this.right);
-
   @override
   T accept<T>(ExpressionVisitor<T> visitor) =>
       visitor.visitBinaryOperationExpression(this);

lib/src/ast/selector.dart

@@ -61,10 +61,7 @@ abstract base class Selector(@override final FileSpan span) implements AstNode {
 }
 
 /// The visitor used to implement [Selector.isInvisible].
-final class const _IsInvisibleVisitor({
-  /// Whether to consider selectors with bogus combinators invisible.
-  required final bool includeBogus,
-}) with AnySelectorVisitor {
+final class const _IsInvisibleVisitor() with AnySelectorVisitor {
   @override
   bool visitSelectorList(SelectorList list) =>
       list.components.every(visitComplexSelector);

lib/src/ast/selector/complex.dart

@@ -23,12 +23,12 @@ import '../selector.dart';
 /// {@category AST}
 /// {@category Parsing}
 final class ComplexSelector(
-  /// This selector's leading combinator, if it has one.
-  final CssValue<Combinator>? leadingCombinator,
-
   Iterable<ComplexSelectorComponent> components,
   super.span, {
 
+  /// This selector's leading combinator, if it has one.
+  final CssValue<Combinator>? leadingCombinator,
+
   /// Whether a line break should be emitted *before* this selector.
   ///
   /// @nodoc

lib/src/async_environment.dart

@@ -984,7 +984,7 @@ final class _EnvironmentModule._(
   @override
   final List<Module> upstream = _environment._allModules;
 
-  _EnvironmentModule factory(
+  factory(
     AsyncEnvironment environment,
     CssStylesheet css,
     Map<Module, List<CssComment>> preModuleComments,

lib/src/callable/async_built_in.dart

@@ -7,7 +7,6 @@ import 'dart:async';
 import 'package:meta/meta.dart';
 
 import '../ast/sass.dart';
-import '../deprecation.dart';
 import '../evaluation_context.dart';
 import '../value.dart';
 import 'async.dart';