Skip to content

Comments

Add AGENTS.md guidance#665

Open
adrian-rpf wants to merge 2 commits intomainfrom
add-agents-md
Open

Add AGENTS.md guidance#665
adrian-rpf wants to merge 2 commits intomainfrom
add-agents-md

Conversation

@adrian-rpf
Copy link
Contributor

Document repo setup, workflows, and guardrails for AI agents.

Status

  • First repo at RPF to set the standard of having a AGENTS.md file

Points for consideration:

  • No code impact

What's changed?

  • Adding AGENTS.md file which should be used by all AI agent tools.

Steps to perform after deploying to production

  • Will never get to production.

Document repo setup, workflows, and guardrails for AI agents.
Copilot AI review requested due to automatic review settings January 29, 2026 17:58
@cla-bot cla-bot bot added the cla-signed label Jan 29, 2026
Copy link
Contributor

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 PR adds AGENTS.md, a comprehensive documentation file designed to guide AI agents (like GitHub Copilot) working with this repository. The document provides a structured overview of the codebase, setup instructions, workflows, and guardrails to help AI tools understand the project context.

Changes:

  • Added AGENTS.md with sections covering project overview, repository structure, quickstart commands, development workflows, testing/CI, code style conventions, security guardrails, common tasks, and references to further reading

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Seeds (dev): `docker compose run --rm api rails projects:create_all` and
`docker compose run --rm api rails for_education:seed_a_school_with_lessons_and_students`
(others in README).
- DB sync (needs Heroku access): `./bin/db-sync/production-to-local.sh` or `staging-to-local.sh`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we be encouraging agents to run this?

## Repository Structure
- `app/` Rails application code (REST, GraphQL, jobs, views, admin).
- `config/` environment, initializers, Puma, CORS, credentials.
- `db/` migrations, seeds, schema helpers; `bin/db-sync/` for pulling Heroku data locally.
Copy link
Contributor

Choose a reason for hiding this comment

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

(same as below) - should be be encouraging agents to run bin/db-sync/?

Copy link
Contributor

@zetter-rpf zetter-rpf left a comment

Choose a reason for hiding this comment

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

Approved - looks like a useful base for us to start with and add things to.

Added a couple of optional comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants