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

1 files changed, 4 insertions, 4 deletions

diff --git a/internal/admin/api_items.go b/internal/admin/api_items.go
index 749e8ff..8798543 100644
--- a/internal/admin/api_items.go
+++ b/internal/admin/api_items.go
@@ -40,11 +40,11 @@ func (s *AdminServer) handleItemByID(w http.ResponseWriter, r *http.Request) {
 
 	switch r.Method {
 	case http.MethodGet:
-		s.getItem(w, r, id)
+		s.getItem(w, id)
 	case http.MethodPut:
 		s.updateItem(w, r, id)
 	case http.MethodDelete:
-		s.deleteItem(w, r, id)
+		s.deleteItem(w, id)
 	default:
 		http.Error(w, `{"error":"method not allowed"}`, http.StatusMethodNotAllowed)
 	}
@@ -66,7 +66,7 @@ func (s *AdminServer) findItemFile(id string) ([]byte, string, error) {
 	return findYAMLFileInSubdirs(s.dataDir, "items", id)
 }
 
-func (s *AdminServer) getItem(w http.ResponseWriter, r *http.Request, id string) {
+func (s *AdminServer) getItem(w http.ResponseWriter, id string) {
 	data, path, err := s.findItemFile(id)
 	if err != nil {
 		http.Error(w, `{"error":"item not found"}`, http.StatusNotFound)
@@ -120,7 +120,7 @@ func (s *AdminServer) updateItem(w http.ResponseWriter, r *http.Request, id stri
 	writeJSON(w, m)
 }
 
-func (s *AdminServer) deleteItem(w http.ResponseWriter, r *http.Request, id string) {
+func (s *AdminServer) deleteItem(w http.ResponseWriter, id string) {
 	_, path, err := s.findItemFile(id)
 	if err != nil {
 		http.Error(w, `{"error":"item not found"}`, http.StatusNotFound)