agentic-delivery: the lander refuses a one-change train, so the queue idles while one change is ready

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#2,249 0 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
76/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ệ
bash, python
Lĩnh vực
tooling

Hướng nghiên cứu

Bắt đầu với .claude/skills/agentic-delivery/scripts/train_land.sh và nội dung hướng dẫn sử dụng của nó, sau đó tìm các bài kiểm thử của chính lander và hướng dẫn xử lý theo lô của delivery overlay. Xác nhận rằng train chỉ có một thay đổi được chấp nhận và kiểm thử, đồng thời việc chờ một thay đổi sẵn sàng khác được mô tả là chính sách của launcher. Issue không nêu tên tệp overlay hay đường dẫn kiểm thử, vì vậy hãy tìm chúng trong repository.

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

Mô tả

The skill's lander refuses a train of one change, so the merge queue idles whenever only one change is ready.

  • Where: .claude/skills/agentic-delivery/scripts/train_land.sh exits 2 unless it gets at least two changes: [ $# -ge 3 ], with the usage line "a train is >= 2 PRs".
  • Field evidence (genericized): one adopter runs its own fork of this lander, and that fork accepts one change ([ $# -ge 2 ]). Over a recent day, about a third of its landed app trains carried a single change. Those changes would have waited for a second ready change, and one did not always come within the hour. The vendored skill copy still carries the two-change rule, so the two copies now disagree, and a reviewer reading the skill copy reports the stale rule as a defect.
  • Why it matters: the union test run for one change costs about the same as for two, and the batch-size floor belongs to the launcher, which knows what else is close to ready. A hard refusal in the lander turns "nothing else is ready" into idle queue time, and merges per hour is the metric the delivery overlay asks adopters to raise.
  • Proposed change:
    1. The lander accepts one change: [ $# -ge 2 ], with the usage line train_land.sh <tag> <change> [change...].
    2. Any batching floor becomes launcher policy, for example "wait up to N minutes for a second change whose checks are about to finish, then launch with what is ready".
  • Done when: the skill's lander lands a one-change train in its own tests, and the delivery overlay describes the batching floor as a launcher wait, not a lander refusal.
Ngôn ngữ chính
Python
Star
2
Fork
1
Merge trung bình
1 giờ 29 phút
Pull request đã merge (30 ngày)
567

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 protocol/perun

Tất cả issue của protocol/perun

Issue tương tự

Thêm issue về Python

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.