.gitignore (1)

39-41: Generalize H2 ignore patterns.
Covers files regardless of location/name variations.

Apply:

-db_dev.mv.db
-db_dev.trace.db
+*.mv.db
+*.trace.db

src/main/java/com/back/domain/wishlist/enums/WishlistStatus.java (1)

3-8: Remove unnecessary Lombok on enum.
@Getter has no effect (no fields); drop it and its import.

-import lombok.Getter;
-
-@Getter
 public enum WishlistStatus {
     ACTIVE,
     DELETED
 }

src/main/java/com/back/domain/wishlist/repository/WishlistRepository.java (1)

5-8: Redundant @repository on Spring Data interface.
Spring auto-detects JpaRepository beans; annotation/import can be removed.

-import org.springframework.stereotype.Repository;
-
-@Repository
 public interface WishlistRepository extends JpaRepository<Wishlist, Long> {
 }

src/main/java/com/back/domain/wishlist/dto/WishlistRequestDto.java (1)

16-21: Tighten validation: userId must be positive; reconsider client‑settable status.

• Add @positive for userId.
• Prefer server-controlled status for creates; split Create/Update DTOs or use validation groups.

 import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Positive;
@@
     @NotNull
+    @Positive
     private Long userId;
 
-    // 생성 시 기본값 ACTIVE, 필요시 상태 지정 업데이트용으로 재사용 가능
-    private WishlistStatus status;
+    // 상태는 서버에서 관리(생성 시 기본 ACTIVE). 업데이트 DTO에서만 허용 권장.
+    private WishlistStatus status;

src/main/java/com/back/domain/wishlist/entity/Wishlist.java (4)

12-16: Add table metadata and auditing listener.
Indexes help common queries; auditing simplifies timestamps.

-@Entity
-@Getter
-@Setter
-@NoArgsConstructor
-public class Wishlist {
+@Entity
+@Table(name = "wishlist",
+       indexes = {
+           @Index(name = "idx_wishlist_user_id", columnList = "user_id"),
+           @Index(name = "idx_wishlist_status", columnList = "status")
+       })
+@EntityListeners(AuditingEntityListener.class)
+@Getter
+@Setter
+@NoArgsConstructor
+public class Wishlist {

Add imports (top of file, alongside others):

+import org.springframework.data.annotation.CreatedDate;
+import org.springframework.data.jpa.domain.support.AuditingEntityListener;

---

22-22: Persist non-null createdAt automatically.
Use Spring Data auditing; mark non-updatable.

-    private LocalDateTime createdAt;
+    @CreatedDate
+    @Column(nullable = false, updatable = false)
+    private LocalDateTime createdAt;

If auditing isn’t enabled, add:

+    @PrePersist
+    void prePersist() {
+        if (createdAt == null) createdAt = LocalDateTime.now();
+        if (status == null) status = WishlistStatus.ACTIVE;
+    }

---

24-26: Set association to LAZY and non-null; name the FK.
Prevents eager loads and enforces integrity.

-    @ManyToOne
-    private User user;  // 찜한 사용자 (위시리스트의 주인)
+    @ManyToOne(fetch = FetchType.LAZY, optional = false)
+    @JoinColumn(name = "user_id", nullable = false)
+    private User user;  // 찜한 사용자 (위시리스트의 주인)

---

31-35: Harden status column mapping.
Name column explicitly; keep non-null; default handled in code.

     @Enumerated(EnumType.STRING)
-    @Column(nullable = false)
+    @Column(name = "status", nullable = false)
     // 위시리스트 상태 - 기본값은 ACTIVE (활성 상태)
     // ACTIVE: 찜한 상태, DELETED: 찜 해제한 상태 (Soft Delete)
     private WishlistStatus status = WishlistStatus.ACTIVE;

Optional: consider soft-delete helpers (@SQLDelete/@where) if you plan to hide DELETED rows by default.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro