DEVX-155: Replace pr_review with spec-driven CI gates and pr-review skill #274

Merged
emil merged 4 commits from DEVX-155-replace-pr-review-with-spec-driven-gates into master 2026-08-24 20:39:10 +00:00
4 Commits
Author SHA1 Message Date
emilandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> 53c4823e04 feat: add refactoring label bypass for check_pr_size
CI / validate (pull_request) Successful in 1m28s
CI / auto-merge (pull_request) Failing after 19s
PRs with the 'refactoring' label bypass the PR size check, allowing
large but legitimate refactoring PRs. The check_pr_size CLI now accepts
--repo and --pr-number to query PR labels via the Gitea API.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
2026-08-24 22:06:51 +02:00
emilandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> 0de18ebb21 fix: skip remaining repo-specific skill tests in CI
CI / validate (pull_request) Failing after 1m34s
CI / auto-merge (pull_request) Skipped
Add _skip_if_repo_missing guard to non-parametrized tests that hardcode
repo names (infra, grm, sso-bridge, mattermost-oidc). These tests only
run when sibling repos are checked out locally.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
2026-08-24 20:09:40 +02:00
emilandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> 83ad963193 fix: skip spec-driven workflow tests when sibling repos not checked out
CI / validate (pull_request) Failing after 42s
CI / auto-merge (pull_request) Skipped
Add _skip_if_repo_missing guard to skill tests that reference sibling
repos (infra, grm, sso-bridge). In CI, only devx is checked out, so
these tests skip instead of failing with FileNotFoundError.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
2026-08-24 20:04:14 +02:00
emilandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> aefc22de57 feat: replace pr_review with spec-driven CI gates and pr-review skill
CI / validate (pull_request) Failing after 41s
CI / auto-merge (pull_request) Skipped
Add validate_spec, check_pr_size, fast_molecule, nightly_gate, and
create_dependency_pr CI modules. Remove the monolithic pr_review module
and its tests. Replace pr_review CI steps with validate_spec + check_pr_size
+ curl-based APPROVE. Add spec-driven-development and pr-review skills.
Remove dead get_reviewer_token. Update translations and AGENTS.md.

Closes DEVX-155

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
2026-08-24 19:56:08 +02:00