feat: Collaboration & Refinement Improvements#19
Conversation
…library management
…nd improve README for clarity
…ormats, and clarify skills usage
There was a problem hiding this comment.
Pull request overview
This PR expands and standardizes the repository's Copilot security "library" content by introducing library-maintenance tooling, improving documentation and catalogues, and normalizing prompt metadata and output formats for better tool/environment compatibility.
Changes:
- Added library-management agents, prompts, instructions, and skills under
.github/to help maintain and evolve the security library - Improved root documentation (README, agents/README, skills/README) with clickable catalogue tables and usage guidance
- Added YAML frontmatter (
agent,name,description) to all root-level security prompts for consistency and discoverability - Strengthened prompt content with evidence-first procedures, deterministic output formats, and quality checks
- Expanded skills with "when to use", inputs, output format, and examples sections
- Added an orchestrator agent as the recommended entry point for AppSec workflows
Reviewed changes
Copilot reviewed 40 out of 41 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/threat-model-lite/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/secure-fix-validation/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/secure-code-review/SKILL.md | Added "when to use" section header |
| skills/secrets-and-logging-hygiene/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/input-validation-hardening/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/genai-acceptance-review/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/dependency-cve-triage/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/authn-authz-review/SKILL.md | Added standard skill sections (when to use, inputs, output format, examples) |
| skills/README.md | Converted skill list to clickable links |
| prompts/validate-input-handling.prompt.md | Added YAML frontmatter and restructured into evidence-first format |
| prompts/threat-model.prompt.md | Drastically simplified from 366 lines to 91 lines; added frontmatter and focused on 4Q core procedure |
| prompts/secure-code-review.prompt.md | Added frontmatter, restructured procedure, clarified logging/redaction guidance |
| prompts/scan-for-insecure-apis.prompt.md | Added frontmatter and evidence-first structure |
| prompts/review-auth-flows.prompt.md | Added frontmatter and evidence-first structure |
| prompts/dependency-cve-triage.prompt.md | Added frontmatter and evidence-first structure with explicit input variables |
| prompts/check-for-unvalidated-genai-acceptances.prompt.md | Added frontmatter and evidence-first structure |
| prompts/check-for-secrets.prompt.md | Added frontmatter and evidence-first structure |
| prompts/check-access-controls.prompt.md | Added frontmatter and evidence-first structure |
| prompts/business-logic-review.prompt.md | Added frontmatter and restructured for clarity |
| prompts/assess-logging.prompt.md | Added frontmatter and evidence-first structure |
| copilot-instructions.md | Added Python support, clarified XSS/sanitization guidance, updated secrets handling guidance |
| agents/application-security-orchestrator.agent.md | New orchestrator agent for routing AppSec workflows |
| agents/application-security-engineer.agent.md | Added "handling missing information" section |
| agents/application-security-architect.agent.md | Added "handling missing information" section |
| agents/README.md | Converted agent list to clickable links and added orchestrator recommendation |
| README.md | Expanded prompt catalogue, added agent/skill overview tables, recommended orchestrator workflow |
| .github/skills/markdown-customizations/SKILL.md | New skill for authoring Copilot customization files |
| .github/prompts/sync-readme-catalogue.prompt.md | New maintenance prompt for README catalogue syncing |
| .github/prompts/review-prompt-fontmatter.prompt.md | New prompt for auditing YAML frontmatter compliance |
| .github/prompts/improve-library-item.prompt.md | New prompt for refactoring library items |
| .github/prompts/create-skill.prompt.md | New prompt for generating new skills |
| .github/prompts/audit-library.prompt.md | New prompt for comprehensive library auditing |
| .github/prompts/add-new-security-prompt.prompt.md | New prompt for creating new security prompts |
| .github/instructions/security-library-authoring.instructions.md | New authoring rules for library content |
| .github/instructions/copilot-customization-files.instructions.md | New authoring rules for customization files |
| .github/copilot-instructions.md | New repo-scoped Copilot instructions |
| .github/agents/security-prompt-engineer.agent.md | New agent for designing security prompts/skills |
| .github/agents/markdown-customizations.agent.md | New agent for maintaining customization files |
| .github/agents/appsec-library-maintainer.agent.md | New agent for library maintenance |
| .env.example | Updated SSL PFX passphrase key name to dot-separated format |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
✅ PR Review Summary (PR Compass - Holistic Pull Request Reviewer)This is a strong, high-signal PR that significantly improves the clarity, consistency, and safety of this repository as a Copilot security content library. Key wins:
The changes are thoughtful, cohesive, and aligned with real-world AppSec workflows. Aside from a few minor nits (typos, newline, and optional clarifications), this PR is ready to merge and sets a solid foundation for future additions to the library. |
Summary
This PR expands and standardizes the repository’s Copilot security “library” content by:
What changed
New library-management content (under .github/)
Root documentation and catalog improvements
New/expanded “orchestrator” entrypoint agent
Prompt metadata normalization (YAML frontmatter)
agent,name,description) across all root-level security prompts to improve clarity and compliance:Prompt content improvements + output format hardening
Skills expansion and enrichment
Minor config/housekeeping
Notes / Compatibility
Testing