diff --git a/docs/specs/GRM-171.md b/docs/specs/GRM-171.md index ac266d0..91e12db 100644 --- a/docs/specs/GRM-171.md +++ b/docs/specs/GRM-171.md @@ -8,17 +8,34 @@ molecule scenario authoring, and Vikunja task lifecycle. `devx-workflow` and `AGENTS.md` document `make create-task -- --title`, which fails because `devx-create-task` forwards no arguments. -## Requirements +## Approach -- REQ-1: Add `runner-ops` skill: RunnerManager/AnsibleExecutor model, +REQ-1: Add `runner-ops` skill: RunnerManager/AnsibleExecutor model, stale-runner cleanup, image pruning, molecule container lifecycle, safe-debugging rules. -- REQ-2: Add `molecule-testing` skill: gitea_runner scenario layout, +REQ-2: Add `molecule-testing` skill: gitea_runner scenario layout, platform overrides, isolation flags, debugging. -- REQ-3: Add `vikunja-tasks` skill: create via module call, query, +REQ-3: Add `vikunja-tasks` skill: create via module call, query, close, spec-collision convention. -- REQ-4: Fix broken `make create-task -- --title` documentation in +REQ-4: Fix broken `make create-task -- --title` documentation in `devx-workflow` skill and `AGENTS.md`. +REQ-5: Add skill validation tests (`tests/unit/test_skills_validation.py`) + + fix stale make-target refs and missing sections in existing skills. + +Preserve the colliding spec as +[GRM-171-kireto-token-historical](GRM-171-kireto-token-historical.md). + +## Test Plan + +- `pytest tests/unit/test_skills_validation.py` passes (12 tests). + +## Deploy Plan + +Documentation/skills only — auto-merge to master; no runtime deploy. + +## Rollback Plan + +Revert the squash-merge commit; skills are inert documentation. ## Acceptance Criteria @@ -26,6 +43,7 @@ which fails because `devx-create-task` forwards no arguments. - [x] REQ-2: `molecule-testing` skill exists. - [x] REQ-3: `vikunja-tasks` skill exists. - [x] REQ-4: create-task docs corrected. +- [x] REQ-5: Skill validation tests added and passing. ## Out of Scope