fix: review updates

Author: cdcabrera

src/options.defaults.ts

@@ -176,7 +176,6 @@ interface ModeOptions {
  */
 interface PatternFlyOptions {
   availableResourceVersions: ('6.0.0')[];
-  // availableSearchVersions: ('current' | 'detected' | 'latest' | 'v3' | 'v4' | 'v5' | 'v6')[];
   availableSearchVersions: ('current' | 'detected' | 'latest' | 'v6')[];
   availableSchemasVersions: ('v6')[];
   default: {

src/patternFly.getResources.ts

@@ -299,8 +299,8 @@ const getPatternFlyMcpResources = async (contextPathOverride?: string): Promise<
         byUri[uriSchemas]?.push(extendedEntry);
       }
 
-      byVersion[entry.version] ??= [];
-      byVersion[entry.version]?.push(extendedEntry);
+      byVersion[version] ??= [];
+      byVersion[version]?.push(extendedEntry);
 
       resource.urls.push(path);
       resource.versions[version].urls.push(path);
@@ -363,7 +363,9 @@ const getPatternFlyComponentSchema = async (componentName: string) => {
     }
 
     return await getComponentSchema(updatedComponentName);
-  } catch {}
+  } catch (error) {
+    log.debug(`Failed to get component schemas for "${componentName}": ${formatUnknownError(error)}`);
+  }
 
   return undefined;
 };

src/patternFly.helpers.ts

@@ -32,7 +32,7 @@ const findClosestPatternFlyVersion = async (
   _contextPathOverride: string | undefined = undefined,
   options = getOptions()
 ): Promise<string> =>
-  options.patternflyOptions?.default?.latestSemVer;
+  options.patternflyOptions.default.latestSemVer;
 
 /**
  * Memoized version of findClosestPatternFlyVersion.
@@ -62,7 +62,7 @@ const getPatternFlyVersionContext = async (
   contextPathOverride: string | undefined = undefined,
   options = getOptions()
 ): Promise<PatternFlyVersionContext> => {
-  const availableSemVer = options.patternflyOptions?.availableResourceVersions;
+  const availableSemVer = options.patternflyOptions?.availableResourceVersions || [];
   const availableVersions = availableSemVer?.map?.(version => {
     const majorVersion = semver.coerce(version)?.major;
 

src/patternFly.search.ts

@@ -77,7 +77,7 @@ const searchPatternFly = async (searchQuery: string, {
   return {
     isSearchWildCardAll,
     firstExactMatch: exactMatches[0],
-    exactMatches: exactMatches,
+    exactMatches,
     searchResults: updatedSearchResults
   };
 };