Skip to content

feat(search): add recency marker to same_entity rows - #268

Merged
liplus-lin-lay merged 2 commits into
mainfrom
issue-267-same-entity-recency
Oct 3, 2026
Merged

liplus-lin-lay merged 2 commits into
mainfrom
issue-267-same-entity-recency

Conversation

@liplus-lin-lay

Copy link
Copy Markdown
Member

Closes #267

検索結果の代表行と same_entity.others に、プール内の updated_at 比較による recency(latest / superseded)を加算的に付与します。順位・score・rerank・graph は変更なし。ツール説明文を更新し、src/aggregate.test.ts に markRecency のテストを追加しました。

spawn record = model same as parent, effort low (reason: issue scope fixed, tests/CI catch errors)

🤖 Generated with Claude Code

https://claude.ai/code/session_01XiAGywnJ6R9ibkNrxg9fry

同じ実体の代表行と same_entity.others の updated_at を比較し、より新しい行がある行に superseded、最新行に latest を付ける加算的フィールドを追加した。順位・score・rerank は変えない。プール内に版が一つしか無い場合は印を付けない。#267

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 8df1a64 Oct 03 2026, 10:29 AM

検索ツールの説明文に recency(superseded / latest)の説明を追加した。前回のコミットで説明文の置換が漏れていたため。#267

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XiAGywnJ6R9ibkNrxg9fry

@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): #267 の受け入れ条件を確認。順位・score・rerank・graph 不変、追加フィールドのみ(recency)、同一実体内で updated_at が 2 種以上のときだけ付与、未付与は最新の主張ではない、ツール説明文に記載済み(初回コミットで説明文の置換が無言で空振りしていたのを差し戻して修正、8df1a64)。テスト 3 件追加、CI 緑。スコープ逸脱なし。ワークスペース側の後回し事項なし。完了条件(デプロイ後の本番検索で印の確認)は PR 単独で満たせないため、マージ後に #267 を再オープンして確認する。auto モードのため直接マージ。

@liplus-lin-lay
liplus-lin-lay merged commit 79f05cd into main Oct 3, 2026
3 checks passed
@liplus-lin-lay
liplus-lin-lay deleted the issue-267-same-entity-recency branch October 3, 2026 10:30
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.

feat(search): mark superseded rows within same_entity by recency

2 participants