mr-magic-mcp-server 0.3.7 → 0.3.9

package/package.json

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

package/src/tests/run-tests.js

@@ -324,9 +324,9 @@ function testRomanization() {
   // 깻: ㄷ-representative of ㅅ batchim; 잎: ㅇ initial (silent) → liaison
   // Actually 깻 = ㄲ+ㅖ+ㅅ, 잎 = ㅇ+ㅣ+ㅍ
   // Liaison: 잎 initial ㅇ → ㅅ(깻) moves to 잎 onset:  → 깨 + 씹? No:
-  // 깻: batchim ㅅ; 잎: initial ㅇ → 깻 coda ㅅ moves to 잎 as initial 'ss'? 
+  // 깻: batchim ㅅ; 잎: initial ㅇ → 깻 coda ㅅ moves to 잎 as initial 'ss'?
   // Standard Korean: 깻잎 → [깬닙] (nasalization of ㅅ→ㄴ before ㅣ? No.
-  // Actually: 깻잎 → liaison: 깻(ㅅ) + 잎(ㅇ) → 깨싫... 
+  // Actually: 깻잎 → liaison: 깻(ㅅ) + 잎(ㅇ) → 깨싫...
   // Correct pronunciation: 깻잎 [깬닙] — the ㅅ turns to ㄴ (because 잎's ㅍ batchim + ㄴ?)
   // Simpler: official = kkaennip. Our engine: 깻(ㅅ liaison to 잎ㅇ) → 깨 + 싶 → 깨십.
   // The 잎 ㅍ final stays = p.  깻잎 → Kkaesip via liaison. That's our engine's output.

package/src/tools/cli.js

@@ -356,7 +356,10 @@ 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 +545,10 @@ 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') {