Repository navigation
chore(#1828): assign CODEOWNERS wildcard to maintainers team - #1844
fullsend-ai-coder[bot] wants to merge 1 commit into
Conversation
Replace @fullsend-ai/core with @fullsend-ai/maintainers on the root CODEOWNERS wildcard rule. The maintainers team was created in fullsend-ai/org#18 (merged), whose peribolos.yaml grants it maintain permission on this repository, which is sufficient for CODEOWNERS entries to take effect. Note: pre-commit could not fetch remote hook repos (TLS error); the applicable pre-commit-hooks v6.0.0 hooks and lint-release-cases were run directly and passed, and gitleaks ran via scan-secrets. make lint (skillsaw) could not run (uvx unavailable) and does not cover CODEOWNERS. The team's live GitHub access was not verified via API. Closes #1828 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
🤖 Finished Review · ✅ Success · Started 9:56 AM UTC · Completed 10:01 AM UTC Commit: Runtime: pi · Model: sonnet → claude-sonnet-5-5 · Effort: high · Cost: $0.58 |
|
<!-- fullsend:risk-assessment --> DetailsMinimal one-line config change by a bot matching the linked issue scope, with low churn and no regressions offsetting the single protected path modification to yield low composite risk. |
ReviewFindingsMedium
|
| @@ -1,2 +1,2 @@ | |||
| # These users have approval access to all resources (wildcard) | |||
| * @fullsend-ai/core | |||
| * @fullsend-ai/maintainers | |||
There was a problem hiding this comment.
[medium] protected-path
This PR modifies the protected path CODEOWNERS (root wildcard owner changed from @fullsend-ai/core to @fullsend-ai/maintainers). It links issue #1828, which authorizes this change, and the description explains the rationale. Human approval is always required for protected-path changes regardless of context. Membership and live repository access of @fullsend-ai/maintainers could not be verified from this review environment (the org config is in another repository).
Suggested fix: Obtain human maintainer approval; confirm @fullsend-ai/maintainers has write-or-higher access to this repository and that GitHub shows no owner errors on CODEOWNERS after merge.
Summary
Changes the root
CODEOWNERSwildcard rule from@fullsend-ai/coreto@fullsend-ai/maintainers.Context
The
maintainersteam was created by fullsend-ai/org#18 (merged 2026-10-29). That PR replaced fullsend-ai/org#16, which was closed without merging. Inperibolos.yamlthe new team hasmaintainpermission onagents, and CODEOWNERS needs at least write access to take effect, so that is enough.Testing
scan-secretspassed on the changed file and on the staged index.pre-commitcould not fetch remote hook repos in the sandbox (TLS error). Instead, the hooks that apply to this file frompre-commit-hooksv6.0.0 (end-of-file, trailing-whitespace, detect-private-key, merge-conflict, mixed-line-ending, large-files) and thelint-release-caseslocal hook were run directly. All passed.make lint(skillsaw) could not run becauseuvxis not installed. It does not cover CODEOWNERS.Reviewer notes
CODEOWNERS.CODEOWNERSis a protected path, so a human must approve this PR.🤖 Generated with Claude Code
Closes #1828
Post-script verification
agent/1828-codeowners-maintainers)ec1ed4607ff6df637d09e7258abb426732f7cac9..HEAD)