fix: refactor parameter handling in OutputsHandler; add URL decoding for output IDs in Delete and Start methods

Author: pablodz

internal/api/handler_outputs.go

@@ -2,6 +2,7 @@ package api
 
 import (
 	"log"
+	"net/url"
 	"sort"
 	"sync"
 	"time"
@@ -91,8 +92,16 @@ func boolStr(v bool) string {
 	return "no"
 }
 
-func (h *OutputsHandler) Delete(c *fiber.Ctx) error {
+func (h *OutputsHandler) paramID(c *fiber.Ctx) string {
 	id := c.Params("id")
+	if decoded, err := url.PathUnescape(id); err == nil {
+		return decoded
+	}
+	return id
+}
+
+func (h *OutputsHandler) Delete(c *fiber.Ctx) error {
+	id := h.paramID(c)
 	h.controller.RemoveOutput(id)
 	h.mu.Lock()
 	delete(h.outputs, id)
@@ -103,7 +112,7 @@ func (h *OutputsHandler) Delete(c *fiber.Ctx) error {
 }
 
 func (h *OutputsHandler) Start(c *fiber.Ctx) error {
-	id := c.Params("id")
+	id := h.paramID(c)
 	var req StartOutputReq
 	if err := c.BodyParser(&req); err != nil {
 		log.Printf("[api] Output %s body parse warning: %v", id, err)
@@ -143,7 +152,7 @@ func (h *OutputsHandler) Start(c *fiber.Ctx) error {
 }
 
 func (h *OutputsHandler) Stop(c *fiber.Ctx) error {
-	id := c.Params("id")
+	id := h.paramID(c)
 	h.controller.RemoveOutput(id)
 	log.Printf("[api] Output stopped: %s", id)
 	return c.JSON(APIResponse{Ok: true})