Skip to content

fix: use local eligibility action in workflows - #2009

Closed
stranske wants to merge 1 commit into
mainfrom
codex-sync-review-1836
Closed

fix: use local eligibility action in workflows#2009
stranske wants to merge 1 commit into
mainfrom
codex-sync-review-1836

Conversation

@stranske

@stranske stranske commented May 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace floating cross-repo agent-event-eligibility action references with the synced local action path
  • add sparse checkout of the local eligibility action before each eligibility step
  • clarify agents-autofix-dispatcher as a legacy compatibility bridge and update the sync manifest wording

Validation

  • python YAML parse for 15 changed YAML files
  • node --test .github/scripts/tests/agent-event-eligibility.test.js
  • pytest -q tests/scripts/test_check_workflow_action_pins.py tests/workflows/test_bot_comment_handler.py::test_template_event_hub_uses_reusable_bot_comment_handler_defaults
  • actionlint with existing allowlist for workflow_job

Copilot AI review requested due to automatic review settings May 5, 2026 01:38
@stranske
stranske temporarily deployed to agent-high-privilege May 5, 2026 01:38 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@stranske-keepalive

Copy link
Copy Markdown
Contributor

Workflow source needed

PR #2009 needs either a linked GitHub issue or one valid non-issue Workflow Source before PR metadata automation can manage it safely.

Please do one of:

  • Add <!-- meta:issue:123 --> or a normal Closes #123 / Related to #123 line.
  • Check one Workflow Source option in the PR body.
  • Add a hidden marker such as <!-- workflow-source:local_request -->, <!-- workflow-source:manual_remote -->, <!-- workflow-source:review_followup -->, <!-- workflow-source:sync_campaign -->, or <!-- workflow-source:dependabot -->.
  • Add a workflow source label such as workflow:source-direct-pr, workflow:source-local-request, workflow:source-review-followup, workflow:source-sync, or workflow:no-automation.

Once a valid source is present, this warning will not be reposted.

@agents-workflows-bot

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: c268662
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: 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

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

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 multiple agent/autofix workflows (both in-repo and consumer templates) to stop depending on a floating cross-repo agent-event-eligibility action reference, and instead use the locally-synced composite action with a lightweight sparse checkout to make that local action available at runtime.

Changes:

  • Replace stranske/Workflows/.github/actions/agent-event-eligibility@main with ./.github/actions/agent-event-eligibility across affected workflows.
  • Add a minimal actions/checkout sparse-checkout step before each eligibility check so local action resolution works without a full repo checkout.
  • Clarify agents-autofix-dispatcher as a legacy compatibility bridge and adjust the sync-manifest description accordingly.

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 Switch eligibility to local action and add sparse checkout for it.
templates/consumer-repo/.github/workflows/agents-guard.yml Switch eligibility to local action and add sparse checkout for it.
templates/consumer-repo/.github/workflows/agents-autofix-dispatcher.yml Add header comment clarifying legacy/bridge role.
templates/consumer-repo/.github/workflows/agents-auto-pilot.yml Switch gate job eligibility to local action and add sparse checkout for it.
templates/consumer-repo/.github/workflows/agents-auto-label.yml Switch eligibility to local action and add sparse checkout for it.
templates/consumer-repo/.github/workflows/agents-80-pr-event-hub.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/autofix.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/agents-moderate-connector.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/agents-guard.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/agents-bot-comment-handler.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/agents-autofix-dispatcher.yml Add header comment clarifying legacy/bridge role.
.github/workflows/agents-auto-pilot.yml Switch gate job eligibility to local action and add sparse checkout for it.
.github/workflows/agents-auto-label.yml Switch eligibility to local action and add sparse checkout for it.
.github/workflows/agents-70-orchestrator.yml Switch eligibility to local action and add sparse checkout for it.
.github/sync-manifest.yml Update description to reflect legacy dispatch bridge behavior.

@stranske

Copy link
Copy Markdown
Owner Author

Closing as superseded by #2116, which brought this local eligibility-action change forward onto current main, addressed the current test expectations, passed Gate/Selftest, and merged in c5924e6.

@stranske stranske closed this May 14, 2026
@stranske
stranske deleted the codex-sync-review-1836 branch May 14, 2026 23:39
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