Fix L3: add apiConfig and JSON helpers; complete handler_notes

Author: Saharyasa

apiconfig.go

@@ -0,0 +1,8 @@
+package main
+
+import "github.com/bootdotdev/learn-cicd-starter/internal/database"
+
+// Minimal apiConfig so handlers with receiver *apiConfig compile.
+type apiConfig struct {
+	DB *database.Queries
+}

handler_notes.go

@@ -9,6 +9,7 @@ import (
 	"github.com/google/uuid"
 )
 
+// handlerNotesGet returns all notes for a user
 func (cfg *apiConfig) handlerNotesGet(w http.ResponseWriter, r *http.Request, user database.User) {
 	posts, err := cfg.DB.GetNotesForUser(r.Context(), user.ID)
 	if err != nil {
@@ -25,15 +26,17 @@ func (cfg *apiConfig) handlerNotesGet(w http.ResponseWriter, r *http.Request, us
 	respondWithJSON(w, http.StatusOK, postsResp)
 }
 
+// handlerNotesCreate creates a new note for a user
 func (cfg *apiConfig) handlerNotesCreate(w http.ResponseWriter, r *http.Request, user database.User) {
 	type parameters struct {
 		Note string `json:"note"`
 	}
+
 	decoder := json.NewDecoder(r.Body)
 	params := parameters{}
 	err := decoder.Decode(&params)
 	if err != nil {
-		respondWithError(w, http.StatusInternalServerError, "Couldn't decode parameters", err)
+		respondWithError(w, http.StatusBadRequest, "Couldn't decode parameters", err)
 		return
 	}
 

helpers.go

@@ -0,0 +1,31 @@
+package main
+
+import (
+	"encoding/json"
+	"net/http"
+)
+
+// respondWithError sends a JSON error with optional internal details.
+func respondWithError(w http.ResponseWriter, code int, message string, err error) {
+	payload := map[string]string{"error": message}
+	if err != nil {
+		payload["details"] = err.Error()
+	}
+	respondWithJSON(w, code, payload)
+}
+
+// respondWithJSON writes v as JSON with proper error handling (gosec-safe).
+func respondWithJSON(w http.ResponseWriter, code int, v any) {
+	w.Header().Set("Content-Type", "application/json")
+	w.WriteHeader(code)
+
+	data, err := json.Marshal(v)
+	if err != nil {
+		http.Error(w, "marshal error", http.StatusInternalServerError)
+		return
+	}
+	if _, err := w.Write(data); err != nil {
+		http.Error(w, "write error", http.StatusInternalServerError)
+		return
+	}
+}