(bug) No CI job runs against the merged tip of main or a release branch

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#6,967 2 bình luận 1 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, shell, typescript, yaml
Lĩnh vực
ci-cd, testing

Hướng nghiên cứu

Bắt đầu với .github/workflows/12-check-unit-tests.yml và kiểm tra các trigger, bộ lọc đường dẫn cùng job kiểm tra kiểu hiện có; nếu cần, hãy so sánh với các workflow khác trong .github/workflows/. Bổ sung phạm vi cho các push lên main và release/**, đồng thời đảm bảo các thay đổi trong hosting/docker-compose/** được CI kiểm tra. Hoàn tất khi các push lên merge tip chạy các bước kiểm tra và thay đổi Docker Compose không còn bị loại trừ.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug report ci/cd devops

Found during release QA for v0.119.0.

What happens. No CI job in this repository type-checks or tests the merged tip of main or a release/** branch. Every workflow that checks application code triggers only on pull_request or workflow_dispatch. Nothing re-runs once a PR merges.

This is not theoretical: during this release, two PRs that were each individually green on their own PR checks broke the build once merged together. The merged tree failed a plain TypeScript compile with two "cannot find name" errors in the runner's turn-settlement code. That would have made every agent turn fail in production with a raw error message, and no CI signal ever caught it, because nothing ever checked the actual merged commit.

Where. .github/workflows/12-check-unit-tests.yml, which owns the pnpm run typecheck step across the web, SDK, API, services and clients packages, triggers only on pull_request and workflow_dispatch. Across the whole .github/workflows/ directory, the only workflows with a push trigger are the docs deploy, the website deploy, and the secret scan; none of them check application code. The concurrency group in 12-check-unit-tests.yml already falls back to github.ref_name, so it needs no other change to accept a push trigger.

A related gap in the same file: hosting/docker-compose/** is not in its path filters at all, so changes there get no CI coverage of any kind (no compose validation, no shellcheck), unlike hosting/kubernetes/helm/**, which is covered.

Suggested fix. Add a push trigger for main and release/** to .github/workflows/12-check-unit-tests.yml so a merge is actually verified, not just a preview of it. Separately, add path coverage (or a dedicated lightweight job) for hosting/docker-compose/**.

Ngôn ngữ chính
TypeScript
Star
4.8k
Fork
690
Merge trung bình
23 giờ 26 phút
Pull request đã merge (30 ngày)
507

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Agenta-AI/agenta

Tất cả issue của Agenta-AI/agenta

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.