fix(fxhttpserver): Fix HTTP handlers/middlewares dependency injection signature (#362)

Author: damnedest

fxhttpserver/reflect.go

@@ -6,14 +6,41 @@ import (
 	"github.com/labstack/echo/v4"
 )
 
-// GetType returns the type of a target.
+// fullTypeID builds a stable identifier for a type in the form "<pkgpath>.<TypeName>".
+func fullTypeID(t reflect.Type) string {
+	if t == nil {
+		return ""
+	}
+
+	// Unwrap pointers to get the underlying named type (if any).
+	for t.Kind() == reflect.Pointer {
+		t = t.Elem()
+	}
+
+	// For named types, PkgPath() + Name() gives a unique and stable identity.
+	if t.Name() != "" && t.PkgPath() != "" {
+		return t.PkgPath() + "." + t.Name()
+	}
+
+	// Fallback for non-named kinds (slices, maps, func, etc.).
+	return t.String()
+}
+
+// GetType returns a stable identifier for the given target’s type.
 func GetType(target any) string {
-	return reflect.TypeOf(target).String()
+	return fullTypeID(reflect.TypeOf(target))
 }
 
-// GetReturnType returns the return type of a target.
+// GetReturnType returns a stable identifier for the return type of constructor-like target.
+// If a target is a function, we examine its first return value (index 0), unwrap pointers, and
+// build an identifier for that named type. For non-function or empty-return cases, we return "".
 func GetReturnType(target any) string {
-	return reflect.TypeOf(target).Out(0).String()
+	t := reflect.TypeOf(target)
+	if t == nil || t.Kind() != reflect.Func || t.NumOut() == 0 {
+		return ""
+	}
+
+	return fullTypeID(t.Out(0))
 }
 
 // IsConcreteMiddleware returns true if the middleware is a concrete [echo.MiddlewareFunc] implementation.

fxhttpserver/reflect_test.go

@@ -15,11 +15,12 @@ func TestGetType(t *testing.T) {
 		target   any
 		expected string
 	}{
+		{nil, ""},
 		{123, "int"},
 		{"test", "string"},
 		{echo.MiddlewareFunc(func(next echo.HandlerFunc) echo.HandlerFunc {
 			return next
-		}), "echo.MiddlewareFunc"},
+		}), "github.com/labstack/echo/v4.MiddlewareFunc"},
 	}
 
 	for _, tt := range tests {
@@ -41,8 +42,12 @@ func TestGetReturnType(t *testing.T) {
 		target   any
 		expected string
 	}{
+		{nil, ""},
 		{func() string { return "test" }, "string"},
 		{func() int { return 123 }, "int"},
+		{echo.MiddlewareFunc(func(next echo.HandlerFunc) echo.HandlerFunc {
+			return next
+		}), "github.com/labstack/echo/v4.HandlerFunc"},
 	}
 
 	for _, tt := range tests {