Emit deprecation

Author: ntkme

lib/src/async_compile.dart

@@ -194,6 +194,19 @@ Future<CompileResult> _compileStylesheet(
       'More info: https://sass-lang.com/d/legacy-js-api',
     );
   }
+  if (sourceMapIncludeSources == SourceMapIncludeSources.true_ ||
+      sourceMapIncludeSources == SourceMapIncludeSources.false_) {
+    logger?.warnForDeprecation(
+      Deprecation.sourceMapIncludeSourcesBoolean,
+      'Passing boolean value for Options.sourceMapIncludeSources is deprecated '
+      'and will be removed in Dart Sass 2.0.0.\n\n'
+      'More info: https://sass-lang.com/d/source-map-include-sources-boolean',
+    );
+    sourceMapIncludeSources =
+        sourceMapIncludeSources == SourceMapIncludeSources.true_
+            ? SourceMapIncludeSources.always
+            : SourceMapIncludeSources.never;
+  }
   var evaluateResult = await evaluateAsync(
     stylesheet,
     importCache: importCache,

lib/src/compile.dart

@@ -5,7 +5,7 @@
 // DO NOT EDIT. This file was generated from async_compile.dart.
 // See tool/grind/synchronize.dart for details.
 //
-// Checksum: 989c62e01aba777bc022501357a6f1be4e683c9c
+// Checksum: f4ec464ef3d537a5d315f3041d5afdfae6dd9911
 //
 // ignore_for_file: unused_import
 
@@ -203,6 +203,19 @@ CompileResult _compileStylesheet(
       'More info: https://sass-lang.com/d/legacy-js-api',
     );
   }
+  if (sourceMapIncludeSources == SourceMapIncludeSources.true_ ||
+      sourceMapIncludeSources == SourceMapIncludeSources.false_) {
+    logger?.warnForDeprecation(
+      Deprecation.sourceMapIncludeSourcesBoolean,
+      'Passing boolean value for Options.sourceMapIncludeSources is deprecated '
+      'and will be removed in Dart Sass 2.0.0.\n\n'
+      'More info: https://sass-lang.com/d/source-map-include-sources-boolean',
+    );
+    sourceMapIncludeSources =
+        sourceMapIncludeSources == SourceMapIncludeSources.true_
+            ? SourceMapIncludeSources.always
+            : SourceMapIncludeSources.never;
+  }
   var evaluateResult = evaluate(
     stylesheet,
     importCache: importCache,

lib/src/deprecation.dart

@@ -15,7 +15,7 @@ enum Deprecation {
   // DO NOT EDIT. This section was generated from the language repo.
   // See tool/grind/generate_deprecations.dart for details.
   //
-  // Checksum: 6fc524360d067b73c243c666e27a9a9ea7e08841
+  // Checksum: d6dc089a2bcab991dd117a6592f2f55d1caafccd
 
   /// Deprecation for passing a string directly to meta.call().
   callString('call-string',
@@ -151,6 +151,12 @@ enum Deprecation {
       deprecatedIn: '1.95.0',
       description: 'The Sass if(\$condition, \$if-true, \$if-false) function.'),
 
+  /// Deprecation for passing a boolean value to as Options.sourceMapIncludeSources.
+  sourceMapIncludeSourcesBoolean('source-map-include-sources-boolean',
+      deprecatedIn: '1.99.0',
+      description:
+          'Passing a boolean value to as Options.sourceMapIncludeSources.'),
+
   // END AUTOGENERATED CODE
 
   /// Used for deprecations coming from user-authored code.

lib/src/js/utils.dart

@@ -276,8 +276,10 @@ SourceMapIncludeSources parseSourceMapIncludeSources(
         Object? sourceMapIncludeSources) =>
     switch (sourceMapIncludeSources) {
       null || 'auto' => SourceMapIncludeSources.auto,
-      true || 'always' => SourceMapIncludeSources.always,
-      false || 'never' => SourceMapIncludeSources.never,
+      'always' => SourceMapIncludeSources.always,
+      'never' => SourceMapIncludeSources.never,
+      true => SourceMapIncludeSources.true_,
+      false => SourceMapIncludeSources.false_,
       _ => jsThrow(JsError(
           'Unknown sourceMapIncludeSources "$sourceMapIncludeSources".')),
     };

lib/src/source_map_include_sources.dart

@@ -14,4 +14,10 @@ enum SourceMapIncludeSources {
 
   /// Never include source contents.
   never,
+
+  @Deprecated('Use always instead.')
+  true_,
+
+  @Deprecated('Use never instead.')
+  false_,
 }

lib/src/util/source_map.dart

@@ -7,9 +7,15 @@ import 'package:source_maps/source_maps.dart';
 import '../source_map_include_sources.dart';
 
 Map<String, dynamic> sourceMapToJson(SingleMapping sourceMap,
-        {required SourceMapIncludeSources sourceMapIncludeSources}) =>
-    sourceMap.toJson(
-        includeSourceContents:
-            sourceMapIncludeSources == SourceMapIncludeSources.always ||
-                (sourceMapIncludeSources == SourceMapIncludeSources.auto &&
-                    sourceMap.files.any((file) => file != null)));
+    {required SourceMapIncludeSources sourceMapIncludeSources}) {
+  if (sourceMapIncludeSources == SourceMapIncludeSources.true_) {
+    sourceMapIncludeSources = SourceMapIncludeSources.always;
+  } else if (sourceMapIncludeSources == SourceMapIncludeSources.false_) {
+    sourceMapIncludeSources = SourceMapIncludeSources.never;
+  }
+  return sourceMap.toJson(
+      includeSourceContents:
+          sourceMapIncludeSources == SourceMapIncludeSources.always ||
+              (sourceMapIncludeSources == SourceMapIncludeSources.auto &&
+                  sourceMap.files.any((file) => file != null)));
+}