Skip to content

chore(ci): Add PR quality signals ahead of automated review - #630

Merged
JWThewes merged 2 commits into
mainfrom
feat/pr-complexity-check
Oct 27, 2026
Merged

JWThewes merged 2 commits into
mainfrom
feat/pr-complexity-check

Conversation

@jeromevdl

Copy link
Copy Markdown
Collaborator

This is a pre-PR for automated reviews. It gives human reviewers—and a future automated reviewer following AGENTS.md—deterministic signals about risky changes. Automated review itself will follow separately.

  • Compare function complexity and backend branch coverage against the merge base, with direct production-importer counts to highlight widely used files.
  • Publish one updated, advisory PR comment. Omit unchanged complexity debt; label missing or partial measurements explicitly.
  • Skip backend coverage suites for unrelated changes, reuse the correct artifacts on selective reruns, and preserve the report if dependency analysis fails. Fork/Dependabot comments use validated artifacts and default-branch publishing code.

Validation: all 6,058 backend tests and 42 focused quality-report tests passed, along with lint, formatting, secret scanning, and workflow validation. A local 60-commit benchmark improved from 45.2 s to 34.7 s with identical reports. Full script suite: 136 passed; two existing local failures came from scratch-package metadata and the unrelated, untracked Terraform test (excluded from this PR). Hosted workflow execution remains to be verified by this PR.

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Complexity and test coverage

Analyzed 0 source files changed by this PR. Advisory threshold: 15.

0 new/unmatched or worsening functions above the threshold.

🟢 Improving · 🟠 Needs attention · 🔴 High risk · 🟣 Highest risk

unknown = unavailable · n/a = no branches · * = incomplete tests

CI run and full report artifacts

@jeromevdl jeromevdl changed the title Add PR quality signals ahead of automated review chore: Add PR quality signals ahead of automated review Oct 27, 2026
@jeromevdl jeromevdl changed the title chore: Add PR quality signals ahead of automated review chore(ci): Add PR quality signals ahead of automated review Oct 27, 2026
@jeromevdl jeromevdl added enhancement New feature or request maintenance labels Oct 27, 2026

@JWThewes JWThewes 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.

That's great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants