Skip to content

Remove Reasonix Dockerfile; reuse Claude Code image for DeepSeek#91

Merged
valITino merged 1 commit into
mainfrom
claude/clever-ride-h9spkq
Jun 12, 2026
Merged

Remove Reasonix Dockerfile; reuse Claude Code image for DeepSeek#91
valITino merged 1 commit into
mainfrom
claude/clever-ride-h9spkq

Conversation

@valITino

Copy link
Copy Markdown
Owner

Summary

Eliminates the separate deepseek.Dockerfile and deepseek-entrypoint.sh by reusing the Claude Code image and routing it to DeepSeek's Anthropic-compatible API. This reduces maintenance burden (one fewer image to build and publish) while preserving full feature parity — same MCP wiring, same skills system, same entrypoint behavior.

Key Changes

  • Removed files:

    • docker/deepseek.Dockerfile (Reasonix-based agent)
    • docker/deepseek-entrypoint.sh (Reasonix startup script)
  • Updated docker-compose.yml:

    • deepseek service now uses image: crhacky/blhackbox:claude-code instead of building a separate image
    • Removed build context for deepseek
    • Replaced DEEPSEEK_API_KEY environment variable with Claude Code's standard Anthropic routing:
      • ANTHROPIC_BASE_URL: "https://api.deepseek.com/anthropic"
      • ANTHROPIC_AUTH_TOKEN: "${DEEPSEEK_API_KEY:-}"
      • Model mapping: ANTHROPIC_MODEL, ANTHROPIC_DEFAULT_OPUS_MODEL, ANTHROPIC_DEFAULT_SONNET_MODEL, ANTHROPIC_DEFAULT_HAIKU_MODEL
    • Added telemetry disable flag: CLAUDE_CODE_DISABLE_NONESSENTIAL_TRAFFIC: "1"
    • Added skills mount: ./.claude/skills:/root/.claude/skills:ro
  • Updated tests (tests/test_compose_integrations.py):

    • Removed assertions checking for Reasonix Dockerfile and entrypoint
    • Added assertions verifying deepseek reuses claude-code image
    • Added assertions verifying DeepSeek API routing via ANTHROPIC_BASE_URL
  • Updated documentation and configuration:

    • README.md: Updated tutorial title and description; clarified that DeepSeek uses Claude Code image + DeepSeek API
    • DOCKER.md: Removed deepseek image from table; updated service description
    • setup.sh: Updated help text to reflect Claude Code + DeepSeek API approach
    • Makefile: Updated deepseek target to pull/build claude-code instead of deepseek
    • .env.example: Clarified that DEEPSEEK_API_KEY is passed as ANTHROPIC_AUTH_TOKEN
    • .github/workflows/build-and-push.yml: Removed deepseek image from CI/CD matrix

Implementation Details

The change follows the official DeepSeek guide for running Claude Code against DeepSeek's Anthropic-compatible API endpoint. The deepseek profile now launches the same Claude Code container with environment variables that redirect API calls to DeepSeek while preserving all MCP server wiring, skills discovery, and CLI behavior. No separate entrypoint or image build is required.

https://claude.ai/code/session_01KYWiyp7SH4bWd31DT7tZgn

The deepseek profile now reuses crhacky/blhackbox:claude-code and points
Claude Code at DeepSeek's Anthropic-compatible API via ANTHROPIC_BASE_URL +
ANTHROPIC_AUTH_TOKEN (per the official DeepSeek guide), instead of building a
separate Reasonix-based image.

This removes a whole image, Dockerfile, entrypoint, and third-party agent
dependency. The DeepSeek experience now runs the same MCP wiring, skills, and
entrypoint as Claude Code — only the model backend differs.

- docker-compose.yml: deepseek service reuses the claude-code image with
  DeepSeek env vars (base URL, auth token from DEEPSEEK_API_KEY, model mapping,
  disable non-essential traffic); mounts skills like claude-code.
- Delete docker/deepseek.Dockerfile and docker/deepseek-entrypoint.sh.
- Makefile / CI: drop the deepseek image build + push; make deepseek pulls the
  claude-code image.
- Tests: assert the deepseek profile reuses the claude-code image and routes at
  the DeepSeek endpoint; assert the removed files are gone.
- Docs (README, DOCKER.md, .env.example, setup.sh): describe the new approach;
  skills now work under DeepSeek.

https://claude.ai/code/session_01KYWiyp7SH4bWd31DT7tZgn
@valITino valITino merged commit 3a9462d into main Jun 12, 2026
2 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