Skip to content

[WIP] Remove redundant live CI re-check from Avenger Step 0 prompt - #50322

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/squad-plan-remove-redundant-ci-check
Closed

pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/squad-plan-remove-redundant-ci-check

Conversation

Copilot AI commented Aug 4, 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.


This section details on the original issue you should resolve

<issue_title>[squad-plan] Remove redundant live CI re-check from Avenger Step 0 prompt</issue_title>
<issue_description>## Objective
Simplify Avenger's Step 0 so it uses the already-provided CI status outputs instead of making a duplicate gh run list call.

Issue context

Issue #50312 found Avenger already receives ${{ needs.check_ci_status.outputs.ci_status }} and ci_run_id from the check_ci_status job, but the prompt tells the agent to re-check live CI via gh run list in Step 0. That extra tool call adds avoidable tokens/cache-read volume and latency (~10-15% estimated AIC savings).

Implementation guidance

  • Edit .github/workflows/avenger.md.
  • Update Step 0 instructions to:
    • immediately noop if the passed-in CI status is success
    • proceed directly to the repair sequence if CI status is failure
  • Remove the redundant live API re-verification block unless there is a strong documented reason to keep a smaller/leaner version (e.g., staleness check only when ambiguous).
  • Keep the prompt concise and consistent with the workflow's existing job outputs.
  • Run make recompile after editing so the .lock.yml is regenerated.

Acceptance criteria

  • Avenger prompt no longer instructs the agent to run a redundant gh run list in Step 0 when CI status is already known.
  • Step 0 clearly uses needs.check_ci_status.outputs.ci_status as the source of truth for initial gating.
  • make recompile succeeds and the generated lock file updates cleanly with no unrelated diffs.

Dependency / ordering notes

Independent of the model-mismatch fix; can be implemented in parallel. Should be validated (recompiled) after any other Avenger markdown edits to avoid merge conflicts in the .lock.yml.

Generated by 🧑‍🤝‍🧑 Squad Plan · auto · 37.1 AIC · ⌖ 2.24 AIC · ⊞ 9.5K · ◷
Comment /squad-plan to run again

Comments on the Issue (you are @copilot in this section)

@pelikhan pelikhan closed this Aug 4, 2026
Copilot AI requested a review from pelikhan August 4, 2026 15:58
Copilot stopped work on behalf of pelikhan due to an error August 4, 2026 15:58
@pelikhan
pelikhan deleted the copilot/squad-plan-remove-redundant-ci-check branch August 18, 2026 23:22
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.

[squad-plan] Remove redundant live CI re-check from Avenger Step 0 prompt

2 participants