Fix deprecation ID displays in JS mode (#2483)

In the pure JS release, these weren't working because the Dart-to-JS
logger didn't know to pass along the deprecation metadata when falling
back to the wrapped StderrLogger. In the embedded release, these
weren't working because we didn't give it an update to match the
stderr logger.

Author: nex3

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.83.2
+
+* Properly display deprecation IDs for the JS Sass API.
+
+* Don't display deprecation IDs for user-defined deprecations.
+
 ## 1.83.1
 
 * Fix a bug where `--quiet-deps` would get deactivated for `@content` blocks,

lib/src/embedded/logger.dart

@@ -44,13 +44,17 @@ final class EmbeddedLogger extends LoggerWithDeprecationType {
       {FileSpan? span, Trace? trace, Deprecation? deprecation}) {
     var formatted = withGlyphs(() {
       var buffer = StringBuffer();
+      var showDeprecation =
+          deprecation != null && deprecation != Deprecation.userAuthored;
       if (_color) {
         buffer.write('\u001b[33m\u001b[1m');
         if (deprecation != null) buffer.write('Deprecation ');
         buffer.write('Warning\u001b[0m');
+        if (showDeprecation) buffer.write(' [\u001b[34m$deprecation\u001b[0m]');
       } else {
         if (deprecation != null) buffer.write('DEPRECATION ');
         buffer.write('WARNING');
+        if (showDeprecation) buffer.write(' [$deprecation]');
       }
       if (span == null) {
         buffer.writeln(': $message');

lib/src/logger/js_to_dart.dart

@@ -41,8 +41,14 @@ final class JSToDartLogger extends LoggerWithDeprecationType {
               deprecationType: deprecations[deprecation?.id]));
     } else {
       _withAscii(() {
-        _fallback.warn(message,
-            span: span, trace: trace, deprecation: deprecation != null);
+        switch (_fallback) {
+          case LoggerWithDeprecationType():
+            _fallback.internalWarn(message,
+                span: span, trace: trace, deprecation: deprecation);
+          case _:
+            _fallback.warn(message,
+                span: span, trace: trace, deprecation: deprecation != null);
+        }
       });
     }
   }

lib/src/logger/stderr.dart

@@ -21,18 +21,18 @@ final class StderrLogger extends LoggerWithDeprecationType {
   void internalWarn(String message,
       {FileSpan? span, Trace? trace, Deprecation? deprecation}) {
     var result = StringBuffer();
+    var showDeprecation =
+        deprecation != null && deprecation != Deprecation.userAuthored;
     if (color) {
       // Bold yellow.
       result.write('\u001b[33m\u001b[1m');
       if (deprecation != null) result.write('Deprecation ');
       result.write('Warning\u001b[0m');
-      if (deprecation != null) {
-        result.write(' [\u001b[34m$deprecation\u001b[0m]');
-      }
+      if (showDeprecation) result.write(' [\u001b[34m$deprecation\u001b[0m]');
     } else {
       if (deprecation != null) result.write('DEPRECATION ');
       result.write('WARNING');
-      if (deprecation != null) result.write(' [$deprecation]');
+      if (showDeprecation) result.write(' [$deprecation]');
     }
 
     if (span == null) {

pkg/sass-parser/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.4.10
+
+* No user-visible changes.
+
 ## 0.4.9
 
 * Add support for parsing the `@include` rule.

pkg/sass-parser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sass-parser",
-  "version": "0.4.9",
+  "version": "0.4.10",
   "description": "A PostCSS-compatible wrapper of the official Sass parser",
   "repository": "sass/sass",
   "author": "Google Inc.",

pkg/sass_api/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 15.0.2
+
+* No user-visible changes.
+
 ## 15.0.1
 
 * No user-visible changes.

pkg/sass_api/pubspec.yaml

@@ -2,15 +2,15 @@ name: sass_api
 # Note: Every time we add a new Sass AST node, we need to bump the *major*
 # version because it's a breaking change for anyone who's implementing the
 # visitor interface(s).
-version: 15.0.1
+version: 15.0.2
 description: Additional APIs for Dart Sass.
 homepage: https://github.com/sass/dart-sass
 
 environment:
   sdk: ">=3.3.0 <4.0.0"
 
 dependencies:
-  sass: 1.83.1
+  sass: 1.83.2
 
 dev_dependencies:
   dartdoc: ^8.0.14

pubspec.yaml

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