Files
grm/docs/specs/GRM-165.md
T
Emil SimeonovandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> 869f2c16bc
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
GRM-165: feat: adopt spec-driven CI gates, create_dependency_pr, and pr-review skill
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>
2026-08-24 23:16:28 +02:00

1.7 KiB

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

  • REQ-1: CI workflow uses validate_spec + check_pr_size + curl APPROVE instead of pr_review
  • REQ-2: post-merge.yml includes create_dependency_pr step targeting oblachno/infra with package grm
  • REQ-3: .devin/skills/spec-driven-development/SKILL.md and .devin/skills/pr-review/SKILL.md exist
  • REQ-4: AGENTS.md documents spec-driven development workflow and pr-review skill