From 345d7a4039cc73d8b913db5351bbf36358414ad8 Mon Sep 17 00:00:00 2001
From: Amolith <amolith@secluded.site>
Date: Fri, 19 Dec 2025 09:03:43 -0700
Subject: [PATCH] refactor(cmd): use net.JoinHostPort for host:port

Handles IPv6 addresses correctly and avoids redundant fmt.Sprintf calls.

Assisted-by: Claude Opus 4.5 via Crush
---
 cmd/lunatask-mcp-server.go | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/cmd/lunatask-mcp-server.go b/cmd/lunatask-mcp-server.go
index 052526aa0b54f8a5a3c90005ac0f6e9de32f1172..cae871cb8d4dbd3d1bd8dd867b57775b477cbec9 100644
--- a/cmd/lunatask-mcp-server.go
+++ b/cmd/lunatask-mcp-server.go
@@ -8,7 +8,9 @@ import (
 	"context"
 	"fmt"
 	"log"
+	"net"
 	"os"
+	"strconv"
 
 	"github.com/BurntSushi/toml"
 	"github.com/mark3labs/mcp-go/mcp"
@@ -131,9 +133,10 @@ func main() {
 
 	mcpServer := NewMCPServer(&config)
 
-	baseURL := fmt.Sprintf("http://%s:%d", config.Server.Host, config.Server.Port)
+	hostPort := net.JoinHostPort(config.Server.Host, strconv.Itoa(config.Server.Port))
+	baseURL := "http://" + hostPort
 	sseServer := server.NewSSEServer(mcpServer, server.WithBaseURL(baseURL))
-	listenAddr := fmt.Sprintf("%s:%d", config.Server.Host, config.Server.Port)
+	listenAddr := hostPort
 	log.Printf("SSE server listening on %s (baseURL: %s)", listenAddr, baseURL)
 
 	if err := sseServer.Start(listenAddr); err != nil {