[WIP] Fix failing GitHub Actions job for CLI Completion by Copilot · Pull Request #42379 · github/gh-aw · GitHub
Skip to content

[WIP] Fix failing GitHub Actions job for CLI Completion#42379

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-cli-completion-job
Jun 30, 2026
Merged

[WIP] Fix failing GitHub Actions job for CLI Completion#42379
pelikhan merged 2 commits into
mainfrom
copilot/fix-cli-completion-job

Conversation

Copilot AI commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Fix the failing GitHub Actions job "Integration: CLI Completion & Other"
Analyze the Actions logs, identify the root cause of the failure, and implement a fix.
Check run ID: 84176756795
Job URL: https://github.com/github/gh-aw/actions/runs/28408665111/job/84176756795

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

@pelikhan pelikhan marked this pull request as ready for review June 30, 2026 01:48
Copilot AI review requested due to automatic review settings June 30, 2026 01:48
@pelikhan pelikhan merged commit 259e66d into main Jun 30, 2026
@pelikhan pelikhan deleted the copilot/fix-cli-completion-job branch June 30, 2026 01:48
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

Hey @app/copilot-swe-agent 👋 — nice catch on the WorkflowStatus struct tags! Changing Status and TimeRemaining from console:"-" (suppressed) to console:"header:state" and console:"header:remaining" is a clean, targeted fix to restore the CLI Completion integration test.

A couple of things would help before this is ready to merge:

  • Add tests — the struct tag change directly affects console rendering output, but no test files were updated. A test confirming that WorkflowStatus renders state and remaining header columns (and doesn’t suppress them) would give reviewers confidence the fix holds and won’t silently regress.
  • Update the PR description — the current body contains the original task prompt rather than an explanation of what was changed and why. A brief summary of the root cause (fields hidden by "-" tag) and the fix applied would make this much easier to review and merge.

If you’d like to continue with a coding agent:

In pkg/cli/status_command.go, WorkflowStatus.Status is now tagged `console:"header:state"` and WorkflowStatus.TimeRemaining is tagged `console:"header:remaining"` (previously both used `console:"-"`, suppressing them from output).

Please:
1. Add tests (in pkg/cli/status_command_test.go, creating the file if it doesn’t exist) that verify a WorkflowStatus value renders a console table with "state" and "remaining" columns visible.
2. Update the PR body to describe: the root cause (Status/TimeRemaining were tagged console:"-", hiding them) and the fix applied.

Generated by ✅ Contribution Check · 362.2 AIC · ⌖ 23.3 AIC · ⊞ 6.3K ·

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 the status command’s console table rendering so workflow state and time remaining are shown in text output, addressing the failing integration job that expected these values to appear.

Changes:

  • Expose WorkflowStatus.Status as a visible console table column (header state).
  • Expose WorkflowStatus.TimeRemaining as a visible console table column (header remaining).
Show a summary per file
File Description
pkg/cli/status_command.go Makes workflow status/state and time remaining visible in the struct-tag-driven console table output.

Review details

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Low

Comment thread pkg/cli/status_command.go
WorkflowListItem
Status string `json:"status" console:"-"`
TimeRemaining string `json:"time_remaining" console:"-"`
Status string `json:"status" console:"header:state"`
@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants