Merge 9351582ba6 into 259e8a11ba

fix(review): address PR #1791 review feedback
- Remove "Task tool" reference per maintainer; use generic "subagents" - Fix nested triple-backtick fencing with four-tick outer fence - Widen location format to support multi-line ranges and hunk refs - Add JSON-safety constraint to guard_snippet field - Tighten input loading to "strictly from provided input" - Replace vague "unreadable" with "cannot be decoded as text" - Replace vague "increased scrutiny" with concrete re-analysis checklist - Resolve HALT-immediately vs re-analysis conflict in LLM instructions Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 01:41:25 +07:00 · 2026-03-01 13:43:26 -07:00 · 2026-03-01 13:22:00 -07:00 · 2026-03-01 13:20:59 -07:00
1 changed files with 10 additions and 0 deletions
--- a/src/core/module.yaml
+++ b/src/core/module.yaml
@ -23,3 +23,13 @@ output_folder:
  prompt: "Where should output files be saved?"
  default: "_bmad-output"
  result: "{project-root}/{value}"
+
+tool_supports_subagents:
+  prompt: "Subagents are supported by the LLM or Tool I will be using?"
+  default: true
+  result: "{value}"
+
+tool_supports_agent_teams:
+  prompt: "Agent Teams are supported by the LLM or Tool I will be using?"
+  default: false
+  result: "{value}"
Author	SHA1	Message	Date
Alex Verkhovsky	196864c814	Merge `9351582ba6` into `259e8a11ba`	2026-03-06 01:41:25 +07:00
Alex Verkhovsky	9351582ba6	fix(review): address PR #1791 review feedback - Remove "Task tool" reference per maintainer; use generic "subagents" - Fix nested triple-backtick fencing with four-tick outer fence - Widen location format to support multi-line ranges and hunk refs - Add JSON-safety constraint to guard_snippet field - Tighten input loading to "strictly from provided input" - Replace vague "unreadable" with "cannot be decoded as text" - Replace vague "increased scrutiny" with concrete re-analysis checklist - Resolve HALT-immediately vs re-analysis conflict in LLM instructions Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-01 13:43:26 -07:00
Alex Verkhovsky	9b9ae44eb9	fix(core): resolve contradictions in edge case hunter task spec - Show array wrapper [{}] in output-format example to match JSON array contract, and document empty array [] as valid output - Consolidate empty-content handling: step 1 now defers to halt-conditions instead of defining separate "ask and abort" behavior - Zero-findings halt no longer contradicts JSON contract: re-analyze once, then return [] instead of ambiguous "HALT or re-analyze" - Soften "Execute ALL steps" to acknowledge halt-conditions can interrupt	2026-03-01 13:22:00 -07:00
Alex Verkhovsky	8e84b05d5b	feat(skills): add edge case hunter as parallel review layer in PR review Wire review-edge-case-hunter.xml into bmad-os-review-pr as a second review layer running in parallel with the adversarial review. Both subagents receive the same PR diff concurrently. Findings are merged, deduplicated, and tagged by source before tone transformation.	2026-03-01 13:20:59 -07:00