agents all indicate hasSidecar true or false, validation requires it, agent builder and validator and editor use the field. Added a better brownfield doc

Author: Brian Madison

docs/modules/bmm-bmad-method/brownfield-guide.md

@@ -5,6 +5,7 @@ agent:
     title: "Commit Message Artisan"
     icon: "📜"
     module: stand-alone
+    hasSidecar: false
 
   persona:
     role: |

samples/sample-custom-modules/sample-unitary-module/agents/commit-poet/commit-poet.agent.yaml

@@ -5,6 +5,7 @@ agent:
     title: "Meditation Guide"
     icon: "🧘"
     module: "mwm"
+    hasSidecar: false
   persona:
     role: "Mindfulness and meditation specialist"
     identity: |

samples/sample-custom-modules/sample-wellness-module/agents/meditation-guide.agent.yaml

@@ -7,6 +7,7 @@ agent:
     name: "BMad Master"
     title: "BMad Master Executor, Knowledge Custodian, and Workflow Orchestrator"
     icon: "🧙"
+    hasSidecar: false
 
   persona:
     role: "Master Task Executor + BMad Expert + Guiding Facilitator Orchestrator"

src/core/agents/bmad-master.agent.yaml

@@ -9,6 +9,7 @@ agent:
     title: Agent Building Expert
     icon: 🤖
     module: bmb
+    hasSidecar: false
 
   persona:
     role: Agent Architecture Specialist + BMAD Compliance Expert
@@ -34,3 +35,7 @@ agent:
     - trigger: EA or fuzzy match on edit-agent
       exec: "{project-root}/_bmad/bmb/workflows/agent/workflow.md"
       description: "[EA] Edit existing BMAD agents while maintaining compliance"
+
+    - trigger: VA or fuzzy match on validate-agent
+      exec: "{project-root}/_bmad/bmb/workflows/agent/workflow.md"
+      description: "[VA] Validate existing BMAD agents and offer to improve deficiencies"

src/modules/bmb/agents/agent-builder.agent.yaml

@@ -9,6 +9,7 @@ agent:
     title: Module Creation Master
     icon: 🏗️
     module: bmb
+    hasSidecar: false
 
   persona:
     role: Module Architecture Specialist + Full-Stack Systems Designer

src/modules/bmb/agents/module-builder.agent.yaml

@@ -9,6 +9,7 @@ agent:
     title: Workflow Building Master
     icon: 🔄
     module: bmb
+    hasSidecar: false
 
   persona:
     role: Workflow Architecture Specialist + Process Design Expert

src/modules/bmb/agents/workflow-builder.agent.yaml

@@ -7,7 +7,8 @@ Validate Expert agents meet BMAD quality standards.
 ## YAML Structure
 
 - [ ] YAML parses without errors
-- [ ] `agent.metadata` includes: `id`, `name`, `title`, `icon`, `module`
+- [ ] `agent.metadata` includes: `id`, `name`, `title`, `icon`, `module`, `hasSidecar`
+- [ ] `agent.metadata.hasSidecar` is `true` (Expert agents have sidecars)
 - [ ] `agent.metadata.module` is `stand-alone` or module code (`bmm`, `cis`, `bmgd`, etc.)
 - [ ] `agent.persona` exists with: `role`, `identity`, `communication_style`, `principles`
 - [ ] `agent.critical_actions` exists (MANDATORY for Expert)

src/modules/bmb/reference/agents/simple-examples/README.md

@@ -34,11 +34,13 @@ Validate Module agents meet BMAD quality standards.
 ### Module Agent Can Be Simple OR Expert
 
 **If Simple-structure Module Agent:**
+- [ ] `agent.metadata.hasSidecar` is `false` (no sidecar)
 - [ ] Single .agent.yaml file (no sidecar)
 - [ ] Uses `exec:` for workflow references
 - [ ] Pass `simple-agent-validation.md` first
 
 **If Expert-structure Module Agent:**
+- [ ] `agent.metadata.hasSidecar` is `true` (has sidecar)
 - [ ] Has sidecar folder
 - [ ] Uses `exec:` for workflow references
 - [ ] Sidecar paths use `{project-root}/_bmad/_memory/{sidecar-folder}/` format