Skip to content

Sync source AWF schema copy with documented schema#3567

Merged
lpcox merged 1 commit into
mainfrom
copilot/debug-test-coverage-reporter
May 22, 2026
Merged

Sync source AWF schema copy with documented schema#3567
lpcox merged 1 commit into
mainfrom
copilot/debug-test-coverage-reporter

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

The Test Coverage Reporter workflow was failing in its coverage step because the schema parity test detected drift between the checked-in runtime schema copy and the documented schema. This change removes that drift so the reporter can complete instead of failing on src/schema.test.ts.

  • Root cause

    • src/awf-config-schema.json had fallen behind docs/awf-config.schema.json.
    • The workflow failure was a test failure, not a coverage-reporting logic failure.
  • Change

    • Sync src/awf-config-schema.json to match the documented schema copy.
    • Bring over the updated excludeEnv/API proxy credential-isolation schema text so the source and docs describe the same contract.
  • Effect

    • Restores the invariant enforced by src/schema.test.ts.
    • Unblocks the Test Coverage Reporter workflow without changing workflow behavior.
{
  "description": "When excluding LLM API credentials via excludeEnv, apiProxy.enabled must also be enabled so the agent receives proxy-backed placeholders instead of losing credentials entirely."
}

Copilot AI linked an issue May 21, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Debug workflow failure for test coverage reporter Sync source AWF schema copy with documented schema May 22, 2026
Copilot finished work on behalf of lpcox May 22, 2026 00:00
Copilot AI requested a review from lpcox May 22, 2026 00:00
@lpcox lpcox marked this pull request as ready for review May 22, 2026 00:09
Copilot AI review requested due to automatic review settings May 22, 2026 00:09
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.

Copilot wasn't able to review any files in this pull request.

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results ✅

Test Result
GitHub API (recent-prs.json) ✅ PASS — 2 PR entries confirmed
GitHub check (playwright_check) ✅ PASS — playwright_check=✅ PASS
File verify (smoke-test-claude) ✅ PASS — file exists
Overall PASS

All smoke tests passed. Claude engine validation successful.

💥 [THE END] — Illustrated by Smoke Claude

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

Test Status
GitHub MCP (github-list_pull_requests)
GitHub.com connectivity
File write/read (smoke-test-copilot-26259941147.txt)

Overall: PASS

PR: "Sync source AWF schema copy with documented schema" — author @Copilot, assignees @lpcox @Copilot

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

✅ GitHub MCP: docs(spec): add port information to /reflect endpoint sections; fix(dind-probe): address review feedback from #3554
❌ Safe Inputs GH CLI: safeinputs-gh unavailable
✅ Playwright: GitHub title verified
❌ Tavily: no registered search tool returned results
✅ File/Bash: smoke-test-codex-26259941150.txt verified
✅ Discussion: oracle comment posted to #3562 via fallback query
✅ Build AWF: 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

🏗️ 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 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 #3567 · ● 5.7M ·

@github-actions
Copy link
Copy Markdown
Contributor

Gemini Engine Smoke Test Results

  • GitHub MCP Testing: ❌ (mcpscripts command not found)
  • GitHub.com Connectivity: ❌ (SSL connect error 35)
  • File Writing Testing: ✅
  • 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

Smoke Test Results

  • Redis PING: ❌ (timeout — host.docker.internal:6379 unreachable)
  • PostgreSQL pg_isready: ❌ (no response on host.docker.internal:5432)
  • 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 running or the hostname is not resolvable.

🔌 Service connectivity validated by Smoke Services

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Offline Mode)

Test Result
GitHub MCP ✅ PR #3567 "Sync source AWF schema copy with documented schema" fetched
GitHub.com connectivity ⚠️ Template var unsubstituted (${{ steps.smoke-data.outputs.SMOKE_HTTP_CODE }})
File write/read ⚠️ Template var unsubstituted (${{ steps.smoke-data.outputs.SMOKE_FILE_PATH }})
BYOK inference (api-proxy → api.githubcopilot.com) ✅ Responding via BYOK offline mode

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

PR author: @Copilot · Assignees: @lpcox, @Copilot

Overall: PARTIAL — BYOK inference and MCP connectivity confirmed ✅; pre-step outputs not substituted ⚠️

🔑 BYOK report filed by Smoke Copilot BYOK

@lpcox lpcox merged commit 91bd370 into main May 22, 2026
37 of 42 checks passed
@lpcox lpcox deleted the copilot/debug-test-coverage-reporter branch May 22, 2026 01:55
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] Test Coverage Reporter failed

3 participants