Fix/coverage exclude filter

[frigini]

Summary by CodeRabbit

• Chores
  • Streamlined CI/CD secrets validation process with uniform defaults and enhanced log security through password redaction
  • Expanded test coverage exclusion patterns to better handle compiler-generated, API-generated, and system artifacts with improved documentation

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Modified .github/workflows/pr-validation.yml to simplify secrets validation by removing strict environment-based checks and replacing with lenient defaults featuring password redaction, while expanding the coverage exclude filter to include additional pattern categories with documented comments.

Changes

Cohort / File(s)	Summary
Secrets validation simplification \.github/workflows/pr-validation\.yml	Removed environment variable declarations (POSTGRES_PASSWORD, POSTGRES_USER, POSTGRES_DB) and replaced complex main-repo-only strict validation with a simple message indicating PostgreSQL credential usage with fallback defaults; passwords redacted in logs.
Coverage exclude filter expansion \.github/workflows/pr-validation\.yml	Replaced EXCLUDE_FILTER hardcoded set with a broader, explicitly documented set of patterns excluding compiler-generated, OpenAPI-generated, and system/regex-generated artifacts; maintains include/exclude pattern logging.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• Ci enable module coverage collection #34 — Also modifies .github/workflows/pr-validation.yml with overlapping changes to secrets validation and coverage collection logic; directly related to this PR's simplification and expansion patterns.

Poem

🐰 Workflows now simpler, secrets kept tight,
Coverage filters expanded just right,
No strict checks to fuss and fight,
Our CI hops faster through the night! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly addresses the main change: expanding the coverage exclude filter with better patterns and documentation, making it the primary modification in this PR.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/coverage-exclude-filter

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8bc8cbe and 71a163b.

📒 Files selected for processing (1)

• .github/workflows/pr-validation.yml (2 hunks)

🔇 Additional comments (2)

.github/workflows/pr-validation.yml (2)

73-78: Verify that simplified validation adequately addresses configuration issues.

The step was previously strict with explicit environment validation, but now relies on lenient defaults and downstream failures (e.g., PostgreSQL connection timeout at lines 107–136) to surface configuration problems. This trades explicit validation feedback for simplicity.

For main repos with configured secrets, this is acceptable. However, for forks or local environments without secrets, failures may surface later (in the test execution step) rather than immediately here, potentially obscuring root causes.

Consider whether this simplification aligns with your team's CI/CD diagnostics expectations.

---

278-285: Excellent documentation and expansion of coverage exclusions.

The EXCLUDE_FILTER now comprehensively excludes compiler-generated, OpenAPI-generated, and test-related artifacts with clear explanatory comments. This improves accuracy of hand-written code coverage metrics.

The patterns follow OpenCover conventions correctly, and the rationale is well-documented (lines 278–280). This is a solid improvement over a minimal filter set.

---

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

_{Comment @coderabbitai help to get the list of available commands and usage tips.}