mallmaverick-store-scraper 0.1.4 → 0.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mallmaverick-store-scraper",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "MCP server + CLI for scraping shopping mall store directories. Hours-first layered pipeline + image classification.",
   "main": "src/main.js",
   "type": "commonjs",

package/src/mcp-server.js

@@ -60,11 +60,11 @@ const TOOLS = [
       '(name, hours, phone, logo, brand image, categories, etc.). Use this ' +
       'when the user wants to capture a directory like ' +
       'https://grasslands.ca/store-directory/.\n\n' +
-      'AFTER RUNNING THIS TOOL: give a short summary — how many stores were ' +
-      'extracted, hours-layer breakdown, and the saved file path. The CSV is ' +
-      'attached as a resource in the tool response (the user can download/' +
-      'preview it from there) — do NOT paste the CSV text into your reply. ' +
-      'Keep your text reply short.',
+      'AFTER RUNNING THIS TOOL: reply with ONE short sentence — the count ' +
+      'and the file path. The CSV file is attached as a resource_link in the ' +
+      'tool response; do NOT paste CSV text into your reply, do NOT print the ' +
+      'JSON, do NOT summarize each store. If the tool response includes an ' +
+      'error block, surface that error verbatim to the user.',
     inputSchema: {
       type: 'object',
       properties: {
@@ -138,7 +138,7 @@ const TOOLS = [
   },
 ];
 
-const PACKAGE_VERSION = '0.1.4';
+const PACKAGE_VERSION = '0.1.5';
 
 const server = new Server(
   { name: 'mall-scraper-mcp', version: PACKAGE_VERSION },
@@ -244,46 +244,73 @@ async function handleScrapeDirectory({ directory_url, max_stores = 10, concurren
       mcp_version: PACKAGE_VERSION,
     };
 
-    // Build a short brief + return the CSV as an embedded resource so
-    // Claude Desktop can render it as an attachment card instead of inline
-    // text. Falls back to inline-rendering if the client doesn't support
-    // resources, but most clients (including Claude Desktop) do.
+    // Response design:
+    //   1. Brief status line (always) — what the user sees in the chat reply
+    //   2. resource_link to the CSV — file attachment with user-priority annotations
+    //   3. ONLY on error: a loud error block so the user knows something failed
+    //
+    // No JSON dump / no inline CSV preview when things succeed — keeps the chat
+    // reply minimal.
     const host = new URL(directory_url).hostname.replace(/^www\./, '');
     const csvFilename = writtenPaths
       ? path.basename(writtenPaths.csv)
       : `stores_v5_${host}.csv`;
     const csvUri = writtenPaths
       ? `file://${writtenPaths.csv}`
-      : `file:///tmp/${csvFilename}`;
+      : null;
+
+    const hasLlmFailure = usage.errors > 0;
+    const hasWriteFailure = !!writeError;
+    const anyFailure = hasLlmFailure || hasWriteFailure;
 
     const brief =
-      `mall-scraper-mcp v${PACKAGE_VERSION}\n` +
-      `Scraped ${stores.length} store${stores.length === 1 ? '' : 's'} from ${host}.\n` +
-      `Hours-layer breakdown: ${Object.entries(bySource).map(([k, v]) => `${k}=${v}`).join(', ')}.\n` +
-      (writtenPaths
-        ? `Saved to: ${writtenPaths.csv}`
-        : `⚠ disk write failed (${writeError}); CSV is in the attached resource only.`);
-
-    return {
-      content: [
-        { type: 'text', text: brief },
-        {
-          type: 'resource',
-          resource: {
-            uri: csvUri,
-            name: csvFilename,
-            mimeType: 'text/csv',
-            text: csvText,
-          },
-        },
-        // Keep the JSON summary at the end for any debugging the user asks for,
-        // but it's far enough down that it doesn't dominate the chat.
-        {
-          type: 'text',
-          text: '\n--- Run summary ---\n' + JSON.stringify(summary, null, 2),
-        },
-      ],
-    };
+      `✅ mall-scraper-mcp v${PACKAGE_VERSION}\n` +
+      `${stores.length} store${stores.length === 1 ? '' : 's'} from ${host}\n` +
+      (writtenPaths ? `📄 ${writtenPaths.csv}` : '⚠ Disk write failed');
+
+    const content = [
+      {
+        type: 'text',
+        text: brief,
+        annotations: { audience: ['user'], priority: 1.0 },
+      },
+    ];
+
+    // resource_link only if we have a real file path (file:// URI must point at
+    // an existing file for the client to do anything useful with it).
+    if (csvUri) {
+      content.push({
+        type: 'resource_link',
+        uri: csvUri,
+        name: csvFilename,
+        description: `Store directory scrape — ${stores.length} stores from ${host}`,
+        mimeType: 'text/csv',
+        annotations: { audience: ['user'], priority: 0.9 },
+      });
+    }
+
+    // Loud error block — only when something failed. The user explicitly asked
+    // for nothing other than a status rundown UNLESS something broke.
+    if (anyFailure) {
+      const errLines = [];
+      if (hasLlmFailure) {
+        errLines.push(
+          `❌ ${usage.errors} LLM call${usage.errors === 1 ? '' : 's'} failed: ${usage.lastError}`,
+          '   → description / categories / location_type fields will be empty.',
+          '   → Run check_status to diagnose (most likely the Worker token doesn\'t match the SHARED_SECRET).',
+        );
+      }
+      if (hasWriteFailure) {
+        errLines.push(`❌ Disk write failed: ${writeError}`);
+      }
+      content.push({
+        type: 'text',
+        text: '\n' + errLines.join('\n'),
+        annotations: { audience: ['user'], priority: 1.0 },
+      });
+    }
+
+    return { content };
   } finally {
     try { await browser.close(); } catch (_) {}
   }