Skip to content

DYN-10239: Add AI agent skills, rules, and IDE configuration#16952

Merged
QilongTang merged 10 commits into
DynamoDS:masterfrom
kalunkuo:DYN-10239
Mar 10, 2026
Merged

DYN-10239: Add AI agent skills, rules, and IDE configuration#16952
QilongTang merged 10 commits into
DynamoDS:masterfrom
kalunkuo:DYN-10239

Conversation

@kalunkuo
Copy link
Copy Markdown
Contributor

@kalunkuo kalunkuo commented Mar 9, 2026

Purpose

Add shared AI agent guidance to the Dynamo repository so that AI-assisted development tools can produce higher-quality, repo-consistent output.

Key changes:

  • New .ai/agents/ directory for all AI guidance
  • 4 skills in .ai/agents/skills/:
    • dynamo-dotnet-expert -- C#/.NET coding patterns, PublicAPI management, NUnit testing, security
    • dynamo-onboarding -- architecture overview, key files, debugging approaches, related projects
    • dynamo-pr-description -- PR description writer matching the Dynamo template
    • dynamo-jira-ticket -- structured Jira ticket creation (includes template.md)
  • 1 rule in .ai/agents/rules/:
    • dynamo-core-rules.md -- coding standards, testing, PublicAPI, security, commit/PR conventions
  • New CLAUDE.md at repo root for Claude Code guidance, pointing to .ai/agents/
  • Updated .github/copilot-instructions.md with a new "Agent Skills and Templates" section referencing .ai/agents/
  • .ai/agents/README.md documenting the folder structure, skill index, and how the files relate to tool-specific mirrors
    No source code, tests, or public API changes are included.

running the skills in claude code: @.ai/agents/skills/dynamo-onboarding/
Screenshot 2026-03-09 at 5 14 32 PM

running the skills in cursor using @.ai/agents/skills/dynamo-onboarding/
Screenshot 2026-03-09 at 5 33 21 PM

Declarations

Check these if you believe they are true

Release Notes

Add AI agent skills, rules, and IDE configuration for Dynamo development.

Reviewers

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

@kalunkuo kalunkuo requested review from a team, QilongTang and Copilot March 9, 2026 22:07
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10239

Copy link
Copy Markdown
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 shared AI agent guidance to the Dynamo repository to improve the quality and consistency of AI-assisted development. It introduces a canonical .ai/agents/ directory containing skills, rules, and templates, along with a CLAUDE.md at the repo root for Claude Code guidance and updates to .github/copilot-instructions.md.

Changes:

  • New .ai/agents/ directory with 4 skills (dynamo-dotnet-expert, dynamo-onboarding, dynamo-pr-description, dynamo-jira-ticket), 1 rule file (dynamo-core-rules.md), and a README.md documenting the structure.
  • New CLAUDE.md at the repo root providing Claude Code with project context and pointers to the .ai/agents/ directory.
  • Updated .github/copilot-instructions.md with an "Agent Skills and Templates" section referencing the new .ai/agents/ directory.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.ai/agents/README.md Canonical index of all agent skills, rules, and templates; documents folder structure and tool-specific mirrors
.ai/agents/skills/dynamo-dotnet-expert/SKILL.md Comprehensive C#/.NET skill covering code design, testing, PublicAPI management, and security
.ai/agents/skills/dynamo-onboarding/SKILL.md Architecture briefing skill covering Dynamo structure, key files, debugging, and related projects
.ai/agents/skills/dynamo-pr-description/SKILL.md Skill for generating PR descriptions matching the Dynamo template
.ai/agents/skills/dynamo-jira-ticket/SKILL.md Skill for creating structured Jira tickets from bug reports or feature requests
.ai/agents/skills/dynamo-jira-ticket/template.md Copy/paste Jira ticket template
.ai/agents/rules/dynamo-core-rules.md Short guardrail rules for C#/.NET, testing, PublicAPI, and security
CLAUDE.md New root-level guidance file for Claude Code
.github/copilot-instructions.md Adds "Agent Skills and Templates" section pointing to the .ai/agents/ directory

Comment thread .ai/agents/README.md Outdated
Comment thread .ai/agents/skills/dynamo-onboarding/SKILL.md Outdated
Comment thread CLAUDE.md Outdated
Comment thread .github/copilot-instructions.md Outdated
Comment thread .ai/agents/skills/dynamo-onboarding/SKILL.md Outdated
@@ -0,0 +1,45 @@
# Dynamo Core Rules
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is a equivalent thing like rules for Cursor in Copilot world, I think it's https://github.com/DynamoDS/Dynamo/blob/master/.github/copilot-instructions.md. Feel free to validate and we may need some thing to keep them in sync. Not sure if there are some other online best practices to do the syncing part

Comment thread .agents/skills/dynamo-jira-ticket/SKILL.md Outdated
Comment thread .github/agents/Dynamo Dotnet Expert.md Outdated
Copy link
Copy Markdown
Contributor

@QilongTang QilongTang left a comment

Choose a reason for hiding this comment

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

A couple comments then LGTM so that we can test

@sonarqubecloud
Copy link
Copy Markdown

@QilongTang QilongTang merged commit 2a18e87 into DynamoDS:master Mar 10, 2026
26 of 29 checks passed
@@ -0,0 +1,129 @@
---
name: dynamo-jira-ticket
Copy link
Copy Markdown
Contributor

@chubakueno chubakueno Mar 12, 2026

Choose a reason for hiding this comment

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

My only comment here would be that this is the public repo but outside devs (non autodeskers) cannot write jira tickets. We could perharps suggest to the user that if he is not an insider he can turn this into a github issue instead?

Anyways it will be 99%+ used by autodeskers so maybe not worth the hassle, but just found it a bit weird to have this on the public repo.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That is a very good point! We should either rename it or move it to a different location.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants