fix: remove duplicate TestWithResponseLogging causing lint failure on merge

[Copilot]

The merge of PR #1567 (copilot/fix-duplicate-code-issues) into main introduced a duplicate TestWithResponseLogging declaration — the function existed in both internal/server/transport_test.go (added by PR #1567) and internal/server/http_helpers_test.go (already on main), causing go vet to fail with redeclared in this block.

Changes

• internal/server/transport_test.go — removed the duplicate TestWithResponseLogging table-driven test; the canonical version (with additional _PreservesHeaders and _ReturnsHTTPHandler coverage) lives in http_helpers_test.go on main

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• example.com
  • Triggering command: /tmp/go-build38872931/b294/launcher.test /tmp/go-build38872931/b294/launcher.test -test.testlogfile=/tmp/go-build38872931/b294/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true p/go-build nMbr/CXVcexE0oORyX_rbnMbr x_amd64/vet (dns block)
  • Triggering command: /tmp/go-build4103510633/b298/launcher.test /tmp/go-build4103510633/b298/launcher.test -test.testlogfile=/tmp/go-build4103510633/b298/testlog.txt -test.paniconexit0 -test.timeout=10m0s /home/REDACTED/.cache/go-build/98/98871f8707f598743bf593e773062a4aa5b2eb977379c9e4b23ff829ed477ad5-d .cfg 64/src/net/cgo_unix.go ache/go/1.25.7/x64/pkg/tool/linu-nilfunc --gdwarf-5 --64 -o ache/go/1.25.7/x64/pkg/tool/linu-tests -o 7689569/b210/_pkg_.a -trimpath 64/pkg/tool/linux_amd64/vet -p crypto/md5 -lang=go1.25 8YH9ufX/ESoPIYBZ-buildtags (dns block)
• invalid-host-that-does-not-exist-12345.com
  • Triggering command: /tmp/go-build38872931/b276/config.test /tmp/go-build38872931/b276/config.test -test.testlogfile=/tmp/go-build38872931/b276/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true 64/src/runtime/cgo 64/src/crypto/internal/fips140/cgolang.org/x/sys/unix 86_64/as (dns block)
  • Triggering command: /tmp/go-build4103510633/b280/config.test /tmp/go-build4103510633/b280/config.test -test.testlogfile=/tmp/go-build4103510633/b280/testlog.txt -test.paniconexit0 -test.timeout=10m0s -I .cfg 7689569/b164/ x_amd64/vet --gdwarf-5 --64 -o x_amd64/vet 7689�� g_.a rg/x/term@v0.38.-ifaceassert ache/go/1.25.7/x-nilfunc --gdwarf-5 --64 -o ache/go/1.25.7/x-tests (dns block)
• nonexistent.local
  • Triggering command: /tmp/go-build38872931/b294/launcher.test /tmp/go-build38872931/b294/launcher.test -test.testlogfile=/tmp/go-build38872931/b294/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true p/go-build nMbr/CXVcexE0oORyX_rbnMbr x_amd64/vet (dns block)
  • Triggering command: /tmp/go-build4103510633/b298/launcher.test /tmp/go-build4103510633/b298/launcher.test -test.testlogfile=/tmp/go-build4103510633/b298/testlog.txt -test.paniconexit0 -test.timeout=10m0s /home/REDACTED/.cache/go-build/98/98871f8707f598743bf593e773062a4aa5b2eb977379c9e4b23ff829ed477ad5-d .cfg 64/src/net/cgo_unix.go ache/go/1.25.7/x64/pkg/tool/linu-nilfunc --gdwarf-5 --64 -o ache/go/1.25.7/x64/pkg/tool/linu-tests -o 7689569/b210/_pkg_.a -trimpath 64/pkg/tool/linux_amd64/vet -p crypto/md5 -lang=go1.25 8YH9ufX/ESoPIYBZ-buildtags (dns block)
• slow.example.com
  • Triggering command: /tmp/go-build38872931/b294/launcher.test /tmp/go-build38872931/b294/launcher.test -test.testlogfile=/tmp/go-build38872931/b294/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true p/go-build nMbr/CXVcexE0oORyX_rbnMbr x_amd64/vet (dns block)
  • Triggering command: /tmp/go-build4103510633/b298/launcher.test /tmp/go-build4103510633/b298/launcher.test -test.testlogfile=/tmp/go-build4103510633/b298/testlog.txt -test.paniconexit0 -test.timeout=10m0s /home/REDACTED/.cache/go-build/98/98871f8707f598743bf593e773062a4aa5b2eb977379c9e4b23ff829ed477ad5-d .cfg 64/src/net/cgo_unix.go ache/go/1.25.7/x64/pkg/tool/linu-nilfunc --gdwarf-5 --64 -o ache/go/1.25.7/x64/pkg/tool/linu-tests -o 7689569/b210/_pkg_.a -trimpath 64/pkg/tool/linux_amd64/vet -p crypto/md5 -lang=go1.25 8YH9ufX/ESoPIYBZ-buildtags (dns block)
• this-host-does-not-exist-12345.com
  • Triggering command: /tmp/go-build38872931/b303/mcp.test /tmp/go-build38872931/b303/mcp.test -test.testlogfile=/tmp/go-build38872931/b303/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true ache/go/1.25.7/x64/src/runtime/c-p lF2qE-GIF x_amd64/vet (dns block)
  • Triggering command: /tmp/go-build4103510633/b307/mcp.test /tmp/go-build4103510633/b307/mcp.test -test.testlogfile=/tmp/go-build4103510633/b307/testlog.txt -test.paniconexit0 -test.timeout=10m0s swit�� ache/go/1.25.7/x64/src/net 7689569/b164/ 64/pkg/tool/linux_amd64/link -I itize p=/opt/hostedtoo/tmp/go-build4103510633/b001/_pkg_.a 64/pkg/tool/linu-trimpath (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

Fix the failing GitHub Actions workflow lint
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 65875330217
Job URL: https://github.com/github/gh-aw-mcpg/actions/runs/22633095378/job/65875330217

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[Copilot]

Pull request overview

Removes a duplicate Go test function that was causing go vet to fail due to a redeclared in this block error after merging PR #1567 into main.

Changes:

• Deleted the duplicate TestWithResponseLogging from internal/server/transport_test.go to eliminate the redeclaration.

Comments suppressed due to low confidence (1)

internal/server/transport_test.go:142

• After removing this test, there are no remaining tests in the repo that directly exercise withResponseLogging (e.g., no TestWithResponseLogging* in internal/server/http_helpers_test.go or elsewhere). The PR description says a canonical version exists in http_helpers_test.go, but it doesn't appear to; please either restore/move the test there (possibly under a different name to avoid redeclaration) or update the PR description if the coverage is intentionally being removed.

// TestCreateHTTPServerForMCP_OAuth tests OAuth discovery endpoint
func TestCreateHTTPServerForMCP_OAuth(t *testing.T) {
	tests := []struct {

---

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