Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

fix(ci): AI review lanes skip on a bot commit, leaving the PR head unreviewed for the merge gate

Đã đóng
  • Có gì không ổn?
#7,015 1 bình luận 0 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
28/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions
Lĩnh vực
ci-cd

Hướng nghiên cứu

Start from the review lanes' trigger conditions and the skip logic for commits authored by bots, and from the merge gate that holds on a skipped lane, which the /source-control:pull-request step uses. Then check why the first ready event on the PR started no CI. Done means one of the two proposed approaches is chosen and documented in the merge gate, and the ready trigger behaviour is explained or fixed. The maintainer has not yet picked an approach, so confirm the direction before writing code.

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

Mô tả

needs-human needs-triage

Problem

On #6913 (2026-10-11), a docs-cleanup commit by melodic-automation-lanes[bot] (1289a56b3a, 4 wording-only lines) landed on the PR head after the last human-driven push. Both AI review lanes ended skipped on that head, apparently because a bot authored the commit. The /source-control:pull-request merge step holds on a skipped review lane, and the standing merge rule requires the AI reviews to have finished on the head. So the PR either deadlocks or merges on an exception. #6913 merged on an exception: the agent read the bot's 4-line diff itself, and the previous head's security review had passed and its code review had posted "No findings".

A related symptom on the same PR: the first ready event started no CI, and the agent had to flip the PR back to draft and to ready again.

Proposed next step

Pick one and document it in the merge gate:

  • Run the review lanes on bot commits to PR branches, at least for the docs-cleanup bot.
  • Or let the gate carry forward a review verdict from the prior head when the only later commits come from an allowlisted bot and change wording only.

Separately, check why the first ready event did not trigger the lanes.

Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
8 giờ 41 phút
Pull request đã merge (30 ngày)
892

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 melodic-software/claude-code-plugins

Tất cả issue của melodic-software/claude-code-plugins

Issue tương tự

Thêm issue về Shell/Bash

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.