Reorder @forward rules in import-only files

Ensures that variables from indirect dependencies are still configurable
via `@import`.

Author: jathak

lib/src/migrators/module.dart

@@ -296,12 +296,12 @@ class _ModuleMigrationVisitor extends MigrationVisitor {
             _absoluteUrlToDependency(entry.key, relativeTo: importOnlyUrl)
                 .item1,
             entry.value)
-    ]..sort((a, b) => a.item2.compareTo(b.item2));
+    ];//..sort((a, b) => a.item2.compareTo(b.item2));
     var forwardLines = [
-      ...entrypointForwards,
       for (var tuple in tuples)
         ..._forwardRulesForShown(tuple.item1, tuple.item2, tuple.item3,
-            hiddenByUrl[tuple.item1] ?? {})
+            hiddenByUrl[tuple.item1] ?? {}),
+      ...entrypointForwards
     ];
     var semicolon = entrypoint.path.endsWith('.sass') ? '' : ';';
     return forwardLines.join('$semicolon\n') + '$semicolon\n';

test/migrators/module/forward_flag/import_only.hrx

@@ -19,5 +19,5 @@ a {
 }
 
 <==> output/entrypoint.import.scss
-@forward "entrypoint";
 @forward "library";
+@forward "entrypoint";

test/migrators/module/forward_flag/import_only_multiple_entrypoints.hrx

@@ -15,8 +15,8 @@ $lib-variable2: 1;
 $variable1: 1 + entrypoint2.$variable2;
 
 <==> output/entrypoint1.import.scss
-@forward "entrypoint1" as lib-*;
 @forward "entrypoint2" as lib-*;
+@forward "entrypoint1" as lib-*;
 
 <==> output/entrypoint2.scss
 $variable2: 1;

test/migrators/module/forward_flag/import_only_order.hrx

@@ -0,0 +1,37 @@
+<==> arguments
+--migrate-deps --forward=import-only
+
+<==> input/entrypoint.scss
+@import "c";
+@import "b";
+
+a {
+  b: $b-variable;
+  c: $c-variable;
+}
+
+<==> input/_b.scss
+$variable: blue !default;
+
+<==> input/_b.import.scss
+@forward "b" as b-*;
+
+<==> input/_c.scss
+$variable: gold !default;
+
+<==> input/_c.import.scss
+@forward "c" as c-*;
+
+<==> output/entrypoint.scss
+@use "c";
+@use "b";
+
+a {
+  b: b.$variable;
+  c: c.$variable;
+}
+
+<==> output/entrypoint.import.scss
+@forward "c" as c-*;
+@forward "b" as b-*;
+@forward "entrypoint";

test/migrators/module/forward_flag/import_only_prefixed.hrx

@@ -16,5 +16,5 @@ $variable: blue;
 $unprefixed: gold;
 
 <==> output/entrypoint.import.scss
-@forward "entrypoint" as lib-*;
 @forward "library" hide $variable;
+@forward "entrypoint" as lib-*;