Skip to content

Add PlexCleaner audit report - #232

Merged
ptr727 merged 2 commits into
developfrom
feature/audit-plexcleaner
Jul 5, 2026
Merged

Add PlexCleaner audit report#232
ptr727 merged 2 commits into
developfrom
feature/audit-plexcleaner

Conversation

@ptr727

@ptr727 ptr727 commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Second per-repo audit (AUDIT.md): ptr727/PlexCleaner main @ e84f142, types csharp+console+docker.

Verdict: operational - no defects, 12 drift findings. Highlights:

  • Publisher builds one branch per run, not a both-branch matrix (schedule rebuilds main only; develop via manual dispatch). Correct by outcome.
  • No changes paths-filter; every push smoke-builds both targets (deliberate over-build).
  • Ruleset context suffixed ...status job; one-directional embedded release gate; dispatch guard skips vs fails fast; NBGV without IGNORE_GITHUB_REF.
  • .editorconfig lacks the global [*] end_of_line default; CODECOV_TOKEN stored but unused; README order; diverged develop (ahead 13, behind 23).

Clean: docker buildcache- (not type=gha), main-only Docker Hub README, console smoke-subset + aggregated release-asset, cspell en-US, analyzer enforcement, SHA-pinning, secrets.

Two recurring cross-repo drifts (PlexCleaner + Utilities): the ruleset context job suffix and the missing global [*] end_of_line default - the report proposes moving these to spec/lint machine-checks. Acting on findings is downstream and maintainer-directed.

🤖 Generated with Claude Code

ptr727 and others added 2 commits July 4, 2026 20:25
Second per-repo audit (csharp+console+docker). Verdict: operational; no defects, 12 drift findings (single-branch publisher vs both-branch matrix, no paths-filter, suffixed ruleset name, skip-not-fail dispatch guard, one-directional embedded release gate, NBGV without IGNORE_GITHUB_REF, no consume-delete on the transfer artifact, .editorconfig lacks global end_of_line default, CA relaxation batch, README order, unused CODECOV_TOKEN, diverged develop). Docker (buildcache-<branch>, main-only Docker Hub README, static secret), console (smoke subset + aggregated release-asset), cspell en-US confirmed clean. Two recurring cross-repo drifts (ruleset context suffix, missing global end_of_line) flagged for a spec check.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
The ruleset-suffix code spans had a leading space inside the span; drop it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 5, 2026 03:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new per-repo audit report for ptr727/PlexCleaner (main @ e84f142) under the reports/ tree, capturing the audit verdict and documenting observed drift items against the template's governance/workflow expectations.

Changes:

  • Add an audit writeup for PlexCleaner (verdict: operational; defects: none).
  • Document drift findings and proposed spec/registry follow-ups based on the audit results.

@ptr727
ptr727 merged commit 47cbf27 into develop Jul 5, 2026
7 checks passed
@ptr727
ptr727 deleted the feature/audit-plexcleaner branch July 5, 2026 03:28
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