Skip to content

meta: fix CODEOWNERS team handle (development → developers) - #64

Merged
heskew merged 1 commit into
mainfrom
meta/codeowners-fix-team-handle
Apr 24, 2026
Merged

meta: fix CODEOWNERS team handle (development → developers)#64
heskew merged 1 commit into
mainfrom
meta/codeowners-fix-team-handle

Conversation

@heskew

@heskew heskew commented Apr 24, 2026

Copy link
Copy Markdown
Member

Summary

Fix a team-handle mismatch in .github/CODEOWNERS. @HarperFast/development doesn't resolve to a real team on GitHub — the actual team (used by HarperFast/harper's CODEOWNERS) is @HarperFast/developers.

Why it matters

When a CODEOWNERS line references a non-existent team, GitHub silently can't resolve an owner for that pattern. If "Require review from Code Owners" is ever enabled on branch protection, PRs will sit unassignable. Even without that requirement, the CODEOWNERS file isn't doing its job of auto-requesting review from the right team.

Coordination

New HarperFast/ai-review-prompts (#3 there) uses @HarperFast/developers too, matching the harper/oauth-fixed-here shape.

🤖 Generated with Claude Code

HarperFast/harper's CODEOWNERS uses `@HarperFast/developers` (plural
noun — the people in the team). oauth was using
`@HarperFast/development` (singular abstract — the discipline).
Only one resolves to an actual team on GitHub; the other silently
fails to assign a reviewer.

Aligning oauth with the harper pattern. New HarperFast/ai-review-prompts
CODEOWNERS (separate PR) uses the same `@HarperFast/developers` handle.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@heskew
heskew requested a review from a team as a code owner April 24, 2026 14:47
@claude

claude Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

No blockers found.

What I traced:

  • Diff scope: .github/CODEOWNERS only — 2 lines, no source code touched
  • CLAUDE.md / agent context files modified: No
  • API contracts, dispatch surfaces, security paths: Unaffected
  • Tests, documentation, session/auth logic: Unaffected

Summary: The fix is correct. @HarperFast/development is a non-existent GitHub team handle; @HarperFast/developers is the real one (verified consistent with HarperFast/harper's CODEOWNERS). Both patterns (* and /.github/) are updated consistently. Pre-existing nit: file still lacks a trailing newline, but that predates this PR — not a blocker.

Generated with Claude Code

@heskew
heskew merged commit f5b65a4 into main Apr 24, 2026
5 checks passed
@heskew
heskew deleted the meta/codeowners-fix-team-handle branch April 24, 2026 15:10
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.

1 participant