Smarter finish: review the turn's changes before saying it is done
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 38/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- typescript
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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:
- Collect the turn's changes: the diffs from the edit tools, which Patch already keeps for Undo.
- Ask a read-only reviewer, the
oraclerole or a newreviewerwithREAD_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. - 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.
- 主要言語
- TypeScript
- スター
- 2
- フォーク
- 2
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
似ている issue
-
unwrapDatabaseError masks real DB errors with TypeError: Cannot convert object to primitive valueオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
NEARBuilders/everything-dev#798 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
vercel-labs/durable-harness#53 ·
メンテナーはふだん 1 日以内に返信
-
[Bug]: Redesign tab bars ignore clicks on Linux: hidden chat layers keep window drag regions対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
experiência: a doc manda subir para o Micro do Supabase, mas não diz que exige plano pago nem quanto custa対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンarea/docs area/ui enhancement
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
melgarafael/DeskcommCRM#4098 ·
メンテナーはふだん 1 日以内に返信
-
dependencies platform: ios
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
software-mansion/react-native-executorch#1617 ·
メンテナーはふだん 1 日以内に返信