argusqa-os 9.6.1 → 9.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argusqa-os",
-  "version": "9.6.1",
+  "version": "9.6.3",
   "mcpName": "io.github.ironclawdevs27/argus",
   "description": "Argus — AI-powered automated dev-testing platform using Chrome DevTools MCP and Claude Code",
   "keywords": [
@@ -33,7 +33,8 @@
   "bin": {
     "argus": "src/cli/init.js",
     "argus-mcp": "src/mcp-server.js",
-    "argusqa-os": "src/mcp-server.js"
+    "argusqa-os": "src/mcp-server.js",
+    "argus-pr-validate": "src/cli/pr-validate.js"
   },
   "scripts": {
     "setup": "node -e \"import('fs').then(fs => fs.default.mkdirSync('./reports', { recursive: true }))\"",

package/src/cli/pr-validate.js

@@ -154,16 +154,14 @@ async function loadRoutes() {
     }
   }
 
-  // 3. Default routes from the package's targets.js
-  try {
-    const { routes } = await import('../config/targets.js');
-    if (Array.isArray(routes) && routes.length > 0) {
-      return routes;
-    }
-  } catch { /* targets.js may not export routes in all environments */ }
-
-  // 4. Final fallback — audit root path only
+  // 3. Final fallback — audit root path only
+  // NOTE: We deliberately do NOT fall back to the package's targets.js here.
+  // targets.js contains the developer's own demo routes with app-specific
+  // waitFor selectors (e.g. [data-testid="dashboard"]) that do not exist on
+  // user apps — falling back to them causes false-positive load_failure
+  // findings and incorrectly blocks merges.
   console.log('[argus] No routes configured — falling back to root path audit');
+  console.log('[argus] Tip: add argus.routes.json to your repo root to audit specific routes.');
   return [{ path: '/', name: 'home' }];
 }
 

package/src/mcp-server.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * Argus MCP Server (v9.6.1)
+ * Argus MCP Server (v9.6.3)
  *
  * Exposes Argus as an MCP server so Claude (or any MCP client) can call
  * argus_audit, argus_audit_full, argus_compare, argus_last_report, and
@@ -447,7 +447,7 @@ async function handleLastReport() {
 // ── Server bootstrap ──────────────────────────────────────────────────────────
 
 const server = new Server(
-  { name: 'argus', version: '9.6.1' },
+  { name: 'argus', version: '9.6.3' },
   { capabilities: { tools: {} } },
 );
 

package/src/utils/mcp-client.js

@@ -57,7 +57,7 @@ const TOOL_TIMEOUT_MS = parseInt(process.env.MCP_TOOL_TIMEOUT_MS ?? '30000', 10)
 export async function createMcpClient() {
   // On Windows, npx is npx.cmd — shell:true resolves this cross-platform.
   const proc = spawn('npx', [
-    '-y', 'chrome-devtools-mcp@latest',
+    '-y', 'chrome-devtools-mcp@1.1.1',
     `--browser-url=${BROWSER_URL}`,
     '--headless=true',
     '--viewport=1920x1080',

package/src/utils/pr-diff-analyzer.js

@@ -36,8 +36,6 @@ export function parsePrUrl(prUrl) {
  */
 export async function fetchPrFiles(prUrl, githubToken) {
   const { owner, repo, prNumber } = parsePrUrl(prUrl);
-  const apiUrl =
-    `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/files?per_page=100`;
   const headers = {
     Accept: 'application/vnd.github+json',
     'X-GitHub-Api-Version': '2022-11-28',
@@ -45,15 +43,41 @@ export async function fetchPrFiles(prUrl, githubToken) {
     ...(githubToken ? { Authorization: `Bearer ${githubToken}` } : {}),
   };
 
-  const res = await fetch(apiUrl, { headers });
-  if (!res.ok) {
-    const body = await res.text().catch(() => '');
-    throw new Error(`GitHub API ${res.status}: ${body || res.statusText}`);
+  const allFiles = [];
+  const MAX_PAGES = 3; // caps at 300 files; avoids runaway requests on mega-PRs
+
+  for (let page = 1; page <= MAX_PAGES; page++) {
+    const apiUrl = `https://api.github.com/repos/${owner}/${repo}/pulls/${prNumber}/files?per_page=100&page=${page}`;
+    const res = await fetch(apiUrl, { headers });
+    if (!res.ok) {
+      const body = await res.text().catch(() => '');
+      throw new Error(`GitHub API ${res.status}: ${body || res.statusText}`);
+    }
+    const files = await res.json();
+    allFiles.push(...files.map(f => f.filename));
+    if (files.length < 100) break; // last page reached
+  }
+
+  if (allFiles.length >= 300) {
+    console.log('::warning::PR has 300+ changed files — Argus analyzed the first 300. Routes affected by later files may be missed.');
   }
-  const files = await res.json();
-  return files.map(f => f.filename);
+
+  return allFiles;
 }
 
+/**
+ * Files that are never relevant to app routes — CI configs, docs, repo metadata.
+ * Changes to ONLY these files cause mapFilesToRoutes to return [] (skip audit).
+ */
+const EXCLUDED_PATTERNS = [
+  /^\.github\//i,
+  /^docs?\//i,
+  /\.md$/i,
+  /^(LICENSE|CHANGELOG|CONTRIBUTING|CODE_OF_CONDUCT|SECURITY)(\..*)?$/i,
+  /^\.gitignore$/i,
+  /^\.gitattributes$/i,
+];
+
 /**
  * Patterns that indicate an infrastructure-level file whose change can affect
  * every route — framework configs, root layouts, global stylesheets, package.json.
@@ -88,15 +112,22 @@ export function mapFilesToRoutes(changedFiles, routes) {
   if (!routes || routes.length === 0) return [];
   if (!changedFiles || changedFiles.length === 0) return routes;
 
+  // Strip files that are never app-route-relevant (CI configs, docs, repo metadata)
+  const appFiles = changedFiles.filter(
+    f => !EXCLUDED_PATTERNS.some(re => re.test(f)),
+  );
+
+  // README-only / CI-only PR — skip the audit entirely
+  if (appFiles.length === 0) return [];
+
   // Infrastructure change → full audit
-  if (changedFiles.some(f => INFRA_PATTERNS.some(re => re.test(f)))) {
+  if (appFiles.some(f => INFRA_PATTERNS.some(re => re.test(f)))) {
     return routes;
   }
 
-  // Build a flat set of lowercase slugs from every changed file path
+  // Build a flat set of lowercase slugs from app-relevant changed files
   const fileSlugs = new Set(
-    changedFiles.flatMap(f =>
-      // Strip extension, split on separators, keep non-trivial tokens
+    appFiles.flatMap(f =>
       f.toLowerCase()
         .replace(/\.[^./\\]+$/, '')
         .split(/[/\\._-]+/)