Skip to content

refactor(api): W7 tools 注册表拆分(RAG / Text2SQL 子模块)#155

Merged
Cyning12 merged 2 commits into
mainfrom
task/api-tools-w7
Jun 9, 2026
Merged

refactor(api): W7 tools 注册表拆分(RAG / Text2SQL 子模块)#155
Cyning12 merged 2 commits into
mainfrom
task/api-tools-w7

Conversation

@Cyning12

@Cyning12 Cyning12 commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • 新增 tool_models.pytools_shared.pytools_rag.pytools_text2sql.py
  • tools.py 瘦身为 registry 入口 + direct_answer_execute + tool_mode_map
  • Epic W7 · api-tools-registry-split

Test plan

  • ruff check api tests
  • pytest tests -m "not intent_eval and not intent_benchmark"(352 passed)
  • CI Required 全绿后 merge

Made with Cursor

CI 状态(自动 · pr-post-ci)

更新:2026-06-09 09:32 UTC · tools/pr_post_ci_update_body.py

Required 全绿:是

Check Status
contract_check success
manifest_check success
pytest success

变更范围(自动统计)

文件数:7

顶层前缀 计数
api/ 5
docs/ 1
tools/ 1

⚠ automerge 路径闸:含 api/tests/.github/workflows/ 等 — 勿打 automerge 或须人审 merge。

命中路径
  • api/tool_models.py
  • api/tools.py
  • api/tools_rag.py
  • api/tools_shared.py
  • api/tools_text2sql.py

tool_models + tools_shared + tools_rag + tools_text2sql;tools.py 保留
get_tool_registry 与 direct_answer,对外 import 路径不变。

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jun 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-ink-brain-api-python Ready Ready Preview, Comment Jun 9, 2026 9:30am

text2sql.phase.* 锚点已迁至 tools_text2sql.py,BACKEND_CONTRACT_SOURCES 同步。

Co-authored-by: Cursor <cursoragent@cursor.com>
@Cyning12 Cyning12 merged commit 5d6babe into main Jun 9, 2026
9 checks passed
Cyning12 added a commit that referenced this pull request Jun 9, 2026
Epic W7 done · PR #155 · 当前棒 W8。

Co-authored-by: cyning <cyning12@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@Cyning12 Cyning12 deleted the task/api-tools-w7 branch June 13, 2026 09:48
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