@calltelemetry/cucm-mcp 0.1.4 → 0.1.6

package/README.md

@@ -89,3 +89,4 @@ Live tests are opt-in via env vars; see `test/live.test.js`.
 
 - Use the platform/OS admin for SSH (`administrator` user on most lab systems)
 - To request a high packet count without specifying an exact number, pass `maxPackets: true` to `packet_capture_start`
+- If traffic is low, a small `count` can still run “forever” waiting for packets; use `packet_capture_stop` to cancel, or set `maxDurationMs` to auto-stop

package/dist/dime.js

@@ -27,8 +27,18 @@ function dimeXmlBytes(contentType, bytes) {
         return bytes;
     const parts = parseMultipartRelated(bytes, boundary);
     const xmlParts = parts.filter((p) => isXmlContentType(p.contentType));
-    if (xmlParts.length === 0)
-        throw new Error("DIME response missing text/xml part");
+    if (xmlParts.length === 0) {
+        // Some CUCM versions have been observed returning multipart bodies where the XML part
+        // does not advertise a classic XML content-type. Fall back to sniffing the payload.
+        const sniffed = parts.find((p) => {
+            const head = p.body.subarray(0, 64).toString("utf8");
+            return head.includes("<?xml") || head.trimStart().startsWith("<");
+        });
+        if (sniffed)
+            return sniffed.body;
+        const partTypes = parts.map((p) => p.contentType).join(", ");
+        throw new Error(`DIME response missing text/xml part (boundary=${boundary}; parts=[${partTypes || "none"}])`);
+    }
     return xmlParts[0].body;
 }
 export function normalizeHost(hostOrUrl) {

package/dist/index.js

@@ -14,7 +14,7 @@ const strictTls = tlsMode === "strict" || tlsMode === "verify";
 // Set CUCM_MCP_TLS_MODE=strict to enforce verification.
 if (!strictTls)
     process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0";
-const server = new McpServer({ name: "cucm", version: "0.1.4" });
+const server = new McpServer({ name: "cucm", version: "0.1.6" });
 const captures = new PacketCaptureManager();
 const captureState = defaultStateStore();
 const dimeAuthSchema = z
@@ -114,7 +114,21 @@ server.tool("packet_capture_start", "Start a packet capture on CUCM via SSH (uti
     size: z.string().optional().describe("Packet size (e.g. all)"),
     hostFilterIp: z.string().optional().describe("Optional filter: host ip <addr>"),
     portFilter: z.number().int().min(1).max(65535).optional().describe("Optional filter: port <num>"),
-}, async ({ host, sshPort, auth, iface, fileBase, count, maxPackets, size, hostFilterIp, portFilter }) => {
+    maxDurationMs: z
+        .number()
+        .int()
+        .min(250)
+        .max(24 * 60 * 60_000)
+        .optional()
+        .describe("Stop after this duration even if packet count isn't reached"),
+    startTimeoutMs: z
+        .number()
+        .int()
+        .min(2000)
+        .max(120_000)
+        .optional()
+        .describe("Timeout for starting capture (SSH connect + command start)"),
+}, async ({ host, sshPort, auth, iface, fileBase, count, maxPackets, size, hostFilterIp, portFilter, maxDurationMs, startTimeoutMs }) => {
     const resolvedCount = count ?? (maxPackets ? 1_000_000 : undefined);
     const result = await captures.start({
         host,
@@ -126,6 +140,8 @@ server.tool("packet_capture_start", "Start a packet capture on CUCM via SSH (uti
         size,
         hostFilterIp,
         portFilter,
+        maxDurationMs,
+        startTimeoutMs,
     });
     return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
 });

package/dist/packetCapture.js

@@ -195,8 +195,31 @@ export class PacketCaptureManager {
             }
         });
         // Wait for prompt before running capture command.
+        // Some CUCM shells don't print a prompt until you send a newline.
+        const promptTimeoutMs = Math.max(2000, Math.trunc(opts.startTimeoutMs ?? 30_000));
         try {
-            await waitFor(channel, session, () => looksLikeCucmPrompt(session.lastStdout) || looksLikeCucmPrompt(session.lastStderr), 30_000, "Timeout waiting for CUCM CLI prompt");
+            try {
+                channel.write("\n");
+            }
+            catch {
+                // ignore
+            }
+            // Nudge a couple times if no prompt yet.
+            void (async () => {
+                const delays = [400, 1200, 2500];
+                for (const d of delays) {
+                    await new Promise((r) => setTimeout(r, d));
+                    if (looksLikeCucmPrompt(session.lastStdout) || looksLikeCucmPrompt(session.lastStderr))
+                        return;
+                    try {
+                        channel.write("\n");
+                    }
+                    catch {
+                        // ignore
+                    }
+                }
+            })();
+            await waitFor(channel, session, () => looksLikeCucmPrompt(session.lastStdout) || looksLikeCucmPrompt(session.lastStderr), promptTimeoutMs, "Timeout waiting for CUCM CLI prompt");
             channel.write(`${cmd}\n`);
         }
         catch (e) {
@@ -217,6 +240,16 @@ export class PacketCaptureManager {
             throw e;
         }
         this.active.set(id, { session, client, channel });
+        // Optional guard: stop after a duration even if packet count isn't reached.
+        if (opts.maxDurationMs != null) {
+            const dur = Math.max(250, Math.trunc(opts.maxDurationMs));
+            const t = setTimeout(() => {
+                void this.stop(id).catch(() => {
+                    // ignore
+                });
+            }, dur);
+            t.unref?.();
+        }
         return session;
     }
     async stop(captureId, timeoutMs = 90_000) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@calltelemetry/cucm-mcp",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "private": false,
   "type": "module",
   "description": "MCP server for CUCM tooling (DIME logs, syslog, packet capture)",