GRM-165: Adopt spec-driven CI gates and pr-review skill #256

Merged
kireto merged 8 commits from GRM-165-fresh into master 2026-08-25 01:40:51 +00:00
Owner

Summary

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.

Requirements

  • 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

Spec

See docs/specs/GRM-165.md for full spec with acceptance criteria.

Test Plan

  • CI workflow YAML passes actionlint
  • post-merge.yml includes create_dependency_pr step with correct DEVX_TASK_PREFIX (GRM)
  • validate_spec and check_pr_size steps reference correct DEVX_TASK_PREFIX (GRM)

Generated with Devin

## Summary 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. ## Requirements - 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 ## Spec See `docs/specs/GRM-165.md` for full spec with acceptance criteria. ## Test Plan - CI workflow YAML passes actionlint - post-merge.yml includes create_dependency_pr step with correct DEVX_TASK_PREFIX (GRM) - validate_spec and check_pr_size steps reference correct DEVX_TASK_PREFIX (GRM) Generated with [Devin](https://devin.ai)
emil added 1 commit 2026-08-24 21:17:13 +00:00
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
869f2c16bc
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>
emil changed title from GRM-165: Adopt spec-driven CI gates, create_dependency_pr, and pr-review skill to GRM-165: Adopt spec-driven CI gates and pr-review skill 2026-08-24 21:21:56 +00:00
emil added 1 commit 2026-08-24 21:22:55 +00:00
chore: trigger CI re-run after PR title fix
CI / validate (pull_request) Failing after 1m46s
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
0e78ed0e5f
emil added 1 commit 2026-08-24 22:26:38 +00:00
fix: bump devx to v0.50.2 for new CI modules
CI / validate (pull_request) Failing after 3m2s
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
7dfd800f39
kireto approved these changes 2026-08-24 22:26:59 +00:00
Dismissed
kireto left a comment
Member

Approved: spec-driven CI gates adoption with devx v0.50.2.

Approved: spec-driven CI gates adoption with devx v0.50.2.
emo added the refactoring label 2026-08-25 00:47:07 +00:00
emil added 1 commit 2026-08-25 00:53:34 +00:00
chore: retrigger CI
CI / validate (pull_request) Successful in 1m42s
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) Failing after 10m4s
a58b64fd49
emil added 1 commit 2026-08-25 01:01:00 +00:00
chore: retrigger CI with refactoring label
CI / validate (pull_request) Successful in 3m21s
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) Failing after 10m4s
97579f6363
emil added 1 commit 2026-08-25 01:17:10 +00:00
fix: handle skipped molecule tests in wait_for_checks
CI / validate (pull_request) Successful in 3m3s
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) Failing after 10m4s
5d0e533bd1
emil dismissed kireto's review 2026-08-25 01:17:10 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

emil added 1 commit 2026-08-25 01:30:23 +00:00
fix: treat skipped molecule tests as terminal in wait_for_checks
CI / validate (pull_request) Successful in 2m27s
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) Failing after 1m52s
07c9322a72
kireto approved these changes 2026-08-25 01:35:53 +00:00
kireto left a comment
Member

Approved: wait_for_checks fix for skipped molecule tests.

Approved: wait_for_checks fix for skipped molecule tests.
emil added 1 commit 2026-08-25 01:36:05 +00:00
chore: retrigger CI after kireto approval
CI / validate (pull_request) Successful in 3m34s
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) Successful in 1m9s
267ed360e4
kireto merged commit eab452ec04 into master 2026-08-25 01:40:51 +00:00
kireto deleted branch GRM-165-fresh 2026-08-25 01:40:52 +00:00
Sign in to join this conversation.