Skip to content

docs: add AI security guidance for security-sensitive contributions#1665

Merged
imran-siddique merged 1 commit into
microsoft:mainfrom
imran-siddique:docs/ai-security-guidance
Apr 30, 2026
Merged

docs: add AI security guidance for security-sensitive contributions#1665
imran-siddique merged 1 commit into
microsoft:mainfrom
imran-siddique:docs/ai-security-guidance

Conversation

@imran-siddique

Copy link
Copy Markdown
Collaborator

Adds detailed guidance for contributors using AI tools on security-sensitive code areas.

New file: \docs/policies/ai-security-guidance.md\ covering:

  • Independent test validation (no circular AI-tests-AI-code)
  • No secrets/credentials in AI tool prompts
  • Common AI anti-patterns checklist (hallucinated packages, deprecated crypto, insecure defaults, timing-unsafe comparisons, incomplete input validation, allowlist ordering)
  • Cryptographic code review checklist
  • Sandbox/isolation change checklist
  • Maintainer review expectations

Updated: CONTRIBUTING.md now links to the detailed guidance doc from its security considerations section.

Gap #6 of 6 from the OpenSSF AI policy alignment analysis.

Add docs/policies/ai-security-guidance.md covering:
- Independent test validation (no circular AI-tests-AI-code)
- No secrets/credentials in AI tool prompts
- Common AI security anti-patterns (hallucinated packages, deprecated
  crypto, insecure defaults, timing-unsafe comparisons)
- Cryptographic code review checklist
- Sandbox and isolation change checklist
- Maintainer review expectations for security PRs

Link from CONTRIBUTING.md security considerations section.

Gap #6 of the OpenSSF AI policy alignment.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@imran-siddique imran-siddique merged commit 4121647 into microsoft:main Apr 30, 2026
19 of 20 checks passed
@imran-siddique imran-siddique deleted the docs/ai-security-guidance branch April 30, 2026 20:26
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 30, 2026
@github-actions

Copy link
Copy Markdown
🤖 AI Agent: breaking-change-detector — View details

No breaking changes detected.

@github-actions

Copy link
Copy Markdown
🤖 AI Agent: test-generator — View details

Test coverage looks good. No gaps identified.

@github-actions

Copy link
Copy Markdown
🤖 AI Agent: security-scanner — View details

No security issues found.

@github-actions

Copy link
Copy Markdown
🤖 AI Agent: docs-sync-checker — Docs Sync

Docs Sync

Documentation is in sync.

@github-actions

Copy link
Copy Markdown
🤖 AI Agent: code-reviewer — View details

No issues found. Clean change.

@github-actions github-actions Bot added the size/M Medium PR (< 200 lines) label Apr 30, 2026
@github-actions

Copy link
Copy Markdown

PR Review Summary

Check Status Details
🔍 Code Review ✅ Passed No issues found
🛡️ Security Scan ✅ Completed Analysis complete
🔄 Breaking Changes ⚠️ Warning See details
📝 Docs Sync ✅ Passed No issues found
🧪 Test Coverage ✅ Completed Analysis complete

Verdict: ⚠️ Ready for human review

MohammadHaroonAbuomar pushed a commit to MohammadHaroonAbuomar/agt-acs that referenced this pull request Jun 1, 2026
…icrosoft#1665)

Add docs/policies/ai-security-guidance.md covering:
- Independent test validation (no circular AI-tests-AI-code)
- No secrets/credentials in AI tool prompts
- Common AI security anti-patterns (hallucinated packages, deprecated
  crypto, insecure defaults, timing-unsafe comparisons)
- Cryptographic code review checklist
- Sandbox and isolation change checklist
- Maintainer review expectations for security PRs

Link from CONTRIBUTING.md security considerations section.

Gap microsoft#6 of the OpenSSF AI policy alignment.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/M Medium PR (< 200 lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant