Merge pull request #116 from aarondandy/issue-114-b

Fix casing issue from suggest early exit

Author: aarondandy

WeCantSpell.Hunspell.Tests/Issue106Tests.cs

@@ -26,18 +26,36 @@ public async ValueTask DisposeAsync() { }
     public void can_suggest_fr_designation()
     {
         var given = "Designation";
-        var actual = _wordListFr.Suggest(given, TestContext.Current.CancellationToken);
-        actual.ShouldContain("Désignation");
+        var actual = _wordListFr.Suggest(
+            given,
+            new QueryOptions()
+            {
+                TimeLimitSuggestStep = TimeSpan.FromSeconds(1),
+                TimeLimitCompoundCheck = TimeSpan.FromSeconds(1),
+                TimeLimitCompoundSuggest = TimeSpan.FromSeconds(1),
+                TimeLimitSuggestGlobal = TimeSpan.FromSeconds(10),
+            },
+            TestContext.Current.CancellationToken);
+        actual.ShouldContain("Désignation", StringComparer.Ordinal);
     }
 
     [Fact]
     public void can_suggest_es_descripcion()
     {
         var given = "Descripcion";
-        var actual = _wordListEs.Suggest(given, TestContext.Current.CancellationToken);
+        var actual = _wordListEs.Suggest(
+            given,
+            new QueryOptions()
+            {
+                TimeLimitSuggestStep = TimeSpan.FromSeconds(1),
+                TimeLimitCompoundCheck = TimeSpan.FromSeconds(1),
+                TimeLimitCompoundSuggest = TimeSpan.FromSeconds(1),
+                TimeLimitSuggestGlobal = TimeSpan.FromSeconds(10),
+            },
+            TestContext.Current.CancellationToken);
         // I don't know why but net48 randomly changes the case of the first letter.
         // It's probably worth investingating further and creating a new test for that
         // when the issue there is understood.
-        actual.ShouldContain("Descripción", StringComparer.InvariantCultureIgnoreCase);
+        actual.ShouldContain("Descripción", StringComparer.Ordinal);
     }
 }

WeCantSpell.Hunspell/WordList.QuerySuggest.cs

@@ -498,6 +498,8 @@ static void removeFromIndexThenInsertAtFront(List<string> list, int removeIndex,
                 }
             }
 
+        result:
+
             // word reversing wrapper for complex prefixes
             if (Affix.ComplexPrefixes)
             {
@@ -567,7 +569,6 @@ static void removeFromIndexThenInsertAtFront(List<string> list, int removeIndex,
             // output conversion
             Affix.OutputConversions.ConvertAll(slst);
 
-        result:
             return slst;
         }
 

changelog.md

@@ -2,6 +2,10 @@
 
 All notable changes to this project will be documented in this file.
 
+## 6.0.4 - 2025-11
+
+- Fix for suggest capitalization after early exit
+
 ## 6.0.3 - 2025-09
 
 - Improves overall performance a tiny bit