@jwilger/pi-development-system 0.10.5 → 0.10.6

package/package.json

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

package/src/core/test-runner.ts

@@ -59,7 +59,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 MASKS_STATUS = /[|;\n]/;
 const FAILURE_MARKERS =
   /^(?:# |ℹ )fail [1-9]|test result: FAILED|\b[1-9]\d* (?:failed|failing)\b|^FAILED\b|^FAIL\b|\bnot ok\b/m;
 

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

@@ -15,6 +15,8 @@ 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|test_case|rstest|proptest|\w+::test)\b/g;
 
+const INLINE_MODULE = /#\[cfg\(test\)\]\s*(?:#\[[^\]]*\]\s*)*mod\s+\w+\s*\{/;
+
 const countInlineTests = (text: string): number => text.match(INLINE_TEST)?.length ?? 0;
 
 const existing = (cwd: string, path: string): string => {
@@ -31,7 +33,8 @@ type EditInput =
 
 /** Is every edit inside the file's existing `#[cfg(test)]` module? Changing a test's expectation is RED itself. */
 const insideTestModule = (edits: ReadonlyArray<{ oldText: string }>, file: string): boolean => {
-  const start = file.indexOf("#[cfg(test)]");
+  // Only an inline module body counts: `mod tests;`, `#[cfg(test)] use ...` and fns are not test code.
+  const start = INLINE_MODULE.exec(file)?.index ?? -1;
   return start >= 0 && edits.every((e) => e.oldText !== "" && file.indexOf(e.oldText) > start);
 };