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): ci-status's hold-lift window counts a removal made through an App

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

Hướng nghiên cứu

Bắt đầu với .github/workflows/pr-require-checks.yml ở khoảng dòng 3587–3588 và đọc bộ lọc hiện có để gỡ nhãn human-gated. Sau đó, kiểm tra scripts/ci-hold-human-gated.test.sh hoặc bài kiểm tra chạy phần thân workflow này. Bổ sung kiểm thử cho thấy việc gỡ nhãn do một App có id đáng tin cậy thực hiện không mở khoảng thời gian gỡ chặn, trong khi việc một người trực tiếp gỡ nhãn vẫn mở khoảng thời gian đó.

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

Mô tả

needs-human status: needs-decision

Problem

ci-status's lift window (.github/workflows/pr-require-checks.yml:3587-3588) opens at the newest unlabeled event for the human-gated label whose actor id is on the trusted list. It does not check performed_via_github_app, so a removal an App made while acting for a listed person counts as that person's lift. The merge policy's own rule refuses App-made removals (babysit_merge_policy.py _class_removed_by_non_human), and #9512 applies it in pr-merge's escalation. Found by the security review on #9512.

Fix

Add and .performed_via_github_app == null to that select, with a case in scripts/ci-hold-human-gated.test.sh (or the test that runs this body) where an App-made removal by a listed id opens no window.

Done when

An App-made removal never opens the lift window, and a person's direct removal still does.

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

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.