-
Notifications
You must be signed in to change notification settings - Fork 212
Add comprehensive AGENTS.md documentation for coding agent onboarding #4
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
Add comprehensive AGENTS.md documentation for coding agent onboarding #4
Conversation
Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
AGENTS.md
Outdated
| In GitHub Codespaces, `GITHUB_TOKEN` is already set, so **no .env file is needed** - scripts will work immediately. | ||
|
|
||
| For local development: | ||
| 1. Create a GitHub Personal Access Token (no scopes needed): https://github.com/settings/tokens |
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.
Agents cant create a GITHUB PAT, but sometimes agent work in environment where theyre available. They can check to see if GITHUB_TOKEN is an env var.
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.
Updated to include agent-facing check for GITHUB_TOKEN environment variable. Commit 35c3d46
AGENTS.md
Outdated
|
|
||
| #### Option 2: Azure OpenAI (requires Azure resources and costs) | ||
|
|
||
| 1. Install Azure Developer CLI: https://aka.ms/install-azd |
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.
Agents cant do this themselves. They could check to see if the AZURE env variables already exist in the environment however.
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.
Updated to include agent-facing check for AZURE_OPENAI_ENDPOINT and AZURE_OPENAI_CHAT_DEPLOYMENT environment variables. Commit 35c3d46
AGENTS.md
Outdated
|
|
||
| ```bash | ||
| cp .env.sample.openai .env | ||
| # Edit .env and set: |
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.
Ditto, all agents can do is check to see if OPENAI_API_KEY env var exists in environment, they cant make it themselves.
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.
Updated to include agent-facing check for OPENAI_API_KEY environment variable. Commit 35c3d46
AGENTS.md
Outdated
|
|
||
| #### Option 4: Ollama (requires local Ollama installation) | ||
|
|
||
| 1. Install Ollama: https://ollama.com/ |
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.
Agents could check to see if ollama is running by running "ollama" and "ollama models list"
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.
Updated to include agent-facing check using "command -v ollama" and "ollama list" to verify if Ollama is installed and running. Commit 35c3d46
AGENTS.md
Outdated
|
|
||
| **Note:** Most scripts are demonstration scripts, not unit-tested. Changes to scripts should be manually verified by running them. | ||
|
|
||
| ## Upgrading Python Dependencies |
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.
This repo does not actually compile dependencies, so this sectio can be entirely removed.
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.
Removed the entire "Upgrading Python Dependencies" section. Commit 35c3d46
pamelafox
left a comment
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.
Please reword the GitHub Models/Azure/OpenAI.com sections to be more agent-facing, per my comments
|
@copilot Address my feedback |
…endency upgrade section Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Overview
This PR adds a comprehensive
AGENTS.mdfile to help coding agents work more efficiently with this repository. The documentation significantly reduces the time needed for agents to understand the codebase, set up their environment, and make changes correctly.What's Included
The
AGENTS.mdfile provides:1. Repository Overview
Clear description of the repository's purpose: a collection of Python scripts demonstrating OpenAI API usage with multiple LLM providers (GitHub Models, Azure OpenAI, OpenAI.com, and Ollama).
2. Code Layout & Architecture
Complete inventory of important files with descriptions:
3. Environment Setup Instructions
Agent-facing guidance for:
GITHUB_TOKENenvironment variableAZURE_OPENAI_ENDPOINTandAZURE_OPENAI_CHAT_DEPLOYMENTOPENAI_API_KEYcommand -v ollamaandollama list4. Testing & Validation
Commands for:
rufffor lintingblackfor code formatting5. Important Notes & Troubleshooting
Documentation of:
6. Agent Workflow Recommendations
Best practices for making code changes:
Benefits
This documentation will help coding agents:
Validation
All commands and instructions in AGENTS.md have been tested and verified to work correctly:
Fixes #3
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.