@axiomatic-labs/claudeflow 2.13.84 → 2.13.85

package/lib/panel.js

@@ -1039,13 +1039,22 @@ function severityFor(id) {
     }
     case 'validators': {
       if (!s.validators || !s.validators.available) return 'info';
-      // Block > error > pass > skip. Block is the actionable state — the
-      // hook actually halted the agent. Surface it the same way Observer
-      // surfaces unresolved incidents (warn = amber dot in the nav).
-      if (s.validators.stats && s.validators.stats.block > 0) return 'warn';
-      if (s.validators.stats && s.validators.stats.error > 0) return 'warn';
-      if (s.validators.stats && s.validators.stats.pass > 0) return 'ok';
-      return 'info';
+      // Compute the CURRENT state per (validator, scope) instead of the
+      // all-time tally. Runs are stored newest-first, so the first
+      // encounter of a given key is the latest run for that combo.
+      // Pre-v2.13.85 we used stats.block > 0, which kept the badge amber
+      // forever once anything had ever blocked — even after the LLM fixed
+      // the underlying bug and subsequent runs passed.
+      const seen = new Set();
+      let currentlyFailing = 0;
+      for (const r of s.validators.runs || []) {
+        const key = (r.validator || '') + '|' + (r.scope || '');
+        if (seen.has(key)) continue;
+        seen.add(key);
+        if (r.status === 'block' || r.status === 'error') currentlyFailing += 1;
+      }
+      if (currentlyFailing > 0) return 'warn';
+      return seen.size > 0 ? 'ok' : 'info';
     }
     case 'setupContext': return !s.setupContext.exists ? 'err' : (s.setupContext.toolingComplete ? 'ok' : 'warn');
     case 'activeRun': return s.activeRun.active ? 'info' : 'info';
@@ -1757,10 +1766,25 @@ function renderValidators() {
     <div style="display:flex;gap:6px;flex-wrap:wrap;">\${pillsHtml}</div>
   </div>\`;
 
+  // Compute "current state" per (validator, scope) — newest run wins, so
+  // the user sees what's failing RIGHT NOW vs what once failed and has
+  // since been fixed. The historical totals remain available below.
+  const currentBy = new Map();
+  for (const r of v.runs) {
+    const key = r.validator + '|' + r.scope;
+    if (!currentBy.has(key)) currentBy.set(key, r);
+  }
+  const currentRuns = [...currentBy.values()];
+  const currentFailing = currentRuns.filter((r) => r.status === 'block' || r.status === 'error');
+  const currentStateLine = currentFailing.length === 0
+    ? \`<span style="color:var(--ok,#3fb950);">✓ All \${currentRuns.length} (validator, scope) combo(s) currently passing</span>\`
+    : \`<span style="color:var(--err,#f85149);">✗ \${currentFailing.length} of \${currentRuns.length} (validator, scope) combo(s) currently failing:</span> \${currentFailing.map((r) => escapeHtml(r.validator + ' @ ' + r.scope)).join(', ')}\`;
+
   return \`<h2>Validators
     <button id="clear-validators" class="logs-clear-btn" style="float:right;" data-scope="\${escapeHtml(validatorsScopeFilter)}">\${clearLabel}</button>
   </h2>
-  <p class="muted" style="margin-top:0;">Last updated \${formatRelativeTime(v.updated_at)} · \${v.stats.total} total run(s): \${v.stats.pass} pass, \${v.stats.block} block, \${v.stats.skip} skip, \${v.stats.error} error.</p>
+  <p style="margin-top:0;">\${currentStateLine}</p>
+  <p class="muted" style="margin-top:4px;font-size:12px;">History: last updated \${formatRelativeTime(v.updated_at)} · \${v.stats.total} total run(s): \${v.stats.pass} pass, \${v.stats.block} block, \${v.stats.skip} skip, \${v.stats.error} error.</p>
   <div class="card" style="padding:10px 14px;margin:12px 0;">
     \${filterRow('Validator', validatorPills)}
     \${filterRow('Scope', scopePills)}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@axiomatic-labs/claudeflow",
-  "version": "2.13.84",
+  "version": "2.13.85",
   "description": "Claudeflow — AI-powered development toolkit for Claude Code. Skills, agents, hooks, and quality gates that ship production apps.",
   "bin": {
     "claudeflow": "./bin/cli.js"