Skip to content

feat(examples): add CodeBuddy Code CLI integration example（refs #644 ） #996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
pei-pei45 wants to merge 12 commits into
base: master
Choose a base branch
from
Open

feat(examples): add CodeBuddy Code CLI integration example（refs #644 ） #996

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
2fa0a4c
feat(examples): add CodeBuddy Code CLI integration example
pei-pei45 Jul 16, 2026
e1b13fa
examples: add CodeBuddy Code CLI integration with session persistence…
pei-pei45 Jul 17, 2026
eba03f5
fix(codebuddy): prefer CUBE_API_URL over E2B_API_URL everywhere
pei-pei45 Jul 18, 2026
82657fe
feat: add sandbox_exec, MCP server, and CodeBuddy hooks
pei-pei45 Jul 22, 2026
3b4abd2
fix: concern about path validation and time running
pei-pei45 Jul 22, 2026
2cc075c
fix：add 2 new tests for pip validation and fix issue
pei-pei45 Jul 22, 2026
7b77eaa
fix:fix issue
pei-pei45 Jul 22, 2026
7070730
fix:the CI workflow uses python3 -m unittest discover which only disc…
pei-pei45 Jul 22, 2026
4840153
fix：README and issue
pei-pei45 Jul 22, 2026
da1cc65
Fix: Fix error path and No check for missing
pei-pei45 Jul 22, 2026
392e564
fix：Missing size and silent rejection of malformed client input
pei-pei45 Jul 22, 2026
c0d2152
fix(codebuddy-integration): address security and quality issues found…
pei-pei45 Jul 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
70 changes: 70 additions & 0 deletions .github/workflows/codebuddy-integration.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
# Copyright (c) 2026 Tencent Inc.
# SPDX-License-Identifier: Apache-2.0

name: codebuddy-integration

on:
pull_request:
paths:
- "examples/codebuddy-integration/**"
- "docs/**/integrations/codebuddy.md"
- ".github/workflows/codebuddy-integration.yml"
push:
branches: [main]
Comment thread
pei-pei45 marked this conversation as resolved.
Outdated

jobs:
static:
name: Static checks (compile + unit tests)
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: "3.11"

- name: Compile host driver scripts
working-directory: examples/codebuddy-integration
run: |
python3 -m py_compile env_utils.py _codebuddy_common.py \
run_codebuddy.py resume_codebuddy.py network_policy.py

- name: Run env_utils unit tests
working-directory: examples/codebuddy-integration
run: python3 -m unittest discover -s . -p test_env_utils.py -v

- name: Smoke-check --help output
working-directory: examples/codebuddy-integration
run: |
python3 run_codebuddy.py --help
python3 resume_codebuddy.py --help
Comment thread
pei-pei45 marked this conversation as resolved.

@cubesandboxbot cubesandboxbot Bot Jul 22, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Critical: CI test discovery silently skips 3 out of 4 test files

The CI uses python3 -m unittest discover which only discovers classes inheriting from unittest.TestCase. Three of the four test files use pytest-style classes (plain object base, bare assert, pytest fixtures) — they are silently skipped with no error or warning:

File Inherits TestCase? Discovered by unittest?
test_env_utils.py ✅ Yes ✅ Yes
test_codebuddy_common.py ❌ No (pytest-style, bare assert) ❌ Silently skipped
test_mcp_server.py ❌ No (pytest-style, free functions) ❌ Silently skipped
test_sandbox_exec.py ❌ No (pytest-style, bare assert) ❌ Silently skipped

Only ~752 lines of test_env_utils.py would run; ~948 lines in the other 3 files are dropped. A developer who checks CI after merging would see a green checkmark but nowhere near the advertised 166-test coverage.

Fix: Change to python3 -m pytest tests/ -v (and add pytest to a requirements file or install step), or convert the pytest-style classes to unittest.TestCase subclasses.

python3 network_policy.py --help

dockerfile:
name: Dockerfile builds cleanly
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Set up Buildx
uses: docker/setup-buildx-action@v3

- name: Build (cache miss is fine; --load is just for smoke)
uses: docker/build-push-action@v6
with:
context: examples/codebuddy-integration
push: false
load: true
tags: codebuddy-cube:ci
cache-from: type=gha
cache-to: type=gha,mode=max

- name: codebuddy --version inside the image
run: |
cid=$(docker run -d --rm codebuddy-cube:ci)
docker exec "$cid" codebuddy --version
# envd /health reachable means the readiness probe will succeed
docker exec "$cid" curl -fsS -o /dev/null \
Comment thread
pei-pei45 marked this conversation as resolved.
Outdated
http://127.0.0.1:49983/health
docker rm -f "$cid"
Loading
Loading