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

1 files changed, 7 insertions, 7 deletions

diff --git a/internal/admin/api_rooms_bulk_delete.go b/internal/admin/api_rooms_bulk_delete.go
index 70c2eb6..5b525f6 100644
--- a/internal/admin/api_rooms_bulk_delete.go
+++ b/internal/admin/api_rooms_bulk_delete.go
@@ -24,18 +24,18 @@ func (s *AdminServer) handleBulkDelete(w http.ResponseWriter, r *http.Request) {
 	body, readErr := io.ReadAll(r.Body)
 	r.Body.Close()
 	if readErr != nil {
-		writeJSON(w, map[string]any{"error": fmt.Sprintf("read body: %v", readErr)})
+		failJSON(w, fmt.Sprintf("read body: %v", readErr))
 		return
 	}
 
 	var req bulkDeleteRequest
 	if err := json.Unmarshal(body, &req); err != nil {
-		writeJSON(w, map[string]any{"error": fmt.Sprintf("invalid json: %v", err)})
+		failJSON(w, fmt.Sprintf("invalid json: %v", err))
 		return
 	}
 
 	if len(req.IDs) == 0 {
-		writeJSON(w, map[string]any{"error": "no room ids"})
+		failJSON(w, "no room ids")
 		return
 	}
 
@@ -45,7 +45,7 @@ func (s *AdminServer) handleBulkDelete(w http.ResponseWriter, r *http.Request) {
 	seenIDs := make(map[int]bool, len(req.IDs))
 	for _, id := range req.IDs {
 		if seenIDs[id] {
-			writeJSON(w, map[string]any{"error": fmt.Sprintf("duplicate room id %d in request", id)})
+			failJSON(w, fmt.Sprintf("duplicate room id %d in request", id))
 			return
 		}
 		seenIDs[id] = true
@@ -61,12 +61,12 @@ func (s *AdminServer) handleBulkDelete(w http.ResponseWriter, r *http.Request) {
 	for _, id := range req.IDs {
 		path, ok := s.world.GetRoomPath(id)
 		if !ok {
-			writeJSON(w, map[string]any{"error": fmt.Sprintf("room %d not found", id)})
+			failJSON(w, fmt.Sprintf("room %d not found", id))
 			return
 		}
 		oldContent, snapErr := snapshotFile(path)
 		if snapErr != nil {
-			writeJSON(w, map[string]any{"error": fmt.Sprintf("snapshot room %d: %v", id, snapErr)})
+			failJSON(w, fmt.Sprintf("snapshot room %d: %v", id, snapErr))
 			return
 		}
 		snapshots = append(snapshots, roomSnapshot{id: id, path: path, oldContent: oldContent})
@@ -85,7 +85,7 @@ func (s *AdminServer) handleBulkDelete(w http.ResponseWriter, r *http.Request) {
 					}
 				}
 			}
-			writeJSON(w, map[string]any{"error": fmt.Sprintf("delete room %d: %v", snap.id, err)})
+			failJSON(w, fmt.Sprintf("delete room %d: %v", snap.id, err))
 			return
 		}
 		deletedSet[snap.id] = true