Comments fix: extensions response format

Author: Natallia Harshunova

src/McpResponse.ts

@@ -551,18 +551,10 @@ Call ${handleDialog.name} to handle it before continuing.`);
       if (data.extensions.length === 0) {
         response.push('No extensions installed.');
       } else {
-        const parts = [];
-        for (const extension of data.extensions) {
-          parts.push(
-            [
-              `Name: ${extension.name}`,
-              `ID: ${extension.id}`,
-              `Version: ${extension.version}`,
-              `Status: ${extension.isEnabled ? 'Enabled' : 'Disabled'}`,
-            ].join('\n'),
-          );
-        }
-        response.push(parts.join('\n\n'));
+        const extensionsMessage = data.extensions.map(extension => {
+          return `id=${extension.id} "${extension.name}" v${extension.version} ${extension.isEnabled ? 'Enabled' : 'Disabled'}`;
+        }).join('\n');
+        response.push(extensionsMessage);
       }
     }
 

src/tools/extensions.ts

@@ -22,7 +22,6 @@ export const installExtension = defineTool({
   schema: {
     path: zod
       .string()
-      .trim()
       .describe('Absolute path to the unpacked extension folder.'),
   },
   handler: async (request, response, context) => {

tests/McpResponse.test.js.snapshot

@@ -484,15 +484,8 @@ No requests found.
 exports[`extensions > lists extensions 1`] = `
 # test response
 ## Extensions
-Name: Extension 1
-ID: id1
-Version: 1.0
-Status: Enabled
-
-Name: Extension 2
-ID: id2
-Version: 2.0
-Status: Disabled
+id=id1 "Extension 1" v1.0 Enabled
+id=id2 "Extension 2" v2.0 Disabled
 `;
 
 exports[`extensions > lists extensions 2`] = `

tests/McpResponse.test.ts

@@ -717,6 +717,14 @@ describe('McpResponse network pagination', () => {
 describe('extensions', () => {
   it('lists extensions', async t => {
     await withMcpContext(async (response, context) => {
+      response.setListExtensions();
+      // Empty state testing 
+      const emptyResult = await response.handle('test', context);
+      const emptyText = getTextContent(emptyResult.content[0]);
+      assert.ok(emptyText.includes('No extensions installed.'), 'Should show message for ampty extensions');
+
+      response.resetResponseLineForTesting();
+      // Testing with extensions
       context.listExtensions = () => [
         {
           id: 'id1',