chore: cleaned up bad architecture file calls, legacy doc references, and case sensitivity issues to remove ambiguity (#718)

src/modules/bmm/workflows/4-implementation/review-story/workflow.yaml

@@ -34,24 +34,11 @@ variables:
     - "{project-root}/docs"
     - "{output_folder}"
   arch_docs_file_names: |
-    - prd.md
-    - epics.md
-    - high-level-architecture.md
-    - tech-stack.md
-    - unified-project-structure.md
-    - coding-standards.md
-    - testing-strategy.md
-    - security-guidelines.md
-    - backend-architecture.md
-    - frontend-architecture.md
-    - data-models.md
-    - database-schema.md
-    - rest-api-spec.md
-    - external-apis.md
+    - solution-architecture.md
   enable_mcp_doc_search: true # Prefer enabled MCP servers for doc/best-practice lookup
   enable_web_fallback: true # Fallback to web search/read-url if MCP not available
-  update_status_on_result: false # If true, update story Status based on review outcome
-  status_on_approve: "Done"
+  update_status_on_result: true # If true, update story Status based on review outcome
+  status_on_approve: "Review Passed"
   status_on_changes_requested: "InProgress"
   # Persistence controls for review action items and notes
   persist_action_items: true
@@ -71,7 +58,7 @@ variables:
 recommended_inputs:
   - story_markdown: "Path to the story markdown file flagged Ready for Review"
   - tech_spec: "Epic technical specification document (auto-discovered if omitted)"
-  - story_context: "Story Context XML/JSON path (auto-discovered if omitted)"
+  - story_context: "Story Context XML path (auto-discovered if omitted)"
 
 # Required tools (conceptual; executor should provide equivalents)
 required_tools: