Allow Node 17 to pass extra args to inspect. (#1631)

Author: Awjin

CHANGELOG.md

@@ -1,3 +1,10 @@
+## 1.49.8
+
+### JS API
+
+* Fix a bug where inspecting the Sass module in the Node.js console crashed on
+  Node 17.
+
 ## 1.49.7
 
 ### Embedded Sass

lib/src/import_cache.dart

@@ -130,9 +130,8 @@ class ImportCache {
           .putIfAbsent(Tuple4(url, forImport, baseImporter, baseUrl), () {
         var resolvedUrl = baseUrl?.resolveUri(url) ?? url;
         var canonicalUrl = _canonicalize(baseImporter, resolvedUrl, forImport);
-        if (canonicalUrl != null) {
-          return Tuple3(baseImporter, canonicalUrl, resolvedUrl);
-        }
+        if (canonicalUrl == null) return null;
+        return Tuple3(baseImporter, canonicalUrl, resolvedUrl);
       });
       if (relativeResult != null) return relativeResult;
     }

lib/src/node/reflection.dart

@@ -69,7 +69,7 @@ extension JSClassExtension on JSClass {
   /// Sets the custom inspect logic for this class to [body].
   void setCustomInspect(String inspect(Object self)) {
     setProperty(prototype, _inspectSymbol,
-        allowInteropCaptureThis((Object self, _, __) => inspect(self)));
+        allowInteropCaptureThis((Object self, _, __, [___]) => inspect(self)));
   }
 
   /// Defines a method with the given [name] and [body].

lib/src/visitor/statement_search.dart

@@ -105,5 +105,6 @@ extension _IterableExtension<E> on Iterable<E> {
       var value = callback(element);
       if (value != null) return value;
     }
+    return null;
   }
 }

pubspec.yaml

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