Repository navigation
fix(devcontainer): theme, login, node_modules isolation, self-contained claude-code config - #58
Merged
Merged
Conversation
Theme wasn't applied because no colorTheme was set. Login was disrupted because init-plugins.sh ran claude CLI commands in postCreateCommand, which executes before extensions install — racing the extension's OAuth. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…Code setup
- Move inline devcontainer.json/docker-compose configs from README into
claude-code/examples/{default,sandbox}/ as actual files
- Add all VS Code extensions with detailed comments (OXC, Bun, Tailwind,
YAML, Docker, Markdown Preview) — OXC was referenced as formatter but
missing from extensions list
- Add monorepo node_modules volume mounts to both variants
- Switch sandbox to passwordless sudo (security comes from the firewall)
- Fix timezone from America/Los_Angeles to America/Edmonton
- Fix stale cross-reference to devcontainer-claude-bun
- Fix stale skill path (sandbox-fetch-docs.md → sandbox-fetch-docs/SKILL.md)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… root Configs belong next to the Dockerfile they reference — no need for a separate examples/ directory. mise.toml goes in the project root where mise expects it. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This was referenced Mar 28, 2026
Closed
gatezh
added a commit
that referenced
this pull request
Sep 23, 2026
…ommand The previous commit wired init-plugins.sh into the default variant's postCreateCommand (#107 option a). That is the configuration #58 removed: before #58 the repo-root default ran init-plugins.sh from postCreateCommand with waitFor already set, and claude CLI calls there raced the Claude Code extension's OAuth sign-in and corrupted auth state. Apply #107 option (b) to the default variant instead: drop the dangling waitFor (its default, updateContentCommand, is correct) and document running the script once after sign-in. The sandbox keeps the script in postCreateCommand, matching the repo-root sandbox, now guarded so a consumer without the optional script doesn't fail container creation.
This was referenced Sep 23, 2026
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
1 participant
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
init-plugins.shfrompostCreateCommand(it ranclaudeCLI commands before extensions installed), addCMD ["sleep", "infinity"].devcontainer/files — complete, batteries-included configs with all VS Code extensions (Claude Code, Bun, OXC, Tailwind, YAML, Docker, Markdown Preview), monorepo node_modules volume mounts, Claude Dark theme, and Edmonton timezone.devcontainer/files, fix stale cross-reference todevcontainer-claude-bun, fix stale skill pathTest plan
claude-code/.devcontainer/files to a test project — verify container starts, extensions install, node_modules stays off host🤖 Generated with Claude Code