Skip to content

Improve devcontainer mounts and add OAuth token support#22

Merged
key merged 1 commit intomainfrom
chore/improve-devcontainer-mounts
Feb 10, 2026
Merged

Improve devcontainer mounts and add OAuth token support#22
key merged 1 commit intomainfrom
chore/improve-devcontainer-mounts

Conversation

@key
Copy link
Member

@key key commented Feb 10, 2026

Summary

  • Replace host-specific bind mounts for .claude directory with a volume mount for better portability
  • Remove .claude.json bind mount (no longer needed)
  • Add containerEnv section to pass through CLAUDE_CODE_OAUTH_TOKEN environment variable

Benefits

  • Portability: Volume mounts work across different host systems and CI environments
  • Security: OAuth token is properly passed through environment variables
  • Simplicity: Removes unnecessary bind mount configuration

Test plan

  • Rebuild devcontainer and verify Claude Code authentication works
  • Verify .claude directory persists across container rebuilds
  • Confirm no breaking changes to existing workflows

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated development container configuration to streamline mount setup and improve environment variable handling for local development workflows.

- Replace host-specific bind mounts for .claude directory with volume mount
- Remove .claude.json bind mount (no longer needed)
- Add containerEnv section to pass through CLAUDE_CODE_OAUTH_TOKEN

This improves portability and follows devcontainer best practices.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

Walkthrough

The dev container configuration was updated to replace two host bind mounts for Claude configuration with a single named volume and to expose a Claude OAuth token environment variable into the container.

Changes

Cohort / File(s) Summary
Dev Container Configuration
.devcontainer/devcontainer.json
Replaced two host bind mounts (${localEnv:HOME}/.claude and ${localEnv:HOME}/.claude.json) with a single named volume mount. Added containerEnv section to expose CLAUDE_CODE_OAUTH_TOKEN from host to container environment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A volume takes the place of two,
Named mounts make mounting cleaner too,
Claude's token flows through container walls,
Dev setup answers duty's calls!


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@key key merged commit 5910cad into main Feb 10, 2026
6 checks passed
@key key deleted the chore/improve-devcontainer-mounts branch February 10, 2026 10:42
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