@jwilger/pi-development-system 0.10.1 → 0.10.3

package/package.json

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

package/skills/tdd-canon/SKILL.md

@@ -39,9 +39,9 @@ This skill is how to work so that gate never fires.
 ## When RED is not required
 
 RED is for **adding or changing first-party production behaviour** when no existing
-failing test already proves the required change. These are exempt. Docs, config, test and generated files are
-exempt by path, so the gate never asks. For the rest, *name* the exemption in a
-recorded departure (scope `slice`; one departure covers the slice):
+failing test already proves the required change. Docs, config, test and generated files are exempt by path, so the gate never asks. For
+any other exemption below, *name* it in a recorded departure (scope `slice`; one
+departure covers the slice):
 
 - docs or metadata only
 - functionality removal (the proof is that nothing still depends on it)

package/src/core/lint-suppression.ts

@@ -31,7 +31,7 @@ const KINDS: readonly Kind[] = [
   {
     marker: "#![allow(",
     pattern: /#!\[allow\(/,
-    inline: () => undefined,
+    inline: (line) => lastGroup(/reason\s*=\s*"([^"]*)"/, line),
   },
   {
     marker: "#[expect(",
@@ -66,7 +66,8 @@ const KINDS: readonly Kind[] = [
   },
 ];
 
-const COMMENT_LINE = /^\s*(?:\/\/+|\/\*+|\*|#+)\s*(.*?)(?:\s*\*\/)?\s*$/;
+// `#` opens a comment in shell/Python, but `#[` / `#!` start Rust attributes and shebangs.
+const COMMENT_LINE = /^\s*(?:\/\/+|\/\*+|\*|#+(?![[!]))\s*(.*?)(?:\s*\*\/)?\s*$/;
 
 const isSuppressionLine = (line: string): boolean => KINDS.some((k) => k.pattern.test(line));
 

package/src/core/path-class.ts

@@ -8,6 +8,7 @@ const SOURCE_EXT =
 const DOCS_EXT = /\.(?:md|mdx|rst|txt|adoc)$/i;
 const DOCS_NAME = /^(?:LICENSE|LICENCE|CHANGELOG|NOTICE|AUTHORS|CONTRIBUTING)(?:\..*)?$/i;
 const CONFIG_EXT = /\.(?:json|jsonc|toml|ya?ml|ini|cfg|conf|env|properties)$/i;
+const TOOL_CONFIG = /(?:^|\/)[\w.-]+\.config\.[cm]?[jt]s$|(?:^|\/)\.[\w-]+rc\.[cm]?[jt]s$/;
 const CONFIG_NAME = /^(?:Dockerfile|Makefile|justfile|Rakefile|Gemfile|Procfile)$/i;
 // Build output directories only count at the repo root; `src/build/` is usually real source.
 const GENERATED_DIR =
@@ -21,12 +22,14 @@ const basename = (path: string): string => path.slice(path.lastIndexOf("/") + 1)
 export function classifyPath(input: string, profile?: TestPathProfile): PathClass {
   const path = input.replace(/^\.\//, "");
   const name = basename(path);
+  if (path.startsWith("../") || path.startsWith("/")) return "other"; // outside the repo
   if (GENERATED_DIR.test(path) || GENERATED_FILE.test(path)) return "generated";
   if (isTestPath(path, profile) && SOURCE_EXT.test(path)) return "test";
   if (DOCS_EXT.test(name) || DOCS_NAME.test(name) || /^docs\//.test(path)) return "docs";
   if (
     CONFIG_EXT.test(name) ||
     CONFIG_NAME.test(name) ||
+    TOOL_CONFIG.test(path) ||
     name.startsWith(".") ||
     /^\.github\//.test(path)
   ) {

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,7 +61,7 @@ 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);

package/src/gates/red-first-guard.ts

@@ -1,4 +1,6 @@
+import { readFileSync } from "node:fs";
 import { homedir } from "node:os";
+import { resolve } from "node:path";
 import { type ExtensionAPI, isToolCallEventType } from "@earendil-works/pi-coding-agent";
 import { classifyPath } from "../core/path-class.ts";
 import { normalizeRepoPath } from "../core/test-paths.ts";
@@ -11,11 +13,26 @@ export type RedFirstGuardDeps = { pi: ExtensionAPI; state: SessionState };
 const GATE_ID = "tdd.red-first";
 
 /** Test code written inside a source file (Rust `#[cfg(test)]`/`#[test]`): that edit is the RED step itself. */
-const INLINE_TEST = /#\[(?:cfg\(test\)|test|tokio::test|rstest)\]/;
+const INLINE_TEST = /#\[(?:cfg\(test\)|test|tokio::test|rstest)\]/g;
 
-const newText = (
+const countInlineTests = (text: string): number => text.match(INLINE_TEST)?.length ?? 0;
+
+const existing = (cwd: string, path: string): string => {
+  try {
+    return readFileSync(resolve(cwd, path), "utf8");
+  } catch {
+    return "";
+  }
+};
+
+/** Does this edit add test code? A whole-file write counts only markers beyond those already on disk. */
+const addsInlineTest = (
   input: { content: string } | { edits: ReadonlyArray<{ newText: string }> },
-): string => ("content" in input ? input.content : input.edits.map((e) => e.newText).join("\n"));
+  before: () => string,
+): boolean =>
+  "content" in input
+    ? countInlineTests(input.content) > countInlineTests(before())
+    : countInlineTests(input.edits.map((e) => e.newText).join("\n")) > 0;
 
 /** Exemption classes a machine cannot see; the departure names which one applies (research 02). */
 const JUDGED_EXEMPTIONS =
@@ -41,7 +58,7 @@ export function registerRedFirstGuard(deps: RedFirstGuardDeps): void {
     if (lastTestRun !== undefined && lastTestRun.exitCode !== 0) return undefined;
     const path = normalizeRepoPath(ctx.cwd, event.input.path, homedir());
     if (classifyPath(path) !== "source") return undefined;
-    if (INLINE_TEST.test(newText(event.input))) return undefined;
+    if (addsInlineTest(event.input, () => existing(ctx.cwd, path))) return undefined;
     if (departure.hasOpen()) {
       departure.consume();
       return undefined;