-
Notifications
You must be signed in to change notification settings - Fork 0
fix(build): use mcp[cli] extra - #2
Merged
Merged
Conversation
pip install -e . (editable/dev install path) reads pyproject.toml's dependency list, not requirements.txt — same missing-typer failure mode as the previous commit if that's the install path the build environment uses.
chenyida7-prog
added a commit
that referenced
this pull request
Sep 2, 2026
MVP 平台 = ClaudeCode + CodeBuddyIDE(client_config.py 注释),共用同一套
configureClient / write_kind / remove_kind / detect_platform,仅落盘目录不同:
ClaudeCode → ~/.claude.json / ~/.claude/settings.json / ~/.claude/agents/
CodeBuddyIDE → ~/.codebuddy/mcp.json / ~/.codebuddy/settings.json / ~/.codebuddy/agents/
hooks matcher: ClaudeCode "Bash|Write|Edit" + curl -d @-;CodeBuddyIDE "*" + hooks_forward
agent frontmatter: CodeBuddyIDE 额外带 agentMode/enabled/enabledAutoRun/mcpServers
改动:
- 重命名(去平台名,与 tests/test_client_config.py 对齐):
tests/test_claude_code_config_verify.py → tests/test_client_config_verify.py
tests/frontend/test_claude_code_config.py → tests/frontend/test_client_config_toggle.py
- test_client_config_verify.py:platform fixture 参数化 ["ClaudeCode","CodeBuddyIDE"],
三次核心检验 + 开关独立性 8 组合矩阵对两平台各跑一遍;新增 TestCrossPlatformIsolation
(同一 home 下两平台配置互不干扰)。27 → 56 例。
- test_client_config_toggle.py:TestConfigToggleBrowser 用 @parametrize("plat",[...])
对两平台跑开关 UI 双向/独立/关开循环/guideExecute 回归;新增 test_full_off_on_cycle_via_ui。
静态断言补两平台 kinds + toggle markup 平台无关。14 → 24 例。
- docs/test-plan-claude-code-config.md:新增 §2.1b CodeBuddy IDE 对照表、§6 打包产物;
三次检验表格补 CodeBuddyIDE 列;发现的问题补「CodeBuddy IDE 未发现新问题」
(问题 #1/#2 同样影响 CodeBuddyIDE,已由同一处 store.js 修复覆盖,新用例参数化验证)。
CodeBuddy IDE 三次检验 + 独立性全部通过,未发现平台专属 bug。
不改动任何既有测试文件。测试结果:
tests/test_client_config_verify.py 56 passed
tests/frontend/test_client_config_toggle.py 24 passed
后端全量 787 passed;tests/frontend/ 88 passed / 115 skipped / 3 pre-existing failed
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SSMDrsmrXR5kr68SUS5D5P
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
打包时 requirements 缺 typer,改用 mcp[cli] extra 修复。已本地验证打包通过。