mr-magic-mcp-server 0.5.1 → 0.5.2

package/package.json

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

package/src/providers/melon.js

@@ -91,7 +91,7 @@ async function buildLyricHeaders(songId) {
 
 function extractSongId(value) {
   if (!value) return null;
-  const goSongMatch = value.match(/goSongDetail\([^0-9]*?(\d+)\)/);
+  const goSongMatch = value.match(/goSongDetail\([^)]*?(\d+)[^)]*\)/);
   if (goSongMatch) return goSongMatch[1];
   const searchLogMatch = value.match(/searchLog\('[^']+','[^']+','[^']+','[^']+','(\d+)'\)/);
   if (searchLogMatch) return searchLogMatch[1];
@@ -100,7 +100,7 @@ function extractSongId(value) {
   return null;
 }
 
-function parseSearchPage(html) {
+export function parseSearchPage(html) {
   const document = parseDocument(html);
   const seenIds = new Set();
   return selectAll('#frm_defaultList > div > table > tbody > tr', document)
@@ -129,7 +129,6 @@ function parseSearchPage(html) {
       if (!title && !artist) return null;
       return { songId, title, artist, album };
     })
-    .get()
     .filter(Boolean);
 }
 

package/src/tests/run-tests.js

@@ -19,6 +19,7 @@ import {
   catalogCacheKey,
   catalogCache
 } from '../services/lyrics-service.js';
+import { parseSearchPage } from '../providers/melon.js';
 import { mcpToolDefinitions, handleMcpTool } from '../transport/mcp-tools.js';
 import { romanizePlainLyrics } from '../utils/lyrics-format.js';
 
@@ -504,6 +505,37 @@ function testCliEnvPathLoadsCustomEnvFile() {
   console.log('CLI --env-path loads and persists custom env files: ok');
 }
 
+function testMelonSearchParserWithoutCheerioCollection() {
+  const html = `
+    <div id="frm_defaultList">
+      <div>
+        <table>
+          <tbody>
+            <tr>
+              <td></td>
+              <td></td>
+              <td><a class="fc_gray" href="javascript:goSongDetail('38914304')">Summer Nights</a></td>
+              <td><div id="artistName"><a href="#artist">St. Lucia</a></div></td>
+              <td><a href="#album">When The Night</a></td>
+            </tr>
+          </tbody>
+        </table>
+      </div>
+    </div>`;
+
+  const results = parseSearchPage(html);
+  assert.equal(results.length, 1, 'Melon parser should return plain arrays, not Cheerio chains');
+  assert.deepEqual(results[0], {
+    songId: '38914304',
+    title: 'Summer Nights',
+    artist: 'St. Lucia',
+    album: 'When The Night'
+  });
+
+  divider();
+  console.log('Melon parser works without Cheerio collection helpers: ok');
+}
+
 async function run() {
   testAutoPickPrefersSynced();
   testAutoPickFallbackWhenNoSynced();
@@ -522,6 +554,7 @@ async function run() {
   await testBuildPayloadFromResultNoCacheKeyWhenNoLyrics();
   testCliExportCommandHelp();
   testCliEnvPathLoadsCustomEnvFile();
+  testMelonSearchParserWithoutCheerioCollection();
   const toolNames = mcpToolDefinitions.map((tool) => tool.name);
   console.log('MCP tooling available:', toolNames.join(', '));
   console.log('All sanity checks passed');