Fix provider search prioritization to show official providers first

[koke1997]

Fixes #178

The search_providers tool was returning community providers before official ones in search results. For example, searching for 'keycloak' would show mrparkers/keycloak (community) before keycloak/keycloak (official).

Changes made:

1. Enhanced search_providers.go to use Registry v2 provider list API

• Fetches provider tier information (official/partner/community)
• Implements tier-based sorting: official > partner > community
• Maintains relevance-based sorting within each tier
• Falls back to original behavior if provider list unavailable

2. Modified registry.go to support test overrides

• Added SendRegistryCallFn variable for mocking in tests
• Allows unit tests to run without network dependencies

3. Added unit tests:

• sort_test.go: Tests tier-based sorting logic
• search_providers_test.go: Integration test with mocked responses
• Ensures prioritization works correctly and prevents regressions

The implementation uses the existing Registry v2 API and maintains backward compatibility. Official and partner providers now appear first in search results, improving discoverability of authoritative providers.

Testing

All existing tests continue to pass, and new tests verify the prioritization logic works correctly.

[hashicorp-cla-app]

All committers have signed the CLA.

[Copilot]

Pull Request Overview

This PR fixes provider search prioritization to display official providers before community providers in search results. The changes implement tier-based sorting using the Registry v2 API to fetch provider tier information and prioritize results accordingly.

Key changes:

• Enhanced search functionality to use Registry v2 provider list API for tier-based sorting
• Added test infrastructure with mocking capabilities for registry calls
• Implemented comprehensive unit tests for the new prioritization logic

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
pkg/tools/registry/sort_test.go	Unit test for tier-based sorting functionality
pkg/tools/registry/search_providers_test.go	Integration test verifying official provider prioritization
pkg/tools/registry/search_providers.go	Core implementation of enhanced provider search with tier-based sorting
pkg/client/registry.go	Added function variable for registry call mocking in tests
CHANGELOG.md	Documentation of the fix

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[gautambaghel]

Hi @koke1997 - the reason we didn't invoke the v2 APIs for search_providers is because it consistently timed out, I think the layering of APIs is too much

could you try it on your end with MCP inspector and perhaps record a short demo if possible because it's timing out on my end

[gautambaghel]

Requesting a test on MCP inspector ^^, with some tests covering keycloak, AWS, Azure & GCP providers