Skip to content

feature: Enhance devcontainer for multi-language support and testing#68

Draft
chefgs wants to merge 3 commits intomainfrom
fix/dev-containers
Draft

feature: Enhance devcontainer for multi-language support and testing#68
chefgs wants to merge 3 commits intomainfrom
fix/dev-containers

Conversation

@chefgs
Copy link
Copy Markdown
Member

@chefgs chefgs commented Apr 16, 2026

This pull request retires the repo-local devcontainer stack and transitions to a new, template-driven approach for generating devcontainer files. The legacy implementation has been archived, and related scripts and documentation have been removed or updated to reflect this change.

Devcontainer Stack Retirement and Migration:

  • .devcontainer/README.md: Updated to announce the retirement of the checked-in devcontainer stack, document the new template-based generation commands, and list the active source and legacy files. All legacy setup, configuration, and troubleshooting instructions have been removed.
  • .devcontainer/configure.py: Removed the configuration script that generated .devcontainer/devcontainer.json based on .devcontainer/devcontainer.env.json.
  • .devcontainer/devcontainer.json: Removed the previously checked-in, static devcontainer configuration file.

Legacy Implementation Archiving:

  • .devcontainer/Dockerfile.legacy/devcontainer/Dockerfile: Moved the devcontainer Dockerfile to a legacy directory. Several language and tool installation sections (for Python, Java, JavaScript, Go, ESLint, Pylint) have been removed, and only unsupported language/toolchain installers remain. [1] [2] [3] [4] [5]

These changes fully deprecate the old, static devcontainer setup in favor of a dynamic, template-based approach, and clean up legacy files and documentation accordingly.## Description

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 📚 Documentation update
  • 🔧 Refactor / code cleanup

Changes Made

  • Explained above.

Testing Done

  • Existing tests pass (python -m pytest cli/test_cli.py mcp_server/test_server.py tests/test_comprehensive.py -v)
  • New tests added to cover the changes
  • Manually tested the feature / fix

Checklist

  • My code follows the existing style and conventions of the project
  • I have self-reviewed my code
  • I have added or updated relevant documentation
  • My changes do not introduce new warnings or errors
  • All CI checks are passing

chefgs added 3 commits April 15, 2026 18:54
- Updated `.devcontainer/devcontainer.env.json` to enable additional languages: Java, Node.js, Ruby, C#, PHP, Rust, Kotlin, C, C++, JavaScript, and Go.
- Modified `.devcontainer/devcontainer.json` to install the newly supported languages and tools, including their respective versions and features.
- Added a new `bootstrap-toolbox.sh` script to install language-specific tools and link the Kubernetes generator.
- Enhanced the `init` command in `cli/devopsos.py` to preserve existing `.devcontainer/` configurations and write generated output to `.devcontainer.generated/`.
- Updated tests in `cli/test_cli.py` to verify the preservation of existing configurations and the correct generation of new files.
- Revised documentation to reflect changes in the devcontainer setup and CLI command behavior.
- Introduced `devcontainer_templates.py` for generating devcontainer configurations using templates.
- Added Dockerfile template for a multi-language development environment.
- Created JSON templates for `devcontainer.env.json` and `devcontainer.json` to facilitate environment variable and configuration management.
- Implemented functions for building environment configurations, features, arguments, extensions, and post-create commands.
- Enhanced support for various programming languages, CI/CD tools, Kubernetes utilities, build tools, code analysis tools, and DevOps tools.
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