fix: review update

Author: cdcabrera

src/resource.patternFlyDocsTemplate.ts

@@ -166,7 +166,7 @@ const resourceCallback = async (passedUri: URL, variables: Record<string, string
       contents: byEntry.filter(entry => entry.uriSchemasId).map(entry => ({
         uri: entry.uriId,
         mimeType: 'text/markdown',
-        text: `# ${entry.displayName}\n\nNo documentation is available for this component, but you can find its [JSON Schema](${entry.uriSchemasId}).`
+        text: `# ${entry.displayName}\n\nNo documentation is available for this component. But a [JSON schema is available](${entry.uriSchemasId}).`
       }))
     };
   }

src/tool.searchPatternFly.ts

@@ -1,7 +1,7 @@
 import { ErrorCode } from '@modelcontextprotocol/sdk/types.js';
 import { z } from 'zod';
 import { type McpTool } from './mcpSdk';
-import { stringJoin, buildSearchString } from './server.helpers';
+import { stringJoin } from './server.helpers';
 import { assertInput, assertInputStringLength, assertInputStringNumberEnumLike } from './server.assertions';
 import { findClosest } from './server.search';
 import { getOptions } from './options.context';
@@ -106,11 +106,9 @@ const searchPatternFlyTool = (options = getOptions()): McpTool => {
           numberRecords += 1;
           recordNames.add(record.displayName);
 
-          const recordUri = `${record.uriId}${buildSearchString({ version: record.version }, { prefix: true })}`;
-
-          results.set(recordUri, {
+          results.set(record.uriId, {
             type: 'resource_link',
-            uri: recordUri,
+            uri: record.uriId,
             name: `${record.displayName} - ${record.displayCategory} (${record.version})`,
             description: record.description,
             mimeType: 'text/markdown',
@@ -122,11 +120,9 @@ const searchPatternFlyTool = (options = getOptions()): McpTool => {
           numberRecords += 1;
           recordNames.add(record.displayName);
 
-          const schemaUri = `${record.uriSchemasId}${buildSearchString({ version: record.version }, { prefix: true })}`;
-
-          results.set(schemaUri, {
+          results.set(record.uriSchemasId, {
             type: 'resource_link',
-            uri: schemaUri,
+            uri: record.uriSchemasId,
             name: `${record.displayName} - JSON Schema (${record.version})`,
             description: `Component JSON schema with property definitions for ${record.displayName}.`,
             mimeType: 'text/markdown',
@@ -135,7 +131,15 @@ const searchPatternFlyTool = (options = getOptions()): McpTool => {
         }
       });
 
-      if (!result.entries.length) {
+      /**
+       * FIXME: Schemas are intended to be part of collections. We're temporarily expanding our limit condition
+       * to include `!result.entries.some(entry => Boolean(entry.path)` to make sure if a collection only has JSON
+       * schemas available, the "collection" doesn't appear; this is specfically to work with the current MCP
+       * resource structure. In the future when we've focused our MCP resources down to "collections" and
+       * "records" we would review dropping this part of the check and allow a "collection" grouping to appear
+       * if all it had were JSON schemas.
+       */
+      if (!result.entries.length || !result.entries.some(entry => Boolean(entry.path))) {
         return;
       }
 
@@ -146,14 +150,10 @@ const searchPatternFlyTool = (options = getOptions()): McpTool => {
       // Track the name used for sorting this entire group
       groupSortNames.set(result.groupId, collectionName.toLowerCase());
 
-      const hasDocs = result.entries.some(entry => Boolean(entry.path));
-      const versionQuery = buildSearchString({ version: updatedVersion }, { prefix: true });
-      const collectionUri = `patternfly://${hasDocs ? 'docs' : 'schemas'}/${result.groupId}${versionQuery}`;
-
       results.set(result.groupId, {
         type: 'resource_link',
-        uri: collectionUri,
-        name: `${collectionName} (${hasDocs ? 'Collection' : 'Schema Collection'})`,
+        uri: `patternfly://docs/${result.groupId}`,
+        name: `${collectionName} (Collection)`,
         description: `A resource collection series for ${collectionNames}`,
         mimeType: 'text/markdown',
         groupId: result.groupId