argusqa-os 9.6.4 → 9.6.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argusqa-os",
-  "version": "9.6.4",
+  "version": "9.6.6",
   "mcpName": "io.github.ironclawdevs27/argus",
   "description": "Argus — AI-powered automated dev-testing platform using Chrome DevTools MCP and Claude Code",
   "keywords": [

package/src/cli/pr-validate.js

@@ -246,8 +246,19 @@ async function main() {
     // Step 5: Audit each affected route via crawlRouteCheap
     // Preserve path prefix (e.g. /project/ in GitHub Pages) — .origin would strip it
     const baseUrl = targetUrl.replace(/\/$/, '');
-    for (const route of affected) {
-      const url = new URL(route.path, targetUrl).href;
+
+    // Normalize route paths — crawlRouteCheap builds URLs via string concat (baseUrl + route.path)
+    // so paths without a leading slash produce malformed URLs like https://example.comlogin
+    const normalizedAffected = affected.map(r => {
+      if (!r.path.startsWith('/')) {
+        console.log(`::warning::Route path "${r.path}" has no leading slash — normalizing to "/${r.path}". Update argus.routes.json to use a leading slash.`);
+        return { ...r, path: `/${r.path}` };
+      }
+      return r;
+    });
+
+    for (const route of normalizedAffected) {
+      const url = `${baseUrl}${route.path}`;
       console.log(`[argus] → Auditing ${url}`);
 
       try {
@@ -276,6 +287,18 @@ async function main() {
       }
     }
 
+    // Guard: if every route failed with an exception, the app was unreachable after
+    // the preflight check (e.g. race condition where app died between check and crawl).
+    // Throwing here causes the step to exit 1, which correctly blocks the merge.
+    const routeFailCount = perRoute.filter(r => r.error).length;
+    if (routeFailCount > 0 && routeFailCount === perRoute.length) {
+      throw new Error(
+        `All ${perRoute.length} route audit(s) failed — Chrome could not reach the app. ` +
+        `Ensure TARGET_DEV_URL is accessible throughout the job. ` +
+        `First error: ${perRoute[0].error}`,
+      );
+    }
+
     // Step 6: Compute aggregate summary and merge-block decision
     const summary = {
       critical: allFindings.filter(f => f.severity === 'critical').length,
@@ -289,13 +312,13 @@ async function main() {
       false;
 
     // Step 7: Write GitHub Actions outputs and step summary
-    writeGithubOutputs({ blocked, summary, affectedRoutes: affected });
-    writeStepSummary(buildStepSummary({ blocked, summary, affectedRoutes: affected, perRoute, findings: allFindings, changedFiles: files, blockOn }));
+    writeGithubOutputs({ blocked, summary, affectedRoutes: normalizedAffected });
+    writeStepSummary(buildStepSummary({ blocked, summary, affectedRoutes: normalizedAffected, perRoute, findings: allFindings, changedFiles: files, blockOn }));
 
     // Step 8: Emit JSON result to stdout for downstream pipeline steps
     const result = {
       prUrl, targetUrl,
-      affectedRoutes: affected.map(r => r.path),
+      affectedRoutes: normalizedAffected.map(r => r.path),
       changedFiles: files,
       findings: allFindings,
       perRoute,
@@ -306,7 +329,10 @@ async function main() {
     console.log(JSON.stringify(result, null, 2));
 
     if (blocked) {
-      console.error(`::error::Argus PR Validator: ${summary.critical} critical finding(s) found. Merge blocked (block-on=${blockOn}).`);
+      const blockReason = blockOn === 'warning'
+        ? `${summary.critical} critical + ${summary.warning} warning finding(s) found`
+        : `${summary.critical} critical finding(s) found`;
+      console.error(`::error::Argus PR Validator: ${blockReason}. Merge blocked (block-on=${blockOn}).`);
       process.exit(1);
     }
 

package/src/mcp-server.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * Argus MCP Server (v9.6.4)
+ * Argus MCP Server (v9.6.6)
  *
  * 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.4' },
+  { name: 'argus', version: '9.6.6' },
   { capabilities: { tools: {} } },
 );