Skip to content

chore(knowledge): remove unused louvain dep & fix wiki_graph description - #975

Open
NIU-123370 wants to merge 1 commit into
TencentCloud:feat/server_teamfrom
NIU-123370:chore/knowledge-remove-unused-louvain
Open

chore(knowledge): remove unused louvain dep & fix wiki_graph description#975
NIU-123370 wants to merge 1 commit into
TencentCloud:feat/server_teamfrom
NIU-123370:chore/knowledge-remove-unused-louvain

Conversation

@NIU-123370

Copy link
Copy Markdown

Closes #973.

graphology-communities-louvain 全仓零 import,communities 恒为空数组。本 PR:

  • 删除未使用的 graphology-communities-louvain 依赖(graphology 主包保留,仍在使用:manager.ts:14 / graph-search.ts:15
  • 修正 wiki_graph / get_graph 工具描述,去掉 communities 承诺(保留 communities: [] 字段维持接口契约)

改动: MemoryKnowledge/package.jsonMemoryKnowledge/src/mcp/tools.tsMemoryKnowledge/src/routes/tools.ts

若维护者计划实现社区发现,可只采纳描述修正、保留依赖——请在 review 时指出,我会调整。

- graphology-communities-louvain 全仓零 import,移除死依赖
  (graphology 主包仍在使用:manager.ts / graph-search.ts)
- wiki_graph / get_graph 工具描述去掉 communities 承诺
  (社区发现未实现,communities 恒为空;保留字段维持接口契约)

Refs: TencentCloud#973
Signed-off-by: 牛瑞博 <912906590@qq.com>
@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

Thank you for your attention and contribution! We will schedule an internal review of this PR, and we will share any feedback right here.

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.

[Bug] graphology-communities-louvain 依赖声明但全仓零 import,communities 恒空却对外宣称返回

2 participants