Add an `AGENTS.md` with shared rules for AI coding tools
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- Half a day
- Newbie friendliness
- 35/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- github, markdown
- Domain
- documentation
Research direction
Start by reading the draft in #950 and the two reference AGENTS.md files from pandas and scikit-learn. The target is a root AGENTS.md; reuse the linked docs under docs/developer_guide/ rather than duplicating them, and reconcile any existing .github/copilot-instructions.md. Done means the file exists, several developers have reviewed it, and the checklist in the issue is satisfied.
Written by the indexing model from the issue text.
Description
Add an AGENTS.md with shared rules for AI coding tools
@genevievestarke already has a draft up in #950 for review!
Developers who use AI tools each configure them differently, so generated code varies in style and quality. Common complaints from the workshop:
- Code that does not follow H2I or OpenMDAO best practices
- Excess code, such as too many
try/exceptblocks, overly long names, and roundabout logic - Unrequested changes to unrelated files and large changes made without human collaboration or review
- Agents lacking an overall view of H2I, so they duplicate existing utilities or break conventions
Proposed solution
Add a root-level AGENTS.md developed collaboratively by H2I developers so everyone's concerns are included.
Use the pandas and scikit-learn files as references:
- https://github.com/pandas-dev/pandas/blob/main/AGENTS.md
- https://github.com/scikit-learn/scikit-learn/blob/main/AGENTS.md
Suggested contents:
- Expectations for humans using agents
- Humans write issues, PR bodies, and review comments
- Disclose AI usage in the PR (see the PR template issue)
- Scope limits for agents
- Agents run
pre-commitand the docs build where appropriate during dev - Make the smallest change that solves the request; no unrelated file edits
- Do not add speculative abstractions (goldplating), broad
try/except, or extra comments - Stop and ask before large or cross-cutting changes
- Agents run
- H2I conventions
- Link to
docs/developer_guide/coding_guidelines.md,class_structure.md,testing.md, andadding_a_new_technology.md - Model registration, config class naming, control signal naming, and units in OpenMDAO variables
- OpenMDAO pitfalls agents commonly get wrong (collect these from the team)
- Link to
- Sensitive framework areas that need extra care and human collaboration (see the area-guardrails issue)
- Facts and numbers: agents can never invent values; cite a primary source.
- Security: no secrets, no sensitive data, no unreviewed network access or new dependencies.
How to know when we're done
-
AGENTS.mdexists at the repo root and is reviewed by several developers (including non-AI users) - Any individual dev's existing
AGENTS.mdfile or.github/copilot-instructions.mdis reconciled with it (single source of truth, or one points to the other) - The file is kept short and links to docs instead of duplicating them
- Dominant language
- Python
- Stars
- 30
- Forks
- 44
- Avg merge
- 3d 12h
- Merged PRs (30d)
- 28
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from NatLabRockies/H2Integrate
-
infrastructure
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
NatLabRockies/H2Integrate#977 ·
Maintainers usually reply within 1 day
-
SimpleThermalNuclearReactorCostModel: variable O&M overstated 1000x (kWh treated as MWh)Possibly taken A pull request linked to this issue is open or already merged. Openbug finance
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
NatLabRockies/H2Integrate#968 ·
Maintainers usually reply within 1 day
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
NatLabRockies/H2Integrate#944 ·
Maintainers usually reply within 1 day
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
NatLabRockies/H2Integrate#894 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
NatLabRockies/H2Integrate#623 ·
Maintainers usually reply within 1 day
All issues in NatLabRockies/H2Integrate
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
BCDA-APS/polar-bits#93 ·
-
infer_audio_type only recognises one MPEG Layer III frame header, so most MP3 files are rejectedPossibly taken @Steve-20-ai claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
[BUG] - ssh_config `Match tagged` statement always appliesPossibly taken @AtharvRG claimed this today. OpenBug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
paramiko/paramiko#2715 · 1 comment ·
Maintainers usually reply within 9 days
-
auto-rejected intake mcp-intake needs-salvage priority:high type:feature
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Ikalus1988/MisakaNet#4221 · 2 comments ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day