Fix Go build syntax error in main.go newServer function

[Copilot]

The Go build was failing due to syntax errors in the newServer function in main.go. The issue was caused by orphaned error handling code that referenced a non-existent error variable and had mismatched braces.

Problem

The build was failing with multiple syntax errors:

./main.go:147:2: syntax error: non-declaration statement outside function body
./main.go:159:44: syntax error: unexpected {, expected (
./main.go:160:23: syntax error: unexpected ( in parameter list; possibly missing comma or )

The problematic code was:

srv := &http.Server{
    Addr:         fmt.Sprintf(":%d", cfg.Server.Port),
    Handler:      h2c.NewHandler(r, h2s),
    ReadTimeout:  cfg.Server.ReadTimeout,
    WriteTimeout: cfg.Server.WriteTimeout,
}
    log.Fatalf("failed to configure HTTP/2 server: %v", err) // ← orphaned error handling
}  // ← mismatched brace

Solution

Removed the orphaned error handling code that was causing the syntax errors. The HTTP server struct definition now properly flows to the CORS configuration without syntax issues.

Verification

• ✅ go build now succeeds without errors
• ✅ All tests pass (24 packages tested successfully)
• ✅ go vet passes with no issues
• ✅ Binary builds correctly

The fix was minimal and surgical - only removed the problematic lines that were preventing compilation without affecting any working functionality.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull Request Overview

This PR fixes critical syntax errors in the newServer function in main.go that were preventing the Go build from succeeding. The issue was caused by orphaned error handling code that referenced a non-existent error variable and had mismatched braces after the HTTP server struct definition.

• Removed orphaned log.Fatalf error handling code that referenced an undefined err variable
• Fixed mismatched brace causing syntax errors in the server configuration flow

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}