Recompile analytics-review with gh-aw v0.86.2 (fixes Copilot CLI ENOENT on toolcache hit) - #151
Merged
Merged
Conversation
…NT on toolcache hit) First workflow_dispatch dry-run (run 32010839654, PR #146) failed at "Execute GitHub Copilot CLI" with `spawn /usr/local/bin/copilot ENOENT`: the v0.85.4 harness hardcoded /usr/local/bin/copilot, but the install step hit the runner toolcache (/opt/hostedtoolcache/copilot-cli) and only appended to GITHUB_PATH. Known upstream bug, fixed in gh-aw v0.86.2 (github/gh-aw#52464): the execute step now resolves the binary via `command -v copilot` and stages it to ${RUNNER_TEMP}/gh-aw/bin/copilot before invoking the harness. Generated files only — source .md workflow/agent files unchanged (frontmatter/body hashes identical in the lock metadata). Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The first post-merge dry-run of the analytics reviewer (run 32010839654,
workflow_dispatchagainst PR #146) failed — but not in our workflow logic. All our pre-steps passed:jira-ticket.mdspawn /usr/local/bin/copilot ENOENTRoot cause (upstream gh-aw bug): the v0.85.4 compiled harness hardcoded
/usr/local/bin/copilot, but on runners with a toolcache hit the install step used/opt/hostedtoolcache/copilot-cli/1.0.78and only appended toGITHUB_PATH— never staging the binary at the hardcoded path. Fixed upstream in gh-aw v0.86.2 — see github/gh-aw#52464 (also #51068, #52327).What changed
Upgraded the gh-aw extension v0.85.4 → v0.86.2 and ran
gh aw compile. Generated files only:.github/workflows/analytics-review.lock.yml— lock metadata now showscompiler_version: v0.86.2;frontmatter_hash/body_hashare identical to before, confirming the source.mdfiles are untouched..github/aw/actions-lock.json—github/gh-aw-actions/setuppin bumped to the v0.86.2 SHA.The fix is visible in the lock's "Execute GitHub Copilot CLI" step: it now resolves the binary with
command -v copilot(which finds toolcache installs via PATH), stages it to${RUNNER_TEMP}/gh-aw/bin/copilot, and the harness invokes that staged path instead of the hardcoded/usr/local/bin/copilot. Fails fast with a clear error (exit 127) if the CLI genuinely isn't installed.Compile output: 0 errors, 0 warnings (one informational tip about
copilot-requests: writebilling — not adopted; we keepCOPILOT_GITHUB_TOKENto match care_fe).After merge
Re-run the dry-run: