@nyx-intelligence/val-mcp 0.4.0 → 0.5.1

package/dist/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { scanSite, checkPage, screenshot } from "./scanner.js";
-import { formatReport } from "./report.js";
+import { formatReport, topFindings } from "./report.js";
 import { session } from "./session.js";
 function fmtAction(r) {
     const lines = [`${r.ok ? "ok" : "warn"} · ${r.title || "(no title)"} · ${r.url}`];
@@ -66,7 +66,7 @@ async function runServer() {
     const { StdioServerTransport } = await import("@modelcontextprotocol/sdk/server/stdio.js");
     const { chromium } = await import("playwright");
     const { z } = await import("zod");
-    const server = new McpServer({ name: "val", version: "0.4.0" });
+    const server = new McpServer({ name: "val", version: "0.5.0" });
     const text = (t) => ({ content: [{ type: "text", text: t }] });
     // ── Passive crawl ──
     server.registerTool("val_scan", {
@@ -75,7 +75,7 @@ async function runServer() {
         inputSchema: { url: z.string(), maxPages: z.number().int().min(1).max(50).optional(), device: z.string().optional().describe("Emulate a mobile device, e.g. 'iPhone 14' or 'Pixel 7'") },
     }, async ({ url, maxPages, device }) => {
         const { findings, pagesVisited } = await scanSite(url, { maxPages: maxPages ?? 12, device });
-        return text(`${formatReport(findings, pagesVisited)}\n\nJSON:\n${JSON.stringify(findings)}`);
+        return text(`${formatReport(findings, pagesVisited)}\n\nJSON:\n${JSON.stringify(topFindings(findings))}`);
     });
     server.registerTool("val_check", {
         title: "Re-check one page (verify a fix)",
@@ -85,7 +85,7 @@ async function runServer() {
         const browser = await chromium.launch({ headless: true });
         try {
             const { findings } = await checkPage(browser, url, { device });
-            return text(`${formatReport(findings, [url])}\n\nJSON:\n${JSON.stringify(findings)}`);
+            return text(`${formatReport(findings, [url])}\n\nJSON:\n${JSON.stringify(topFindings(findings))}`);
         }
         finally {
             await browser.close();

package/dist/report.js

@@ -1,5 +1,11 @@
 const RANK = { high: 0, medium: 1, low: 2 };
-/** Render findings as a readable, agent-friendly report. */
+// Cap how many findings we surface so a very broken site can't blow up the
+// calling agent's token budget. The full count is always reported; only the
+// itemized list (and the JSON the agent parses) is capped to the top N.
+const MAX_REPORT = 50;
+export function topFindings(findings, n = MAX_REPORT) {
+    return [...findings].sort((a, b) => RANK[a.severity] - RANK[b.severity]).slice(0, n);
+}
 export function formatReport(findings, pagesVisited) {
     const counts = { high: 0, medium: 0, low: 0 };
     for (const f of findings)
@@ -12,11 +18,15 @@ export function formatReport(findings, pagesVisited) {
     }
     else {
         lines.push("");
-        const sorted = [...findings].sort((a, b) => RANK[a.severity] - RANK[b.severity]);
-        for (const f of sorted) {
+        const shown = topFindings(findings);
+        for (const f of shown) {
             lines.push(`[${f.severity.toUpperCase()}] ${f.type} — ${f.page}`);
             lines.push(`   ${f.detail}${f.evidence ? `  (${f.evidence})` : ""}`);
         }
+        if (findings.length > shown.length) {
+            lines.push("");
+            lines.push(`… and ${findings.length - shown.length} more (capped to keep this small). Fix the above, then re-scan.`);
+        }
     }
     lines.push("");
     lines.push("Pages crawled:");

package/dist/session.js

@@ -153,9 +153,41 @@ class ValSession {
                 const r = el.getBoundingClientRect();
                 return r.width > 0 && r.height > 0;
             };
-            const els = Array.from(document.querySelectorAll("button, [role=button]")).filter(isVisible);
+            // Skip buttons that are toggles / segmented-control items / already-active
+            // controls — clicking them is a legitimate no-op, not a dead control.
+            const TOGGLE_ROLES = ["tab", "switch", "radio", "option", "menuitemradio", "menuitemcheckbox"];
+            const CONTAINER_ROLES = ["tablist", "radiogroup", "listbox", "menu", "tabs"];
+            const ACTIVE_CLASS = /(^|\s|-)(active|selected|is-active|is-selected|current|is-current)(\s|$|-)/i;
+            const isToggleLike = (el) => {
+                if (el.hasAttribute("disabled") || el.getAttribute("aria-disabled") === "true")
+                    return true;
+                if (el.hasAttribute("aria-pressed") || el.hasAttribute("aria-selected"))
+                    return true;
+                const ac = el.getAttribute("aria-current");
+                if (ac && ac !== "false")
+                    return true;
+                const role = el.getAttribute("role");
+                if (role && TOGGLE_ROLES.indexOf(role) >= 0)
+                    return true;
+                if (ACTIVE_CLASS.test(el.getAttribute("class") || ""))
+                    return true;
+                let p = el.parentElement;
+                while (p) {
+                    const r = p.getAttribute("role");
+                    if (r && CONTAINER_ROLES.indexOf(r) >= 0)
+                        return true;
+                    p = p.parentElement;
+                }
+                return false;
+            };
+            const els = Array.from(document.querySelectorAll("button, [role=button]"))
+                .filter(isVisible)
+                .filter((el) => !isToggleLike(el));
             els.forEach((el, i) => el.setAttribute("data-val-ex", `e${i}`));
-            return els.map((el, i) => ({ ref: `e${i}`, text: (el.innerText || el.getAttribute("aria-label") || "").trim().slice(0, 40) }));
+            return els.map((el, i) => ({
+                ref: `e${i}`,
+                text: (el.innerText || el.getAttribute("aria-label") || "").trim().slice(0, 40),
+            }));
         });
         const buttons = await tag();
         const dead = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nyx-intelligence/val-mcp",
-  "version": "0.4.0",
+  "version": "0.5.1",
   "description": "Val — a 100% MCP QA agent for vibecoders. Drives a real browser to catch UX bugs (broken links, 404s, console errors, broken images) so your coding agent can fix them.",
   "type": "module",
   "license": "UNLICENSED",