Smarter finish: review the turn's changes before saying it is done

Đã đóng
  • Có gì không ổn?
#83 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

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
38/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
ai-infra-agents

Hướng nghiên cứu

Start by reading the turn completion flow and the edit-tool/Patch Undo change tracking, then inspect how subagents, oracle or READ_ONLY_TOOLS, and per-turn cost reporting work. Check the handling of AGENTS.md and the benchmark referenced by #49. Done means file-changing turns can use the requested off/multi-file/always settings, perform at most one read-only review, address concrete findings before finishing, and report its cost and time.

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

Mô tả

enhancement priority: medium severity: low

Today

When the model decides it has finished, it writes a summary and the turn ends. Nothing checks the changes against what the user asked. Missed requirements, a leftover debug line or an edit to the wrong file show up only when the user reads the diff.

Proposal

For turns that changed files, run a short self-review before the final answer:

  1. Collect the turn's changes: the diffs from the edit tools, which Patch already keeps for Undo.
  2. Ask a read-only reviewer, the oracle role or a new reviewer with READ_ONLY_TOOLS, to compare them with the user's request and the project's conventions (AGENTS.md). It should answer with concrete problems only: missing parts, wrong behavior, unrelated changes, failing checks.
  3. If it finds problems, the main agent fixes them in the same turn; if not, it finishes. At most one review round, so it can't loop.

Settings: off / on for multi-file changes / always. The review's cost and time show on the turn like any subagent.

Measure

Benchmark success rate (#49) against the extra cost per task. If it doesn't pay for itself on the cheaper reviewer model (#78), keep it off by default.

Ngôn ngữ chính
TypeScript
Star
2
Fork
2
Không có pull request nào được merge trong 30 ngày

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 tương tự

Thêm issue về TypeScript

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.