feat: implement custom HTTP error handling

Add a custom HTTP error handler to improve error response formatting and ensure internal errors are properly reported. Disable error forwarding in the request logger middleware to allow for centralized error handling.

internal/port/httpserver/middleware.go

@@ -123,7 +123,7 @@ func requestLoggerMiddleware() echo.MiddlewareFunc {
 		LogRequestID:  true,
 		LogReferer:    true,
 		LogHeaders:    []string{},
-		HandleError:   true, // forwards error to the global error handler, so it can decide appropriate status code
+		HandleError:   false,
 		LogValuesFunc: logValuesFunc,
 	})
 }

internal/port/httpserver/model.go

@@ -29,12 +29,29 @@ func JsonResponse(c echo.Context, code int, data interface{}) error {
 }
 
 func ErrorResponse(c echo.Context, code int, err error) error {
+	return echo.NewHTTPError(code).SetInternal(err)
+}
+
+func customHTTPErrorHandler(err error, c echo.Context) {
+	if c.Response().Committed {
+		return
+	}
+
+	code := http.StatusInternalServerError
+	msg := err.Error()
+
+	he, ok := err.(*echo.HTTPError)
+	if ok {
+		code = he.Code
+		if he.Internal != nil {
+			msg = he.Internal.Error()
+		}
+	}
 	_ = c.JSON(code, JSONResult{
 		Success: false,
 		Code:    code,
-		Message: err.Error(),
+		Message: msg,
 	})
-	return err
 }
 
 // bindAndValidate binds the request data to the provided struct and validates it.

internal/port/httpserver/server.go

@@ -61,6 +61,7 @@ func New(pool *db.Pool, llm *llms.LLM, queue *queue.Queue, opts ...Option) (*Ser
 		queue:  queue,
 	}
 	s.Server.Validator = &CustomValidator{validator: validator.New()}
+	s.Server.HTTPErrorHandler = customHTTPErrorHandler
 	// if config.Settings.DebugUI {
 	// 	logger.Default.Info("debug ui enabled")
 	// 	s.uiCmd = exec.Command("bun", "run", "dev")