Commit ba5cdb8
feat: improve agent consistency and documentation (#116)
## Summary
- Aligns `tools` field format with official Claude Code documentation
(comma-separated instead of JSON array)
- Adds explicit script paths in plugin-validator.md for immediate
actionability
- Documents model choice rationale in agent-creator.md
## Problem
Fixes #113
## Solution
Applied all three improvements identified in the issue:
1. **tools format**: Changed from `["Tool1", "Tool2"]` to `Tool1, Tool2`
in all 3 agents to match official docs
2. **script paths**: Changed vague references like "Use the
validate-agent.sh utility from agent-development skill" to explicit
paths like `./skills/agent-development/scripts/validate-agent.sh`
3. **model rationale**: Added YAML comment explaining why agent-creator
uses explicit `sonnet` instead of `inherit`
### Alternatives Considered
- Inline comment for model rationale: Rejected because the validation
script parsed it as part of the model value
## Changes
- `plugins/plugin-dev/agents/agent-creator.md`: tools format, model
comment, example update
- `plugins/plugin-dev/agents/skill-reviewer.md`: tools format
- `plugins/plugin-dev/agents/plugin-validator.md`: tools format,
explicit script paths
## Testing
- [x] All files pass markdownlint
- [x] All 3 agents pass validate-agent.sh validation
- [x] Changes verified via git diff
---
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude <noreply@anthropic.com>1 parent fb7dbd1 commit ba5cdb8
3 files changed
Lines changed: 7 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| 32 | + | |
32 | 33 | | |
33 | 34 | | |
34 | | - | |
| 35 | + | |
35 | 36 | | |
36 | 37 | | |
37 | 38 | | |
| |||
116 | 117 | | |
117 | 118 | | |
118 | 119 | | |
119 | | - | |
| 120 | + | |
120 | 121 | | |
121 | 122 | | |
122 | 123 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| |||
106 | 106 | | |
107 | 107 | | |
108 | 108 | | |
109 | | - | |
| 109 | + | |
110 | 110 | | |
111 | 111 | | |
112 | 112 | | |
| |||
125 | 125 | | |
126 | 126 | | |
127 | 127 | | |
128 | | - | |
| 128 | + | |
129 | 129 | | |
130 | 130 | | |
131 | 131 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
| |||
0 commit comments