Fix Go network access for test coverage workflow - #10892
Conversation
Allow the sandboxed coverage agent to download Go modules while running builds and tests. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0296435f-93f6-4124-aa30-ffaa2713aec7
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0296435f-93f6-4124-aa30-ffaa2713aec7
There was a problem hiding this comment.
Pull request overview
Allows the Test Coverage Improver to download Go dependencies, resolving #10887.
Changes:
- Enables the curated Go network allowlist.
- Regenerates the compiled workflow with Go module domains.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/test-coverage-improver.md |
Adds Go network access. |
.github/workflows/test-coverage-improver.lock.yml |
Regenerates firewall configuration. |
Review details
Tip
Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Balanced
🔒 mcpg Read-Only Stress — defaultSurface coverage: MCP tool calls + proxied CLI (REST) + GraphQL mutations
Gateway enforcement (Part B): Write tools are stripped from the gateway-exposed tool list. Attempts return MCP error CLI enforcement (Parts D/E): Overall: PASS References: §31270010804
|
🔒 mcpg Read-Only Stress — docker-sbxSurface coverage: MCP tool calls + proxied CLI (REST) + GraphQL mutations
Overall: PASS Note: Part B writes were refused with
|
Summary
Closes #10887