@windyroad/tdd 0.4.3-preview.565 → 0.4.4-preview.576

package/.claude-plugin/plugin.json

@@ -101,5 +101,5 @@
     }
   },
   "name": "wr-tdd",
-  "version": "0.4.3"
+  "version": "0.4.4"
 }

package/README.md

@@ -49,6 +49,20 @@ Once active, the workflow is enforced on every edit:
 
 Test files and config/doc files are always writable regardless of state.
 
+## Supported test layouts
+
+The hook associates an implementation file with its tracked test by matching one of the following path shapes (per [P201](../../docs/problems/verifying/201-windyroad-tdd-hook-only-recognises-same-dir-or-tests-test-associations.md)):
+
+| Shape | Impl path | Test path |
+|---|---|---|
+| Same-directory | `src/foo.js` | `src/foo.test.js` or `src/foo.spec.js` |
+| `__tests__/`-adjacent | `src/foo.js` | `src/__tests__/foo.test.js` |
+| Parent `__tests__/` | `src/components/Hero.tsx` | `src/__tests__/Hero.test.tsx` |
+| `test/`-mirror (Vitest default; many Jest setups) | `src/foo.js`, `src/a/b/foo.js`, `packages/<pkg>/src/foo.js` | `test/foo.test.js`, `test/a/b/foo.test.js`, `packages/<pkg>/test/foo.test.js` |
+| Cucumber step-definitions | `features/step_definitions/checkout.steps.js` | `features/checkout.feature` |
+
+The `test/`-mirror rule replaces the **last** `src` path segment with `test`, so it works at any nesting depth and for monorepo workspace layouts.
+
 ## How It Works
 
 | Hook | Trigger | What it does |

package/hooks/lib/tdd-gate.sh

@@ -157,6 +157,22 @@ tdd_find_test_for_impl() {
     return
   fi
 
+  # test/-mirror layout (P201): src/foo.js → test/foo.test.js, recursive for
+  # nested src/a/b/foo.js → test/a/b/foo.test.js, and workspace
+  # packages/<pkg>/src/foo.js → packages/<pkg>/test/foo.test.js.
+  # Compute the mirror once — depends only on DIR (the impl's directory).
+  # Replaces the LAST `src` path segment with `test`.
+  local MIRROR_DIR=""
+  case "$DIR" in
+    src)        MIRROR_DIR="test" ;;
+    src/*)      MIRROR_DIR="test/${DIR#src/}" ;;
+    */src)      MIRROR_DIR="${DIR%/src}/test" ;;
+    */src/*)
+      local _mirror_prefix="${DIR%/src/*}"
+      MIRROR_DIR="${_mirror_prefix}/test/${DIR#"${_mirror_prefix}"/src/}"
+      ;;
+  esac
+
   # Check tracked test files for a match
   # Priority: exact match in tracked files (any convention)
   while IFS= read -r tracked; do
@@ -188,6 +204,13 @@ tdd_find_test_for_impl() {
         ;;
     esac
 
+    # test/-mirror layout (P201): match when tracked_dir is the computed mirror
+    if [ -n "$MIRROR_DIR" ] && [ "$tracked_dir" = "$MIRROR_DIR" ]; then
+      case "$tracked_base" in
+        "${STEM}.test."*|"${STEM}.spec."*) echo "$tracked"; return ;;
+      esac
+    fi
+
     # Cucumber: features/step_definitions/foo.steps.js → features/foo.feature
     # If this impl is inside a step_definitions/ directory, look in the parent for a .feature file
     case "$DIR" in

package/hooks/test/tdd-gate.bats

@@ -238,6 +238,61 @@ teardown() {
   [ -z "$result" ]
 }
 
+# --- tdd_find_test_for_impl: test/-mirror layout (P201) ---
+# Vitest default + many Jest setups mirror src/ under a sibling test/ tree.
+# A new behavioural rule: replace the LAST `src` path segment with `test`
+# to map an impl path to its mirrored test directory.
+
+@test "find_test_for_impl: test/-mirror at top level (src/foo.js → test/foo.test.js)" {
+  tdd_add_test_file "$TEST_SESSION" "test/foo.test.js"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "src/foo.js")
+  [ "$result" = "test/foo.test.js" ]
+}
+
+@test "find_test_for_impl: test/-mirror with .spec variant (src/foo.js → test/foo.spec.js)" {
+  tdd_add_test_file "$TEST_SESSION" "test/foo.spec.js"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "src/foo.js")
+  [ "$result" = "test/foo.spec.js" ]
+}
+
+@test "find_test_for_impl: test/-mirror preserves .tsx (src/Hero.tsx → test/Hero.test.tsx)" {
+  tdd_add_test_file "$TEST_SESSION" "test/Hero.test.tsx"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "src/Hero.tsx")
+  [ "$result" = "test/Hero.test.tsx" ]
+}
+
+@test "find_test_for_impl: test/-mirror recursive nested (src/a/b/foo.js → test/a/b/foo.test.js)" {
+  tdd_add_test_file "$TEST_SESSION" "test/a/b/foo.test.js"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "src/a/b/foo.js")
+  [ "$result" = "test/a/b/foo.test.js" ]
+}
+
+@test "find_test_for_impl: test/-mirror workspace layout (packages/foo/src/x.ts → packages/foo/test/x.test.ts)" {
+  tdd_add_test_file "$TEST_SESSION" "packages/foo/test/x.test.ts"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "packages/foo/src/x.ts")
+  [ "$result" = "packages/foo/test/x.test.ts" ]
+}
+
+@test "find_test_for_impl: test/-mirror workspace nested (packages/foo/src/a/b/x.ts → packages/foo/test/a/b/x.test.ts)" {
+  tdd_add_test_file "$TEST_SESSION" "packages/foo/test/a/b/x.test.ts"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "packages/foo/src/a/b/x.ts")
+  [ "$result" = "packages/foo/test/a/b/x.test.ts" ]
+}
+
+@test "find_test_for_impl: test/-mirror does NOT match wrong stem (src/foo.js + test/bar.test.js → empty)" {
+  tdd_add_test_file "$TEST_SESSION" "test/bar.test.js"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "src/foo.js")
+  [ -z "$result" ]
+}
+
+@test "find_test_for_impl: test/-mirror does NOT match without src/ anchor (lib/foo.js + test/foo.test.js → empty)" {
+  # Ticket scope is explicitly src/-anchored mirror; lib/foo.js → test/foo.test.js
+  # is NOT a recognised pairing (the ticket frames it as "src/foo.js → test/foo.test.js").
+  tdd_add_test_file "$TEST_SESSION" "test/foo.test.js"
+  result=$(tdd_find_test_for_impl "$TEST_SESSION" "lib/foo.js")
+  [ -z "$result" ]
+}
+
 # --- State for impl files (via association) ---
 
 @test "read_state_for_impl: returns IDLE when no associated test" {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/tdd",
-  "version": "0.4.3-preview.565",
+  "version": "0.4.4-preview.576",
   "description": "TDD state machine enforcement (Red-Green-Refactor cycle)",
   "bin": {
     "windyroad-tdd": "./bin/install.mjs"