Code review + make this a -dev release

lib/src/ast/sass/expression.dart

@@ -50,5 +50,5 @@ abstract class Expression implements SassNode {
   ///
   /// Throws a [SassFormatException] if parsing fails.
   factory Expression.parse(String contents, {Object? url}) =>
-      ScssParser(contents, url: url).parseExpression().node;
+      ScssParser(contents, url: url).parseExpression().$1;
 }

lib/src/ast/sass/statement/use_rule.dart

@@ -56,7 +56,7 @@ final class UseRule extends Statement implements SassDependency {
   /// @nodoc
   @internal
   factory UseRule.parse(String contents, {Object? url}) =>
-      ScssParser(contents, url: url).parseUseRule().node;
+      ScssParser(contents, url: url).parseUseRule().$1;
 
   T accept<T>(StatementVisitor<T> visitor) => visitor.visitUseRule(this);
 

lib/src/ast/sass/statement/variable_declaration.dart

@@ -81,7 +81,7 @@ final class VariableDeclaration extends Statement implements SassDeclaration {
   /// @nodoc
   @internal
   factory VariableDeclaration.parse(String contents, {Object? url}) =>
-      ScssParser(contents, url: url).parseVariableDeclaration().node;
+      ScssParser(contents, url: url).parseVariableDeclaration().$1;
 
   T accept<T>(StatementVisitor<T> visitor) =>
       visitor.visitVariableDeclaration(this);

lib/src/async_import_cache.dart

@@ -233,6 +233,9 @@ final class AsyncImportCache {
 
     if (result == null) return (null, cacheable);
 
+    // Relative canonical URLs (empty scheme) should throw an error starting in
+    // Dart Sass 2.0.0, but for now, they only emit a deprecation warning in
+    // the evaluator.
     if (result.scheme != '' &&
         await importer.isNonCanonicalScheme(result.scheme)) {
       throw "Importer $importer canonicalized $url to $result, which uses a "

lib/src/executable/repl.dart

@@ -49,20 +49,20 @@ Future<void> repl(ExecutableOptions options) async {
     if (line.trim().isEmpty) continue;
     try {
       if (line.startsWith("@")) {
-        var (:node, :warnings) = ScssParser(line).parseUseRule();
+        var (node, warnings) = ScssParser(line).parseUseRule();
         warnings.forEach(warn);
         evaluator.use(node);
         continue;
       }
 
       if (Parser.isVariableDeclarationLike(line)) {
-        var (:node, :warnings) = ScssParser(line).parseVariableDeclaration();
+        var (node, warnings) = ScssParser(line).parseVariableDeclaration();
         warnings.forEach(warn);
         evaluator.setVariable(node);
         print(evaluator.evaluate(VariableExpression(node.name, node.span,
             namespace: node.namespace)));
       } else {
-        var (:node, :warnings) = ScssParser(line).parseExpression();
+        var (node, warnings) = ScssParser(line).parseExpression();
         warnings.forEach(warn);
         print(evaluator.evaluate(node));
       }

lib/src/parse/stylesheet.dart

@@ -123,31 +123,30 @@ abstract class StylesheetParser extends Parser {
         return arguments;
       });
 
-  ({Expression node, List<ParseTimeWarning> warnings}) parseExpression() =>
-      (node: _parseSingleProduction(_expression), warnings: warnings);
+  (Expression, List<ParseTimeWarning>) parseExpression() =>
+      (_parseSingleProduction(_expression), warnings);
 
   SassNumber parseNumber() {
     var expression = _parseSingleProduction(_number);
     return SassNumber(expression.value, expression.unit);
   }
 
-  ({VariableDeclaration node, List<ParseTimeWarning> warnings})
-      parseVariableDeclaration() => (
-            node: _parseSingleProduction(() => lookingAtIdentifier()
-                ? _variableDeclarationWithNamespace()
-                : variableDeclarationWithoutNamespace()),
-            warnings: warnings
-          );
+  (VariableDeclaration, List<ParseTimeWarning>) parseVariableDeclaration() => (
+        _parseSingleProduction(() => lookingAtIdentifier()
+            ? _variableDeclarationWithNamespace()
+            : variableDeclarationWithoutNamespace()),
+        warnings
+      );
 
-  ({UseRule node, List<ParseTimeWarning> warnings}) parseUseRule() => (
-        node: _parseSingleProduction(() {
+  (UseRule, List<ParseTimeWarning>) parseUseRule() => (
+        _parseSingleProduction(() {
           var start = scanner.state;
           scanner.expectChar($at, name: "@-rule");
           expectIdentifier("use");
           whitespace();
           return _useRule(start);
         }),
-        warnings: warnings
+        warnings
       );
 
   /// Parses and returns [production] as the entire contents of [scanner].

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass
-version: 1.80.3
+version: 1.80.3-dev
 description: A Sass implementation in Dart.
 homepage: https://github.com/sass/dart-sass