Skip to content

fix: use local eligibility action in workflows - #2116

Merged
stranske merged 2 commits into
mainfrom
codex/integration-autofix-pin-sync
May 14, 2026
Merged

stranske merged 2 commits into
mainfrom
codex/integration-autofix-pin-sync

Conversation

@stranske

@stranske stranske commented May 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • bring forward the non-conflicted replacement for fix: use local eligibility action in workflows #2009 onto current main
  • switch agent eligibility checks to the synced local action and sparse-checkout only that action before use
  • keep consumer templates and sync manifest aligned

Validation

  • python scripts/validate_template_sync.py
  • python scripts/validate_template_completeness.py
  • node --test .github/scripts/tests/agent-event-eligibility.test.js
  • python YAML parse for changed workflow/template files
  • git diff --check HEAD~1..HEAD

Supersedes #2009.

Copilot AI review requested due to automatic review settings May 14, 2026 23:28
@stranske-keepalive

stranske-keepalive Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Workflow source detected

PR #2116 now has valid workflow source context (origin=review_followup).

No linked GitHub issue is required for this PR.

@agents-workflows-bot

agents-workflows-bot Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: 0ddd032
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / guard
Required: core tests (3.12): ⏳ pending, core tests (3.13): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending

Coverage Overview

  • Coverage history entries: 1

Coverage Trend

Metric Value
Current 93.12%
Baseline 85.00%
Delta +8.12%
Minimum 70.00%
Status ✅ Pass

Top Coverage Hotspots (lowest coverage)

File Coverage Missing
src/cli_parser.py 81.8% 4
src/percentile_calculator.py 95.0% 1
src/aggregator.py 95.0% 2
src/__init__.py 100.0% 0
src/ndjson_parser.py 100.0% 0

Updated automatically; will refresh on subsequent CI/Docker completions.


Keepalive checklist

Scope

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

Copilot AI 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.

Pull request overview

This PR updates both the Workflows repo and the consumer templates to stop using the cross-repo agent-event-eligibility action reference and instead run eligibility checks via the synced local action (./.github/actions/agent-event-eligibility). To support local action execution without pulling the full repo, it adds a minimal actions/checkout sparse-checkout step for the action directory immediately before each eligibility step, and it updates legacy-autofix bridge messaging in the workflow header and sync manifest.

Changes:

  • Replace stranske/Workflows/.github/actions/agent-event-eligibility@main with ./.github/actions/agent-event-eligibility across affected workflows.
  • Add a sparse-checkout actions/checkout step to fetch only .github/actions/agent-event-eligibility before running the local action.
  • Clarify the legacy role of agents-autofix-dispatcher.yml and align .github/sync-manifest.yml description wording.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
templates/consumer-repo/.github/workflows/autofix.yml Sparse-checkout the local eligibility action and switch eligibility step to ./.github/actions/agent-event-eligibility.
templates/consumer-repo/.github/workflows/agents-guard.yml Same local eligibility action + sparse-checkout pattern for guard gating.
templates/consumer-repo/.github/workflows/agents-autofix-dispatcher.yml Add header comment clarifying legacy/compatibility purpose.
templates/consumer-repo/.github/workflows/agents-auto-pilot.yml Gate job now checks out and uses the local eligibility action.
templates/consumer-repo/.github/workflows/agents-auto-label.yml Add sparse-checkout + local eligibility action before running auto-label flow.
templates/consumer-repo/.github/workflows/agents-80-pr-event-hub.yml Add sparse-checkout + local eligibility action for consolidated event hub gating.
.github/workflows/autofix.yml Internal workflow now uses the local eligibility action via sparse-checkout.
.github/workflows/agents-moderate-connector.yml Internal workflow now uses the local eligibility action via sparse-checkout.
.github/workflows/agents-guard.yml Internal workflow now uses the local eligibility action via sparse-checkout.
.github/workflows/agents-bot-comment-handler.yml Internal workflow now uses the local eligibility action via sparse-checkout.
.github/workflows/agents-autofix-dispatcher.yml Add header comment clarifying legacy/compatibility purpose.
.github/workflows/agents-auto-pilot.yml Gate job now checks out and uses the local eligibility action.
.github/workflows/agents-auto-label.yml Add sparse-checkout + local eligibility action before running auto-label flow.
.github/workflows/agents-70-orchestrator.yml Eligibility job now checks out and uses the local eligibility action.
.github/sync-manifest.yml Update manifest description to reflect legacy nature of the autofix dispatcher bridge.

@stranske
stranske temporarily deployed to agent-high-privilege May 14, 2026 23:33 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants