Update sass_api to 15.4.0 (#277)

Author: jathak

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.3.2
+
+* Update to be compatible with the latest version of the Dart Sass AST.
+
 ## 2.3.1
 
 ### Color Function Migrator

lib/src/migrators/color.dart

@@ -167,7 +167,7 @@ class _ColorMigrationVisitor extends MigrationVisitor {
       {required String channel, bool negate = false, String? space}) sync* {
     yield Patch(node.nameSpan, 'adjust');
     switch (node.arguments) {
-      case ArgumentInvocation(positional: [_, var adjustment]):
+      case ArgumentList(positional: [_, var adjustment]):
         yield patchBefore(adjustment, '\$$channel: ${negate ? '-' : ''}');
         if (negate && adjustment.needsParens) {
           yield patchBefore(adjustment, '(');
@@ -177,7 +177,7 @@ class _ColorMigrationVisitor extends MigrationVisitor {
           yield patchAfter(adjustment, ', \$space: $space');
         }
 
-      case ArgumentInvocation(
+      case ArgumentList(
           named: {'amount': var adjustment} || {'degrees': var adjustment}
         ):
         var start = adjustment.span.start.offset - 1;

lib/src/migrators/division.dart

@@ -165,11 +165,10 @@ class _DivisionMigrationVisitor extends MigrationVisitor {
     return '$namespace.';
   }
 
-  /// Allows division within this argument invocation.
+  /// Allows division within this argument list.
   @override
-  void visitArgumentInvocation(ArgumentInvocation invocation,
-      {bool inCalcContext = false}) {
-    _withContext(() => super.visitArgumentInvocation(invocation),
+  void visitArgumentList(ArgumentList arguments, {bool inCalcContext = false}) {
+    _withContext(() => super.visitArgumentList(arguments),
         isDivisionAllowed: true, inCalcContext: inCalcContext);
   }
 
@@ -195,7 +194,7 @@ class _DivisionMigrationVisitor extends MigrationVisitor {
   void visitFunctionExpression(FunctionExpression node) {
     if (_tryColorFunction(node)) return;
     var validCalcs = _calcFunctions.difference(currentScope.allFunctionNames);
-    visitArgumentInvocation(node.arguments,
+    visitArgumentList(node.arguments,
         inCalcContext:
             node.namespace == null && validCalcs.contains(node.name));
   }
@@ -274,15 +273,12 @@ class _DivisionMigrationVisitor extends MigrationVisitor {
     }
 
     var channels = switch (node.arguments) {
-      ArgumentInvocation(
+      ArgumentList(
         positional: [ListExpression arg],
         named: Map(isEmpty: true)
       ) =>
         arg,
-      ArgumentInvocation(
-        positional: [],
-        named: {r'$channels': ListExpression arg}
-      ) =>
+      ArgumentList(positional: [], named: {r'$channels': ListExpression arg}) =>
         arg,
       _ => null,
     };

lib/src/util/scoped_ast_visitor.dart

@@ -62,9 +62,9 @@ abstract class ScopedAstVisitor
   @override
   void visitCallableDeclaration(CallableDeclaration node) {
     scoped(() {
-      for (var argument in node.arguments.arguments) {
-        currentScope.variables[argument.name] = MemberDeclaration(argument);
-        if (argument.defaultValue case var defaultValue?) {
+      for (var parameter in node.parameters.parameters) {
+        currentScope.variables[parameter.name] = MemberDeclaration(parameter);
+        if (parameter.defaultValue case var defaultValue?) {
           visitExpression(defaultValue);
         }
       }

lib/src/utils.dart

@@ -157,13 +157,13 @@ void emitWarning(String message, [FileSpan? context]) {
   }
 }
 
-/// Returns the only argument in [invocation], or null if [invocation] does not
+/// Returns the only argument in [arguments], or null if [arguments] does not
 /// contain exactly one argument.
-Expression? getOnlyArgument(ArgumentInvocation invocation) {
-  if (invocation.positional.length == 0 && invocation.named.length == 1) {
-    return invocation.named.values.first;
-  } else if (invocation.positional.length == 1 && invocation.named.isEmpty) {
-    return invocation.positional.first;
+Expression? getOnlyArgument(ArgumentList arguments) {
+  if (arguments.positional.length == 0 && arguments.named.length == 1) {
+    return arguments.named.values.first;
+  } else if (arguments.positional.length == 1 && arguments.named.isEmpty) {
+    return arguments.positional.first;
   } else {
     return null;
   }

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass_migrator
-version: 2.3.1
+version: 2.3.2
 description: A tool for running migrations on Sass files
 homepage: https://github.com/sass/migrator
 
@@ -17,7 +17,7 @@ dependencies:
   node_interop: ^2.0.2
   node_io: ^2.3.0
   path: ^1.8.0
-  sass_api: ^14.4.0
+  sass_api: ^15.4.0
   source_span: ^1.8.1
   stack_trace: ^1.10.0
   string_scanner: ^1.1.0