Skip to content

Recompile analytics-review with gh-aw v0.86.2 (fixes Copilot CLI ENOENT on toolcache hit) - #151

Merged
amjithtitus09 merged 1 commit into
mainfrom
amjithtitus09-analytics-review-bot
Aug 17, 2026
Merged

Recompile analytics-review with gh-aw v0.86.2 (fixes Copilot CLI ENOENT on toolcache hit)#151
amjithtitus09 merged 1 commit into
mainfrom
amjithtitus09-analytics-review-bot

Conversation

@amjithtitus09

Copy link
Copy Markdown
Member

Why

The first post-merge dry-run of the analytics reviewer (run 32010839654, workflow_dispatch against PR #146) failed — but not in our workflow logic. All our pre-steps passed:

  • ✅ JIRA fetch wrote jira-ticket.md
  • ✅ Skills checkout (pinned SHA) OK
  • ✅ sqlglot lint OK
  • ❌ "Execute GitHub Copilot CLI" → spawn /usr/local/bin/copilot ENOENT

Root 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.78 and only appended to GITHUB_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 shows compiler_version: v0.86.2; frontmatter_hash/body_hash are identical to before, confirming the source .md files are untouched.
  • .github/aw/actions-lock.jsongithub/gh-aw-actions/setup pin 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: write billing — not adopted; we keep COPILOT_GITHUB_TOKEN to match care_fe).

After merge

Re-run the dry-run:

gh workflow run analytics-review.lock.yml -f pr_number=146

…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>
@amjithtitus09
amjithtitus09 merged commit 8ce12f2 into main Aug 17, 2026
4 of 5 checks passed
@amjithtitus09
amjithtitus09 deleted the amjithtitus09-analytics-review-bot branch August 17, 2026 09:25
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.

1 participant