fix: address review feedback - init ToolNameMap eagerly, log collisions, add collision test

internal/util/sanitize_test.go

@@ -93,6 +93,20 @@ func TestSanitizedToolNameMap(t *testing.T) {
 			t.Error("expected nil for nil input")
 		}
 	})
+
+	t.Run("collision keeps first mapping", func(t *testing.T) {
+		raw := []byte(`{"tools":[
+			{"name":"read/file","input_schema":{}},
+			{"name":"read@file","input_schema":{}}
+		]}`)
+		m := SanitizedToolNameMap(raw)
+		if m == nil {
+			t.Fatal("expected non-nil map")
+		}
+		if m["read_file"] != "read/file" {
+			t.Errorf("expected first mapping read/file, got %q", m["read_file"])
+		}
+	})
 }
 
 func TestRestoreSanitizedToolName(t *testing.T) {

internal/util/translator.go

@@ -8,6 +8,7 @@ import (
 	"fmt"
 	"strings"
 
+	log "github.com/sirupsen/logrus"
 	"github.com/tidwall/gjson"
 	"github.com/tidwall/sjson"
 )
@@ -298,6 +299,8 @@ func SanitizedToolNameMap(rawJSON []byte) map[string]string {
 		}
 		if _, exists := out[sanitized]; !exists {
 			out[sanitized] = name
+		} else {
+			log.Warnf("sanitized tool name collision: %q and %q both map to %q, keeping first", out[sanitized], name, sanitized)
 		}
 		return true
 	})