@jwilger/pi-development-system 0.10.2 → 0.10.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.10.2",
+  "version": "0.10.4",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/src/core/test-paths.ts

@@ -14,6 +14,8 @@ const PATTERNS: readonly RegExp[] = [
   /_test\.(go|rb|py|exs?|rs)$/,
   /(^|\/)test_[^/]+\.py$/,
   /_spec\.rb$/,
+  /(^|\/)tests?\.rs$/,
+  /_tests\.rs$/,
 ];
 
 /** Glob match without building a RegExp: `**` spans directories, `*` stays within a segment. */

package/src/core/test-runner.ts

@@ -27,6 +27,8 @@ function isRunner(words: readonly string[]): boolean {
   if (head === undefined) return false;
   const args = rest.filter((w) => !w.startsWith("+"));
   const first = args[0];
+  // Compile-only and listing runs prove nothing about RED/GREEN, so they must not overwrite a real result.
+  if (rest.some((w) => /^--(?:no-run|list|help)$/.test(w) || w === "-h")) return false;
   if (head === "cargo") return first === "test" || first === "nextest";
   if (head === "go" || head === "deno") return first === "test";
   if (head === "node") return rest.includes("--test");
@@ -59,22 +61,27 @@ export type ResultLike = {
 /** A pipe or sequence makes the shell report the last command's status, not the runner's. */
 const MASKS_STATUS = /\||;/;
 const FAILURE_MARKERS =
-  /^# fail [1-9]|test result: FAILED|\b[1-9]\d* (?:failed|failing)\b|^FAILED\b|^FAIL\b|\bnot ok\b/m;
+  /^(?:# |ℹ )fail [1-9]|test result: FAILED|\b[1-9]\d* (?:failed|failing)\b|^FAILED\b|^FAIL\b|\bnot ok\b/m;
 
 const isRecord = (v: unknown): v is Record<string, unknown> =>
   typeof v === "object" && v !== null && !Array.isArray(v);
 
 /** Exit code of a bash tool result: structured `exit_code`, else pi's status line, else error/not. */
 export function exitCodeOf(result: ResultLike): number {
+  const reported = reportedExit(result);
+  if (reported !== 0) return reported;
+  // `npm test | tail` exits 0 whatever the tests did, even in the structured status; trust the output.
+  const masked = result.command !== undefined && MASKS_STATUS.test(result.command);
+  return masked && FAILURE_MARKERS.test(result.text) ? 1 : 0;
+}
+
+function reportedExit(result: ResultLike): number {
   if (isRecord(result.structured) && typeof result.structured.exit_code === "number") {
     return result.structured.exit_code;
   }
   const line = /Command exited with code (\d+)\s*$/.exec(result.text);
   if (line?.[1] !== undefined) return Number(line[1]);
-  if (result.isError) return 1;
-  // `npm test | tail` exits 0 whatever the tests did; trust the output over the masked status.
-  const masked = result.command !== undefined && MASKS_STATUS.test(result.command);
-  return masked && FAILURE_MARKERS.test(result.text) ? 1 : 0;
+  return result.isError ? 1 : 0;
 }
 
 /** The last few non-empty output lines, redacted and bounded: what the runner said at the end. */