From 1d49c1e4e4311b26d4ad2fc614b6998ae3f078c8 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Mon, 16 Mar 2026 04:01:14 +0200 Subject: Use filepath.Join instead of fmt.Sprintf for path construction Replace string-based path building with filepath.Join in LSP and MCP server log path helpers for platform-correct path separators. Co-Authored-By: Claude Opus 4.6 --- cmd/hexai-lsp-server/main.go | 2 +- cmd/hexai-mcp-server/main.go | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) (limited to 'cmd') diff --git a/cmd/hexai-lsp-server/main.go b/cmd/hexai-lsp-server/main.go index 6577bc2..3f7ed60 100644 --- a/cmd/hexai-lsp-server/main.go +++ b/cmd/hexai-lsp-server/main.go @@ -39,5 +39,5 @@ func defaultLogPath() string { if err != nil { return filepath.Join(os.TempDir(), "hexai-lsp-server.log") } - return fmt.Sprintf("%s/hexai-lsp-server.log", stateDir) + return filepath.Join(stateDir, "hexai-lsp-server.log") } diff --git a/cmd/hexai-mcp-server/main.go b/cmd/hexai-mcp-server/main.go index d2495e6..959c99f 100644 --- a/cmd/hexai-mcp-server/main.go +++ b/cmd/hexai-mcp-server/main.go @@ -6,6 +6,7 @@ import ( "fmt" "io" "os" + "path/filepath" "codeberg.org/snonux/hexai/internal" "codeberg.org/snonux/hexai/internal/appconfig" @@ -120,5 +121,5 @@ func defaultLogPath() (string, error) { if err != nil { return "", fmt.Errorf("cannot create state directory: %w", err) } - return fmt.Sprintf("%s/hexai-mcp-server.log", stateDir), nil + return filepath.Join(stateDir, "hexai-mcp-server.log"), nil } -- cgit v1.2.3