Fixing merge issues

.github/actions/spell-check/expect.txt

@@ -1221,6 +1221,7 @@ opensource
 openxmlformats
 ollama
 onnx
+openurl
 OPTIMIZEFORINVOKE
 ORPHANEDDIALOGTITLE
 ORSCANS

src/modules/cmdpal/Microsoft.CmdPal.UI.ViewModels/Commands/MainListPage.cs

@@ -162,11 +162,19 @@ public partial class MainListPage : DynamicListPage,
             }
             else
             {
+                var validScoredFallbacks = _scoredFallbackItems?
+                    .Where(s => !string.IsNullOrWhiteSpace(s.Item.Title))
+                    .ToList();
+
+                var validFallbacks = _fallbackItems?
+                    .Where(s => !string.IsNullOrWhiteSpace(s.Item.Title))
+                    .ToList();
+
                 return MainListPageResultFactory.Create(
                     _filteredItems,
-                    _scoredFallbackItems?.ToList(),
+                    validScoredFallbacks,
                     _filteredApps,
-                    _fallbackItems?.ToList(),
+                    validFallbacks,
                     _appResultLimit);
             }
         }

src/modules/cmdpal/Microsoft.CmdPal.UI.ViewModels/Commands/MainListPageResultFactory.cs

@@ -30,22 +30,17 @@ internal static class MainListPageResultFactory
 
         int len1 = filteredItems?.Count ?? 0;
 
-        // Scored or not, fallbacks may contain empty items that we want to skip.
+        // Empty fallbacks are removed prior to this merge.
-        int len2 = GetNonEmptyFallbackItemsCount(scoredFallbackItems);
+        int len2 = scoredFallbackItems?.Count ?? 0;
 
         // Apps are pre-sorted, so we just need to take the top N, limited by appResultLimit.
         int len3 = Math.Min(filteredApps?.Count ?? 0, appResultLimit);
 
-        int nonEmptyFallbackCount = GetNonEmptyFallbackItemsCount(fallbackItems);
+        int nonEmptyFallbackCount = fallbackItems?.Count ?? 0;
 
         // Allocate the exact size of the result array.
-        int totalCount = len1 + len2 + len3 + nonEmptyFallbackCount;
+        // We'll add an extra slot for the fallbacks section header if needed.
-
+        int totalCount = len1 + len2 + len3 + nonEmptyFallbackCount + (nonEmptyFallbackCount > 0 ? 1 : 0);
-        // If there are non-empty fallbacks, we'll be adding a separator "Section"
-        if (nonEmptyFallbackCount > 0)
-        {
-            totalCount++;
-        }
 
         var result = new IListItem[totalCount];
 
@@ -134,7 +129,10 @@ internal static class MainListPageResultFactory
         if (fallbackItems is not null)
         {
             // Create the fallbacks section header
-            result[writePos++] = new Separator(Properties.Resources.fallbacks);
+            if (fallbackItems.Count > 0)
+            {
+                result[writePos++] = new Separator(Properties.Resources.fallbacks);
+            }
 
             for (int i = 0; i < fallbackItems.Count; i++)
             {

src/modules/cmdpal/Tests/Microsoft.CmdPal.UI.ViewModels.UnitTests/MainListPageResultFactoryTests.cs

@@ -96,7 +96,7 @@ public partial class MainListPageResultFactoryTests
         var titles = result.Select(r => r.Title).ToArray();
 #pragma warning disable CA1861 // Avoid constant arrays as arguments
         CollectionAssert.AreEqual(
-            new[] { "F1", "SF1", "A1", "SF2", "A2", "F2", "FB1", "FB2" },
+            new[] { "F1", "SF1", "A1", "SF2", "A2", "F2", "Fallbacks", "FB1", "FB2" },
             titles);
 #pragma warning restore CA1861 // Avoid constant arrays as arguments
     }
@@ -129,7 +129,6 @@ public partial class MainListPageResultFactoryTests
         var fallbacks = new List<Scored<IListItem>>
         {
             S("FB1", 0),
-            S(string.Empty, 0),
             S("FB3", 0),
         };
 
@@ -140,9 +139,10 @@ public partial class MainListPageResultFactoryTests
             fallbacks,
             appResultLimit: 10);
 
-        Assert.AreEqual(2, result.Length);
+        Assert.AreEqual(3, result.Length);
-        Assert.AreEqual("FB1", result[0].Title);
+        Assert.AreEqual("Fallbacks", result[0].Title);
-        Assert.AreEqual("FB3", result[1].Title);
+        Assert.AreEqual("FB1", result[1].Title);
+        Assert.AreEqual("FB3", result[2].Title);
     }
 
     [TestMethod]