@velaro/mcp-server 0.6.97 → 0.6.99

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@velaro/mcp-server",
-  "version": "0.6.97",
+  "version": "0.6.99",
   "description": "Velaro MCP server — connect Claude and other AI agents directly to your Velaro account: KB, workflows, bots, conversations, contacts, routing, and more.",
   "type": "module",
   "bin": {

package/server.js

@@ -40,6 +40,7 @@
  *   VELARO_MCP_MODE=http PORT=3000 node server.js
  */
 
+import { describeRawMeta, fetchRetry429 } from './log-search.js';
 import { Server }            from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import {
@@ -119,11 +120,12 @@ function parseDuration(s) {
   return n * 24 * 3600 * 1000;
 }
 
-async function messagingApi(path, opts = {}) {
+async function messagingApi(path, opts = {}) {  // opts.timeoutMs: log searches pass 40000 (server scan budget 12 s + 6 s AI search)
   if (!MESSAGING_API) throw new Error(
     'VELARO_MESSAGING_API is not configured. Add "VELARO_MESSAGING_API": "https://velaro-messaging-api-staging.azurewebsites.net" to your MCP env config.'
   );
-  const res = await fetch(`${MESSAGING_API}${path}`, {
+  // 429 = the server's per-instance scan gate is full: retry honouring Retry-After (GETs only; a retried POST could repeat a write).
+  const init = () => ({
     method: opts.method ?? 'GET',
     headers: {
       Authorization: authHeader(),
@@ -131,8 +133,9 @@ async function messagingApi(path, opts = {}) {
       ...(opts.body ? { 'Content-Type': 'application/json' } : {}),
     },
     body: opts.body ? JSON.stringify(opts.body) : undefined,
-    signal: AbortSignal.timeout(20000),
+    signal: AbortSignal.timeout(opts.timeoutMs ?? 20000),
   });
+  const res = (opts.method ?? 'GET') === 'GET' ? await fetchRetry429(`${MESSAGING_API}${path}`, init) : await fetch(`${MESSAGING_API}${path}`, init());
   if (!res.ok) {
     const text = await res.text().catch(() => '');
     throw new Error(`Velaro Logger ${path} -> ${res.status}: ${text.slice(0, 300)}`);
@@ -7569,9 +7572,18 @@ async function handleTool(name, args) {
       const from1h = new Date(Date.now() - 3600000).toISOString();
       const velaroLoggerP = safeJson(async () => {
         if (!MESSAGING_API) return { _error: 'VELARO_MESSAGING_API not set — add it to MCP env config' };
-        const q = encodeURIComponent('ERROR|WF-AI-ROUTE|PERF-SLOW');
-        const res = await messagingApi(`/superadmin/logs/search?q=${q}&from=${encodeURIComponent(from1h)}&take=30&regex=true`);
-        return extractLogEntries(res);
+        // Two honest reads (Opus review 2026-10-14): ERRORS use the server-side level=ERROR filter; PATTERNS are the WF-AI-ROUTE / PERF-SLOW regex (no level).
+        const f = encodeURIComponent(from1h);
+        // One at a time: the server runs at most 2 exact scans per instance.
+        const errRes = await messagingApi(`/superadmin/logs/search?level=ERROR&from=${f}&take=30`, { timeoutMs: 40000 });
+        const patRes = await messagingApi(`/superadmin/logs/search?q=${encodeURIComponent('WF-AI-ROUTE|PERF-SLOW')}&regex=true&from=${f}&take=30`, { timeoutMs: 40000 });
+        const byId = new Map();
+        for (const e of [...extractLogEntries(errRes), ...extractLogEntries(patRes)]) byId.set(e.id ?? `${e.timestamp}|${e.message}`, e);
+        const found = [...byId.values()].sort((x, y) => (Date.parse(y.timestamp ?? y.LogTimestamp) || 0) - (Date.parse(x.timestamp ?? x.LogTimestamp) || 0));
+        const verified = (r) => !!(r && !Array.isArray(r) && r.truncated === false);
+        found._verified = verified(errRes) && verified(patRes);
+        found._note = `errors: ${describeRawMeta(errRes)} | patterns: ${describeRawMeta(patRes)}`;
+        return found;
       });
 
       // ── Layer 4b: Loggly ERROR + SLOW + AI-routing logs (last 1h) — SECONDARY (migrating away)
@@ -7647,9 +7659,11 @@ async function handleTool(name, args) {
       if (velaroLogs?._error) {
         out.push(section('Velaro Logger (1h) [PRIMARY]', [`  ⚠ ${velaroLogs._error}`]));
       } else if (!Array.isArray(velaroLogs) || !velaroLogs.length) {
-        out.push(section('Velaro Logger (1h) [PRIMARY]', ['  ✅ No errors, slow responses, or AI routing events']));
+        out.push(section('Velaro Logger (1h) [PRIMARY]', velaroLogs?._verified
+          ? [`  ✅ No errors, slow responses, or AI routing events (${velaroLogs._note})`]
+          : [`  ⚠ No matches, but the scan is NOT verified complete: ${velaroLogs?._note ?? 'no scan metadata'}. Use velaro_logs_search before concluding all is well.`]));
       } else {
-        out.push(section(`Velaro Logger (1h) [PRIMARY] — ${velaroLogs.length} entries`, velaroLogs.map(fmtVelaroEntry)));
+        out.push(section(`Velaro Logger (1h) [PRIMARY] — ${velaroLogs.length} entries${velaroLogs._verified ? '' : ` (scan not verified complete: ${velaroLogs._note})`}`, velaroLogs.map(fmtVelaroEntry)));
       }
 
       // ── Loggly (secondary — being phased out; check if Velaro Logger is empty/errored)
@@ -8611,9 +8625,11 @@ async function handleTool(name, args) {
       if (args.environment)   params.set('environment',   args.environment);
       if (args.correlationId) params.set('correlationId', args.correlationId);
       if (args.regex)         params.set('regex',         'true');
-      const entries = extractLogEntries(await messagingApi(`/superadmin/logs/search?${params}`));
-      if (!entries.length) return 'No results.';
-      const lines = [`${entries.length} result(s) from ${from.toISOString().slice(0, 16)} UTC\n`];
+      const resp = await messagingApi(`/superadmin/logs/search?${params}`, { timeoutMs: 40000 });
+      const entries = extractLogEntries(resp);
+      const meta = describeRawMeta(resp);   // searchPath/scannedRows/truncated: a 0 or short list is never ambiguous
+      if (!entries.length) return `No results. ${meta}`;
+      const lines = [`${entries.length} result(s) from ${from.toISOString().slice(0, 16)} UTC. ${meta}\n`];
       for (const e of entries) {
         const ts    = new Date(e.timestamp ?? e.LogTimestamp).toISOString().replace('T', ' ').slice(0, 19);
         const lvl   = (e.Level ?? e.level ?? '?').padEnd(5);