Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

feat(skill-pr-addresser): detect updated comments after addressing

オープン
#796 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

@aRustyDev がすでに取り組んでいます。

2026年1月2日 から。

評価

この issue はまだ評価されていません。

説明

enhancement skill-pr-addresser

Summary

The skill-pr-addresser agent tracks addressed feedback by ID to avoid duplicate work. However, if a reviewer updates their comment after it's been addressed, the agent will still skip it because the ID hasn't changed.

Current Behavior

# discovery.py:288-291
def not_addressed(item) -> bool:
    item_id = getattr(item, "id", None)
    return item_id is None or item_id not in addressed_ids

Only checks if the ID exists in addressed_ids. Content changes are not detected.

Proposed Solution

Store a content hash alongside the addressed ID:

# In session.results["addressed_feedback"]
{
    "comment-123": {
        "id": "comment-123",
        "content_hash": "sha256:abc123...",
        "addressed_at": "2025-01-02T12:00:00Z"
    }
}

Then compare content hash when filtering:

def not_addressed(item) -> bool:
    item_id = getattr(item, "id", None)
    if item_id is None:
        return True
    
    addressed = addressed_feedback.get(item_id)
    if not addressed:
        return True
    
    # Check if content changed since addressing
    current_hash = hash_content(item.body)
    return current_hash != addressed.get("content_hash")

Acceptance Criteria

  • Store content hash when marking feedback as addressed
  • Compare hash when filtering already-addressed feedback
  • Re-process comments whose content has changed
  • Add tests for updated comment detection

Notes

  • Review threads don't have persistent IDs in our model, so they're always reprocessed
  • This only affects PR comments and review comments with IDs
主要言語
Jupyter Notebook
スター
8
フォーク
3
PR マージ指標
30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

aRustyDev/agents のほかの issue

aRustyDev/agents の issue をすべて見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。