mr-magic-mcp-server 0.3.7 → 0.3.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mr-magic-mcp-server",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "Lyrics MCP server connecting LRCLIB, Genius, Musixmatch, and Melon",
   "type": "module",
   "main": "src/index.js",

package/src/tools/cli.js

@@ -356,7 +356,7 @@ program
     },
     []
   )
-  .option('--output <dir>', 'Directory to write exports (requires --export)', 'exports')
+  .option('--output <dir>', 'Directory to write exports (requires --export; defaults to MR_MAGIC_EXPORT_DIR or ./exports)')
   .option('--no-romanize', 'Disable romanized lyrics', false)
   .action(async (options) => {
     const track = buildTrackFromOptions(options);
@@ -542,7 +542,7 @@ program
     },
     []
   )
-  .option('--output <dir>', 'Directory for exports (requires --export)', 'exports')
+  .option('--output <dir>', 'Directory for exports (requires --export; defaults to MR_MAGIC_EXPORT_DIR or ./exports)')
   .option('--no-romanize', 'Disable romanized lyrics', false)
   .action(async (options) => {
     const track = buildTrackFromOptions(options);

package/src/transport/mcp-tools.js

@@ -6,7 +6,6 @@ import {
   buildActionContext,
   buildPayloadFromResult,
   buildCatalogPayload,
-  exportBestResult,
   formatRecord,
   catalogCache,
   catalogCacheKey
@@ -403,8 +402,9 @@ export async function handleMcpTool(name, args = {}) {
     const result = await runFind(track, options);
     const context = buildActionContext({ ...options, export: true });
     const payload = await buildPayloadFromResult(result, context);
-    const exports = await exportBestResult(result, context);
-    return { result: payload, exports };
+    // buildPayloadFromResult already calls exportBestResult internally when
+    // context.shouldExport is true — avoid exporting twice by reusing payload.exports.
+    return { result: payload, exports: payload.exports ?? null };
   }
 
   if (name === 'format_lyrics') {