Skip to content

Daily Test Coverage Improver - Fix Coverage Steps Network Issues#1996

Merged
devantler merged 2 commits intomainfrom
test/coverage-steps-network-fix-7bacd189982c4e71
Jan 30, 2026
Merged

Daily Test Coverage Improver - Fix Coverage Steps Network Issues#1996
devantler merged 2 commits intomainfrom
test/coverage-steps-network-fix-7bacd189982c4e71

Conversation

@botantler
Copy link
Contributor

@botantler botantler bot commented Jan 30, 2026

Summary

This PR fixes network connectivity issues preventing the coverage steps action from building the project and running tests.

Problem

The coverage steps action was failing with Forbidden errors when downloading Go modules from storage.googleapis.com proxy. Multiple critical packages could not be fetched:

  • modernc.org/libc@v1.66.10
  • modernc.org/sqlite@v1.40.0
  • google.golang.org/api@v0.252.0
  • github.com/klauspost/compress@v1.18.2
  • github.com/diskfs/go-diskfs@v1.7.0
  • Other transitive dependencies

This prevented Phase 3 of the Daily Test Coverage Improver workflow from proceeding.

Solution

Enable GitHub Actions' built-in Go module caching by adding two parameters to the setup-go action:

cache: true
cache-dependency-path: go.sum

This leverages GitHub's module cache to avoid repeated network downloads and works around the proxy connectivity issues.

Changes

  • Updated .github/actions/daily-test-improver/coverage-steps/action.yml
  • Added cache: true to setup-go step
  • Added cache-dependency-path: go.sum to setup-go step

Testing

The fix follows GitHub Actions best practices documented at:
https://github.com/actions/setup-go#caching-dependency-files-and-build-outputs

Once merged, the next workflow run should successfully:

  1. Cache Go modules on first run
  2. Reuse cached modules on subsequent runs
  3. Complete the build step without network errors
  4. Run tests and generate coverage reports

Related


What Happens Next

Once this PR is merged:

  • The coverage steps action will use module caching
  • Phase 3 of Daily Test Coverage Improver can proceed
  • Actual test coverage improvements can begin
  • The workflow will be able to systematically improve repository test coverage

AI generated by Daily Test Coverage Improver

Add cache: true and cache-dependency-path to setup-go action to prevent
network issues when downloading Go modules during coverage testing.

This fixes Forbidden errors from storage.googleapis.com proxy by
utilizing GitHub Actions' built-in Go module caching mechanism.
@botantler botantler bot added the bug label Jan 30, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

⚠️MegaLinter analysis: Success with warnings

⚠️ MARKDOWN / markdownlint - 46 errors
.github/aw/create-agentic-workflow.md:88 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Read the gh-aw instructions"]
.github/aw/create-agentic-workflow.md:248 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/create-shared-agentic-workflow.md:16 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Build on agentic workflows"]
.github/aw/create-shared-agentic-workflow.md:23 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Prefer Docker Solutions"]
.github/aw/create-shared-agentic-workflow.md:30 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Support Read-Only Tools"]
.github/aw/create-shared-agentic-workflow.md:37 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Move Write Operations to Safe ..."]
.github/aw/create-shared-agentic-workflow.md:44 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Process Agent Output in Safe J..."]
.github/aw/create-shared-agentic-workflow.md:57 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Documentation"]
.github/aw/create-shared-agentic-workflow.md:67 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:70 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:306 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:310 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:411 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:415 error MD003/heading-style Heading style [Expected: atx; Actual: setext]
.github/aw/create-shared-agentic-workflow.md:442 error MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "Good - specific version"]
.github/aw/create-shared-agentic-workflow.md:447 error MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "Good - SHA for immutability"]
.github/aw/create-shared-agentic-workflow.md:452 error MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "Acceptable - latest for develo..."]
.github/aw/create-shared-agentic-workflow.md:458 error MD001/heading-increment Heading levels should only increment by one level at a time [Expected: h2; Actual: h3]
.github/aw/create-shared-agentic-workflow.md:462 error MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "Read-only workspace mount"]
.github/aw/create-shared-agentic-workflow.md:472 error MD001/heading-increment Heading levels should only increment by one level at a time [Expected: h2; Actual: h3]
.github/aw/create-shared-agentic-workflow.md:476 error MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "Pattern: Pass through Docker w..."]
.github/aw/debug-agentic-workflow.md:19 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Example: Debugging from a work..."]
.github/aw/debug-agentic-workflow.md:25 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:39 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:53 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Prerequisites"]
.github/aw/debug-agentic-workflow.md:60 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Key Commands Available"]
.github/aw/debug-agentic-workflow.md:91 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:124 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:162 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:199 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:208 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:218 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:256 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/debug-agentic-workflow.md:468 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/update-agentic-workflow.md:48 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Read the gh-aw instructions"]
.github/aw/update-agentic-workflow.md:240 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/update-agentic-workflow.md:509 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/aw/upgrade-agentic-workflows.md:13 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Prerequisites"]
.github/aw/upgrade-agentic-workflows.md:20 error MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "Key Commands Available"]
.github/aw/upgrade-agentic-workflows.md:265 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/skills/refactor/SKILL.md:37 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/skills/refactor/SKILL.md:563 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
.github/workflows/update-docs.md:152 error MD028/no-blanks-blockquote Blank line inside blockquote
docs/src/content/docs/faq.md:204 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "``````"]
pkg/svc/chat/docs/faq.md:204 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "``````"]
vsce/README.md:100 error MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]

✅ Linters with no issues

actionlint, bash-exec, git_diff, hadolint, jscpd, jsonlint, lychee, markdown-table-formatter, prettier, prettier, shellcheck, shfmt, stylelint, syft, trivy-sbom, trufflehog, v8r, v8r, yamllint

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security

@devantler devantler marked this pull request as ready for review January 30, 2026 08:57
@devantler devantler self-requested a review as a code owner January 30, 2026 08:57
Copilot AI review requested due to automatic review settings January 30, 2026 08:57
Copy link
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 fixes network connectivity issues in the Daily Test Coverage Improver workflow by enabling GitHub Actions' built-in Go module caching to avoid repeated downloads from the Google proxy.

Changes:

  • Added cache: true and cache-dependency-path: go.sum to the setup-go step in the coverage-steps action

@botantler botantler bot enabled auto-merge January 30, 2026 09:03
@devantler devantler disabled auto-merge January 30, 2026 09:46
@devantler devantler added this pull request to the merge queue Jan 30, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2026
@devantler devantler added this pull request to the merge queue Jan 30, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2026
@devantler devantler added this pull request to the merge queue Jan 30, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2026
@devantler devantler added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 806c3c3 Jan 30, 2026
17 checks passed
@devantler devantler deleted the test/coverage-steps-network-fix-7bacd189982c4e71 branch January 30, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant