[ptr727]

Summary

Promotes the following pull requests merged into develop since main's last promotion: #1249, #1252, #1251, #1245, #1243.

This is a main-based promotion branch because develop and main conflict on the hub pin. main pins hub release 2.0.1366 and develop pins 2.0.1401.

Conflicted file	Resolution
.github/workflows/merge-bot-pull-request.yml	Kept develop: hub pin 2.0.1401 over 2.0.1366
.github/workflows/publish-release.yml	Kept develop in all four hunks: hub pin 2.0.1401 over 2.0.1366
.github/workflows/test-pull-request.yml	Kept develop in both hunks: hub pin 2.0.1401 over 2.0.1366

Every hunk differed only in the pin, and develop is a strict superset of main for these files.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Updated development and test tools and libraries, including the command-line parsing package.
  • Refreshed the versions of shared release, validation, build, and merge workflows.
  • Corrected the integration coverage filter to match the repository’s coverage configuration.
• Documentation
  • Updated architecture and coverage instructions to reflect the current command-line package and coverage configuration file.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The pull request updates .NET tool and package versions, reusable workflow references, and Codecov configuration paths. It also reformats C# code and tests. The summarized processing and schema-deserialization behavior remains unchanged.

Changes: Repository maintenance

Layer / File(s)	Summary
Tool and package versions .config/dotnet-tools.json, Directory.Packages.props, ARCHITECTURE.md	Several tool and package versions are updated. The architecture documentation now refers to System.CommandLine v3.x.
Reusable workflow revisions .github/workflows/merge-bot-pull-request.yml, .github/workflows/publish-release.yml, .github/workflows/test-pull-request.yml	The merge-bot, release, validation, and smoke-build workflow references now use revision 2.0.1401.
Codecov configuration references .github/actions/integration-coverage/action.yml, .github/workflows/test-pull-request.yml, OPERATIONS.md, RegressionTests/README.md	The integration filter, coverage action comment, runbook, and regression instructions now refer to .codecov.yml.
C# formatting and track tag lookup PlexCleaner/ConfigFileJsonSchema.cs, PlexCleaner/Process.cs, PlexCleaner/ProcessDriver.cs, PlexCleaner/ProcessFile.cs, PlexCleaner/SidecarFileJsonSchema.cs, PlexCleaner/TrackProps.cs, PlexCleanerTests/*	Processing callbacks, schema deserialization calls, track-selection expressions, and test setup are reformatted. The FFprobe track parser looks up title and language in Tags with case-insensitive keys and retains empty-string fallbacks.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Possibly related issues

• Issue 1177 concerns a conflict in the publish-release.yml publish job and describes the hub pin used by this promotion.

Possibly related PRs

• ptr727/PlexCleaner#1172: Adds the Codecov configuration and tool manifest entries that this pull request updates.
• ptr727/PlexCleaner#1210: Adds the integration coverage action and workflow filter whose Codecov path this pull request changes.
• ptr727/PlexCleaner#1064: Updates the same reusable workflow pin locations in earlier revisions.

Suggested labels: chore

Merge Risk: Minimal · up to 9a854

This promotion updates dependency and tool versions, workflow pins, and Codecov config references, and reformats code without changing behavior. No actionable merge-blocking risk was identified.

Pre-merge checks: 4

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title names two changes in the promotion: the hub reusable-workflow pin moving to 2.0.1401 and the rename of codecov.yml to .codecov.yml. Both appear in the changeset: the workflow files pin 2.0.1…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title names two changes in the promotion: the hub reusable-workflow pin moving to 2.0.1401 and the rename of codecov.yml to .codecov.yml. Both appear in the changeset: the workflow files pin 2.0.1401, and the integration path filter, OPERATIONS.md, RegressionTests/README.md, and the action file reference .codecov.yml. The title is specific and matches the main objective, which is promoting develop to main. It omits the dependency updates (System.CommandLine 3.0.0, csharpier, and others), which is expected for a title. The title also uses the 'Promote Develop' prefix, which is clear to teammates scanning history.

Finishing Touches

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

Recent review info

Run configuration

• Source mode: Auto
• Repository: ptr727/PlexCleaner
• Branch: promote/develop-to-main-hub-2.0.1401
• File: .coderabbit.yaml
• Commit: 9a8540e
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: a543b0d0-77dc-4242-904d-c5497b934884

Commits

Reviewing files that changed from the base of the PR and between 24225ae and 9a8540e.

Files selected for processing: 19

• .codecov.yml
• .config/dotnet-tools.json
• .github/actions/integration-coverage/action.yml
• .github/workflows/merge-bot-pull-request.yml
• .github/workflows/publish-release.yml
• .github/workflows/test-pull-request.yml
• ARCHITECTURE.md
• Directory.Packages.props
• OPERATIONS.md
• PlexCleaner/ConfigFileJsonSchema.cs
• PlexCleaner/Process.cs
• PlexCleaner/ProcessDriver.cs
• PlexCleaner/ProcessFile.cs
• PlexCleaner/SidecarFileJsonSchema.cs
• PlexCleaner/TrackProps.cs
• PlexCleanerTests/CleanupStepRepeatTests.cs
• PlexCleanerTests/PerFileExceptionTests.cs
• PlexCleanerTests/ReplaceOriginalTests.cs
• RegressionTests/README.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review.

[qodo-free-for-open-source-projects]

PR Summary by Qodo

Promote hub 2.0.1401, dependency updates, and Codecov config rename

⚙️ Configuration changes ✨ Enhancement 📝 Documentation 🕐 20-40 Minutes

AI Description

• Promote reusable CI and release workflows to hub 2.0.1401.
• Rename Codecov configuration to .codecov.yml and update workflow detection and references.
• Refresh .NET packages and tools; apply the newer CSharpier formatting.

Diagram

graph TD
  PR["PR checks"] --> Hub(["Hub tasks"]) --> Validation["Build validation"] --> Upload["Coverage upload"] --> Codecov(["Codecov service"])
  Release["Release workflow"] --> Hub
  Config["Codecov config"] --> Codecov
  Packages["Package pins"] --> Validation

Loading

High-Level Assessment

Keep the explicit, matching SHA pins across reusable workflows and the conventional .codecov.yml location. Central package versions already provide a single place for .NET dependency updates; a broader workflow or configuration redesign is not warranted.

Files changed (18) +349 / -407

Refactor (6) +286 / -327

ConfigFileJsonSchema.csApply CSharpier formatting to config deserialization +20/-23

Apply CSharpier formatting to config deserialization

• Reformats versioned JSON deserialization expressions without changing schema selection or error handling.

PlexCleaner/ConfigFileJsonSchema.cs

Process.csReformat the file-processing callback +50/-55

Reformat the file-processing callback

• Adjusts CSharpier layout around the processing callback; cancellation, metrics, and result collection remain unchanged.

PlexCleaner/Process.cs

ProcessDriver.csReformat media-processing callbacks +174/-196

Reformat media-processing callbacks

• Reflows callbacks for tag mapping and media/tool inspection. The checks, warnings, and return paths remain the same.

PlexCleaner/ProcessDriver.cs

ProcessFile.csReformat language-track selection predicates +8/-11

Reformat language-track selection predicates

• Changes the layout of unknown- and unwanted-language selection expressions without altering their predicates.

PlexCleaner/ProcessFile.cs

SidecarFileJsonSchema.csApply CSharpier formatting to sidecar deserialization +24/-30

Apply CSharpier formatting to sidecar deserialization

• Reformats versioned sidecar JSON deserialization expressions without changing schema behavior.

PlexCleaner/SidecarFileJsonSchema.cs

TrackProps.csReformat FFmpeg track tag lookups +10/-12

Reformat FFmpeg track tag lookups

• Reflows the title and language tag assignments without changing their lookup or fallback behavior.

PlexCleaner/TrackProps.cs

Tests (3) +43 / -60

CleanupStepRepeatTests.csReformat cleanup test fixtures +27/-34

Reformat cleanup test fixtures

• Adjusts lambda and invocation layout in repeated-cleanup tests; assertions and fixture mutations remain unchanged.

PlexCleanerTests/CleanupStepRepeatTests.cs

PerFileExceptionTests.csReformat per-file exception test callback +5/-10

Reformat per-file exception test callback

• Reflows the test's 'ProcessFiles' invocation without changing the exception scenario or assertions.

PlexCleanerTests/PerFileExceptionTests.cs

ReplaceOriginalTests.csReformat replacement test callbacks +11/-16

Reformat replacement test callbacks

• Adjusts fake-tool callback layout in replacement and language-restoration tests without changing test behavior.

PlexCleanerTests/ReplaceOriginalTests.cs

Documentation (3) +4 / -4

action.ymlCorrect the Codecov configuration reference +1/-1

Correct the Codecov configuration reference

• Updates the coverage-upload comment to identify '.codecov.yml' as the source of the '/Builder' path fix.

.github/actions/integration-coverage/action.yml

OPERATIONS.mdPoint coverage guidance to the renamed config +1/-1

Point coverage guidance to the renamed config

• Updates the Codecov configuration link in the coverage operations guide.

OPERATIONS.md

README.mdUpdate regression coverage links +2/-2

Update regression coverage links

• Points the coverage explanation and reference link to '.codecov.yml'.

RegressionTests/README.md

Other (6) +16 / -16

.codecov.ymlMove Codecov configuration to its conventional filename +0/-0

Move Codecov configuration to its conventional filename

• Renames the existing coverage-status, flag, and source-path configuration to '.codecov.yml' without changing its contents.

.codecov.yml

dotnet-tools.jsonRefresh pinned .NET tools +3/-3

Refresh pinned .NET tools

• Bumps CSharpier to 1.4.0, dotnet-outdated to 4.9.0, and ReportGenerator to 5.6.0.

.config/dotnet-tools.json

merge-bot-pull-request.ymlPin merge-bot task to hub 2.0.1401 +1/-1

Pin merge-bot task to hub 2.0.1401

• Advances the reusable merge-bot workflow from hub 2.0.1366 to the 2.0.1401 SHA.

.github/workflows/merge-bot-pull-request.yml

publish-release.ymlPin release-chain tasks to hub 2.0.1401 +4/-4

Pin release-chain tasks to hub 2.0.1401

• Advances the reusable planning, validation, build/publish, and Docker README tasks to the same 2.0.1401 SHA.

.github/workflows/publish-release.yml

test-pull-request.ymlUpdate PR task pins and Codecov path detection +3/-3

Update PR task pins and Codecov path detection

• Advances validation and smoke-build tasks to hub 2.0.1401. Changes the integration path filter to detect '.codecov.yml'.

.github/workflows/test-pull-request.yml

Directory.Packages.propsRefresh centrally managed .NET packages +5/-5

Refresh centrally managed .NET packages

• Bumps five package versions, including Microsoft.NET.Test.Sdk, SourceLink, ptr727 libraries, and System.CommandLine 3.0.0.

Directory.Packages.props

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

1. CI guidance points to a deleted file 🐞 Bug ⚙ Maintainability

Description

WORKFLOW.md still names codecov.yml in three descriptions of the integration coverage contract,
although the configuration and workflow filter now use .codecov.yml. Someone following that
guidance to inspect or update the coverage settings encounters a nonexistent path and an incorrect
description of which file triggers the integration job.

Code

.github/workflows/test-pull-request.yml[66]

+              - '.codecov.yml'

Evidence

The changed integration filter watches .codecov.yml, which exists at the repository root, while
the CI guidance still names the former path three times.

.github/workflows/test-pull-request.yml[63-66]
.codecov.yml[16-30]
WORKFLOW.md[413-415]
WORKFLOW.md[598-601]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The coverage configuration was renamed to `.codecov.yml`, but three references in the CI guidance retain its old filename.
## Fix Focus Areas
- WORKFLOW.md[413-415]
- WORKFLOW.md[601-601]
## Recommended Fix
Replace the three stale `codecov.yml` references with `.codecov.yml` so the documented filter and coverage configuration match the repository.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

Tip of the day

💡 Did you know, you can open the rule or relationship behind a finding in one click

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[Copilot]

🟡 Changes recommended

An active documentation inconsistency remains: ARCHITECTURE.md still describes System.CommandLine as v2.x after the dependency was upgraded to 3.0.0.

1 open issue

• Update documentation for System.CommandLine 3.0 · New

What changed in this PR

This promotion brings the validated develop changes into main, including the hub workflow pin, Codecov configuration rename, dependency updates, and CSharpier formatting.

Changes:

• Updates reusable workflow references to hub release 2.0.1401.
• Renames the Codecov configuration to .codecov.yml and updates related references.
• Refreshes .NET packages and tooling, including System.CommandLine 3.0.0 and CSharpier 1.4.0.
• Applies CSharpier formatting across production and test code.
• Records a documentation follow-up for the outdated System.CommandLine version description.

File	Description
RegressionTests/​README.md	Updates regression coverage documentation and Codecov references.
PlexCleanerTests/​ReplaceOriginalTests.cs	Applies CSharpier formatting to replacement tests.
PlexCleanerTests/​PerFileExceptionTests.cs	Applies CSharpier formatting to exception-processing tests.
PlexCleanerTests/​CleanupStepRepeatTests.cs	Applies CSharpier formatting to cleanup-step tests.
PlexCleaner/​TrackProps.cs	Applies CSharpier formatting to track property extraction.
PlexCleaner/​SidecarFileJsonSchema.cs	Applies CSharpier formatting to sidecar schema deserialization.
PlexCleaner/​ProcessFile.cs	Applies CSharpier formatting to track-selection callbacks.
PlexCleaner/​ProcessDriver.cs	Applies CSharpier formatting to processing driver callbacks.
PlexCleaner/​Process.cs	Applies CSharpier formatting to file-processing code.
PlexCleaner/​ConfigFileJsonSchema.cs	Applies CSharpier formatting to configuration schema deserialization.
OPERATIONS.md	Updates operational documentation to reference .codecov.yml.
Directory.Packages.props	Updates central .NET package versions, including System.CommandLine.
.github/​workflows/​test-pull-request.yml	Updates Codecov path filtering and hub workflow pins.
.github/​workflows/​publish-release.yml	Updates reusable release workflow pins to hub release 2.0.1401.
.github/​workflows/​merge-bot-pull-request.yml	Updates the merge-bot reusable workflow pin to hub release 2.0.1401.
.github/​actions/​integration-coverage/​action.yml	Updates the Codecov configuration reference.
.config/​dotnet-tools.json	Updates CSharpier and supporting global tools.
.codecov.yml	Adds the renamed Codecov configuration file.

_{Review effort: Lite}

[ptr727]

@qodo-free-for-open-source-projects please dismiss finding 1 (CI guidance points to a deleted file). The stale codecov.yml references are in WORKFLOW.md and .github/skills/**, which are carried verbatim from the hub (ptr727/ProjectTemplate) and are not edited here. The hub already renamed the file in its own copy (hub PR #4077, merged into hub develop), and this repository receives the new wording at its next resync.

[qodo-free-for-open-source-projects]

free-for-open-source-projects please dismiss finding 1 (CI guidance points to a deleted file). The stale codecov.yml references are in WORKFLOW.md and .github/skills/**, which are carried verbatim from the hub (ptr727/ProjectTemplate) and are not edited here. The hub already renamed the file in its own copy (hub PR #4077, merged into hub develop), and this repository receives the new wording at its next resync.

I haven't dismissed finding 1. The findings index for this PR is empty, so I can't map "finding 1" to a stable ID, and I won't close a finding without one.

Your reasoning makes sense: the codecov.yml references are carried verbatim from the hub, and the hub has already renamed the file. If you want this closed, you can:

• Dismiss it in the Qodo portal, or
• Ask me again on the next review run, once the finding appears in the index. I can then dismiss it by ID. Only the PR author can dismiss findings.

[Copilot]

🟢 Approval recommended

No unresolved review issues are identified across the dependency, workflow, configuration, documentation, and formatting updates.

No open issues 🎉

1 resolved since last review

• Update documentation for System.CommandLine 3.0

_{Review effort: Lite}

[ptr727]

@coderabbitai review

[coderabbitai]

Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[ptr727]

A recorded local strict-review pass covers head 9a8540e2ac41124c7b025e65abf227306c1869c9, the content this pull request carries at that commit against main, and it recorded 0 findings.