feat: add Goose client support for MCP server installation

- Add js-yaml dependency for YAML parsing/writing
- Extend client-config.ts to support YAML format files
- Add Goose configuration path (~/.config/goose/config.yaml)
- Update install command to handle Goose''s specific config structure
- Goose extensions include: name, cmd, args, enabled, envs, type, timeout

Resolves #4

Co-authored-by: Dhravya Shah <Dhravya@users.noreply.github.com>

Author: claude[bot]

package.json

@@ -49,6 +49,7 @@
     "@jest/globals": "^29.7.0",
     "@tsconfig/node20": "^20.1.4",
     "@types/jest": "^29.5.12",
+    "@types/js-yaml": "^4.0.9",
     "@types/node": "^20.12.12",
     "@types/prompts": "^2.4.9",
     "@types/signale": "^1.4.7",
@@ -76,6 +77,7 @@
     "consola": "^3.2.3",
     "dotenv": "^16.4.5",
     "giget": "^1.2.3",
+    "js-yaml": "^4.1.0",
     "picocolors": "^1.0.1",
     "yargs": "^17.7.2"
   },

src/client-config.ts

@@ -2,6 +2,7 @@ import fs from 'node:fs'
 import os from 'node:os'
 import path from 'node:path'
 import process from 'node:process'
+import yaml from 'js-yaml'
 
 import { verbose } from './logger'
 // import { execFileSync } from "node:child_process"
@@ -16,6 +17,7 @@ interface ClientFileTarget {
   path: string
   localPath?: string
   configKey: string
+  format?: 'json' | 'yaml' // Add format property for different file types
 }
 type ClientInstallTarget = ClientFileTarget
 
@@ -117,6 +119,12 @@ function getClientPaths(): { [key: string]: ClientInstallTarget } {
       localPath: path.join(process.cwd(), '.mcp.json'),
       configKey: 'mcpServers',
     },
+    goose: {
+      type: 'file',
+      path: path.join(baseDir, 'goose', 'config.yaml'),
+      configKey: 'extensions',
+      format: 'yaml',
+    },
   }
 }
 
@@ -132,6 +140,7 @@ export const clientNames = [
   'gemini-cli',
   'vscode',
   'claude-code',
+  'goose',
 ]
 
 // Helper function to get nested value from an object using dot notation
@@ -197,7 +206,15 @@ export function readConfig(client: string, local?: boolean): ClientConfig {
     }
 
     verbose('Reading config file content')
-    const rawConfig = JSON.parse(fs.readFileSync(configPath.path, 'utf8'))
+    const fileContent = fs.readFileSync(configPath.path, 'utf8')
+    
+    let rawConfig: ClientConfig
+    if (configPath.format === 'yaml') {
+      rawConfig = yaml.load(fileContent) as ClientConfig || {}
+    } else {
+      rawConfig = JSON.parse(fileContent)
+    }
+    
     verbose(`Config loaded successfully: ${JSON.stringify(rawConfig, null, 2)}`)
 
     // Ensure the nested structure exists
@@ -261,7 +278,14 @@ function writeConfigFile(config: ClientConfig, target: ClientFileTarget): void {
   try {
     if (fs.existsSync(target.path)) {
       verbose('Reading existing config file for merging')
-      existingConfig = JSON.parse(fs.readFileSync(target.path, 'utf8'))
+      const fileContent = fs.readFileSync(target.path, 'utf8')
+      
+      if (target.format === 'yaml') {
+        existingConfig = yaml.load(fileContent) as ClientConfig || {}
+      } else {
+        existingConfig = JSON.parse(fileContent)
+      }
+      
       verbose(`Existing config loaded: ${JSON.stringify(existingConfig, null, 2)}`)
     }
   } catch (error) {
@@ -274,6 +298,18 @@ function writeConfigFile(config: ClientConfig, target: ClientFileTarget): void {
   verbose(`Merged config: ${JSON.stringify(mergedConfig, null, 2)}`)
 
   verbose(`Writing config to file: ${target.path}`)
-  fs.writeFileSync(target.path, JSON.stringify(mergedConfig, null, 2))
+  
+  let configContent: string
+  if (target.format === 'yaml') {
+    configContent = yaml.dump(mergedConfig, { 
+      indent: 2,
+      lineWidth: -1,
+      noRefs: true
+    })
+  } else {
+    configContent = JSON.stringify(mergedConfig, null, 2)
+  }
+  
+  fs.writeFileSync(target.path, configContent)
   verbose('Config successfully written')
 }

src/commands/install.ts

@@ -18,6 +18,7 @@ function setServerConfig(
   configKey: string,
   serverName: string,
   serverConfig: ClientConfig,
+  client: string,
 ): void {
   // Get or create the nested config object
   let servers = getNestedValue(config, configKey)
@@ -28,7 +29,21 @@ function setServerConfig(
 
   // Set the server config
   if (servers) {
-    servers[serverName] = serverConfig
+    if (client === 'goose') {
+      // Goose has a different config structure
+      servers[serverName] = {
+        name: serverName,
+        cmd: serverConfig.command,
+        args: serverConfig.args,
+        enabled: true,
+        envs: {},
+        type: 'stdio',
+        timeout: 300,
+        ...serverConfig, // Allow overriding defaults
+      }
+    } else {
+      servers[serverName] = serverConfig
+    }
   }
 }
 
@@ -259,14 +274,14 @@ export async function handler(argv: ArgumentsCamelCase<InstallArgv>) {
         setServerConfig(config, configKey, name, {
           command: 'npx',
           args: args,
-        })
+        }, argv.client)
       } else {
         // Command-based installation (including simple package names)
         const cmdParts = command.split(' ')
         setServerConfig(config, configKey, name, {
           command: cmdParts[0],
           args: cmdParts.slice(1),
-        })
+        }, argv.client)
       }
 
       writeConfig(config, argv.client, argv.local)