Skip to content

fix: update workflow test SHA assertions after recompile#4720

Merged
lpcox merged 1 commit into
mainfrom
fix-workflow-test-sha-assertions
Jun 11, 2026
Merged

fix: update workflow test SHA assertions after recompile#4720
lpcox merged 1 commit into
mainfrom
fix-workflow-test-sha-assertions

Conversation

@lpcox

@lpcox lpcox commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

PR #4714 recompiled all workflows, bumping github/gh-aw-actions/setup from v0.79.2 (9b1d7307) to v0.79.4 (d059700c), but the test assertions in security-guard-workflow.test.ts and test-coverage-improver-workflow.test.ts weren't updated — causing npm test to fail on main.

This fixes both tests to match the current lock file content.

PR #4714 recompiled all workflows, bumping the pinned action from
v0.79.2 (9b1d7307) to v0.79.4 (d059700c) but didn't update the
test assertions that verify the lock files contain the correct SHA.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 11, 2026 00:16

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

Updates workflow-lockfile string assertions in CI tests to match the newly recompiled workflow lock files (from PR #4714), where github/gh-aw-actions/setup was bumped from v0.79.2 to v0.79.4 and re-pinned to a new commit SHA. This unblocks npm test on main by aligning the tests with the current locked workflow content.

Changes:

  • Update test-coverage-improver-workflow.test.ts to expect github/gh-aw-actions/setup@d059700c… # v0.79.4.
  • Update security-guard-workflow.test.ts to expect github/gh-aw-actions/setup@d059700c… # v0.79.4.
Show a summary per file
File Description
scripts/ci/test-coverage-improver-workflow.test.ts Updates the pinned gh-aw-actions/setup SHA/version assertion to match the current lock workflow content.
scripts/ci/security-guard-workflow.test.ts Updates the pinned gh-aw-actions/setup SHA/version assertion to match the current lock workflow content.

Copilot's findings

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 96.44% 96.48% 📈 +0.04%
Statements 96.35% 96.39% 📈 +0.04%
Functions 98.77% 98.77% ➡️ +0.00%
Branches 90.74% 90.77% 📈 +0.03%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/config-writer.ts 89.3% → 90.9% (+1.65%) 89.3% → 90.9% (+1.65%)

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

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Direct Mode) — PASS

✅ GitHub MCP connectivity — verified
✅ GitHub.com connectivity (HTTP 200)
✅ BYOK inference path active (api-proxy → api.githubcopilot.com)
✅ Direct BYOK mode confirmed (COPILOT_PROVIDER_API_KEY via api-proxy sidecar)

Running in direct BYOK mode with real API key held by sidecar, agent receives placeholder credential.

Author: @lpcox

🔑 BYOK report filed by Smoke Copilot BYOK

@github-actions

Copy link
Copy Markdown
Contributor

@lpcox

  • GitHub MCP Testing: ✅
  • GitHub.com Connectivity: ✅
  • File Write/Read Test: ✅
  • BYOK Inference Test: ✅

Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw)

Overall: PASS

🔑 BYOK (AOAI api-key) report filed by Smoke Copilot BYOK AOAI (api-key)

@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

@lpcox
PR titles:

  • fix: update workflow test SHA assertions after recompile
    GitHub MCP Testing: ✅
    GitHub.com Connectivity: ✅
    File Write/Read Test: ✅
    BYOK Inference Test: ✅
    Running in direct BYOK mode (AWF_AUTH_TYPE=github-oidc + AWF_AUTH_AZURE_* + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) authenticated via Microsoft Entra
    Overall: PASS

Warning

Firewall blocked 1 domain

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

  • api.openai.com

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

network:
  allowed:
    - defaults
    - "api.openai.com"

See Network Configuration for more information.

🪪 BYOK (AOAI Entra) report filed by Smoke Copilot BYOK AOAI (Entra)

@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

Last merged PRs:

  • chore: recompile all workflows with latest gh-aw
  • feat: emit AI credits as OTEL span attributes

GitHub read: ❌
Safe Inputs GH CLI: ❌
Playwright: ✅
File write/read: ✅
Discussion: ✅
Build: ✅
Overall: 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

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: GitHub Actions Services Connectivity

Check Result
Redis PING ❌ No response (connection blocked/timeout)
PostgreSQL pg_isready no response
PostgreSQL SELECT 1 ❌ No response (connection blocked/timeout)

Overall: ❌ FAIL

host.docker.internal resolves to 172.17.0.1 but all connections to ports 6379 and 5432 time out — the AWF firewall is blocking non-HTTP/HTTPS traffic to these ports.

🔌 Service connectivity validated by Smoke Services

@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 passed ✅ PASS
Go env passed ✅ PASS
Go uuid passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx passed ✅ PASS
Node.js execa passed ✅ PASS
Node.js p-limit 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 #4720 · 559.3 AIC · ⊞ 33.8K ·

@github-actions

Copy link
Copy Markdown
Contributor

GitHub API: ✅ PASS
GitHub check: ✅ PASS
File verify: ✅ PASS

Total: PASS

💥 [THE END] — Illustrated by Smoke Claude

@lpcox lpcox merged commit 6c1f2b6 into main Jun 11, 2026
75 of 77 checks passed
@lpcox lpcox deleted the fix-workflow-test-sha-assertions branch June 11, 2026 01:19
@github-actions github-actions Bot mentioned this pull request Jun 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot PAT Auth — Results

Test Status
GitHub MCP connectivity ✅ PR #4720 fetched
GitHub.com HTTP connectivity ✅ (pre-step confirmed)
File write/read (smoke-test-copilot-pat-27318057018.txt) ✅ "Smoke test passed for Copilot PAT"

Overall: PASS 🟢

Author: @lpcox · Auth mode: PAT (COPILOT_GITHUB_TOKEN)

🔑 PAT report filed by Smoke Copilot PAT

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test Results

Overall: PASS · Author: @lpcox

📰 BREAKING: Report filed by Smoke Copilot

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.

2 participants