docs: focus coding-agents guide on Claude Code with translation proxy - #258
Conversation
- Narrow scope to Claude Code only; remove opencode and Codex CLI sections - Add how to configure reasoning effort when starting the InferenceService (server-side --reasoning-effort flag and request-time override) - Update Claude Code section with corrected proxy setup for LiteLLM and claude-code-router (config-driven, ccr code startup command) - Qwen3.6 and Gemma 4 recommendations and Unsloth quantized model list already present; no change needed
WalkthroughClarifies that tuning should be agent-driven, adds a GPU memory headroom SLO, expands benchmark metrics to include TTFT/ITL/TPS and reasoning overhead, provides richer one-change proposals and GPU-level monitoring, and adds guardrails for measuring and bounding reasoning costs. ChangesAgent-driven tuning workflow revisions
🎯 3 (Moderate) | ⏱️ ~20 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/en/agentic_mlops/coding-agents-with-inference-service.mdx`:
- Line 106: Clarify that the chat template path
`examples/tool_chat_template_deepseekr1.jinja` referenced in the DeepSeek R1
table is relative to the vLLM repository root, or replace it with a full
external URL (e.g., the GitHub or HuggingFace raw file URL) so users can locate
the template; update the table cell for DeepSeek R1 / `deepseek_v3` to either
prepend "vLLM repo root:" before the relative path or supply the canonical
remote link to the template.
- Around line 135-143: The example JSON incorrectly nests the reasoning_effort
setting inside an SDK-specific extra_body object; update the JSON example so
reasoning_effort is a top-level field (remove the extra_body wrapper) — locate
the snippet that defines "extra_body" and "reasoning_effort" and move
reasoning_effort out to be a sibling of "model" and "messages" so the request
body matches the raw HTTP API.
- Around line 104-111: Update the model table entries so the vLLM parser flags
match documented support: remove `deepseek_v3` from the DeepSeek R1 row and set
its `--reasoning-parser` to `deepseek_r1` (no `--tool-call-parser`), change the
QwQ / Qwen row to use `--tool-call-parser hermes` and `--reasoning-parser
deepseek_r1`, change Hunyuan-A13B-Instruct to use `--tool-call-parser
hunyuan_a13b` and `--reasoning-parser hunyuan_a13b` (replace `deepseek_v3`), and
change Cohere Command A Reasoning to `--tool-call-parser cohere_command3` and
`--reasoning-parser cohere_command3` (note the optional `cohere_melody`
dependency); keep the guidance to consult model cards and the vLLM tool calling
docs for other families.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 09ee6c7b-8d23-4d8c-8c3d-4c48b4e1439f
📒 Files selected for processing (1)
docs/en/agentic_mlops/coding-agents-with-inference-service.mdx
|
|
||
| | Model family | `--tool-call-parser` | `--reasoning-parser` | Notes | | ||
| | --- | --- | --- | --- | | ||
| | DeepSeek R1 (`deepseek-ai/DeepSeek-R1-*`) | `deepseek_v3` | *(none required)* | Also needs `--chat-template examples/tool_chat_template_deepseekr1.jinja` | |
There was a problem hiding this comment.
Clarify the chat template path.
The path examples/tool_chat_template_deepseekr1.jinja is relative without context. Users won't know where to find this file. Clarify that this path is relative to the vLLM repository root, or provide the full HuggingFace/GitHub URL where users can locate the template.
📝 Suggested clarification
-| DeepSeek R1 (`deepseek-ai/DeepSeek-R1-*`) | `deepseek_v3` | *(none required)* | Also needs `--chat-template examples/tool_chat_template_deepseekr1.jinja` |
+| DeepSeek R1 (`deepseek-ai/DeepSeek-R1-*`) | `deepseek_v3` | *(none required)* | Also needs `--chat-template` pointing to `examples/tool_chat_template_deepseekr1.jinja` from the vLLM repository |📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| | DeepSeek R1 (`deepseek-ai/DeepSeek-R1-*`) | `deepseek_v3` | *(none required)* | Also needs `--chat-template examples/tool_chat_template_deepseekr1.jinja` | | |
| | DeepSeek R1 (`deepseek-ai/DeepSeek-R1-*`) | `deepseek_v3` | *(none required)* | Also needs `--chat-template` pointing to `examples/tool_chat_template_deepseekr1.jinja` from the vLLM repository | |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/en/agentic_mlops/coding-agents-with-inference-service.mdx` at line 106,
Clarify that the chat template path
`examples/tool_chat_template_deepseekr1.jinja` referenced in the DeepSeek R1
table is relative to the vLLM repository root, or replace it with a full
external URL (e.g., the GitHub or HuggingFace raw file URL) so users can locate
the template; update the table cell for DeepSeek R1 / `deepseek_v3` to either
prepend "vLLM repo root:" before the relative path or supply the canonical
remote link to the template.
Deploying alauda-ai with
|
| Latest commit: |
b18b5cd
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2b48441d.alauda-ai.pages.dev |
| Branch Preview URL: | https://docs-coding-agents-inference.alauda-ai.pages.dev |
The flag does not exist in vLLM. Replaced with accurate guidance about server-wide control via --chat-template and request-level parameters.
Summary
Extend the coding agents guide to cover reasoning models, Claude Code proxy setup, and model recommendations for on-premise LLMs. Also update the MLOps tuning guide to cover llama.cpp (GGUF) and GPU-level observability.
Files changed
docs/en/agentic_mlops/coding-agents-with-inference-service.mdxdocs/en/agentic_mlops/mlops-with-coding-agents.mdxcoding-agents-with-inference-service.mdxchangesClaude Code translation proxy
/v1/messagesendpoint~/.claude-code-router/config.jsonStep 2b: Configure reasoning models and reasoning effort
--tool-call-parser+--reasoning-parservalues--chat-templatedefaults, request-timechat_template_kwargs,thinking_token_budget, andreasoning_effort(where supported)--default-chat-template-kwargsflag (non-existent in vLLM)Model recommendations
Architecture update
mlops-with-coding-agents.mdxchangesExtended tuning guide for llama.cpp (GGUF)
--cache-type-k,--cache-type-v,--ctx-size,--parallel,--reasoning/--reasoning-budget)Reasoning overhead as a benchmarking metric
reasoning_tokens / content_tokensGPU-level metrics in performance tuning
Observability guidance
/metricsendpoint and llama.cpp--metricsflagPR history
This PR builds on the initial coding agents documentation (#245) by extending coverage to reasoning models, Claude Code proxy setup, and model recommendations.