[PR #149] [MERGED] feat: add DevOps Engineer base starter collection #151

Closed
opened 2026-09-02 07:34:54 +02:00 by zaph0d · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/niels-emmer/myace/pull/149
Author: @niels-emmer
Created: 9/1/2026
Status: Merged
Merged: 9/1/2026
Merged by: @niels-emmer

Base: mainHead: feat/devops-engineer-base-collection


📝 Commits (2)

  • 5afc716 feat: add DevOps Engineer base starter collection
  • 596b581 docs: fix starter-pack tag count in README

📊 Changes

22 files changed (+626 additions, -7 deletions)

View changed files

📝 README.md (+6 -5)
📝 backend/app/services/seed_collections.py (+10 -0)
📝 backend/tests/test_seed_collections.py (+59 -2)
collections/base/devops-engineer/AGENTS.md (+49 -0)
collections/base/devops-engineer/agents/builder.md (+30 -0)
collections/base/devops-engineer/agents/code-reviewer.md (+27 -0)
collections/base/devops-engineer/agents/debugger.md (+29 -0)
collections/base/devops-engineer/agents/docs-writer.md (+28 -0)
collections/base/devops-engineer/agents/orchestrator.md (+28 -0)
collections/base/devops-engineer/agents/security-auditor.md (+26 -0)
collections/base/devops-engineer/agents/verifier.md (+27 -0)
collections/base/devops-engineer/commands/plan.md (+14 -0)
collections/base/devops-engineer/commands/verify.md (+15 -0)
collections/base/devops-engineer/skills/devsecops-checklist/SKILL.md (+31 -0)
collections/base/devops-engineer/skills/iac-patterns/SKILL.md (+30 -0)
collections/base/devops-engineer/skills/image-build/SKILL.md (+30 -0)
collections/base/devops-engineer/skills/incident-management/SKILL.md (+28 -0)
collections/base/devops-engineer/skills/memory-system/SKILL.md (+43 -0)
collections/base/devops-engineer/skills/observability/SKILL.md (+29 -0)
collections/base/devops-engineer/skills/pipeline-design/SKILL.md (+30 -0)

...and 2 more files

📄 Description

What does this change?

Adds a new DevOps Engineer base starter collection (collections/base/devops-engineer/), modeled on the existing software-engineer base and optimized for infrastructure, CI/CD, and platform work — IaC (Terraform/Bicep), DevSecOps, observability, and incident response.

The collection ships 30 artifacts: 12 rules (AGENTS.md), 7 agents, 9 skills, and 2 commands. It reuses the software-engineer pipeline agent names (orchestrator primary + builder, verifier, security-auditor, code-reviewer, docs-writer, debugger) with DevOps-specialized content, so the additional collections' handoff_to references to verifier/security-auditor/code-reviewer resolve against this base too. Skill and rule names are deliberately chosen to avoid rule-29 name collisions with the infra-related additions (devops, iac-expert, azure, aws, gcp, scaleway, auditor) it's most naturally composed with.

Registered in STARTER_COLLECTIONS (seeded on backend boot), README counts updated (23 collections / 278 artifacts / 133 tags), and a new regression test guards all 19 additional collections compiling clean on the new base.

N/A — new starter collection.

Type of change

  • Bug fix
  • New feature
  • Breaking change (fix or feature that would cause existing behavior to change)
  • Documentation only
  • Chore / refactor

How was this tested?

  • pytest tests/test_seed_collections.py — 8 passed, including the new test_additional_collections_compile_clean_on_devops_engineer (all 19 additional collections compile with zero name_collision / dangling_handoff warnings on the new base) and the updated test_base_profiles_compile_without_orchestration_warnings (now covers 4 bases).
  • Full backend suite — 356 passed.
  • ruff check . — clean.
  • Rule-29 collision check across the whole starter set — zero collisions between devops-engineer and any additional collection (the only overlaps are with software-engineer, which is intentional: bases never compose with each other, and the shared agent names are what let additional collections' handoffs resolve).
  • All 30 files parse correctly via the scanner's per-file parsers.

Checklist

  • I read CONTRIBUTING.md
  • Tests pass locally (pytest / npm run test / npx tsc -b as applicable)
  • I added tests for new behavior, or explained why none are needed
  • Schema changes include an Alembic migration with a working downgrade() (N/A — no schema change)
  • I updated README.md, AGENTS.md/CLAUDE.md, and/or docs/ if this change affects what they document (README starter-pack counts updated — collections, artifacts, and the previously-stale tag count)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/niels-emmer/myace/pull/149 **Author:** [@niels-emmer](https://github.com/niels-emmer) **Created:** 9/1/2026 **Status:** ✅ Merged **Merged:** 9/1/2026 **Merged by:** [@niels-emmer](https://github.com/niels-emmer) **Base:** `main` ← **Head:** `feat/devops-engineer-base-collection` --- ### 📝 Commits (2) - [`5afc716`](https://github.com/niels-emmer/myace/commit/5afc716ac3e89d85ab5680193f1e462b2b9faae0) feat: add DevOps Engineer base starter collection - [`596b581`](https://github.com/niels-emmer/myace/commit/596b581ca9715aea1288a0ec31c33f702d90d397) docs: fix starter-pack tag count in README ### 📊 Changes **22 files changed** (+626 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+6 -5) 📝 `backend/app/services/seed_collections.py` (+10 -0) 📝 `backend/tests/test_seed_collections.py` (+59 -2) ➕ `collections/base/devops-engineer/AGENTS.md` (+49 -0) ➕ `collections/base/devops-engineer/agents/builder.md` (+30 -0) ➕ `collections/base/devops-engineer/agents/code-reviewer.md` (+27 -0) ➕ `collections/base/devops-engineer/agents/debugger.md` (+29 -0) ➕ `collections/base/devops-engineer/agents/docs-writer.md` (+28 -0) ➕ `collections/base/devops-engineer/agents/orchestrator.md` (+28 -0) ➕ `collections/base/devops-engineer/agents/security-auditor.md` (+26 -0) ➕ `collections/base/devops-engineer/agents/verifier.md` (+27 -0) ➕ `collections/base/devops-engineer/commands/plan.md` (+14 -0) ➕ `collections/base/devops-engineer/commands/verify.md` (+15 -0) ➕ `collections/base/devops-engineer/skills/devsecops-checklist/SKILL.md` (+31 -0) ➕ `collections/base/devops-engineer/skills/iac-patterns/SKILL.md` (+30 -0) ➕ `collections/base/devops-engineer/skills/image-build/SKILL.md` (+30 -0) ➕ `collections/base/devops-engineer/skills/incident-management/SKILL.md` (+28 -0) ➕ `collections/base/devops-engineer/skills/memory-system/SKILL.md` (+43 -0) ➕ `collections/base/devops-engineer/skills/observability/SKILL.md` (+29 -0) ➕ `collections/base/devops-engineer/skills/pipeline-design/SKILL.md` (+30 -0) _...and 2 more files_ </details> ### 📄 Description ## What does this change? Adds a new **DevOps Engineer** base starter collection (`collections/base/devops-engineer/`), modeled on the existing `software-engineer` base and optimized for infrastructure, CI/CD, and platform work — IaC (Terraform/Bicep), DevSecOps, observability, and incident response. The collection ships 30 artifacts: 12 rules (AGENTS.md), 7 agents, 9 skills, and 2 commands. It reuses the software-engineer pipeline agent names (`orchestrator` primary + `builder`, `verifier`, `security-auditor`, `code-reviewer`, `docs-writer`, `debugger`) with DevOps-specialized content, so the additional collections' `handoff_to` references to `verifier`/`security-auditor`/`code-reviewer` resolve against this base too. Skill and rule names are deliberately chosen to avoid rule-29 name collisions with the infra-related additions (`devops`, `iac-expert`, `azure`, `aws`, `gcp`, `scaleway`, `auditor`) it's most naturally composed with. Registered in `STARTER_COLLECTIONS` (seeded on backend boot), README counts updated (23 collections / 278 artifacts / 133 tags), and a new regression test guards all 19 additional collections compiling clean on the new base. ## Related issue N/A — new starter collection. ## Type of change - [ ] Bug fix - [x] New feature - [ ] Breaking change (fix or feature that would cause existing behavior to change) - [ ] Documentation only - [ ] Chore / refactor ## How was this tested? - `pytest tests/test_seed_collections.py` — 8 passed, including the new `test_additional_collections_compile_clean_on_devops_engineer` (all 19 additional collections compile with zero `name_collision` / `dangling_handoff` warnings on the new base) and the updated `test_base_profiles_compile_without_orchestration_warnings` (now covers 4 bases). - Full backend suite — 356 passed. - `ruff check .` — clean. - Rule-29 collision check across the whole starter set — zero collisions between `devops-engineer` and any additional collection (the only overlaps are with `software-engineer`, which is intentional: bases never compose with each other, and the shared agent names are what let additional collections' handoffs resolve). - All 30 files parse correctly via the scanner's per-file parsers. ## Checklist - [x] I read [`CONTRIBUTING.md`](../CONTRIBUTING.md) - [x] Tests pass locally (`pytest` / `npm run test` / `npx tsc -b` as applicable) - [x] I added tests for new behavior, or explained why none are needed - [ ] Schema changes include an Alembic migration with a working `downgrade()` (N/A — no schema change) - [x] I updated `README.md`, `AGENTS.md`/`CLAUDE.md`, and/or `docs/` if this change affects what they document (README starter-pack counts updated — collections, artifacts, and the previously-stale tag count) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
zaph0d 2026-09-02 07:34:54 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
github-mirrors/myace#151
No description provided.