Repository navigation
feat(code-tools): CodeTool 插件化架构 (UFO-176) - #396
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
PR Summary by Qodofeat(code-tools): Introduce CodeTool adapter registry (Phase 1)
AI Description
Diagram
High-Level Assessment
Files changed (18)
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
Code Review by Qodo
1.
|
482061a to
9b2ab7c
Compare
9b2ab7c to
7a15a6d
Compare
…/ Codex Route init, update, uninstall, menu, and config-switch through the built-in registry so command handlers no longer branch on tool IDs.
Lock built-in tools, aliases, banners, and unknown-type rejection to the current main behavior.
Keep the conservative in-process adapter boundary and Claude Code / Codex compatibility constraints.
064524c to
a35e67d
Compare
Keep skip/output-style, illegal -T fallback, Codex menu update, and provider import feedback aligned with main, and stop adapters from importing command handlers.
Keep Claude/Codex --api-configs success summary, sink tool-owned menu actions to adapters, and import providers as ProviderProfile.
Fail invalid Codex -o before writes, switch profiles via adapter, and derive Claude/Codex path constants from definitions.
Keep the real CLI path from writing on invalid -o, and stop Claude menu switch from re-entering the command handler.
Keep the adapter-to-Codex boundary on CodexFullInitOptions, and send reachable Claude/Codex menu and uninstall paths through CodeToolDefinition.
Summary
CodeToolAdapter与src/code-tools/注册表,仅内置 Claude Code / Codexinit/update/uninstall/ 菜单 / config-switch 改经 registry 分发,去掉命令层按工具 ID 分支main一致的 skip / output-style、非法-T回退、Codex 菜单更新与 provider 导入反馈docs/adr/001-code-tool-plugin-architecture.md范围
main:Claude Code / Codex 在相同输入下的命令、交互、输出、配置文件与副作用保持一致Test plan
pnpm installpnpm typecheck/lint/buildpnpm test:run(143 files / 2541 tests)tests/unit/code-tools/main-parity.test.tsCloses UFO-176