Cleaning up the name of a lot of agents#651
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request aims to standardize agent naming by replacing the title field with a name field in agent and prompt frontmatter. The PR also updates the README generation scripts to use only the name field instead of title.
Changes:
- Replaces
title:withname:in 4 prompt files - Adds
name:field to 54 agent files - Removes
titlefield handling fromeng/update-readme.mjsandeng/generate-website-data.mjs - Updates
docs/README.agents.mdwith new agent names - Updates
AGENTS.mdguidelines to document thenamefield requirement
Reviewed changes
Copilot reviewed 54 out of 55 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| eng/update-readme.mjs | Removes title field handling, but incorrectly transforms name field values |
| eng/generate-website-data.mjs | Removes title field handling, but incorrectly transforms name field values |
| prompts/*.prompt.md | Correctly changes title to name field in 4 prompt files |
| agents/*.agent.md | Correctly adds name field to 54 agent files with proper human-readable formatting |
| docs/README.agents.md | Auto-generated file with updated agent names |
| AGENTS.md | Adds guideline documenting the name field requirement |
Comments suppressed due to low confidence (2)
eng/update-readme.mjs:137
- The script incorrectly transforms the
namefield by splitting on hyphens and capitalizing each word. Thenamefield should already contain the human-readable name in the correct format and should be used as-is without transformation.
For example, the agent file has name: 'AEM Front-End Specialist' but the script would transform this to "Aem Front End Specialist" (splitting on hyphens). The name field should be used directly without any transformation since it's already in the intended human-readable format.
eng/generate-website-data.mjs:51
- Same issue as in update-readme.mjs - the script incorrectly transforms the
namefield by splitting on hyphens and capitalizing each word. Thenamefield should already contain the human-readable name in the correct format and should be used as-is without transformation.
The transformation logic on lines 48-51 should be removed, and the name field should be returned directly: return frontmatter.name;
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.Description
Type of Contribution
Additional Notes
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.