Skip to content

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

Description

@liplus-lin-lay

purpose

#267(PR #268)で入れた recency 印が、版の関係にない行にまで付く誤りを直す。

premise

  • 2026-10-03 10:35Z の本番検索「webhook ownership test own external events」(repo=Liplus-Project/liplus-language)で、issue #2007 の本文行が recency=latest、同じ実体に畳まれたコメント 4 件が recency=superseded になった。#1890、#1997 も同じ形。
  • same_entity は issue/PR とそのコメント・レビューを一つの実体として畳む(src/mcp.ts ツール説明「an issue or PR and its comments / reviews are one result」)。feat(search): add recency marker to same_entity rows #268 の markRecency は種類を区別せず updated_at だけで比べている(src/aggregate.ts markRecency)。
  • コメントは本文の旧版ではないので、本文の更新日時が新しいことを理由に「この後に新しい変更あり」とするのは誤り。

constraints

  • 印の比較対象は、同じファイルの版の関係にある行(type が diff と doc)に限る。issue / pull_request / issue_comment / pr_review / pr_review_comment / release / wiki_doc の行には付けない。
  • 順位・score・rerank・graph は変えない。
  • ツール説明文の recency の記述を、対象が差分と doc に限られることが分かる形に直す。
  • テストに「issue 本文 + コメント」の組で印が付かないケースを足す。

completion condition (PR 単独では満たせない)

  • デプロイ後、上記の本番検索で #2007 のコメントに recency が付かず、かつ「subagent model effort selection criteria spawn」で c6f16c3 の差分に superseded が付いたままであることを確認する。

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bug動いていない、壊れているready本文が実装開始できる形まで収束している状態。ただし更新は継続可能review-pending実装フェーズ完了。orchestration (brake eval / review / merge / close) 待ち

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions