fix: keep all resources registered for resources/read requests

The ForMCPRequest optimization was incorrectly filtering resources by
doing an exact string match between the URI template pattern and the
concrete URI. This would never match because templates like
'repo://{owner}/{repo}/contents{/path*}' don't match concrete URIs
like 'repo://owner/repo/contents/file.py'.

Instead of implementing template matching in the inventory, we simply
keep all resources registered for resources/read requests and let the
SDK handle URI template matching internally (which it already does
correctly via uritemplate.Regexp().MatchString()).

This fixes resources/read returning 'Resource not found' for valid URIs.

Author: SamMorrowDrums

pkg/inventory/registry.go

@@ -91,7 +91,7 @@ const (
 //   - MCPMethodToolsList: All available tools (no resources/prompts)
 //   - MCPMethodToolsCall: Only the named tool
 //   - MCPMethodResourcesList, MCPMethodResourcesTemplatesList: All available resources (no tools/prompts)
-//   - MCPMethodResourcesRead: Only the named resource template
+//   - MCPMethodResourcesRead: All resources (SDK handles URI template matching)
 //   - MCPMethodPromptsList: All available prompts (no tools/resources)
 //   - MCPMethodPromptsGet: Only the named prompt
 //   - Unknown methods: Empty (no items registered)
@@ -134,10 +134,8 @@ func (r *Inventory) ForMCPRequest(method string, itemName string) *Inventory {
 	case MCPMethodResourcesList, MCPMethodResourcesTemplatesList:
 		result.tools, result.prompts = nil, nil
 	case MCPMethodResourcesRead:
+		// Keep all resources registered - SDK handles URI template matching internally
 		result.tools, result.prompts = nil, nil
-		if itemName != "" {
-			result.resourceTemplates = r.filterResourcesByURI(itemName)
-		}
 	case MCPMethodPromptsList:
 		result.tools, result.resourceTemplates = nil, nil
 	case MCPMethodPromptsGet:

pkg/inventory/registry_test.go

@@ -775,17 +775,15 @@ func TestForMCPRequest_ResourcesRead(t *testing.T) {
 	}
 
 	reg := NewBuilder().SetResources(resources).WithToolsets([]string{"all"}).Build()
-	filtered := reg.ForMCPRequest(MCPMethodResourcesRead, "repo://{owner}/{repo}")
+	// Pass a concrete URI - all resources remain registered, SDK handles matching
+	filtered := reg.ForMCPRequest(MCPMethodResourcesRead, "repo://owner/repo")
 
+	// All resources should be available - SDK handles URI template matching internally
 	available := filtered.AvailableResourceTemplates(context.Background())
-	if len(available) != 1 {
-		t.Fatalf("Expected 1 resource for resources/read, got %d", len(available))
-	}
-	if available[0].Template.URITemplate != "repo://{owner}/{repo}" {
-		t.Errorf("Expected URI template 'repo://{owner}/{repo}', got %q", available[0].Template.URITemplate)
+	if len(available) != 2 {
+		t.Fatalf("Expected 2 resources for resources/read (SDK handles matching), got %d", len(available))
 	}
 }
-
 func TestForMCPRequest_PromptsList(t *testing.T) {
 	tools := []ServerTool{
 		mockTool("tool1", "repos", true),