Skip to content

Harden gh-aw extension install in CI/CD gaps assessment workflow#2904

Merged
lpcox merged 3 commits into
mainfrom
copilot/fix-ci-cd-pipelines-integration-tests
May 11, 2026
Merged

Harden gh-aw extension install in CI/CD gaps assessment workflow#2904
lpcox merged 3 commits into
mainfrom
copilot/fix-ci-cd-pipelines-integration-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 11, 2026

Bug Fix

What was the bug?

The scheduled CI/CD Pipelines and Integration Tests Gap Assessment workflow failed on main because gh extension install github/gh-aw exited on HTTP 401: Bad credentials during extension install, aborting the agent job before execution.

How did you fix it?

  • Install-path resilience in workflow lock file
    • Updated .github/workflows/ci-cd-gaps-assessment.lock.yml to keep the current authenticated install path first, then retry once without GH_TOKEN if the initial install fails with token-related auth issues.
  • Failure-mode containment
    • Preserved existing behavior for successful authenticated installs.
    • Added only a narrow fallback around extension installation; no other workflow stages or permissions were changed.
if ! gh extension install github/gh-aw; then
  echo "Initial install failed, retrying without GH_TOKEN..."
  GH_TOKEN="" gh extension install github/gh-aw
fi

Testing

  • Workflow-only corrective change; no new product behavior or interface changes.

Copilot AI changed the title [WIP] Fix CI/CD pipelines and integration tests gap assessment failure Harden gh-aw extension install in CI/CD gaps assessment workflow May 11, 2026
Copilot finished work on behalf of lpcox May 11, 2026 13:59
Copilot AI requested a review from lpcox May 11, 2026 13:59
@lpcox lpcox marked this pull request as ready for review May 11, 2026 14:59
@lpcox lpcox requested a review from Mossaka as a code owner May 11, 2026 14:59
Copilot AI review requested due to automatic review settings May 11, 2026 14:59
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 11, 2026

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 87.84% 87.91% 📈 +0.07%
Statements 87.77% 87.85% 📈 +0.08%
Functions 82.78% 82.78% ➡️ +0.00%
Branches 79.72% 79.76% 📈 +0.04%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/container-lifecycle.ts 87.1% → 88.2% (+1.14%) 87.5% → 88.6% (+1.11%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

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 hardens the scheduled CI/CD Pipelines and Integration Tests Gap Assessment workflow by making gh-aw extension installation more resilient when an authenticated install fails (e.g., due to HTTP 401: Bad credentials), reducing the chance the job aborts before running the agent.

Changes:

  • Wraps gh extension install github/gh-aw with a single retry that clears GH_TOKEN on failure.
  • Removes an extraneous trailing blank line at the end of the workflow file.
Show a summary per file
File Description
.github/workflows/ci-cd-gaps-assessment.lock.yml Adds a fallback retry path for gh-aw extension install to mitigate token-auth failures in CI.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread .github/workflows/ci-cd-gaps-assessment.lock.yml Outdated
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

Test Result
GitHub MCP (last 2 merged PRs) ❌ Auth failed
Playwright (github.com title) ✅ PASS
File Writing ✅ PASS
Bash Tool Verification ✅ PASS

Overall: FAIL — GitHub MCP auth issue (401). Playwright, file ops, and bash all working.

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK — Results

Test Result
GitHub MCP connectivity ❌ 401 Bad credentials
GitHub.com HTTP ⚠️ Pre-step data unavailable (template vars not expanded)
File write/read ⚠️ Pre-step data unavailable (template vars not expanded)
BYOK inference (agent → api-proxy → api.githubcopilot.com)

Running in BYOK offline mode (COPILOT_OFFLINE=true) via api-proxy → api.githubcopilot.com.

Overall: FAIL — GitHub MCP returned 401; pre-step outputs not injected into agent environment.

🔑 BYOK report filed by Smoke Copilot BYOK

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Smoke Test Results

Test Status
GitHub MCP connectivity ❌ (401 - token not available to agent)
GitHub.com HTTP connectivity
File write/read (smoke-test-copilot-25679185053.txt)

Overall: PASS (MCP auth is a known agent limitation; infra tests pass)

cc @lpcox

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color 1/1 passed ✅ PASS
Go env 1/1 passed ✅ PASS
Go uuid 1/1 passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx All passed ✅ PASS
Node.js execa All passed ✅ PASS
Node.js p-limit All passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #2904 · ● 489.5K ·

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

  • Redis PING: ❌ (connection timeout)
  • PostgreSQL pg_isready: ❌ (no response)
  • PostgreSQL SELECT 1: ❌ (skipped due to pg_isready failure)

Overall: FAILhost.docker.internal is not reachable from this runner environment. Service containers may not be configured or the hostname is not resolvable.

🔌 Service connectivity validated by Smoke Services

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: FAIL
✅ Last merged PRs: refactor: split src/cli.ts into focused modules; [docs] Add --anthropic-auto-cache and --anthropic-cache-tail-ttl documentation
❌ SafeInputs GH: tool unavailable; fallback PRs: Harden gh-aw extension install in CI/CD gaps assessment workflow; refactor: split agent-environment.test.ts into 4 focused test modules
✅ Playwright: GitHub title verified
❌ Tavily: no search command exposed
✅ File/bash: smoke-test-codex-25679185380.txt verified
❌ Discussion query: tool unavailable; ✅ oracle comment added to #2913 via fallback
✅ Build: npm ci && npm run build
Overall status: FAIL

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "registry.npmjs.org"

See Network Configuration for more information.

🔮 The oracle has spoken through Smoke Codex

@lpcox lpcox merged commit 5955193 into main May 11, 2026
60 of 64 checks passed
@lpcox lpcox deleted the copilot/fix-ci-cd-pipelines-integration-tests branch May 11, 2026 15:46
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.

[aw] CI/CD Pipelines and Integration Tests Gap Assessment failed

3 participants