Skip to content

feat(install): add CodeBuddy Code support#633

Merged
tirth8205 merged 1 commit into
mainfrom
codex/port-codebuddy-584
Jul 17, 2026
Merged

feat(install): add CodeBuddy Code support#633
tirth8205 merged 1 commit into
mainfrom
codex/port-codebuddy-584

Conversation

@tirth8205

Copy link
Copy Markdown
Owner

Summary

Ports the viable CodeBuddy Code integration from #584 onto the current installer while preserving the original contributor's attribution.

  • registers the official project .mcp.json / mcpServers contract and keeps the current JSONC-safe, no-data-loss merge behavior
  • deduplicates only the exact Claude Code / CodeBuddy shared contract; unrelated clients that happen to share a path are still processed
  • writes CODEBUDDY.md and .codebuddy/skills/<name>/SKILL.md
  • installs .codebuddy/settings.json hooks for Edit|Write|Bash, preserving user settings and resolving the repository at hook runtime rather than embedding an install-machine path
  • adds current CLI dispatch, installer regression coverage, and user/architecture documentation

The layout is checked against CodeBuddy's current official MCP, skills, and hooks documentation.

Attribution

Source contribution: #584 by @chuckiefan, exact reviewed head b92f4bd69b9aa7fed0873acad187daeb808535b3.

The implementation commit includes:

Co-Authored-By: Chuckiefan <chuckiefan@163.com>

Validation

  • TDD: the nine new CodeBuddy contract tests failed before implementation and passed afterward
  • focused installer/CLI/docs: 184 passed
  • complete Python suite: 1492 passed, 2 xpassed
    • on macOS, the installed native FSEvents extension bus-faults during watcher teardown; the complete run used watchdog's polling observer as a test-only backend and exercised all watcher tests
  • changed-file Ruff: clean
  • git diff --check: clean
  • graph review: risk 0.40, no affected execution flows

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

code-review-graph review

Overall risk: 0.80 (HIGH) — 24 changed function(s)/class(es), 0 affected flow(s), 13 test gap(s)

Risk-scored changes

Risk Level Symbol Location Tested
0.80 high code_review_graph/cli.py::_handle_init code_review_graph/cli.py:189 no
0.65 medium code_review_graph/skills.py::_merge_hooks_into_settings code_review_graph/skills.py:889 no
0.55 medium code_review_graph/skills.py::install_codebuddy_hooks code_review_graph/skills.py:948 no
0.40 medium code_review_graph/skills.py::install_platform_configs code_review_graph/skills.py:412 no
0.40 medium code_review_graph/skills.py::_record_configured code_review_graph/skills.py:459 no
0.40 medium code_review_graph/skills.py::install_hooks code_review_graph/skills.py:930 no
0.30 low code_review_graph/skills.py::install_codebuddy_skills code_review_graph/skills.py:1327 no
0.30 low tests/test_cli_install.py::_generate_skills tests/test_cli_install.py:130 no
0.30 low tests/test_cli_install.py::_install_codebuddy_skills tests/test_cli_install.py:134 no
0.30 low tests/test_cli_install.py::_install_codebuddy_hooks tests/test_cli_install.py:138 no

Test gaps

  • code_review_graph/cli.py::_handle_init (code_review_graph/cli.py:189)
  • code_review_graph/skills.py::install_platform_configs (code_review_graph/skills.py:412)
  • code_review_graph/skills.py::_record_configured (code_review_graph/skills.py:459)
  • code_review_graph/skills.py::_merge_hooks_into_settings (code_review_graph/skills.py:889)
  • code_review_graph/skills.py::install_hooks (code_review_graph/skills.py:930)
  • ...and 8 more without direct tests

Token savings: this graph-backed report used ~62,262 fewer tokens (~93%) than reading every changed file in full (estimated, chars/4 approximation).


Powered by code-review-graph — local-first analysis; no code leaves the CI runner.

Port the viable CodeBuddy integration from PR #584 onto the current JSONC-safe installer, with narrow shared-config deduplication, runtime-resolved hooks, current tests, and user documentation.

Co-Authored-By: Chuckiefan <chuckiefan@163.com>
@tirth8205
tirth8205 force-pushed the codex/port-codebuddy-584 branch from 4a313b6 to 496cae4 Compare July 17, 2026 14:45
@tirth8205
tirth8205 merged commit 976d0bf into main Jul 17, 2026
11 checks passed
@tirth8205
tirth8205 deleted the codex/port-codebuddy-584 branch July 17, 2026 14:54
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.

1 participant