fix: review update

Author: cdcabrera

src/resource.helpers.ts

@@ -17,12 +17,23 @@ const paramCompletion = async (filters: FilterPatternFlyFilters) => {
   const schemas = new Set<string>();
 
   for (const entry of byEntry) {
-    names.add(entry.name);
-    categories.add(entry.category);
-    sections.add(entry.section);
-    versions.add(entry.version);
+    if (typeof entry.name === 'string') {
+      names.add(entry.name);
+    }
+
+    if (typeof entry.category === 'string') {
+      categories.add(entry.category);
+    }
+
+    if (typeof entry.section === 'string') {
+      sections.add(entry.section);
+    }
+
+    if (typeof entry.version === 'string') {
+      versions.add(entry.version);
+    }
 
-    if (entry.uriSchemas !== undefined) {
+    if (entry.uriSchemas !== undefined && typeof entry.name === 'string') {
       schemas.add(entry.name);
     }
   }

src/resource.patternFlyComponentsIndex.ts

@@ -46,7 +46,7 @@ const CONFIG = {
  * @note We use "byVersionComponentNames" instead of "byVersion" because it's specific to components.
  * Docs resources don't necessarily contain all components.
  *
- * @returns {Promise<PatterFlyListResourceResult>} The list of available resources.
+ * @returns {Promise<PatternFlyListResourceResult>} The list of available resources.
  */
 const listResources = async () => {
   const { availableVersions, byVersionComponentNames } = await getPatternFlyMcpResources.memo();
@@ -103,7 +103,7 @@ const uriCategoryComplete: ExtendedCompleteResourceTemplateCallback = async (cat
 uriCategoryComplete.memo = memo(uriCategoryComplete);
 
 /**
- * Name completion callback for the URI template.
+ * Version completion callback for the URI template.
  *
  * @param version - The value to complete.
  * @param context - The completion context containing arguments for the URI template.

src/resource.patternFlyDocsIndex.ts

@@ -66,7 +66,7 @@ const CONFIG = {
  *
  * @note It's important to keep lists focused and concise, avoid listing all resources.
  *
- * @returns {Promise<PatterFlyListResourceResult>} The list of available resources.
+ * @returns {Promise<PatternFlyListResourceResult>} The list of available resources.
  */
 const listResources = async () => {
   const { availableVersions, byVersion } = await getPatternFlyMcpResources.memo();

src/resource.patternFlySchemasIndex.ts

@@ -40,7 +40,7 @@ const CONFIG = {
 /**
  * List resources callback for the URI template.
  *
- * @returns {Promise<PatterFlyListResourceResult>} The list of available resources.
+ * @returns {Promise<PatternFlyListResourceResult>} The list of available resources.
  */
 const listResources = async () => {
   const { availableSchemasVersions, byVersionComponentNames } = await getPatternFlyMcpResources.memo();