Skip to content

chore: remove QA agent (Quinn) and migrate to Developer agent#2179

Merged
alexeyv merged 1 commit intomainfrom
chore/remove-qa-agent
Apr 2, 2026
Merged

chore: remove QA agent (Quinn) and migrate to Developer agent#2179
alexeyv merged 1 commit intomainfrom
chore/remove-qa-agent

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Apr 1, 2026

Summary

  • Delete the Quinn (bmad-agent-qa) agent wrapper — SKILL.md and manifest
  • Add QA test-generation capability (QA menu code) to Amelia (bmad-agent-dev)
  • Update agent tables and testing docs across EN, ZH-CN, and FR
  • Remove bmad-agent-qa from marketplace.json
  • Clean up Quinn references in party-mode and QA checklist

Follows the same pattern as the Barry removal in #2177.

Test plan

  • grep -ri quinn src/ docs/ website/ .claude-plugin/ | grep -v CHANGELOG → zero matches
  • grep -ri bmad-agent-qa src/ docs/ website/ .claude-plugin/ | grep -v CHANGELOG → zero matches
  • npm run quality passes
  • Dev agent menu shows QA capability after reinstall

🤖 Generated with Claude Code

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Apr 1, 2026

🤖 Augment PR Summary

Summary: This PR removes the standalone QA agent wrapper (Quinn) and consolidates QA test-generation entrypoints under the Developer agent (Amelia).

Changes:

  • Removed bmad-agent-qa skill wrapper (SKILL.md + manifest) and dropped it from the Claude plugin marketplace list.
  • Added a QA menu code to the Developer agent that routes to bmad-qa-generate-e2e-tests.
  • Updated the main Agents reference table to reflect QA being invoked via the Developer agent (and removed the Quinn row).
  • Refreshed Testing docs (EN/FR/ZH-CN) to describe the built-in QA capability as a workflow rather than a dedicated agent.
  • Renamed QA validation checklist heading from “Quinn Automate” to “QA Automate”.
  • Updated Party Mode examples to reference Amelia instead of Quinn.

Technical Notes: The underlying QA workflow skill (bmad-qa-generate-e2e-tests) remains installed/available; only the dedicated agent wrapper was removed.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: acd1fd59-fa77-4aa5-bc06-b98c726d1fb5

📥 Commits

Reviewing files that changed from the base of the PR and between 1aa0903 and 6b3c4b2.

📒 Files selected for processing (11)
  • .claude-plugin/marketplace.json
  • docs/fr/reference/testing.md
  • docs/reference/agents.md
  • docs/reference/testing.md
  • docs/zh-cn/reference/agents.md
  • docs/zh-cn/reference/testing.md
  • src/bmm-skills/4-implementation/bmad-agent-dev/SKILL.md
  • src/bmm-skills/4-implementation/bmad-agent-qa/SKILL.md
  • src/bmm-skills/4-implementation/bmad-agent-qa/bmad-skill-manifest.yaml
  • src/bmm-skills/4-implementation/bmad-qa-generate-e2e-tests/checklist.md
  • src/core-skills/bmad-party-mode/SKILL.md
💤 Files with no reviewable changes (3)
  • .claude-plugin/marketplace.json
  • src/bmm-skills/4-implementation/bmad-agent-qa/bmad-skill-manifest.yaml
  • src/bmm-skills/4-implementation/bmad-agent-qa/SKILL.md

📝 Walkthrough

Walkthrough

This PR removes the standalone QA agent (bmad-agent-qa / Quinn) and consolidates QA test generation capabilities into the Developer agent. The bmad-qa-generate-e2e-tests workflow is now invoked via the Developer agent rather than through a dedicated QA agent, with corresponding documentation and configuration updates across multiple languages.

Changes

Cohort / File(s) Summary
Agent Removal
.claude-plugin/marketplace.json, src/bmm-skills/4-implementation/bmad-agent-qa/SKILL.md, src/bmm-skills/4-implementation/bmad-agent-qa/bmad-skill-manifest.yaml
Removed standalone bmad-agent-qa skill entry from plugin manifest and deleted associated skill documentation and manifest files, including persona ("Quinn" QA engineer), capabilities, and activation procedures.
Developer Agent Enhancement
src/bmm-skills/4-implementation/bmad-agent-dev/SKILL.md
Added QA capability mapping to bmad-qa-generate-e2e-tests workflow, enabling Developer agent to handle API and E2E test generation for existing features.
English Documentation Updates
docs/reference/agents.md, docs/reference/testing.md
Updated agent descriptions to remove dedicated QA Engineer entry and extend Developer agent with QA trigger; replaced "QA Agent (Quinn)" terminology with "QA Workflow"; clarified that QA test generation occurs via Developer agent and the bmad-qa-generate-e2e-tests workflow.
French Documentation Updates
docs/fr/reference/testing.md
Updated workflow trigger descriptions to indicate QA functionality is provided via the Developer agent alongside the bmad-qa-generate-e2e-tests workflow identifier.
Chinese Documentation Updates
docs/zh-cn/reference/agents.md, docs/zh-cn/reference/testing.md
Reframed QA from dedicated "Quinn" agent to "built-in QA workflow"; updated Developer agent trigger set to include QA; clarified that test generation is handled by bmad-qa-generate-e2e-tests workflow invoked through Developer agent.
QA Workflow & Party Mode Updates
src/bmm-skills/4-implementation/bmad-qa-generate-e2e-tests/checklist.md, src/core-skills/bmad-party-mode/SKILL.md
Updated QA workflow checklist title from "Quinn Automate" to "QA Automate"; modified party mode to spawn Amelia instead of Quinn when requesting QA assistance.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

  • PR #1508: Modifies QA agent surface by renaming/updating Quinn agent, directly related to this agent consolidation effort.
  • PR #2160: Refactors party-mode skill to handle subagent spawning; overlaps with party-mode changes in this PR.
  • PR #2177: Performs similar agent removal/migration pattern by deleting agent entry from marketplace and moving capability into Developer agent.

Suggested reviewers

  • bmadcode
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main objective: removing the QA agent (Quinn) and migrating its functionality to the Developer agent.
Description check ✅ Passed The description comprehensively outlines the changes, including removal of the QA agent, migration to Developer agent, documentation updates across multiple languages, and references the related PR pattern.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-qa-agent
⚔️ Resolve merge conflicts
  • Resolve merge conflict in branch chore/remove-qa-agent

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Delete the Quinn (bmad-agent-qa) agent wrapper and add QA test-generation
capability to Amelia (bmad-agent-dev). Update agent tables, testing docs
(EN/ZH-CN/FR), marketplace.json, party-mode, and checklist references.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@alexeyv alexeyv force-pushed the chore/remove-qa-agent branch from 6b3c4b2 to f06795b Compare April 2, 2026 14:12
@alexeyv alexeyv merged commit 48c2324 into main Apr 2, 2026
5 checks passed
@alexeyv alexeyv deleted the chore/remove-qa-agent branch April 2, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant