-
Notifications
You must be signed in to change notification settings - Fork 4
feat: add Claude Code devcontainer feature #79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: jsburckhardt <18494471+jsburckhardt@users.noreply.github.com>
Co-authored-by: jsburckhardt <18494471+jsburckhardt@users.noreply.github.com>
|
@copilot have you seen the other READMEs in the other features - they all say "autogenerated" - why did you create the file? |
Co-authored-by: jsburckhardt <18494471+jsburckhardt@users.noreply.github.com>
You're absolutely right - I removed the manually created README.md file in commit 4e4d1f2. The README.md should be auto-generated from the devcontainer-feature.json, following the pattern used by all other features in the repository. |
|
@copilot claude gets install but it gets installed in the root path. you'll need to move it to the user path! |
Co-authored-by: jsburckhardt <18494471+jsburckhardt@users.noreply.github.com>
Fixed in commit c487129. The install script now moves the claude binary from root-specific paths (like /root/.local/bin) to /usr/local/bin to ensure it's accessible in the user path. The script checks common installation locations and relocates the binary as needed. |
|
@copilot run the tests or trigger the ci - claude is not working as expected. Please review the script being used. don't come back till you fix it |
Co-authored-by: jsburckhardt <18494471+jsburckhardt@users.noreply.github.com>
Fixed in commit cc9d736. The installation script now uses the official npm package The script:
Test output shows successful installation and verification. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Claude Code Feature Implementation Plan
Create README.md for the feature(removed - should be auto-generated)Summary
Successfully implemented Claude Code devcontainer feature following the pattern of existing features. The feature uses the official npm package
@anthropic-ai/claude-codefor installation.Files Created:
src/claude-code/devcontainer-feature.json- Feature metadatasrc/claude-code/install.sh- Installation script using npm with SSL fallbacktest/claude-code/test.sh- Basic feature testtest/_global/claude-code-specific-version.sh- Version-specific testFiles Modified:
README.md- Added Claude Code to features table and usage sectiontest/_global/all-tools.sh- Added validationtest/_global/scenarios.json- Added test scenarios.github/workflows/test.yaml- Added to CI/CD matrixInstallation Method:
@anthropic-ai/claude-code(official installation method)Security Summary:
✅ No security vulnerabilities detected by CodeQL
✅ No hardcoded secrets
✅ Uses official Anthropic npm package
Test Results:
✅ Feature test passed - claude command available and working (version 2.1.5)
Note: The README.md in src/claude-code/ will be auto-generated from devcontainer-feature.json as per repository conventions.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.