Plan contradiction: force push is both an extracted action and excluded as network egress
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
- 74/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- documentation, security
Hướng nghiên cứu
Đọc docs/plans/2026-09-19-intent-aware-judgment.md cùng với literal-match.ts, bắt đầu từ bảng § Literal match và đoạn về loại trừ. Giải quyết mâu thuẫn bằng cách khiến kế hoạch mô tả nhất quán chính sách force-push đã chọn và lý do của chính sách đó; hoàn thành khi bảng và phần loại trừ không còn mâu thuẫn với hành vi hiện tại hoặc với một thay đổi được biện minh rõ ràng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The intent-aware plan (docs/plans/2026-09-19-intent-aware-judgment.md) says two things that cannot both hold:
- The action table (§ Literal match) lists "Force push, branch delete" as an extracted action that can match literally.
- The paragraph under it says: "These always go to the reviewer and never match literally: any network egress, any use of a secret, privilege (
sudo,doas), and any segment that is not extracted or inert."
A force push is network egress, so the table row is unreachable under the exclusion.
literal-match.ts resolves it the conservative way: it does not extract any push, so a force push goes to the reviewer. Branch delete is local and still matches. Pick one and make the plan say it:
- Keep the exclusion as written and drop force push from the table (what the code does today).
- Carve out git push to its own remote as not "egress" for this purpose, and say why the carve-out is safe.
Option 1 costs a dialog on git push --force-with-lease to a branch the user named. Option 2 needs an argument for why a push is different from scp, and the plan does not make one.
- Ngôn ngữ chính
- TypeScript
- Star
- 0
- Fork
- 1
- Merge trung bình
- 1 giờ 15 phút
- Pull request đã merge (30 ngày)
- 49
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của STRML/omp-classifier
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 80/100
STRML/omp-classifier#111 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
STRML/omp-classifier#107 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
STRML/omp-classifier#88 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
STRML/omp-classifier#81 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
STRML/omp-classifier#80 ·
Tất cả issue của STRML/omp-classifier
Issue tương tự
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Crush Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/cln-application#167 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Quantco/pnpm-licenses#17 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100