-
Notifications
You must be signed in to change notification settings - Fork 36
Add markdown formatting guidelines to create-agentic-workflow template #10580
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
pelikhan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot make sure prompt it compiled in lock files with others
pkg/workflow/prompts/markdown.md
Outdated
| @@ -0,0 +1,363 @@ | |||
| # Markdown Formatting Guidelines | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot wrap in markdown xmk tags
Make it super concise, do not provide structural guidelines beyond heading level
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 8d99c69. Wrapped in <markdown-guidelines> XML tags and made super concise (11 lines). Removed all structural guidelines beyond heading level - kept only: GFM, headers start at h3, checkboxes, progressive disclosure, and workflow run links.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
| **Progressive Disclosure**: Use `<details><summary><b>Bold Summary Text</b></summary>` to collapse long content. | ||
| **Workflow Run Links**: Format as `[§12345](https://github.com/owner/repo/actions/runs/12345)`. Include up to 3 most relevant runs under `**References:**` at the end. Do NOT add footer attribution (system adds automatically). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot remove references sentence
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 2cca6f1. Removed the "Include up to 3 most relevant runs under **References:** at the end." sentence from the workflow run links guideline. The line now only specifies the format and footer warning.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
Smoke Test: Claude✅ GitHub MCP | ❌ Serena (unavailable) | ✅ Playwright | ✅ Tavily | ✅ File Write | ✅ Bash Status: FAIL (1 tool unavailable)
|
|
GitHub MCP ✅ Fix broken documentation links to skill instruction files; Fix intermittent Copilot CLI schema validation bug and improve add_labels behavior
|
Smoke Test Results: PASS
|
|
@copilot migrate the markdown instructions to the create-agentic-workflows.md file and revert adding a new prompt file that's built in. Recompile all the workflows. |
…lt-in prompt Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in b074df4. The markdown guidelines have been migrated to |
| - The workflow has been created and compiled successfully. | ||
| - Commit and push the changes to activate it. | ||
|
|
||
| ## Markdown Formatting Guidelines |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot so, we will move this section under the guidelines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 765909c. Moved the markdown formatting guidelines from a separate section to a bullet point under the "Guidelines" section.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Add Markdown Formatting Guidelines to Workflow Creation Template ✅
Completed Tasks
.github/aw/create-agentic-workflow.mdtemplateSummary
Added markdown formatting guidelines to the
.github/aw/create-agentic-workflow.mdtemplate file as a bullet point under the "Guidelines" section. This provides guidance to the workflow creation agent without embedding the guidelines into every compiled workflow.Key Changes
1. Added Markdown Formatting Guidelines as Guideline Item
Added markdown formatting guidelines to
.github/aw/create-agentic-workflow.mdunder the "Guidelines" section:2. Recompiled All Workflows
Rationale
This approach is better than embedding guidelines in every compiled workflow because:
Testing Performed
✅ Binary built successfully
✅ All 129 workflows compiled successfully
✅ Verified markdown guidelines appear in create-agentic-workflow.md under Guidelines section
✅ Verified lock files no longer contain embedded guidelines
✅ Code formatting passed (
make fmt)Files Changed
.github/aw/create-agentic-workflow.md- Added markdown formatting guidelines to Guidelines section.github/workflows/*.lock.yml- Recompiled (129 files) - no embedded guidelines.github/aw/campaign-generator.lock.yml- Recompiled - no embedded guidelinesOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.