Skip to content

chore: recompile test-coverage-improver lock file#3287

Merged
lpcox merged 1 commit into
mainfrom
chore/recompile-test-improver
May 17, 2026
Merged

chore: recompile test-coverage-improver lock file#3287
lpcox merged 1 commit into
mainfrom
chore/recompile-test-improver

Conversation

@lpcox
Copy link
Copy Markdown
Collaborator

@lpcox lpcox commented May 17, 2026

Recompile and post-process the test-coverage-improver.lock.yml so CI uses a local npm build (npm ci && npm run build) instead of downloading a released binary.

Problem: Run #25976313126 failed with a 404 because the lock file on main tries to download v0.25.29 from GitHub Releases, which doesn't exist.

Fix: Post-processing replaces the binary download step with a local build, matching how all other smoke/test workflows work.

Recompile and post-process the lock file so CI uses a local npm
build instead of downloading a released binary (v0.25.29 does not
exist, causing 404 failures on main).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 17, 2026 00:30
@lpcox lpcox requested a review from Mossaka as a code owner May 17, 2026 00:30
@lpcox lpcox merged commit c7f8c7a into main May 17, 2026
46 of 49 checks passed
@lpcox lpcox deleted the chore/recompile-test-improver branch May 17, 2026 00:30
@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

  1. GitHub API: Failed — HTTP 401: Bad credentials (gh CLI not authenticated in this environment)
  2. Playwright: Passed — GitHub homepage loads, title contains 'GitHub'
  3. File verify: Passed — smoke-test-claude-25976924910.txt exists with expected content

Overall: FAIL (2/3 tests passed)

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

🔬 Smoke Test Results

Test Result
GitHub MCP connectivity ❌ 401 Bad credentials
GitHub.com HTTP ❌ Template vars unresolved (${{ steps.smoke-data.outputs.SMOKE_HTTP_CODE }})
File write/read ❌ Template vars unresolved (${{ steps.smoke-data.outputs.SMOKE_FILE_PATH }})

Overall: FAIL — Pre-computed step outputs were not substituted into the workflow prompt before agent execution. The ${{ steps.smoke-data.outputs.* }} expressions are unresolved, and GitHub MCP returned 401.

Author: @lpcox

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Offline) Mode

Test Result
GitHub MCP connectivity ❌ 401 Bad credentials
GitHub.com HTTP connectivity ⚠️ Template vars unexpanded
File write/read ⚠️ Template vars unexpanded
BYOK inference (agent responding)

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

Overall: FAIL — GitHub MCP returned 401; workflow template variables were not interpolated before agent execution.

🔑 BYOK report filed by Smoke Copilot BYOK

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

Updates the generated test-coverage-improver.lock.yml workflow so CI builds and uses the local AWF CLI (npm ci && npm run build) instead of attempting to download a non-existent released AWF binary, preventing the 404 failure seen in CI.

Changes:

  • Replaces the “install AWF binary from GitHub Releases” step with a local npm ci + npm run build and a /usr/local/bin/awf wrapper that runs dist/cli.js.
  • Adds --session-state-dir to the awf invocation and copies the resulting Copilot session state into the workflow logs directory.
  • Removes the explicit sparse-checkout path list from the activation job checkout configuration.
Show a summary per file
File Description
.github/workflows/test-coverage-improver.lock.yml Switches AWF installation from release download to local build and updates session-state handling for Copilot runs.

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: 0

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: FAIL
PRs: chore: recompile test-coverage-improver lock file; chore: run test coverage improver twice daily
GitHub MCP last 2 merged PRs: ❌
safeinputs-gh PR query: ❌
Playwright title contains GitHub: ✅
Tavily search: ❌
File write/read: ✅
Discussion interaction: ❌
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

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results: Gemini Engine

  1. GitHub MCP Testing: ❌ (mcpscripts command not found)
  2. GitHub.com Connectivity: ❌ (Status 000, error 35)
  3. File Writing Testing: ✅
  4. Bash Tool Testing: ✅

Overall status: FAIL

Warning

Firewall blocked 1 domain

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

  • localhost

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

network:
  allowed:
    - defaults
    - "localhost"

See Network Configuration for more information.

💎 Faceted by Smoke Gemini

@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 #3287 · ● 8.4M ·

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results — FAIL

Check Result
Redis PING ❌ Timeout (no response)
PostgreSQL pg_isready ❌ No response
PostgreSQL SELECT 1 ❌ No response

host.docker.internal resolves to 172.17.0.1 but ports 6379 and 5432 are unreachable (connection timeout). Service containers may not be running or are not bound to the expected host interface.

🔌 Service connectivity validated by Smoke Services

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