Convert qa-generate-e2e-tests workflow to native skill packaging

[alexeyv]

Summary

• mechanically converted qa-generate-e2e-tests into native skill packaging under src/bmm/workflows/bmad-qa-generate-e2e-tests/
• switched manifest to type: skill and added SKILL.md with matching frontmatter
• updated dependent references in src/bmm/module-help.csv and src/bmm/agents/qa.agent.yaml

Validation

• npm test
• before/after equivalence checks on workflow execution section and checklist content (2 repeat runs; byte-identical hashes)
• installer smoke run against harness directory: node tools/cli/bmad-cli.js install --directory /Users/alex/src/bmad --modules bmm --tools claude-code --yes

[augmentcode]

🤖 Augment PR Summary

Summary: Migrates the QA “generate e2e tests” workflow to native skill packaging by renaming it to bmad-qa-generate-e2e-tests and adding SKILL.md + a type: skill manifest.

Changes: Updates the QA agent’s menu reference and aligns workflow paths/metadata with the new skill directory name.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 1 suggestion posted.

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

[augmentcode]

It looks like the YAML frontmatter opening delimiter (---) was removed but the closing --- remains, so tooling that discovers/parses workflows via ^---\n...\n--- will skip this file and metadata won’t be read. If this file is still meant to be a workflow entry, consider restoring valid frontmatter (or removing the dangling delimiter + raw name:/description: lines if the intent is to make this plain markdown). (Guideline: workflow_name_required)

Severity: medium

_{🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.}

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Converting the QA workflow into a native skill package by adding skill metadata files (SKILL.md and skill manifest), updating workflow path resolution to relative paths, switching the agent menu trigger from a file-path reference to a skill-based exec, and removing old workflow manifest declarations.

Changes

Cohort / File(s)	Summary
Agent Configuration src/bmm/agents/qa.agent.yaml	Updated menu trigger exec from a relative workflow.md path to skill-based reference skill:bmad-qa-generate-e2e-tests.
New Skill Package Files src/bmm/workflows/bmad-qa-generate-e2e-tests/SKILL.md, src/bmm/workflows/bmad-qa-generate-e2e-tests/bmad-skill-manifest.yaml	Added skill metadata files: SKILL.md with frontmatter defining the skill name and description, and bmad-skill-manifest.yaml declaring type: skill.
Workflow Path Resolution src/bmm/workflows/bmad-qa-generate-e2e-tests/workflow.md	Updated path references to use relative paths: installed_path changed to "." and checklist reference updated from project-root format to "./checklist.md".
Old Workflow Manifest Cleanup src/bmm/workflows/qa-generate-e2e-tests/bmad-skill-manifest.yaml	Removed workflow manifest entries including canonicalId, type, and description fields from the old workflow package.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Convert correct-course workflow to native skill packaging #1942: Performs the same code-level transformation—converting a workflow into native skill packaging and updating agent exec references to point to the new skill.
• convert dev-story workflow to native skill package #1940: Applies the identical conversion pattern—replacing workflow file exec targets with skill references, adding SKILL.md and bmad-skill-manifest.yaml, and updating workflow path fields.
• refactor(bmm): convert create-product-brief to native skill package #1933: Executes the same refactoring pattern—converting a workflow into a native skill package with updated agent execs and relative path resolution.

Suggested reviewers

• bmadcode

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: converting a workflow to native skill packaging, which is the primary objective reflected in the file changes.
Description check	✅ Passed	The description is directly related to the changeset, providing clear details about the mechanical conversion, manifest changes, and validation performed.
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
• Post copyable unit tests in a comment
• Commit unit tests in branch convert-qa-generate-e2e-tests-skill

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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