feat(docker): add system dependencies for Claude Code support#241
Closed
mg-dev25 wants to merge 7 commits intosuperagent-ai:mainfrom
Closed
feat(docker): add system dependencies for Claude Code support#241mg-dev25 wants to merge 7 commits intosuperagent-ai:mainfrom
mg-dev25 wants to merge 7 commits intosuperagent-ai:mainfrom
Conversation
…ctories Add procps package to both Claude and CLI Docker containers to provide process management utilities like ps, top, and kill. Also ensure PATH environment variable includes system directories (/usr/local/bin, /usr/bin, /bin) so that process utilities and other system commands can be found and executed properly within containers. This resolves issues where process monitoring commands fail due to missing utilities or incorrect PATH configuration.
…upport - Add neovim: Required by Claude Code for editing - Add Node.js LTS: Required for MCP servers to run - Create /root/.claude directory: Needed for Claude configuration files These system dependencies complement the procps package already added, ensuring Claude Code has all required tools to function properly.
|
@mg-dev25 is attempting to deploy a commit to the Superagent Team on Vercel. A member of the Team first needs to authorize it. |
Install ccusage CLI tool to enable Claude Code usage statistics and model detection in the sandbox environment. This resolves the "unknown" model display issue in the status bar. Tries multiple installation methods (bun, npm) with graceful fallback.
- Add tini as PID 1 for proper subprocess and signal handling - Install jq for JSON processing in container - Switch ccusage to npm install for better compatibility - Add VIBEKIT_SANDBOX_ACTIVE=1 environment variable to identify sandbox - Clean up whitespace in claude-auth-helper.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds system dependencies and process management infrastructure needed for Claude Code to work properly inside the Docker sandbox. Without these packages and configurations, Claude Code crashes or outputs errors that break the workflow.
Commit 1: procps package and PATH configuration
procpspackage - providespscommand that Claude Code uses for diagnosticsCommit 2: Node.js and neovim
Commit 3: EDITOR environment variable
EDITOR=nvimfor proper editor detection in Claude CodeCommit 4: ccusage installation
Commit 5: Merge latest main
Commit 6: Container process management and tooling
SIGTERMreaches all processes for clean container shutdownWhy these belong in the Dockerfile
These are system-level dependencies and infrastructure that need to be baked into the Docker image:
Testing
Tested with Claude Code inside the sandbox:
Related Issue
Fixes #240
Checklist