fix: review update

Author: cdcabrera

src/__tests__/server.resourceMeta.test.ts

@@ -81,6 +81,29 @@ describe('setMetadataOptions', () => {
     expect(content).toContain('# Test Config Metadata');
   });
 
+  it('should merge values from multiple complete callbacks', async () => {
+    const completeVersion = jest.fn().mockResolvedValue(['v1']);
+    const completeCategory = jest.fn().mockResolvedValue(['cat1']);
+    const options = setMetadataOptions({
+      name: 'test',
+      baseUri: 'test://uri',
+      searchParams: [],
+      config: { title: 'Test Multiple Callbacks' } as any,
+      metaConfig: {},
+      complete: { version: completeVersion, category: completeCategory },
+      registerAllSearchCombinations: undefined
+    });
+
+    const content = await options.metaHandler({});
+
+    expect(completeVersion).toHaveBeenCalledTimes(1);
+    expect(completeCategory).toHaveBeenCalledTimes(1);
+    expect(content).toContain('version');
+    expect(content).toContain('category');
+    expect(content).toContain('v1');
+    expect(content).toContain('cat1');
+  });
+
   it('should fall back to empty values when a complete callback throws', async () => {
     const throwingComplete = jest.fn().mockRejectedValue(new Error('network error'));
     const options = setMetadataOptions({
@@ -252,6 +275,7 @@ describe('setMetaResources', () => {
 
     expect(callback).toHaveBeenCalledTimes(1);
     expect(result.contents).toHaveLength(2);
+    expect(result.contents[0]).toBe(originalContent.contents[0]);
     expect(result.contents[0].text).toBe('original');
     expect(result.contents[1].text).toContain('Test Metadata');
   });

src/server.resourceMeta.ts

@@ -401,18 +401,25 @@ const setMetaResources = (resources: McpResourceCreator[], options = getOptions(
         runWithOptions(opts, async () => {
           const result = await callback(passedUri, variables);
 
-          if (result.contents) {
-            const updatedText = await resolveMetaText(variables);
-            const queryString = buildSearchString(variables, { prefix: true });
-
-            result.contents.push({
-              uri: queryString ? `${uriBreakdown.metaBaseUri}${queryString}` : uriBreakdown.metaBaseUri,
-              mimeType: metaMimeType,
-              text: updatedText
-            });
+          if (!isPlainObject(result) || !Array.isArray(result.contents)) {
+            return result;
           }
 
-          return result;
+          const updatedText = await resolveMetaText(variables);
+          const queryString = buildSearchString(variables, { prefix: true });
+          const metaContentUri = queryString ? `${uriBreakdown.metaBaseUri}${queryString}` : uriBreakdown.metaBaseUri;
+
+          return {
+            ...result,
+            contents: [
+              ...result.contents,
+              {
+                uri: metaContentUri,
+                mimeType: metaMimeType,
+                text: updatedText
+              }
+            ]
+          };
         });
 
       return [name, uriOrTemplate, config, metaEnhancedCallback, metadata];