Skip to content

Rename outside-in-testing skill to qa-team#3021

Merged
rysweet merged 5 commits intomainfrom
qa-team-skill
Mar 10, 2026
Merged

Rename outside-in-testing skill to qa-team#3021
rysweet merged 5 commits intomainfrom
qa-team-skill

Conversation

@rysweet
Copy link
Owner

@rysweet rysweet commented Mar 10, 2026

Summary

  • rename the primary outside-in-testing skill to qa-team across shipped mirrors
  • keep outside-in-testing as a compatibility alias with symlinked docs/examples/tests
  • add parity/shadow guidance plus tests and update workflow surfaces to prefer qa-team

Testing

  • uv run pytest tests/skills/test_qa_team_skill.py .claude/skills/qa-team/tests/test_skill_examples.py -q

Notes

  • tests/test_known_skills.py::TestKnownSkillsRegistry::test_registry_matches_actual_filesystem still fails on pre-existing registry drift unrelated to this PR

@github-actions
Copy link
Contributor

🤖 Auto-fixed version bump

The version in pyproject.toml has been automatically bumped to the next patch version.

If you need a minor or major version bump instead, please update pyproject.toml manually and push the change.

@github-actions
Copy link
Contributor

Repo Guardian - Passed

All 30 changed files in this PR were reviewed and no ephemeral content was detected.

Summary of changes analyzed:

  • .claude/skills/qa-team/ — New skill directory with durable SKILL.md, README.md, example YAML fixtures, a documented/parameterized utility script (check-freshness.py), and test files
  • .claude/skills/outside-in-testing/ — Updated to act as a compatibility alias pointing to the new qa-team skill
  • .claude/skills/e2e-outside-in-test-generator/SKILL.md, .claude/skills/shadow-testing/, .claude/skills/smart-test/ — Reference updates reflecting the rename
  • .claude/profiles/coding.yaml — Configuration update

All files are durable reference material (skill definitions, documentation, reusable examples, tests, and tooling scripts). No temporal indicators, no one-off scripts, no meeting notes or status snapshots were found.

Generated by Repo Guardian for issue #3021 ·

@rysweet
Copy link
Owner Author

rysweet commented Mar 10, 2026

Workflow remediation update:

Ubuntu and others added 2 commits March 10, 2026 15:14
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

🤖 Auto-fixed version bump

The version in pyproject.toml has been automatically bumped to the next patch version.

If you need a minor or major version bump instead, please update pyproject.toml manually and push the change.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

Repo Guardian - Passed

All 92 changed files in this PR were reviewed and no ephemeral content was detected.

Summary of changes analyzed:

  • .claude/skills/qa-team/ — New skill directory (renamed from outside-in-testing) containing durable SKILL.md, README.md, reusable YAML example fixtures, a parameterized check-freshness.py utility script, and test files
  • .claude/skills/outside-in-testing/ — Updated to a compatibility alias pointing to the new qa-team skill
  • .claude/skills/smart-test/, .claude/skills/shadow-testing/, .claude/skills/e2e-outside-in-test-generator/ — Reference updates reflecting the rename
  • .claude/profiles/coding.yaml, .claude/workflow/DEFAULT_WORKFLOW.md — Configuration/workflow updates (reference updates only)
  • amplifier-bundle/skills/qa-team/, amplifier-bundle/skills/outside-in-testing/, amplifier-bundle/skills/smart-test/ — Bundle sync of the same rename
  • docs/claude/skills/qa-team/, docs/claude/skills/outside-in-testing/ — Documentation sync
  • amplifier-bundle/skills/pm-architect/scripts/generate_daily_status.py, generate_roadmap_review.py, triage_pr.py — Pre-existing toolchain scripts with a single-line fix (print warning removed); not new additions
  • amplifier-bundle/recipes/, amplifier-bundle/skills/common/, amplifier-bundle/skills/context-management/ — Reference/bundle sync updates
  • pyproject.toml, src/amplihack/known_skills.py, tests/skills/test_qa_team_skill.py — Version bump, known-skills registry update, and durable test file

All changes are durable reference material — skill definitions, documentation, reusable examples, tests, and tooling scripts. No temporal indicators, no one-off scripts, no meeting notes or status snapshots were found.

Generated by Repo Guardian for issue #3021 ·

@github-actions
Copy link
Contributor

PR Triage Report

Risk: 🟡 Medium | Action: Needs Review

Summary

Renames the outside-in-testing skill to qa-team across all shipped mirrors, with a backward-compatibility alias kept in place. Large file count (92 files, +8840/-8440) reflects mirror sync rather than novel logic.

Assessment

Dimension Finding
Author rysweet (repo owner)
CI ✅ All 27 checks pass
Repo Guardian ✅ Passed — all 92 files confirmed durable content
Scope Skill rename + backward-compat alias + tests
Tests ✅ Targeted tests added (test_qa_team_skill.py)
Known issue Pre-existing test_known_skills.py registry drift on main (unrelated)

Risk Factors

  • Large changeset (92 files) due to mirror sync across .claude/skills/, amplifier-bundle/, docs/
  • Backward compatibility maintained via alias — low regression risk
  • Workflow/profile references updated consistently

Recommendation

Approve and merge after a spot-check of the outside-in-testing alias and at least one mirror bundle directory. No blocking issues found.

Generated by PR Triage Agent ·

@rysweet rysweet merged commit 0938d71 into main Mar 10, 2026
14 of 15 checks passed
@github-actions
Copy link
Contributor

🤖 Auto-fixed version bump

The version in pyproject.toml has been automatically bumped to the next patch version.

If you need a minor or major version bump instead, please update pyproject.toml manually and push the change.

@github-actions
Copy link
Contributor

Repo Guardian - Passed

All 95 changed files in this PR were reviewed and no ephemeral content was detected.

Summary of changes analyzed:

  • .claude/skills/qa-team/ — New skill directory (renamed from outside-in-testing) containing durable SKILL.md, README.md, reusable YAML example fixtures, a parameterized check-freshness.py utility script, and test files
  • .claude/skills/outside-in-testing/ — Updated to a backward-compatibility alias pointing to the new qa-team skill
  • .claude/skills/smart-test/, .claude/skills/shadow-testing/, .claude/skills/e2e-outside-in-test-generator/ — Reference updates reflecting the rename
  • .claude/profiles/coding.yaml, .claude/workflow/DEFAULT_WORKFLOW.md — Configuration/workflow reference updates
  • amplifier-bundle/skills/qa-team/, amplifier-bundle/skills/outside-in-testing/, amplifier-bundle/skills/smart-test/ — Bundle mirror sync of the same rename
  • amplifier-bundle/skills/pm-architect/scripts/generate_daily_status.py, generate_roadmap_review.py, triage_pr.py — Reusable parameterized Claude Agent SDK toolchain scripts (not one-off); single-line fix applied
  • docs/claude/skills/qa-team/, docs/claude/skills/outside-in-testing/ — Documentation mirror sync
  • pyproject.toml, src/amplihack/known_skills.py, tests/skills/test_qa_team_skill.py — Version bump, known-skills registry update, and durable test file

All files are durable reference material — skill definitions, documentation, reusable examples, tests, and tooling scripts. No temporal indicators, no one-off scripts, no meeting notes or status snapshots were found.

Generated by Repo Guardian for issue #3021 ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant