fix: clean up command templates (specify, analyze)#1810
fix: clean up command templates (specify, analyze)#1810pierluigilenoci wants to merge 1 commit intogithub:mainfrom
Conversation
- specify.md: fix self-referential step number (step 6c → proceed to step 7)
- specify.md: remove empty "General Guidelines" heading before "Quick Guidelines"
- analyze.md: deduplicate {ARGS} — already present in "User Input" section at top
dhilipkumars
left a comment
dhilipkumars
left a comment
There was a problem hiding this comment.
some comments from myside
cc: @mnriem
| c. **Handle Validation Results**: | ||
|
|
||
| - **If all items pass**: Mark checklist complete and proceed to step 6 | ||
| - **If all items pass**: Mark checklist complete and proceed to step 7 |
| - **Use examples over exhaustive rules** (cite specific instances, not generic patterns) | ||
| - **Report zero issues gracefully** (emit success report with coverage statistics) | ||
|
|
||
| ## Context |
There was a problem hiding this comment.
i would still leave the ## Context to keep the hierarchy and importance when LLM interprets this
There was a problem hiding this comment.
Pull request overview
Cleans up command prompt templates under templates/commands/ to remove minor inconsistencies/redundancy and improve instruction clarity for the specify and analyze commands.
Changes:
- Fixes an incorrect step reference in
specify.md(validation pass now proceeds to step 7, matching “Report completion”). - Removes an empty
## General Guidelinesheading fromspecify.md. - Removes a duplicated
{ARGS}block inanalyze.md, keeping context passed via## User Inputwhile still retaining a single{ARGS}reference.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| templates/commands/specify.md | Corrects step cross-reference and removes an empty heading for cleaner instructions. |
| templates/commands/analyze.md | Eliminates redundant {ARGS} section and replaces it with a single inline context line. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please address the feedback given by @dhilipkumars and Copilot. If not applicable, please explain why.
Summary
## General Guidelinesheading that has no content before## Quick Guidelines{ARGS}— the## Contextsection at the bottom repeats the same arguments already present in the## User Inputsection at the topTest plan
specify.mdstep 6c correctly references step 7specify.mdanalyze.mdstill passes{ARGS}context correctly via the## User Inputsection