Skip to content

Conversation

@mjun0812
Copy link
Owner

This pull request updates the development container configuration to improve environment setup and user experience. The main changes include refining environment variable settings, expanding installed packages, adjusting mount points for configuration sharing, and specifying user settings for the container.

Environment configuration improvements:

  • Changed the UV_CACHE_DIR environment variable to use a fixed path (/home/vscode/.cache/uv) and added a new CLAUDE_CONFIG_DIR variable for Claude configuration (/home/vscode/.claude).

Package and feature enhancements:

  • Added fd-find to the list of installed packages for enhanced file searching capabilities.

User and mount point adjustments:

  • Set both remoteUser and containerUser to vscode to ensure consistent user context within the container.
  • Updated mount points to bind local .claude and .codex configuration directories and files into the container, and added a shell history volume for improved developer workflow.

Post-creation and startup commands:

  • Enhanced the postCreateCommand to install the latest version of @openai/codex globally via npm before syncing Python dependencies.

Copilot AI review requested due to automatic review settings September 26, 2025 16:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request enhances the development container configuration to integrate Codex support and improve the developer experience. The changes streamline environment setup by adding Codex-related configurations, package installations, and mount points for better configuration sharing.

  • Added Codex integration with global npm installation and configuration directory mounting
  • Enhanced environment configuration with fixed cache paths and Claude config directory
  • Improved developer workflow with additional packages and shell history persistence

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mjun0812 mjun0812 merged commit ee8c422 into main Sep 26, 2025
5 checks passed
@mjun0812 mjun0812 deleted the dev/mjun branch September 26, 2025 17:15
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.

2 participants