Design: configurable second-reviewer loop
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ó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 45/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ệ
- python
- Lĩnh vực
- documentation
Hướng nghiên cứu
Start by reading the parent issue #1232 and the existing material in docs/design/ to understand the project’s design conventions. Write a design doc covering the reviewer interface, severity scale, round limits, finding ledger, stop rules, cost defaults, and gates; include sample .plate configs for Copilot and a local model, mark the multi-provider council as reference only, and list proposed follow-up issues. Done means every requirement in the issue is addressed in that document.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Part of #1232.
Summary
Design a bounded second-reviewer loop. The ideas come from tstack's sol-loop and council skills (t3dotgg/tstack, MIT, credit Theo Browne). We write our own design.
Andrew's decision
Yes, but configurable. The reviewer is an adopter option: Copilot, a local model (for example via Ollama), OpenHands, Devin, or another tool. PLATE ships the interface, the round limit and the written-reason rules. PLATE does not hard-code a vendor. The three-model council stays reference only.
What the design must cover
- Interface. How a reviewer is set (
.platesetting), what it gets (base SHA, diff, intent, out of scope, check results), and the reply shape (verdict plus findings with stable ids). - Severity scale. blocker (intended case wrong), should (a real bug callers will hit), nit (the rest).
- Round limit. Default 3 rounds. Up to 5 only for security, auth, data loss or concurrency while a blocker or should stays open. Stop early on nits only.
- Ledger. Each finding gets an id, a severity and an outcome (fixed, disagree with evidence, or declined with a written reason).
- Stop rules. Approved, or stuck after two replies on one item, or out of budget. Running out of budget is not approval. A timeout is not approval.
- Cost defaults. Off or cheap by default. Read-only reviewers. One reviewer at a time.
- Gates. The loop never merges and never overrides AGENTS.md approval gates.
Definition of done
- A design doc in
docs/design/covers all points above, with a sample.plateconfig for at least Copilot and a local model. - It says the multi-provider council is reference only.
- It lists follow-up implementation issues (proposed, not filed).
- Ngôn ngữ chính
- Python
- Star
- 1
- Fork
- 0
- Merge trung bình
- 2 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 40
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
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 akasper/plate
-
Documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
conservative import-payload dry-run lists plate-ci.yml as would_create although ci.yml existsĐang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
akasper/plate#1260 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bootstrap commit message cites an issue number from PLATE's own repoCó thể đã có người làm @akasper đã nhận 1 ngày trước. Đang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bootstrap creates its Epic and Curiosity Question issues without labelsCó thể đã có người làm @akasper đã nhận 1 ngày trước. Đang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của akasper/plate
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
aws-samples/sample-per-model-bedrock#639 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
awslabs/visual-asset-management-system#937 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
project-codeflare/codeflare-sdk#1195 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Bump numpy dependencyĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
pandas-dev/pandas-stubs#2051 ·
Maintainer thường phản hồi trong vòng 1 ngày