KofTwentyTwo/gclo
2
0 Fork 0
Code Issues 28 Pull Requests 1 Actions Packages Projects Releases 22 Wiki Activity

Best-practices/KISS audit + repo enforcement #5

New Issue
Closed
opened 2026-07-03 13:59:22 +00:00 by KofTwentyTwo · 0 comments
KofTwentyTwo commented 2026-07-03 13:59:22 +00:00 (Migrated from github.com)
Copy link
Copy source

Agent audit of the final codebase shape, then enforcement so it stays true.

Audit (adversarially verified findings, then fixes): design simplicity / over-engineering, coding standards + analyzer findings, documentation accuracy vs actual code, comment quality (why-not-what; XML docs on public library surface). Include: Octokit wrapper coverage gap, TryDeleteDirectory defensive dead path.

Enforcement: .editorconfig + Directory.Build.props (TreatWarningsAsErrors, analyzers latest-recommended, EnforceCodeStyleInBuild — style violations fail local builds, not just CI); CODEOWNERS; PR + issue templates; SECURITY.md; required status checks added to the main ruleset (build-test, format, CodeQL) now that check names exist. No required PR approvals (solo maintainer) unless the owner opts in.

Agent audit of the final codebase shape, then enforcement so it stays true. **Audit** (adversarially verified findings, then fixes): design simplicity / over-engineering, coding standards + analyzer findings, documentation accuracy vs actual code, comment quality (why-not-what; XML docs on public library surface). Include: Octokit wrapper coverage gap, TryDeleteDirectory defensive dead path. **Enforcement**: `.editorconfig` + `Directory.Build.props` (TreatWarningsAsErrors, analyzers latest-recommended, EnforceCodeStyleInBuild — style violations fail local builds, not just CI); CODEOWNERS; PR + issue templates; SECURITY.md; **required status checks added to the `main` ruleset** (build-test, format, CodeQL) now that check names exist. No required PR approvals (solo maintainer) unless the owner opts in.
Sign in to join this conversation.
No Branch/Tag Specified
Labels
Clear labels
.NET
Pull requests that update .NET code
 bug
Something isn't working
 deferred
Agreed follow-up, not scheduled
 dependencies
Pull requests that update a dependency file
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 github_actions
Pull requests that update GitHub Actions code
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 needs-owner
Blocked on an action only the owner can take
 priority
Owner-flagged: work this first
 quality
Standards, simplicity, enforcement
 question
Further information is requested
 refactor
Restructuring without behavior change
 release
Release pipeline / distribution
 wontfix
This will not be worked on
No labels quality
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
forge-recovery (Forge Recovery) james.maes (James Maes)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/gclo#5