specrails-core 6.2.1 → 6.2.2

package/dist/agent-runtime/engine/pieces/verify.js

@@ -3,7 +3,7 @@ import { contentDigest } from '../canonical-json.js';
 import { advisoryMemory } from './project-memory.js';
 import { executeVerification, validateVerificationRequest } from '../../../pipeline/pipeline-state.js';
 import { withScopeDefault } from '../../change-scope.js';
-import { verificationFailureSummary } from '../../verification-diagnostics.js';
+import { verificationDiagnosticPriority, verificationFailureSummary } from '../../verification-diagnostics.js';
 import { boundedText, json, paramsSchema, positiveInteger } from './shared.js';
 /** Keep real subprocess diagnostics in committed outputs for subsequent agents.
  * The full command evidence remains in the receipt; prioritize failed checks
@@ -17,15 +17,23 @@ function verificationDiagnostics(receipt) {
         const limit = Math.min(8_000, remaining);
         const source = stripVTControlCharacters([command.stdout ?? '', command.stderr ?? '', command.output].join('\n'));
         const lines = source.split('\n');
-        const failures = [];
-        let used = 0;
-        for (let i = 0; command.exitCode !== 0 && i < lines.length && used < 6_000; i++) {
-            if (!/^\s*(?:not ok\b|[✖✗]\s|test at\b|FAIL\b|Error:|AssertionError\b|error:)/.test(lines[i]))
+        // Reserve the bounded facts before contextual blocks: a source dump or a
+        // long warning next to the first failure must not hide later diagnostics.
+        const failures = command.exitCode !== 0 ? verificationFailureSummary(command) : [];
+        let used = failures.reduce((total, line) => total + line.length + 1, 0);
+        const anchors = command.exitCode !== 0 ? lines.map((line, index) => ({ line, index, priority: verificationDiagnosticPriority(line) }))
+            .filter((anchor) => anchor.priority !== undefined)
+            .sort((a, b) => a.priority - b.priority || a.index - b.index) : [];
+        const selected = new Set();
+        for (const { line, index } of anchors) {
+            if (used >= 6_000)
+                break;
+            if (selected.has(line.trim()))
                 continue;
-            const excerpt = lines.slice(Math.max(0, i - 2), i + 28).join('\n').slice(0, 6_000 - used);
+            selected.add(line.trim());
+            const excerpt = lines.slice(Math.max(0, index - 2), index + 28).join('\n').slice(0, 6_000 - used);
             failures.push(excerpt);
-            used += excerpt.length;
-            i += 27;
+            used += excerpt.length + 1;
         }
         const failureText = failures.join('\n').slice(0, Math.min(6_000, limit));
         const tailBudget = Math.max(0, limit - failureText.length - (failureText ? 1 : 0));

package/dist/agent-runtime/verification-diagnostics.d.ts

@@ -1,4 +1,8 @@
 import type { CommandReceipt } from '../pipeline/pipeline-state.js';
+/** Shared ordering for summary facts and contextual excerpts. Input is an
+ * ANSI-normalized output line; ordinary warnings and quoted source are not
+ * failure anchors. Lower ranks reserve space for the actual error first. */
+export declare function verificationDiagnosticPriority(raw: string): number | undefined;
 /** Exact subprocess facts, not an AI diagnosis or a waiver of the failed check.
  * Separate them from source dumps, reserving space for each kind of fact. */
 export declare function verificationFailureSummary(command: Pick<CommandReceipt, 'stdout' | 'stderr' | 'output'>): string[];

package/dist/agent-runtime/verification-diagnostics.js

@@ -1,24 +1,38 @@
 import { stripVTControlCharacters } from 'node:util';
+/** Shared ordering for summary facts and contextual excerpts. Input is an
+ * ANSI-normalized output line; ordinary warnings and quoted source are not
+ * failure anchors. Lower ranks reserve space for the actual error first. */
+export function verificationDiagnosticPriority(raw) {
+    const line = raw.trim();
+    if (/node:internal\//.test(line) || /^[✖✗]\s+failing tests:/.test(line)
+        || /^(?:[✖✗]\s+)?(?:\d+\s+problems?\s+\(\s*)?0 errors?(?:,\s*\d+ warnings?)?\)?$/.test(line))
+        return undefined;
+    const compiler = /^(?!['"`]|(?:actual|expected|source):).+(?:\(\d+,\d+\)|:\d+(?::\d+)?)(?::|\s+-)?\s+(?:fatal\s+)?error\b/.test(line)
+        || /^(?:fatal\s+)?error(?:\s+[A-Z][A-Z0-9]*\d+)?\s*:/.test(line);
+    if (compiler || /^(?:AssertionError\b|\w*Error(?:\s+\[[^\]]+\])?:|code:)/.test(line))
+        return 0;
+    if (/^(?:not ok\b|[✖✗]\s|FAIL\b)/.test(line))
+        return 1;
+    if (/^expected:/.test(line))
+        return 2;
+    if (/^(?:test at\b|location:|at .+:\d+(?::\d+)?\)?$)/.test(line))
+        return 3;
+    return undefined;
+}
 /** Exact subprocess facts, not an AI diagnosis or a waiver of the failed check.
  * Separate them from source dumps, reserving space for each kind of fact. */
 export function verificationFailureSummary(command) {
     const source = stripVTControlCharacters([command.stdout ?? '', command.stderr ?? '', command.output].join('\n'));
     const lines = source.split('\n').map(line => line.trim());
-    const groups = [
-        /^(?:not ok\b|[✖✗]\s|FAIL\b)/,
-        /^(?:AssertionError\b|\w*Error:|error:|code:)/,
-        /^expected:/,
-        /^(?:test at\b|location:|at .+:\d+(?::\d+)?\)?$)/,
-    ];
     const selected = new Set();
     const facts = [];
-    for (const pattern of groups) {
+    for (const priority of [0, 1, 2, 3]) {
         let remaining = 750;
         let count = 0;
         for (const line of lines) {
             if (remaining <= 0 || count >= 3)
                 break;
-            if (!pattern.test(line) || /node:internal\//.test(line) || /^[✖✗]\s+failing tests:/.test(line) || selected.has(line))
+            if (verificationDiagnosticPriority(line) !== priority || selected.has(line))
                 continue;
             const bounded = line.slice(0, Math.min(512, remaining - 1));
             if (!bounded)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specrails-core",
-  "version": "6.2.1",
+  "version": "6.2.2",
   "description": "Agent workflow engine for specrails-desktop: provider-independent installer and programmatic agent runtime for Claude Code and Codex",
   "type": "module",
   "exports": {