-
Notifications
You must be signed in to change notification settings - Fork 35
feat: add security-plan-auditor agent #337
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
base: main
Are you sure you want to change the base?
feat: add security-plan-auditor agent #337
Conversation
0617be4 to
f49f9ee
Compare
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.
Pull request overview
This pull request adds a new security-plan-auditor.agent.md agent that complements the existing security-plan-creator.agent.md by providing audit capabilities for existing security plans. The agent validates security plans against system changes such as infrastructure updates, new features, or compliance requirements, identifying gaps and generating actionable remediation recommendations.
Changes:
- Adds conversational security audit agent with five-phase workflow
- Implements systematic gap analysis between security plans and current infrastructure state
- Provides audit category framework and finding severity classification
Add security audit expert agent for validating and updating security plans against changed requirements, new features, or updated infrastructure configurations. Key capabilities: - Systematic gap analysis between security plans and current state - Audit category framework with finding severity levels - Five-phase workflow from scope definition to report generation - Integration with existing security-plan-outputs directory Fixes microsoft#336
f49f9ee to
76f673b
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #337 +/- ##
==========================================
- Coverage 41.08% 41.04% -0.04%
==========================================
Files 15 15
Lines 2870 2870
==========================================
- Hits 1179 1178 -1
- Misses 1691 1692 +1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
| "author": "Microsoft", | ||
| "license": "MIT" | ||
| } | ||
| } No newline at end of file |
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.
Can you add a new line to the end of the file?
|
@omer-demir this is a really nice baseline review agent ... thanks for the contribution! I'm not sure how the agent would respond to this change, but you may want to reference a few of the core standards like we do in the repo's threat model (probably shouldn't be this specific list - go more generic) e.g. @katriendg and/or @agreaves-ms - can you give this a once over? |
|
First of all thanks for adding it! I have a few points at first look at this new agent.
|
Pull Request
Description
Add security audit expert agent for validating and updating security plans against changed requirements, new features, or updated infrastructure configurations.
Key capabilities:
Related Issue(s)
Fixes #336
Type of Change
Select all that apply:
Code & Documentation:
AI Artifacts:
.github/agents/*.agent.md)Sample Prompts (for AI Artifact Contributions)
or
Execution Flow:
security-plan-outputs/, presents options, and asks user to select a plan and describe the audit trigger (changed features, new requirements, or infrastructure updates)security-plan-outputs/with findings, recommendations, and plan update suggestionsTesting
Required Checks
AI Artifact Contributions
/prompt-analyzeto review contributionprompt-builderreviewRequired Automated Checks
The following validation commands must pass before merging:
npm run lint:mdnpm run spell-checknpm run lint:frontmatterSecurity Considerations
Additional Notes
This agent complements the existing
security-plan-creator.agent.mdby providing audit capabilities for existing security plans. While the creator generates new plans from blueprints, the auditor validates existing plans against system changes, ensuring security documentation stays current with evolving infrastructure.