Skip to content

test: add direct codex test workflow#108

Closed
stranske wants to merge 1 commit intomainfrom
test-codex-direct
Closed

test: add direct codex test workflow#108
stranske wants to merge 1 commit intomainfrom
test-codex-direct

Conversation

@stranske
Copy link
Copy Markdown
Owner

Test workflow to debug why reusable workflow calls aren't creating jobs.

This is a minimal workflow with workflow_dispatch trigger to test the reusable-codex-run.yml directly.

Copilot AI review requested due to automatic review settings December 24, 2025 12:58
@agents-workflows-bot
Copy link
Copy Markdown
Contributor

⚠️ Action Required: Unable to determine source issue for PR #108. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 24, 2025

Automated Status Summary

Head SHA: a02cc34
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / Enforce agents workflow protections
Required: core tests (3.11): ⏳ pending, core tests (3.12): ⏳ 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
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 24, 2025

Keepalive loop status for PR #108

  • Action: wait (missing-agent-label)
  • Gate conclusion: failure
  • Tasks: 0/0 complete
  • Iteration: 0/5
  • Keepalive enabled: no
  • Autofix enabled: no

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 adds a test workflow to diagnose issues with reusable workflow calls not creating jobs. It provides a minimal, manually-triggerable workflow that directly calls the reusable-codex-run.yml workflow.

  • Adds a new workflow_dispatch-triggered workflow for testing the Codex reusable workflow
  • Implements a preflight job to check secret availability before execution
  • Includes a summary job to report results of the test run

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

run: |
echo "CODEX_AUTH_JSON present: $HAS_CODEX_AUTH"
echo "WORKFLOWS_APP_ID present: $HAS_APP_ID"
echo "secrets_ok=true" >> "$GITHUB_OUTPUT"
Copy link

Copilot AI Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The preflight job always sets secrets_ok=true regardless of whether the secrets are actually present. This means the test-codex job will always run even if the required secrets are missing, which may not provide useful test results. Consider adding conditional logic similar to the production workflow to only set secrets_ok=true when at least one of the required secrets is present, or document that this is intentional for testing purposes.

Suggested change
echo "secrets_ok=true" >> "$GITHUB_OUTPUT"
if [ "$HAS_CODEX_AUTH" = "true" ] || [ "$HAS_APP_ID" = "true" ]; then
echo "secrets_ok=true" >> "$GITHUB_OUTPUT"
else
echo "secrets_ok=false" >> "$GITHUB_OUTPUT"
fi

Copilot uses AI. Check for mistakes.
@stranske stranske temporarily deployed to agent-high-privilege December 24, 2025 13:02 — with GitHub Actions Inactive
Copy link
Copy Markdown
Owner Author

Abandoning this test workflow approach. Found the root cause: PR #107 added a preflight dependency to run-codex which caused GitHub Actions to not create the reusable workflow job. See PR #109 for the fix.

@stranske stranske closed this Dec 24, 2025
@stranske stranske deleted the test-codex-direct branch December 25, 2025 23:00
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