Skip to content

refactor: update devcontainer to use template's pre-built image#144

Merged
fx merged 4 commits intomainfrom
feat/template-update-20250729
Jul 29, 2025
Merged

refactor: update devcontainer to use template's pre-built image#144
fx merged 4 commits intomainfrom
feat/template-update-20250729

Conversation

@fx
Copy link
Copy Markdown
Owner

@fx fx commented Jul 29, 2025

Summary

  • Switched from Microsoft's base devcontainer image to fx/docker/devcontainer:latest pre-built image
  • Replaced simple install-deps.sh with template's comprehensive post-start scripts
  • Removed unnecessary customizations that are already in the base image

Changes Made

  • Updated: .devcontainer/devcontainer.json to use template image
  • Added: .devcontainer/post-start-wrapper.sh for execution logging
  • Added: .devcontainer/post-start.sh with mise, claude-code, and npm setup
  • Removed: .devcontainer/install-deps.sh (functionality moved to post-start scripts)
  • Removed: .coder.yaml (no longer in template)
  • Removed: custom-setup.sh (packages already in base image)
  • Removed: Fake Claude settings configuration

Testing Checklist

  • Devcontainer builds successfully
  • Post-start scripts execute without errors
  • Required tools available: mise ✓, claude-code ✓, npm ✓
  • Project dependencies install correctly

Note: The devcontainer now follows the template pattern cleanly without unnecessary customizations.

fx added 4 commits July 29, 2025 00:12
- Switch from Microsoft base image to fx/docker/devcontainer:latest
- Replace install-deps.sh with template's post-start scripts
- Extract custom installations to custom-setup.sh
- Add .coder.yaml for Coder workspace support
- Enable Docker socket mounting for container operations
@fx fx merged commit 57cda77 into main Jul 29, 2025
1 of 2 checks passed
@fx fx deleted the feat/template-update-20250729 branch July 29, 2025 01:59
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.

1 participant