[jbampton]

Add a separate workflow for the manual hooks

Description

This PR refs #14093

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)
• Build/CI
• Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

• Major
• Minor

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

[Copilot]

🟡 Changes recommended

The manual workflow duplicates the Run pre-commit check name on pull requests, which can create ambiguous results and block merging if branch protection requires that check.

1 open issue

• Rename duplicate pre-commit job to avoid ambiguous checks · New

What changed in this PR

This PR advances the migration from pre-commit to prek described in #14093 and separates manual-stage hooks into their own CI workflow.

Changes:

• Replace explicit Python setup, installation, and caching with j178/prek-action.
• Run manual-stage hooks in a separate pull-request workflow.
• Update checkout and adjust cancellation behavior in both workflows.

File	Description
.github/​workflows/​pre-commit.yml	Switch normal hooks to prek-action and remove manual-stage execution.
.github/​workflows/​pre-commit-manual.yml	Add a dedicated workflow for running manual-stage hooks with prek.

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are recorded for the migration to prek and separation of manual-stage hooks.

No open issues 🎉

1 resolved since last review

• Rename duplicate pre-commit job to avoid ambiguous checks

_{Review effort: Balanced}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.78%. Comparing base (92e44bb) to head (140128a).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #14549      +/-   ##
============================================
+ Coverage     20.76%   20.78%   +0.01%     
- Complexity    21905    21931      +26     
============================================
  Files          6496     6496              
  Lines        591605   591657      +52     
  Branches      72583    72592       +9     
============================================
+ Hits         122854   122966     +112     
+ Misses       455414   455346      -68     
- Partials      13337    13345       +8     

Flag	Coverage Δ
uitests	3.88% <ø> (ø)
unittests	22.09% <ø> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

🟢 Approval recommended

No active review issues are reported for the migration to separate standard and manual-stage prek workflows.

No open issues 🎉

_{Review effort: Balanced}

[DaanHoogland]

@jbampton , the prek standard job fails (also after retried) can you have a look?

[Copilot]

🟢 Approval recommended

The pull-request hook migration from pre-commit to prek has no active review issues.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The manual workflow can populate the shared cache with incomplete hook environments, forcing subsequent standard runs to reinstall dependencies without being able to update the cache on exact hits.

1 open issue

• Disable caching for the install-only manual job · New

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Human assessment is needed because an unpublished workflow-concurrency finding indicates obsolete hook runs may compete with current checks for runner capacity.

No open issues 🎉

1 resolved since last review

• Disable caching for the install-only manual job

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

Human review is needed to assess the runner-capacity impact of replacement workflows no longer canceling superseded PR runs.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The standard-hooks job fails before running any hooks because extra-args duplicates options already supplied by the pinned prek-action; use only --all-files or omit extra-args.

1 open issue

• Remove duplicate prek options from extra-args · New

_{Review effort: Balanced}