Skip to content

chore(runtime): remove the unused cc (Claude Code) runtime#594

Merged
zakahan merged 1 commit into
mainfrom
chore/remove-cc-runtime
Jun 9, 2026
Merged

chore(runtime): remove the unused cc (Claude Code) runtime#594
zakahan merged 1 commit into
mainfrom
chore/remove-cc-runtime

Conversation

@yaozheng-fang

Copy link
Copy Markdown
Collaborator

What

Removes the cc (Claude Code) runtime from veadk/runtime/. The project standardizes on two runtimes: adk (default) and codex. The cc runtime was never a documented/installable path — it has no PyPI extra and depends on claude_agent_sdk, which isn't declared anywhere.

Changes

  • Delete veadk/runtime/cc/runtime.py, proxy.py, translate.py, __init__.py.
  • veadk/runtime/__init__.py — drop the cc branch in get_runtime and remove cc from the package docstring. Unknown "cc" now raises ValueError like any other unknown runtime.
  • veadk/agent.pyruntime: Literal["adk", "codex"] (was ["adk", "cc", "codex"]) and updated the field docstring.
  • veadk/runtime/base_runtime.py — docstring example uses "codex".
  • veadk/runtime/codex/translate.py — module docstring no longer references the removed cc runtime.

No standalone docs referenced cc (only built docs/out / node_modules artifacts, untouched).

Verification

  • get_runtime("cc")ValueError: Unknown runtime: 'cc'; codex runtime imports unchanged.
  • No cc-runtime references remain in any *.py.
  • ruff + pyright clean.

The `cc` runtime was never a documented/usable path (no PyPI extra, depends on
claude_agent_sdk) and the project standardizes on `adk` (default) and `codex`.
Remove it and all references:

- delete veadk/runtime/cc/ (runtime, proxy, translate)
- drop the `cc` branch from get_runtime and the runtime package docstring
- Agent.runtime: Literal["adk", "codex"] (was ["adk", "cc", "codex"]) + docstring
- base_runtime docstring example -> "codex"
- codex/translate docstring no longer references the cc runtime
@zakahan zakahan merged commit c953e74 into main Jun 9, 2026
16 checks passed
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