-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Codespace cautious journey wxr777p742gwrr #1867
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
Codespace cautious journey wxr777p742gwrr #1867
Conversation
modified: examples/debate.py modified: examples/hello_world.py modified: setup.py
There was a problem hiding this 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 PR appears to be creating a development environment setup with configuration files. Based on the title "Codespace cautious journey", this seems to be setting up a codespace or development container environment.
- Adds LLM configuration for Claude API integration
- Updates development container settings for spell checking
Reviewed Changes
Copilot reviewed 2 out of 6 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| metagpt/config2.yaml | Adds Claude LLM configuration with API credentials |
| .devcontainer/devcontainer.json | Updates VS Code settings to include spell checker words |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| api_type: "claude" # or azure / ollama / groq etc. Check LLMType for more options | ||
| model: "claude-3-5-sonnet-20240620" | ||
| base_url: "https://api.anthropic.com/v1/messages" # or forward url / other llm url | ||
| api_key: "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" |
Copilot
AI
Sep 2, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
API key is hardcoded and exposed in the configuration file. This should be moved to environment variables or a secure secrets management system to prevent credential exposure.
| api_key: "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" | |
| api_key: "${ANTHROPIC_API_KEY}" # Set this environment variable with your API key |
| "vscode": { | ||
| "settings": {}, | ||
| "settings": { | ||
| "cSpell.words": ["metagpt", "metagpt"] |
Copilot
AI
Sep 2, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The word 'metagpt' is duplicated in the spell checker words array. Remove the duplicate entry.
| "cSpell.words": ["metagpt", "metagpt"] | |
| "cSpell.words": ["metagpt"] |
Features
Feature Docs
Influence
Result
Other