Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 12, 2025

Vault filtering/searching was completely broken - all searches returned empty results regardless of query.

works as expeected:

image

Root Cause

FilterService.Filter() created filtered subscription objects but discarded them immediately, then passed an empty list to the sort function:

// Before: filtered results were created but never collected
foreach (var subscription in allSubscriptions)
{
    // ... filtering logic ...
    if (filteredResourceGroups.Count != 0)
    {
        var filteredSubscription = new KvSubscriptionModel { /* ... */ };
        new List<KvSubscriptionModel>().Add(filteredSubscription); // ❌ discarded
    }
}
var sortedSubscriptions = SortService.SortSubscriptions(new List<KvSubscriptionModel>()); // ❌ empty list

Changes

  • Added filteredSubscriptions list to accumulate results
  • Changed to add filtered subscriptions to the collection
  • Pass collected subscriptions to sort function
// After: filtered results are properly collected and returned
var filteredSubscriptions = new List<KvSubscriptionModel>();
foreach (var subscription in allSubscriptions)
{
    // ... filtering logic ...
    if (filteredResourceGroups.Count != 0)
    {
        var filteredSubscription = new KvSubscriptionModel { /* ... */ };
        filteredSubscriptions.Add(filteredSubscription); // ✅ collected
    }
}
var sortedSubscriptions = SortService.SortSubscriptions(filteredSubscriptions); // ✅ actual results

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • av-build-tel-api-v1.avaloniaui.net
    • Triggering command: /usr/share/dotnet/dotnet dotnet exec --runtimeconfig /home/REDACTED/.nuget/packages/avalonia.buildservices/0.0.31/tools/netstandard2.0/runtimeconfig.json /home/REDACTED/.nuget/packages/avalonia.buildservices/0.0.31/tools/netstandard2.0/Avalonia.BuildServices.Collector.dll (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Filtering/Searching of vaults is broken</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)

Copilot AI changed the title [WIP] Fix broken filtering and searching of vaults Fix vault filtering returning empty results Dec 12, 2025
Copilot AI requested a review from cricketthomas December 12, 2025 04:43
@cricketthomas cricketthomas force-pushed the copilot/fix-vaults-search-filtering branch from 5b4786f to 3a92595 Compare December 12, 2025 13:06
@cricketthomas cricketthomas marked this pull request as ready for review December 12, 2025 13:06
@cricketthomas cricketthomas merged commit b820774 into master Dec 12, 2025
2 checks passed
@cricketthomas cricketthomas deleted the copilot/fix-vaults-search-filtering branch December 12, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants