Skip to content

fix(search): limit same_entity recency marker to diff/doc rows - #271

Merged
liplus-lin-lay merged 1 commit into
mainfrom
fix/269-recency-diff-doc-only
Oct 3, 2026
Merged

liplus-lin-lay merged 1 commit into
mainfrom
fix/269-recency-diff-doc-only

Conversation

@liplus-lin-lay

Copy link
Copy Markdown
Member

Closes #269

変更内容

  • markRecency(src/aggregate.ts)が各行の type を受け取り、diff / doc 行のみで updated_at を比較するよう変更。その他の type には marker を付けず、比較にも参加させない。
  • src/mcp.ts の呼び出しで type を渡すよう変更し、ツール説明文の recency の文を diff/doc 行限定に更新。
  • テスト追加: issue + コメント群で marker が付かないこと、混在グループで diff/doc のみ比較されること。

spawn record = model same as parent, effort low (reason: scoped fix, tests/CI catch errors)

🤖 Generated with Claude Code

https://claude.ai/code/session_01XiAGywnJ6R9ibkNrxg9fry

markRecency now takes each row's type and only compares diff and doc rows.
Issue/PR/comment/review rows get no marker and do not enter the comparison.
issue とコメントは同一 entity の版ではないため、recency は diff/doc 行に限定する。
ツール説明文も同様に更新し、issue + コメント群で marker が付かないテストを追加。

Refs #269

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XiAGywnJ6R9ibkNrxg9fry
@liplus-lin-lay liplus-lin-lay linked an issue Oct 3, 2026 that may be closed by this pull request
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
github-rag-mcp 2d94244 Oct 03 2026, 10:37 AM

@liplus-lin-lay liplus-lin-lay left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

セルフレビュー(Lin): #269 の受け入れ条件を確認。印の比較対象を diff/doc に限定、issue・PR・コメント・レビューには付かない。順位・score・rerank・graph 不変。ツール説明文も更新済み(diff で確認)。テスト 2 件追加、既存テストに type を付与、CI 緑。スコープ逸脱なし。ワークスペース側の後回し事項なし。完了条件(本番検索での確認)は PR 単独で満たせないため、マージ後に #269 を再オープンして確認する。auto モードのため直接マージ。

@liplus-lin-lay
liplus-lin-lay merged commit 3529ed8 into main Oct 3, 2026
3 checks passed
@liplus-lin-lay
liplus-lin-lay deleted the fix/269-recency-diff-doc-only branch October 3, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(search): limit recency marker to file-version rows (diff/doc)

2 participants