Fix workflow execution: coverage/build on workflow_dispatch, build on push to main

[Copilot]

Three critical workflow issues prevented CI from running correctly:

1. coverage.yaml didn't run when manually triggered via workflow_dispatch
2. cmake-build.yaml didn't run when manually triggered via workflow_dispatch
3. cmake-build.yaml didn't run on push to main/develop branches

Root Causes

Incorrect detect-changes handling: Jobs checked explicit event names (github.event_name == 'workflow_dispatch') instead of testing whether detect-changes was skipped. When detect-changes skips (workflow_dispatch, act, etc.), its result is 'skipped' not 'success', so conditions failed.

Missing push event: The pre-check job enumerated specific events but omitted push despite the workflow being triggered by push events.

Changes

coverage.yaml

# Before: explicit event checks
coverage:
  if: >
    github.event_name == 'workflow_dispatch' ||
    needs.pre-check.outputs.is_act == 'true' ||
    (needs.detect-changes.result == 'success' && ...)

# After: result-based pattern
coverage:
  if: >
    needs.detect-changes.result == 'skipped' ||
    (needs.detect-changes.result == 'success' && ...)

cmake-build.yaml

# Fix 1: Same result-based pattern for build job
build:
  if: >
    needs.detect-changes.result == 'skipped' ||
    (needs.detect-changes.result == 'success' && ...)

# Fix 2: Add push to pre-check condition
pre-check:
  if: >
    github.event_name == 'workflow_dispatch' ||
    github.event_name == 'pull_request' ||
    github.event_name == 'push' ||  # ← Added
    github.event_name == 'workflow_call' ||
    (issue_comment...)

This pattern matches all other check workflows (python-check, clang-tidy-check, etc.) and is more maintainable—no need to update conditions when adding new trigger events.

Original prompt

According to https://github.com/Framework-R-D/phlex/actions/runs/21765323541 (workflow_dispatch), we are not running the coverage unconditionally as expected. Please check and fix the workflow logic in coverage.yaml and other workflows if appropriate.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

Adjusts the Code Coverage GitHub Actions workflow so the coverage job runs correctly when the workflow is manually triggered via workflow_dispatch, even though detect-changes is skipped in that event mode.

Changes:

• Update coverage job gating to run when detect-changes is skipped, or when it ran and detected relevant changes.
• Add a needs.pre-check.result == 'success' guard to detect-changes and coverage-skipped for consistency with other workflows.