From 11b2cffaee398b96f5d8f31a91e10201fe051d4d Mon Sep 17 00:00:00 2001
From: Murat Ozcan <murat@mac.lan>
Date: Tue, 11 Nov 2025 09:42:29 -0600
Subject: [PATCH] addressed review comments

---
 .bmad/bmm/docs/test-architecture.md                        | 2 +-
 .claude/settings.local.json                                | 7 -------
 .gitignore                                                 | 3 ++-
 .../bmm/workflows/testarch/test-design/workflow.yaml       | 2 +-
 4 files changed, 4 insertions(+), 10 deletions(-)
 delete mode 100644 .claude/settings.local.json

diff --git a/.bmad/bmm/docs/test-architecture.md b/.bmad/bmm/docs/test-architecture.md
index ce7304e0..6ef0ffec 100644
--- a/.bmad/bmm/docs/test-architecture.md
+++ b/.bmad/bmm/docs/test-architecture.md
@@ -167,7 +167,7 @@ Epic/Release Gate → TEA: *nfr-assess, *trace Phase 2 (release decision)
 | **Phase 4 Per-Story** | *atdd, *automate, \*test-review, \*trace | Per story        | Test implementation and quality validation     |
 | **Release Gate**      | *nfr-assess, *trace (Phase 2: gate)      | Per epic/release | Go/no-go decision                              |
 
-**Note**: Like `*trace`, `*test-design` is now a two-phase workflow: Phase 1 (system-level testability review in Phase 3) + Phase 2 (epic-level test planning in Phase 4). Auto-detects mode based on project phase.
+**Note**: Like `*trace`, `*test-design` is now a dual-mode workflow: system-level mode (testability review in Phase 3) and epic-level mode (test planning in Phase 4). Auto-detects mode based on project phase.
 
 ### Unique Directory Architecture
 
diff --git a/.claude/settings.local.json b/.claude/settings.local.json
deleted file mode 100644
index ff1d37c7..00000000
--- a/.claude/settings.local.json
+++ /dev/null
@@ -1,7 +0,0 @@
-{
-  "permissions": {
-    "allow": ["Bash(tree:*)"],
-    "deny": [],
-    "ask": []
-  }
-}
diff --git a/.gitignore b/.gitignore
index 9eff8ef1..d1aa4849 100644
--- a/.gitignore
+++ b/.gitignore
@@ -41,6 +41,7 @@ cursor
 .mcp.json
 CLAUDE.local.md
 .serena/
+.claude/settings.local.json
 
 # Project-specific
 .bmad-core
@@ -58,4 +59,4 @@ tools/template-test-generator/test-scenarios/
 
 # Test Install Output
 
-z*/
+z*/.claude/settings.local.json
diff --git a/src/modules/bmm/workflows/testarch/test-design/workflow.yaml b/src/modules/bmm/workflows/testarch/test-design/workflow.yaml
index 3d86a0cc..202551ba 100644
--- a/src/modules/bmm/workflows/testarch/test-design/workflow.yaml
+++ b/src/modules/bmm/workflows/testarch/test-design/workflow.yaml
@@ -1,6 +1,6 @@
 # Test Architect workflow: test-design
 name: testarch-test-design
-description: "Two-phase workflow: (1) System-level testability review in Phase 3, or (2) Epic-level test planning in Phase 4. Auto-detects mode based on project phase."
+description: "Dual-mode workflow: (1) System-level testability review in Solutioning phase, or (2) Epic-level test planning in Implementation phase. Auto-detects mode based on project phase."
 author: "BMad"
 
 # Critical variables from config