Skip to content

ci(quality): isolate PR concurrency by repository - #1827

Merged
seonghobae merged 1 commit into
mainfrom
codex/pr-quality-repository-concurrency
Sep 4, 2026
Merged

seonghobae merged 1 commit into
mainfrom
codex/pr-quality-repository-concurrency

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

변경 사항

현재 origin/main에서 PR-triggered quality workflow의 concurrency group을 의미 기반으로 다시 감사했습니다. 서로 다른 repository의 같은 PR 번호가 동일 group을 공유하던 다음 6개 workflow에 ${{ github.repository }}를 추가했습니다.

  • Agent Mention Router Quality CI
  • Cloudflare DNS
  • Contextual Orchestrator Review Repair Quality CI
  • JavaScript Coverage Gate Quality CI
  • Organization Commercial Readiness Loop Quality CI
  • Trusted uv Materializer Quality CI

PR 실행은 workflow-repository-PR로 분리되고 기존 cancel-in-progress 계약을 유지합니다. Cloudflare의 push/repository_dispatch 비취소 정책과 각 workflow의 push/ref fallback은 변경하지 않았습니다.

범위

  • workflow 6개
  • 기존 exact 계약 갱신 2개
  • 여섯 workflow를 함께 감사하는 queue 계약 테스트 1개

repository-metadata-reconcile은 quality workflow가 아니므로 제외했습니다. OpenCode, Strix, Current Head Run Coalescer는 job-level/queue-level 별도 계약이라 이번 PR에서 변경하지 않았습니다.

검증

  • 관련 pytest: 98 passed
  • 수정 workflow 6개 actionlint: passed, no output
  • 후속 의미 감사: repository/PR 식별자 누락 0개
  • git diff --check origin/main...HEAD: passed

Exact revision

  • base: 4024a8afd7339b306a03752839a13bc7d8ea4733
  • head: db7f945d976ecb77a5d9c572e5857615dc8a6336

Summary by CodeRabbit

  • 개선 사항

    • 여러 저장소에서 실행되는 자동화 및 품질 검사 작업의 동시 실행 그룹을 저장소별로 분리했습니다.
    • 이제 서로 다른 저장소의 동일한 풀 리퀘스트 번호나 브랜치 참조가 실행 취소 또는 대체 처리에 영향을 주지 않습니다.
    • 풀 리퀘스트와 일반 브랜치 실행에 대한 동시성 관리가 더 일관되게 적용됩니다.
  • 테스트

    • 저장소별 동시성 격리와 풀 리퀘스트 실행 취소 동작을 검증하는 테스트를 추가하고 업데이트했습니다.

Signed-off-by: Seongho Bae <me@seonghobae.me>
@seonghobae
seonghobae merged commit 5ec781d into main Sep 4, 2026
5 of 26 checks passed
@seonghobae
seonghobae deleted the codex/pr-quality-repository-concurrency branch September 4, 2026 04:57
@coderabbitai

coderabbitai Bot commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: addc14e0-e254-47ba-a80e-7288c09d5641

📥 Commits

Reviewing files that changed from the base of the PR and between 4024a8a and db7f945.

📒 Files selected for processing (9)
  • .github/workflows/agent-mention-router-quality-ci.yml
  • .github/workflows/cloudflare-dns.yml
  • .github/workflows/hourly-nvidia-nim-review-repair.yml
  • .github/workflows/javascript-coverage-quality-ci.yml
  • .github/workflows/organization-commercial-readiness-loop-quality-ci.yml
  • .github/workflows/trusted-uv-materializer-quality-ci.yml
  • tests/test_cloudflare_dns_contract.py
  • tests/test_hourly_scheduler_runtime_budget.py
  • tests/test_required_workflow_queue_contract.py

📝 Walkthrough

Walkthrough

여섯 개 GitHub Actions 워크플로의 동시성 그룹 키에 저장소 이름을 추가했습니다. 관련 계약 테스트는 저장소별 그룹 형식과 워크플로별 cancel-in-progress 조건을 검증합니다.

Changes

저장소별 동시성 그룹 격리

Layer / File(s) Summary
워크플로 동시성 그룹 범위 변경
.github/workflows/*quality-ci.yml, .github/workflows/cloudflare-dns.yml
여섯 개 워크플로의 concurrency.group 값에 ${{ github.repository }}를 추가했습니다. 기존 PR 번호 또는 ref 값은 유지했습니다.
동시성 계약 검증 갱신
tests/test_cloudflare_dns_contract.py, tests/test_hourly_scheduler_runtime_budget.py, tests/test_required_workflow_queue_contract.py
기존 기대값을 갱신했습니다. 여섯 개 워크플로의 저장소별 그룹 형식과 cancel-in-progress 조건을 검증하는 테스트를 추가했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/pr-quality-repository-concurrency

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant