Improve Skill syncing process in Dynamo#16977
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR canonicalizes Dynamo “agent skills” under .agents/skills/ and wires up wrapper regeneration so tool-facing agent files in .github/agents/ stay in sync.
Changes:
- Added new canonical skills (and supporting assets/pattern catalog) under
.agents/skills/. - Updated the wrapper sync script to generate/validate wrappers for the full canonical skill set.
- Regenerated multiple
.github/agents/*.agent.mdwrappers and updated docs to reflect the new locations.
Reviewed changes
Copilot reviewed 16 out of 37 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| AGENTS.md | Lists the expanded set of canonical skills. |
| .github/scripts/sync_agent_wrappers.ps1 | Adds new canonical skills to wrapper generation/checking. |
| .github/copilot-instructions.md | Updates skill list reference to include new skills. |
| .github/agents/dynamo-webview-component-scaffold.agent.md | Replaced full content with canonical-skill wrapper. |
| .github/agents/dynamo-ux-designer.agent.md | Replaced full content with canonical-skill wrapper. |
| .github/agents/dynamo-ecosystem-reviewer.agent.md | Replaced full content with canonical-skill wrapper. |
| .github/agents/dynamo-dotnet-janitor.agent.md | Replaced full content with canonical-skill wrapper. |
| .github/agents/dynamo-content-designer.agent.md | Replaced full content with canonical-skill wrapper. |
| .github/agents/dynamo-codebase-patterns.agent.md | Replaced full content with canonical-skill wrapper. |
| .agents/skills/dynamo-webview-component-scaffold/SKILL.md | Adds canonical skill content for scaffold workflow. |
| .agents/skills/dynamo-ux-designer/SKILL.md | Adds canonical UX designer skill content. |
| .agents/skills/dynamo-ecosystem-reviewer/SKILL.md | Adds canonical ecosystem review guidance. |
| .agents/skills/dynamo-dotnet-janitor/SKILL.md | Adds canonical janitor skill content. |
| .agents/skills/dynamo-content-designer/SKILL.md | Adds canonical content designer skill + references. |
| .agents/skills/dynamo-content-designer/assets/ui-content.md | Adds UI content writing guidance asset. |
| .agents/skills/dynamo-content-designer/assets/tutorials-user-guides.md | Adds tutorial/user guide templates asset. |
| .agents/skills/dynamo-content-designer/assets/release-notes.md | Adds release-notes + node-description guidance asset. |
| .agents/skills/dynamo-content-designer/assets/node-errors-warnings.md | Adds node warning/error messaging guidance asset. |
| .agents/skills/dynamo-content-designer/assets/node-descriptions.md | Adds node description + in-depth help guidance asset. |
| .agents/skills/dynamo-content-designer/assets/feature-documentation.md | Adds feature documentation writing guidance asset. |
| .agents/skills/dynamo-content-designer/assets/blog-posts.md | Adds blog-post structure and tone guidance asset. |
| .agents/skills/dynamo-codebase-patterns/SKILL.md | Adds canonical codebase-patterns skill definition. |
| .agents/skills/dynamo-codebase-patterns/patterns/README.md | Introduces pattern-catalog README. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-001-register-all-ports.md | Adds confirmed Dynamo node-port registration pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-002-raises-modification-events.md | Adds candidate batch-modification suppression pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-003-dynamowebview2-init-sequence.md | Adds confirmed WebView2 initialization pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-006-asynctask-two-phase-init.md | Adds confirmed AsyncTask state-capture pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-007-task-completion-scheduler-thread.md | Adds confirmed scheduler-thread completion handler pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-008-viewextension-lifecycle.md | Adds confirmed view-extension lifecycle pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-009-workspace-type-guard.md | Adds confirmed workspace type-guard pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-010-keepreference-ffi.md | Adds confirmed FFI KeepReference pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-011-arbitrary-dimension-array.md | Adds confirmed ArbitraryDimensionArrayImport pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-012-dispose-derived-geometry.md | Adds confirmed derived-geometry disposal pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-013-isvisibleindynamolibrary.md | Adds confirmed library visibility attribute pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-014-asynctask-canmergewith.md | Adds candidate AsyncTask merge/coalescing pattern. |
| .agents/skills/dynamo-codebase-patterns/patterns/dp-015-property-change-manager.md | Adds confirmed PropertyChangeManager suppression pattern. |
| .agents/README.md | Updates parity matrix, quick reference, and structure docs for new skills/wrappers. |
avidit
requested changes
Mar 23, 2026
Contributor
avidit
left a comment
avidit
left a comment
There was a problem hiding this comment.
can we move the dynamo-codebase-patterns' assets to assets folder?
Contributor
Author
Seems not recommended for now.. 
|
Contributor
Sounds good. I have updated the skill-validator script to add |
avidit
approved these changes
Mar 24, 2026
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
The unsynced skills under .github folder are now canonicalized under .agents/skills, plus wrapper regeneration wiring. So they (The Arch skills, content writing skill, UX design skill) can be synced and leveraged across VSCode/ Cursor / Claude Code. FYI: @hwahlstrom @mjkkirschner
Asset/pattern relocation (no-content-change moves)
patternsassetsDeclarations
Check these if you believe they are true
Release Notes
N/A
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