Skip to content

[SPVS] Add CODEOWNERS for security-sensitive file review#57

Open
lmrb-1968 wants to merge 1 commit intomainfrom
spvs/codeowners
Open

[SPVS] Add CODEOWNERS for security-sensitive file review#57
lmrb-1968 wants to merge 1 commit intomainfrom
spvs/codeowners

Conversation

@lmrb-1968
Copy link
Copy Markdown
Member

Summary

Add .github/CODEOWNERS to route changes to security-sensitive files to the appropriate review teams.

Protected file categories:

  • CI/CD workflows → xygeni/security + xygeni/release-engineering
  • Dependency descriptors (pom.xml, package.json, etc.) → xygeni/security + repo team
  • Container definitions → xygeni/security + xygeni/release-engineering
  • Xygenibot config, pre-commit hooks, .github/ config → xygeni/security

The org rulesets now have require_code_owner_review: true, so this activates the moment the PR merges.

Test plan

  • After merge, open a PR modifying a .github/workflows/ file — verify security + release-engineering are auto-requested
  • Open a PR modifying regular code — verify only repo team is requested

Ref: xygeni/product-backlog#4431

🤖 Generated with Claude Code

OWASP SPVS V2.3.1: route changes to CI/CD workflows, dependency
descriptors, container definitions, and security configs to the
appropriate teams (xygeni/security, xygeni/release-engineering,
repo maintainers).

Ref: xygeni/product-backlog#4431

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lmrb-1968 lmrb-1968 self-assigned this Apr 10, 2026
@lmrb-1968 lmrb-1968 requested review from a team, Garcel and fcarnicero April 10, 2026 05:24
Copy link
Copy Markdown
Contributor

@Garcel Garcel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants