Prompt formatting/wording changes#210
Open
aditya-arcot wants to merge 4 commits intoKeygraphHQ:mainfrom
Open
Conversation
There was a problem hiding this comment.
Pull request overview
Restores and normalizes prompt formatting/wording changes (primarily whitespace/indentation cleanup) that were previously present in #204 and lost after a force-push, improving consistency across vulnerability analysis/exploitation prompt templates.
Changes:
- Normalized Markdown whitespace/indentation across multiple prompt files.
- Fixed/standardized several headings and list items in vulnerability prompt templates.
- Adjusted some instructional wording/formatting in injection prompts and shared snippets.
Reviewed changes
Copilot reviewed 1 out of 17 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| prompts/vuln-xss.txt | Whitespace/indentation normalization in XSS analysis template sections. |
| prompts/vuln-ssrf.txt | Whitespace cleanup in SSRF analysis methodology formatting. |
| prompts/vuln-injection.txt | Formatting/wording updates to injection analysis template; includes a changed queue instruction block. |
| prompts/vuln-authz.txt | Whitespace cleanup in authz analysis template. |
| prompts/shared/login-instructions.txt | Whitespace cleanup and line alignment in shared login instructions. |
| prompts/shared/_vuln-scope.txt | Removed leading indentation for consistent include rendering. |
| prompts/shared/_target.txt | Removed leading indentation for consistent include rendering. |
| prompts/shared/_rules.txt | Removed leading indentation for consistent include rendering. |
| prompts/shared/_exploit-scope.txt | Whitespace cleanup in exploitation scope snippet. |
| prompts/report-executive.txt | Removed trailing whitespace in objective section. |
| prompts/recon.txt | Whitespace cleanup in recon prompt instructions. |
| prompts/pre-recon-code.txt | Whitespace cleanup and line wrapping normalization in pre-recon code prompt. |
| prompts/exploit-xss.txt | Whitespace cleanup in XSS exploitation prompt. |
| prompts/exploit-ssrf.txt | Whitespace cleanup in SSRF exploitation prompt. |
| prompts/exploit-injection.txt | Whitespace cleanup in injection exploitation prompt. |
| prompts/exploit-authz.txt | Whitespace cleanup in authz exploitation prompt. |
| prompts/exploit-auth.txt | Whitespace cleanup in auth exploitation prompt. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR restores the 3 commits that were previously in #204 and disappeared after the fork
mainbranch was force-pushed