timothy/ersatztv
1
0
Fork 0
Code Issues 58 Pull Requests 8 Actions Packages Projects Releases 9 Wiki Activity

docs(698): correct the Renovate auto-pass rule in CLAUDE.md #710

Merged
timothy merged 1 commits from docs/698-claudemd-renovate-rule into main 2026-07-30 21:58:04 +02:00
Conversation 0 Commits 1 Files Changed 1
+1 -1
timothy commented 2026-07-30 21:56:25 +02:00
Owner
Copy Link
Copy Source

refs #698

Docs-only. CLAUDE.md still described the Renovate review-verdict/h10 auto-pass as identity-only — "auto-passed as a bot PR unless they touch .claude//.gitea//.husky//scripts//docker/ci/".

Since #698 that is no longer the rule. The auto-pass now requires both: no protected path and every changed path being a dependency manifest (Directory.Packages.props, .config/dotnet-tools.json). A bot account does not attribute the code at a head — pull_request.user.login is the PR's immutable creator, while the head it points at is not.

CLAUDE.md is loaded into every session, so a stale rule there propagates further than a stale doc would. Caught by the /done docs-impact check after #705 merged; docs/ci-cd.md and the decision records were already correct.

Practical effect worth knowing: a Renovate PR touching a .csproj or a source file is not blocked, it simply needs a real verdict.

refs #698 Docs-only. `CLAUDE.md` still described the Renovate `review-verdict/h10` auto-pass as identity-only — "auto-passed as a bot PR unless they touch `.claude/`/`.gitea/`/`.husky/`/`scripts/`/`docker/ci/`". Since #698 that is no longer the rule. The auto-pass now requires **both**: no protected path **and** every changed path being a dependency manifest (`Directory.Packages.props`, `.config/dotnet-tools.json`). A bot *account* does not attribute the *code* at a head — `pull_request.user.login` is the PR's immutable creator, while the head it points at is not. `CLAUDE.md` is loaded into every session, so a stale rule there propagates further than a stale doc would. Caught by the `/done` docs-impact check after #705 merged; `docs/ci-cd.md` and the decision records were already correct. Practical effect worth knowing: a Renovate PR touching a `.csproj` or a source file is not blocked, it simply needs a real verdict.
timothy added 1 commit 2026-07-30 21:56:26 +02:00
docs(698): correct the Renovate auto-pass rule in CLAUDE.md
PR Gates / CI image pin matches docker/ci (pull_request) Successful in 16s
Details
PR Gates / Docs update reminder (pull_request) Successful in 18s
Details
Build ErsatzTV Image / Build & test (.NET) (pull_request) Successful in 19s
Details
PR Gates / decisions lifecycle (pull_request) Successful in 20s
Details
Build ErsatzTV Image / EF migration integrity (SQLite + MySql) (pull_request) Successful in 39s
Details
Build ErsatzTV Image / Functional E2E (curl + UI contracts) (pull_request) Successful in 38s
Details
review-verdict/h10 Exempt: docs-only change (no code, no protected path)
Details
Build ErsatzTV Image / Formatting (changed .cs conform to .editorconfig) (pull_request) Successful in 38s
Details
Build ErsatzTV Image / API docs in sync (OpenAPI + endpoint index) (pull_request) Successful in 38s
Details
Review verdict / Set review-verdict status (pull_request_target) Successful in 24s
Details
Build ErsatzTV Image / Build & push image (amd64) (pull_request) Has been skipped
Details
PR Gates / Script tests (pytest) (pull_request) Successful in 1m0s
Details
1eca9b0c11 
CLAUDE.md still described the bot exemption as identity-only (auto-passed unless a
protected path is touched). Since #698 it also requires EVERY changed path to be a
dependency manifest — a bot account does not attribute the code at a head. CLAUDE.md is
loaded every session, so a stale rule here is worse than a stale doc.

Refs: #698
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
timothy merged commit 81be685df9 into main 2026-07-30 21:58:04 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
ad-hoc
One-off / ad-hoc work not tracked by a dedicated issue
 api
REST API / HTTP endpoints
 bug
Something isn't working
 ci-cd
Build, test, deploy pipeline
 content
Channel content / schedules / playlists
 dependencies
Dependency updates (Renovate)
 enhancement
New feature or improvement
 frontend
ChicoryTV React SPA frontend
 in-progress
Claimed by an active session — do not pick up
 jellyfin
Jellyfin tuner / IPTV integration
 parked
Excluded from automatic queue pickup; work only when explicitly selected
 priority: high
priority: low
priority: medium
review
Adversarial review finding
 security
Security / vulnerability fix
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate timothy (Timothy)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: timothy/ersatztv#710