GRM-165: feat: adopt spec-driven CI gates, create_dependency_pr, and pr-review skill
CI / validate (pull_request) Failing after 1m41s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
CI / validate (pull_request) Failing after 1m41s
CI / molecule-tests (1) (pull_request) Skipped
CI / molecule-tests (2) (pull_request) Skipped
CI / molecule-tests (3) (pull_request) Skipped
CI / molecule-tests (4) (pull_request) Skipped
CI / auto-merge (pull_request) Skipped
Replace pr_review CI steps with validate_spec + check_pr_size + curl-based APPROVE. Add create_dependency_pr step to post-merge for auto-creating infra PR to bump grm version. Add spec-driven-development and pr-review skills. Update AGENTS.md. Implements: REQ-1, REQ-2, REQ-3, REQ-4 Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
co-authored by
Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
parent
9609ef2505
commit
869f2c16bc
@@ -0,0 +1,34 @@
|
||||
# GRM-165: Adopt spec-driven CI gates and pr-review skill
|
||||
|
||||
## Problem
|
||||
grm uses the old `devx.ci.pr_review` CI step and lacks the new spec-driven
|
||||
CI gates (validate_spec, check_pr_size). It also needs a create_dependency_pr
|
||||
step in post-merge to auto-create infra PRs when grm publishes a new version.
|
||||
|
||||
## Approach
|
||||
Replace pr_review CI steps with validate_spec + check_pr_size + curl-based
|
||||
APPROVE. Add create_dependency_pr step to post-merge. Add the spec-driven-
|
||||
development and pr-review skills. Update AGENTS.md.
|
||||
|
||||
REQ-1: Replace pr_review CI steps with validate_spec + check_pr_size + curl-based APPROVE
|
||||
REQ-2: Add create_dependency_pr step to post-merge for auto-creating infra PR to bump grm version
|
||||
REQ-3: Add spec-driven-development and pr-review skills under `.devin/skills/`
|
||||
REQ-4: Update AGENTS.md to document spec-driven development workflow and pr-review skill
|
||||
|
||||
## Test Plan
|
||||
- Verify CI workflow YAML passes actionlint
|
||||
- Verify post-merge.yml includes create_dependency_pr step with correct DEVX_TASK_PREFIX (GRM)
|
||||
- Verify validate_spec and check_pr_size steps reference correct DEVX_TASK_PREFIX (GRM)
|
||||
|
||||
## Deploy Plan
|
||||
- Merge to master via auto-merge workflow
|
||||
- Post-merge workflow handles release + publish + dependency PR automatically
|
||||
|
||||
## Rollback Plan
|
||||
- Revert the merge commit; CI reverts to pr_review-based workflow
|
||||
|
||||
## Acceptance Criteria
|
||||
- [x] REQ-1: CI workflow uses validate_spec + check_pr_size + curl APPROVE instead of pr_review
|
||||
- [x] REQ-2: post-merge.yml includes create_dependency_pr step targeting oblachno/infra with package grm
|
||||
- [x] REQ-3: `.devin/skills/spec-driven-development/SKILL.md` and `.devin/skills/pr-review/SKILL.md` exist
|
||||
- [x] REQ-4: AGENTS.md documents spec-driven development workflow and pr-review skill
|
||||
Reference in New Issue
Block a user