Use generic "private" prefix instead of generated variable counter

Author: jathak

CHANGELOG.md

@@ -1,3 +1,11 @@
+## 2.3.3
+
+### Module System Migration
+
+* Fix some bugs in the conversion of private names that are references across
+  files to public names, especially when `--remove-prefix` and/or multiple
+  leading dashes/underscores are involved.
+
 ## 2.3.2
 
 * Update to be compatible with the latest version of the Dart Sass AST.

lib/src/migrators/module.dart

@@ -192,9 +192,6 @@ class _ModuleMigrationVisitor extends MigrationVisitor {
           "Can't access _configuredVariables when not visiting a dependency."));
   Set<MemberDeclaration<VariableDeclaration>>? __configuredVariables;
 
-  /// The number of variables that have been generated from whole cloth.
-  var _generatedVariables = 0;
-
   /// A mapping between member declarations and references.
   ///
   /// This performs an initial pass to determine how a declaration seen in the
@@ -384,8 +381,7 @@ class _ModuleMigrationVisitor extends MigrationVisitor {
         }
       }
 
-      _generatedVariables++;
-      return 'var${_generatedVariables}';
+      return 'private$identifier';
     }
     return identifier;
   }

pubspec.yaml

@@ -1,5 +1,5 @@
 name: sass_migrator
-version: 2.3.2
+version: 2.3.3
 description: A tool for running migrations on Sass files
 homepage: https://github.com/sass/migrator
 

test/migrators/module/scope/pseudoprivate/dashes_only.hrx

@@ -12,8 +12,8 @@ $---: red;
 <==> output/entrypoint.scss
 @use "library";
 a {
-  color: library.$var1;
+  color: library.$private---;
 }
 
 <==> output/_library.scss
-$var1: red;
+$private---: red;