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

Exec remediation preview lists files with uncommitted user changes that the apply will not touch

Đang mở
  • Có gì không ổn?
#543 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/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ệ
python
Lĩnh vực
backend

Hướng nghiên cứu

Start by reading the preview and apply paths for exec remediation on the 043 branch, then compare how other handlers represent dirty files as action = "none" with reason user_changes_present (T055d). Trace how previewed paths control whether apply runs the command. Done means the preview excludes user changes while apply still refuses to run when recorded dirty paths conflict; verify against FR-021 and the relevant tests.

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

Mô tả

module-core needs triage quality

On the 043 branch, a previewable exec remediation (effects = "working_tree", offline = true, for example the zizmor fixes) is previewed in a scratch copy of the checkout. The preview lists every file the command would change.

If one of those files has uncommitted user changes, the preview still lists it as a modification. The apply then refuses to run the command and reports the conflict.

Nothing is lost, but the preview overstates what will happen, which violates FR-021 (the preview is what the apply will do). Other handlers already preview such files as action = "none", reason = "user_changes_present" (T055d).

Marking the exec paths none in the plan is not enough on its own: the apply uses the previewed paths to decide whether to run the command, so it would then run and touch the user's file. The fix has to keep that stop. One option is to record the dirty paths on the plan item, mark the step as "will not run", and have the apply re-check the recorded paths before running.

Found while implementing feature 043 (remediation safety, branch 043-remediation-safety, not yet merged).

Ngôn ngữ chính
Python
Star
22
Fork
27
Merge trung bình
1 ngày 20 giờ
Pull request đã merge (30 ngày)
39

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 darnitdevorg/darnit

Tất cả issue của darnitdevorg/darnit

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.