Skip to content

refactor: antv skill refactor#90

Merged
lxfu1 merged 5 commits into
feat-context-servicefrom
refactor/skill
Jul 1, 2026
Merged

refactor: antv skill refactor#90
lxfu1 merged 5 commits into
feat-context-servicefrom
refactor/skill

Conversation

@lxfu1

@lxfu1 lxfu1 commented Jun 30, 2026

Copy link
Copy Markdown
Member

Content markdown 作为唯一知识源 → Build 脚本生成索引 → zvec 提供混合检索 → CLI/API/HTTP 多层暴露

  1. 简化 skills ,内容移入 src/content
  2. 移除 bm25 召回逻辑
  3. 基于 @zvec/zvec@huggingface/transformers 重构整个检索体现

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the retrieval and evaluation system, transitioning from file-path-based BM25 and tool-call strategies to an asynchronous, vector-based hybrid search using an in-process vector database (zvec) and the @huggingface/transformers library. Key changes include updating the embedding dimension to 512, adding a model pre-download script, refactoring the evaluation manager and CLI to support zvec retrieval, and rewriting the SKILL.md files for G2, G6, and X6 to provide streamlined quick references and critical rules. Additionally, comprehensive test suites were added for the embedder, synonyms, and token budget modules. The reviewer identified a critical inconsistency in the SKILL.md files where the library constraints endpoint was incorrectly documented as POST /info instead of POST /api/v1/info, which would cause LLM agent calls to fail.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread skills/antv-g2-chart/SKILL.md Outdated
Comment thread skills/antv-g6-graph/SKILL.md
Comment thread skills/antv-x6-editor/SKILL.md
@lxfu1
lxfu1 requested a review from hustcc June 30, 2026 11:32
@lxfu1
lxfu1 merged commit da1c60b into feat-context-service Jul 1, 2026
4 checks passed
@lxfu1
lxfu1 deleted the refactor/skill branch July 1, 2026 02:16
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.

2 participants