Skip to content

feat: adding Agents.md creation as skill#24

Merged
rombert merged 5 commits intoadobe:betafrom
abhishekgarg18:dev_add_agent_creation_skill
Mar 19, 2026
Merged

feat: adding Agents.md creation as skill#24
rombert merged 5 commits intoadobe:betafrom
abhishekgarg18:dev_add_agent_creation_skill

Conversation

@abhishekgarg18
Copy link
Copy Markdown
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@abhishekgarg18 abhishekgarg18 changed the title feat: adding Agents.md creation as skill [WIP - DO NOT MERGE] feat: adding Agents.md creation as skill Mar 17, 2026
@abhishekgarg18 abhishekgarg18 changed the title [WIP - DO NOT MERGE] feat: adding Agents.md creation as skill feat: adding Agents.md creation as skill Mar 17, 2026
@abhishekgarg18
Copy link
Copy Markdown
Collaborator Author

Screenshot 2026-03-18 at 9 09 41 AM Screenshot 2026-03-18 at 9 10 13 AM

Copy link
Copy Markdown
Collaborator

@rombert rombert left a comment

Choose a reason for hiding this comment

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

LGTM but had a question regarding MCPs. I am not sure that we should reinforce usage of MCPs/skills in AGENTS.md , it's probably simpler to let the agent discover them.

If MCPs/skills are not picked up properly without an AGENTS.md we should figure out why (metadata?) and fix that.

- Include dispatcher validate command only if `dispatcher` module exists.
- For React/Angular SPA, note that `npm start` requires AEM running.

#### e. Dispatcher MCP section
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I am not sure if we should include Dispatcher MCP instructions. Coding assistants should be able to discover them directly based on the metadata.

@rombert rombert merged commit 35f26b2 into adobe:beta Mar 19, 2026
2 checks passed
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.

2 participants