From 2af943b51d1c968bae9a4a4b689b44570d0ba104 Mon Sep 17 00:00:00 2001 From: Hongming Wang Date: Wed, 15 Apr 2026 01:53:47 -0700 Subject: [PATCH] =?UTF-8?q?fix(template):=20#133=20=E2=80=94=20add=20code-?= =?UTF-8?q?review=20plugins=20to=20Dev=20Lead=20+=20QA=20Engineer?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Closes #133. Both roles previously inherited defaults only (ecc, molecule-dev, superpowers, careful-bash, prompt-watchdog, audit-trail, session-context, cron-learnings, update-docs) — no review skill. Dev Lead enforces PR quality gates per triage SKILL.md; QA Engineer reviews test coverage against acceptance criteria. Both need the 16-criteria code-review rubric and llm-judge to operate deterministically. Mirrors Security Auditor's existing \`[molecule-skill-code-review, molecule-skill-cross-vendor-review, molecule-skill-llm-judge]\` override. Dropped cross-vendor from these two since it's a noteworthy-PR tool — the workflow-triage entry in defaults already gates that for the ticks that need it. Co-Authored-By: Claude Opus 4.6 (1M context) --- org-templates/molecule-dev/org.yaml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/org-templates/molecule-dev/org.yaml b/org-templates/molecule-dev/org.yaml index ca79f8f7..977d7bf6 100644 --- a/org-templates/molecule-dev/org.yaml +++ b/org-templates/molecule-dev/org.yaml @@ -211,6 +211,12 @@ workspaces: tier: 3 model: opus files_dir: dev-lead + # Dev Lead enforces PR quality gates (see gate 2a in + # .claude/skills/triage/SKILL.md) and reviews engineering output + # before handoff to PM. The code-review skill surfaces the + # 16-criteria rubric — without it Dev Lead falls back to ad-hoc + # review prompts. Issue #133. + plugins: [molecule-skill-code-review, molecule-skill-llm-judge] canvas: { x: 650, y: 250 } initial_prompt: | You just started as Dev Lead. Set up silently — do NOT contact other agents. @@ -497,6 +503,9 @@ workspaces: tier: 3 model: opus files_dir: qa-engineer + # QA reviews test coverage + runs llm-judge on whether test + # deliverables actually match acceptance criteria. Issue #133. + plugins: [molecule-skill-code-review, molecule-skill-llm-judge] initial_prompt: | You just started as QA Engineer. Set up silently — do NOT contact other agents. 1. Clone the repo: git clone https://github.com/${GITHUB_REPO}.git /workspace/repo 2>/dev/null || (cd /workspace/repo && git pull)