Skip to content

fix(search): filter invalid result tags - #4153

Merged
qin-ctx merged 1 commit into
mainfrom
fix/filter-invalid-attr-tags
Aug 21, 2026
Merged

fix(search): filter invalid result tags#4153
qin-ctx merged 1 commit into
mainfrom
fix/filter-invalid-attr-tags

Conversation

@zhoujh01

@zhoujh01 zhoujh01 commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

变更说明

  • 修正 session 自动提交策略的默认值,使其与发布配置保持一致:
    • pending_token_threshold: 150000
    • message_count_threshold: 100
    • keep_recent_count: 0
  • 读取历史向量记录的标签时,仅展示严格 key=value 格式的业务标签。
    • fs/attrs/api/v1/search/find 和默认 list 模式的 /api/v1/search/search 都会过滤裸标签(例如 default)和非法标签(例如 bad=)。
    • 合法标签会保持原有规范化行为;该改动只影响读取结果,不会回写或迁移存量向量数据。
  • 补充检索转换、公共结果序列化及两个 HTTP 搜索接口的回归测试。

验证

  • uv run --active pytest tests/retrieve/test_hierarchical_retriever_rerank.py tests/retrieve/test_provenance.py tests/server/test_api_search.py::test_search_endpoints_filter_invalid_result_tags -q
    • 32 passed
  • uv run --active ruff check openviking/retrieve/hierarchical_retriever.py openviking_cli/retrieve/types.py tests/retrieve/test_hierarchical_retriever_rerank.py tests/retrieve/test_provenance.py tests/server/test_api_search.py
  • git diff --check

Co-authored-by: TRAE CLI <traecli@bytedance.com>
@zhoujh01
zhoujh01 marked this pull request as ready for review August 20, 2026 07:32
@qin-ctx
qin-ctx merged commit 499995f into main Aug 21, 2026
6 checks passed
@qin-ctx
qin-ctx deleted the fix/filter-invalid-attr-tags branch August 21, 2026 06:59
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants