skill-pr-addresser: Improve feedback deduplication and batching
@aRustyDev がすでに取り組んでいます。
2026年1月2日 から。
評価
この issue はまだ評価されていません。
説明
Problem
The agent times out when processing PRs with many review comments because:
- No deduplication of similar feedback - "Move to examples" appearing at 4 different lines creates 4 separate work items
- Guidance mixed with actionable items - Meta-guidance reviews (e.g., "follow progressive delivery patterns") are treated as change requests
- Checkbox explosion - A single review with 6 checkboxes becomes 6 separate items
- No batching - All items processed in a single fixer run
Example from PR #795
| Source | Items Generated |
|---|---|
| Review 1 (6 checkboxes) | 6 items |
| Review 2 (guidance) | 1 item |
| Review 3 (guidance) | 1 item |
| 11 review threads | 11 items |
| Total | ~17-20 items |
Many threads are actually the same feedback at different locations:
- 4 threads say "move to examples/*"
- 3 threads say "move to reference/*.md"
- 3 threads say "keep high-level, point to references"
Proposed Solutions
1. Deduplication in Analyzer
Update feedback-analyzer prompt to consolidate similar feedback:
{
"feedback_items": [
{
"id": "group-examples",
"type": "change_request",
"description": "Move code blocks to examples/",
"locations": [
{"file": "SKILL.md", "line": 671},
{"file": "SKILL.md", "line": 733},
{"file": "SKILL.md", "line": 398}
],
"priority": "high"
}
]
}
2. Separate Guidance from Actions
Add a guidance type that the fixer can use for context but not treat as individual work items:
{
"guidance": [
"SKILL.md MUST follow progressive delivery patterns",
"Adopt best-practices & anti-patterns pattern"
],
"feedback_items": [...]
}
3. Batch Processing
Instead of sending all items to fixer at once:
- Group by type (move-to-examples, move-to-references, etc.)
- Process one group at a time
- Commit after each batch
4. Priority Ordering
Process high-priority items first, skip lower priority if time/turn budget exhausted.
Related
- #796 - Detect updated comments
- Observed in PR #795 (10 items all timed out)
- 主要言語
- Jupyter Notebook
- スター
- 8
- フォーク
- 3
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
aRustyDev/agents のほかの issue
-
dependencies github-actions
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
-
dependencies github-actions
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
-
dependencies github-actions
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
-
dependencies github-actions
難易度 1/5 1時間未満 初心者へのやさしさ 25/100
-
dependencies github-actions
難易度 1/5 1〜3時間 初心者へのやさしさ 45/100