refactor: remove dead code, consolidate save/error handling, add lint gate

1 files changed, 5 insertions, 5 deletions

diff --git a/internal/admin/api_mobs.go b/internal/admin/api_mobs.go
index 8531e40..c3e47bb 100644
--- a/internal/admin/api_mobs.go
+++ b/internal/admin/api_mobs.go
@@ -13,7 +13,7 @@ func (s *AdminServer) handleMobs(w http.ResponseWriter, r *http.Request) {
 	case http.MethodGet:
 		tree, err := listYAMLTree(s.dataDir, "mobs")
 		if err != nil {
-			writeJSON(w, map[string]any{"error": err.Error()})
+			failJSON(w, err.Error())
 			return
 		}
 		if tree == nil {
@@ -42,11 +42,11 @@ func (s *AdminServer) handleMobByID(w http.ResponseWriter, r *http.Request) {
 
 	switch r.Method {
 	case http.MethodGet:
-		s.getMob(w, r, id)
+		s.getMob(w, id)
 	case http.MethodPut:
 		s.updateMob(w, r, id)
 	case http.MethodDelete:
-		s.deleteMob(w, r, id)
+		s.deleteMob(w, id)
 	default:
 		http.Error(w, `{"error":"method not allowed"}`, http.StatusMethodNotAllowed)
 	}
@@ -60,7 +60,7 @@ func (s *AdminServer) findMobFile(id string) ([]byte, string, error) {
 	return findYAMLFileInSubdirs(s.dataDir, "mobs", id)
 }
 
-func (s *AdminServer) getMob(w http.ResponseWriter, r *http.Request, id string) {
+func (s *AdminServer) getMob(w http.ResponseWriter, id string) {
 	data, path, err := s.findMobFile(id)
 	if err != nil {
 		http.Error(w, `{"error":"mob not found"}`, http.StatusNotFound)
@@ -110,7 +110,7 @@ func (s *AdminServer) updateMob(w http.ResponseWriter, r *http.Request, id strin
 	writeJSON(w, m)
 }
 
-func (s *AdminServer) deleteMob(w http.ResponseWriter, r *http.Request, id string) {
+func (s *AdminServer) deleteMob(w http.ResponseWriter, id string) {
 	_, path, err := s.findMobFile(id)
 	if err != nil {
 		http.Error(w, `{"error":"mob not found"}`, http.StatusNotFound)