-
Notifications
You must be signed in to change notification settings - Fork 36
Centralize markdown instructions in dedicated markdown.md prompt file #10709
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
…t.md Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Instruct agents to use 6 backticks instead of 3 when generating markdown to avoid unbalanced code regions. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
All unit tests, integration tests, formatting, and linting passed successfully. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot review all custom prompts injected related to markdown and centralize in a single markdown.md file |
- Created new actions/setup/md/markdown.md with markdown generation instructions - Removed markdown-generation section from temp_folder_prompt.md - Added markdownPromptFile constant in pkg/workflow/sh.go - Updated collectPromptSections to include markdown prompt (always included) - Recompiled all workflows to include the new markdown.md prompt The markdown instructions are now centralized in a dedicated file instead of being mixed with temporary folder instructions. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Centralized all markdown-related prompt instructions into The new file contains:
Removed the markdown section from |
| cat "/opt/gh-aw/prompts/temp_folder_prompt.md" >> "$GH_AW_PROMPT" | ||
| cat "/opt/gh-aw/prompts/markdown.md" >> "$GH_AW_PROMPT" |
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 markdown.txt should have been concatenanted in the system prompt
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.
The markdown.md file is correctly being concatenated inside the <system> tags (lines 521-589 in the lock file). The structure is:
- Line 521:
<system>tag opens - Lines 523-524:
temp_folder_prompt.mdandmarkdown.mdare concatenated - Lines 525-587: Additional system prompt content
- Line 589:
</system>tag closes - Lines 591+: User prompt content begins
All built-in prompts (including markdown.md) are wrapped in <system> tags as intended by the unified prompt system in pkg/workflow/unified_prompt_step.go:434-438.
Agents generating markdown with embedded code blocks can create unbalanced regions where opening/closing delimiters desync, breaking rendering. This adds an instruction to use 6 backticks instead of 3 when generating markdown text.
Changes
actions/setup/md/markdown.md: Created new dedicated prompt file with markdown generation instructions and examplesactions/setup/md/temp_folder_prompt.md: Removed markdown-generation section (keeping only temporary folder and file editing instructions)pkg/workflow/sh.go: AddedmarkdownPromptFileconstant for the new prompt filepkg/workflow/unified_prompt_step.go: UpdatedcollectPromptSections()to include markdown prompt as section Add workflow: githubnext/agentics/weekly-research #2 (always included, right after temp folder instructions)The instruction is injected into all workflow prompts at runtime via the unified prompt system. All markdown-related instructions are now centralized in a single file for better maintainability.
Markdown Prompt Content