feat(cli): add devcontainer integration support #49
Conversation
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
Current behavior: N/A - new documentation New behavior: Added a design document exploring approaches for integrating devcontainer support into Headjack. The document covers three integration approaches: 1. Wrapping the official @devcontainers/cli (recommended) 2. Parsing devcontainer.json directly in Go 3. Hybrid approach using CLI for builds, native runtime for lifecycle The document recommends Option 1 (CLI wrapper) due to full spec compliance, automatic feature support, and reduced maintenance burden.
Current behavior: Initial design document proposed multiple integration approaches with separate code paths for devcontainer vs vanilla image handling. New behavior: Revised design to use a decorator pattern where DevcontainerRuntime wraps an underlying Docker/Podman runtime and implements the same containerRuntime interface. Key changes: - No changes needed to Instance Manager - DevcontainerRuntime delegates lifecycle ops to underlying runtime - Run() calls devcontainer up, Exec() calls devcontainer exec - Auto-detection: devcontainer.json used unless --image flag passed - Added WorkspaceFolder field to RunConfig - Documented Apple Containerization limitation
Current behavior: Headjack only supported vanilla OCI images for container creation, requiring users to manually configure container environments. New behavior: Added devcontainer support that automatically detects and uses .devcontainer/devcontainer.json when present. Key changes: - Added DevcontainerRuntime that wraps underlying Docker/Podman runtime - DevcontainerRuntime uses devcontainer CLI for Run/Exec operations - Lifecycle operations (Stop/Start/Remove) delegate to underlying runtime - Auto-detection: devcontainer mode used unless --base flag is passed - Added WorkspaceFolder field to container.RunConfig - Added RemoteUser/RemoteWorkdir fields to catalog.Entry - Extended Manager with Runtime/Executor accessors for runtime composition - Apple Containerization not supported (warning printed, falls back to vanilla) Closes: #49
Current behavior: Code had linter errors: unchecked type assertion in manager.go, overly permissive file permissions in tests, and unnecessary fmt.Errorf usage. New behavior: - Added explicit ok check for type assertion in NewManager - Changed test file permissions to 0o750/0o600 with new-style octal literals - Replaced fmt.Errorf with errors.New where no formatting needed 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
headjack | 6cdd150 | Commit Preview URL Branch Preview URL |
Jan 04 2026, 09:49 PM |
Current behavior: N/A - new feature New behavior: Added devcontainer configuration for consistent development environments. Includes Go 1.23, Node.js 22, Docker-in-Docker, golangci-lint, just, hadolint, and GitHub CLI. Post-create script installs dependencies and verifies tooling. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.
Summary
--devcontainerflag for explicit devcontainer modeTest plan
just checkto verify all lints and tests pass.devcontainer/devcontainer.json--devcontainerflag usage🤖 Generated with Claude Code