Merge pull request #30 from Han-Spoon/refactor/#29-card

[Refactor] 카드 저장 텍스트 스냅샷 단순화 + Flyway 마이그레이션 스쿼시

Author: xYunaL

src/main/java/com/hanspoon/backend_api/domain/card/dto/SaveCardRequest.java

@@ -1,57 +1,24 @@
 package com.hanspoon.backend_api.domain.card.dto;
 
-import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.hanspoon.backend_api.domain.card.entity.CardType;
 import io.swagger.v3.oas.annotations.media.Schema;
 import jakarta.validation.Valid;
-import jakarta.validation.constraints.AssertTrue;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Size;
-import java.util.List;
 import java.util.UUID;
 
 /**
- * 카드 저장 요청. 문구/재료는 FE 가 분석 결과(hits/ownerCard)로 구성해 전달한다.
+ * 카드 저장 요청. FE 가 화면에 보이는 문구를 그대로 {@code text} 로 전달한다(언어 스냅샷 — 서버는 저장만).
  *
- * <p>타입별 페이로드가 다르다:
- * <ul>
- *   <li>order: menuNameKo 만.
- *   <li>ingredient_check / exclude: menuNameKo + ingredients(hit 코드).
- *   <li>owner_question: menuNameKo + text(AI 완성 문구) + flag(선택).
- * </ul>
- *
- * @param type 카드 종류 (order | ingredient_check | exclude | owner_question)
+ * @param type 카드 종류 (order | ingredient_check | exclude)
  * @param menuNameKo 한국어 메뉴명(사장님 제시용)
- * @param ingredients hit 태그 코드 목록(템플릿형 전용)
- * @param text 표시 문구(다국어, owner_question 전용). ko 필수
- * @param flag 애매 사유 플래그(owner_question 선택)
+ * @param text 표시 문구(다국어). ko 필수 + 사용자 언어 1개
  * @param scanId 출처 스캔 id(선택). 제공 시 본인 소유 검증
  */
 @Schema(description = "카드 저장 요청")
 public record SaveCardRequest(
         @NotNull CardType type,
         @NotBlank @Size(max = 255) String menuNameKo,
-        List<String> ingredients,
-        @Valid CardText text,
-        @Size(max = 50) String flag,
-        UUID scanId) {
-
-    /** ingredient_check / exclude 는 ingredients 가 필수(non-empty). */
-    @JsonIgnore
-    @AssertTrue(message = "ingredients is required for ingredient_check/exclude cards.") public boolean isIngredientsValid() {
-        if (type != CardType.INGREDIENT_CHECK && type != CardType.EXCLUDE) {
-            return true;
-        }
-        return ingredients != null && !ingredients.isEmpty();
-    }
-
-    /** owner_question 은 text(다국어 완성 문구)가 필수. */
-    @JsonIgnore
-    @AssertTrue(message = "text is required for owner_question cards.") public boolean isTextValid() {
-        if (type != CardType.OWNER_QUESTION) {
-            return true;
-        }
-        return text != null;
-    }
-}
+        @NotNull @Valid CardText text,
+        UUID scanId) {}

src/main/java/com/hanspoon/backend_api/domain/card/dto/SavedCardResponse.java

@@ -4,39 +4,22 @@
 import com.hanspoon.backend_api.domain.card.entity.SavedCard;
 import io.swagger.v3.oas.annotations.media.Schema;
 import java.time.Instant;
-import java.util.List;
 import java.util.UUID;
 
 /**
  * 저장 카드 1건 응답. 목록은 최신 저장순(lastSavedAt desc)으로 정렬된다.
- * 타입별로 채워지는 필드가 다르다(템플릿형=ingredients, owner_question=text+flag).
  *
  * @param cardId 카드 id
  * @param type 카드 종류
  * @param menuNameKo 한국어 메뉴명
- * @param ingredients hit 태그 코드(템플릿형), 그 외 null
- * @param text 표시 문구(owner_question), 그 외 null
- * @param flag 애매 사유(owner_question), 그 외 null
+ * @param text 표시 문구(다국어, 저장 시점 스냅샷)
  * @param createdAt 최초 저장 시각
  */
 @Schema(description = "저장 카드 항목")
-public record SavedCardResponse(
-        UUID cardId,
-        CardType type,
-        String menuNameKo,
-        List<String> ingredients,
-        CardText text,
-        String flag,
-        Instant createdAt) {
+public record SavedCardResponse(UUID cardId, CardType type, String menuNameKo, CardText text, Instant createdAt) {
 
     public static SavedCardResponse from(SavedCard card) {
         return new SavedCardResponse(
-                card.getId(),
-                card.getCardType(),
-                card.getMenuNameKo(),
-                card.getIngredients(),
-                card.getText(),
-                card.getFlag(),
-                card.getCreatedAt());
+                card.getId(), card.getCardType(), card.getMenuNameKo(), card.getText(), card.getCreatedAt());
     }
 }

src/main/java/com/hanspoon/backend_api/domain/card/entity/CardType.java

@@ -8,8 +8,7 @@
 public enum CardType {
     ORDER("order"),
     INGREDIENT_CHECK("ingredient_check"),
-    EXCLUDE("exclude"),
-    OWNER_QUESTION("owner_question");
+    EXCLUDE("exclude");
 
     private final String code;
 

src/main/java/com/hanspoon/backend_api/domain/card/entity/SavedCard.java

@@ -7,7 +7,6 @@
 import jakarta.persistence.Id;
 import jakarta.persistence.Table;
 import java.time.Instant;
-import java.util.List;
 import java.util.UUID;
 import lombok.AccessLevel;
 import lombok.Getter;
@@ -18,13 +17,10 @@
 /**
  * 자주 쓰는 카드(사장님 소통 카드 저장). users 와 N:1 (FK ON DELETE CASCADE).
  * 출처 스캔(scanSessionId)은 선택이며, 스캔 삭제 시 ON DELETE SET NULL 로 카드는 보존된다.
- * 문구 생성은 FE 책임 — 서버는 저장/목록/삭제만 담당한다.
  *
- * <p>저장 형태가 타입별로 다르다:
- * <ul>
- *   <li>order/ingredient_check/exclude: {@code ingredients}(hit 코드)만 저장 → FE 가 언어별 문장 재렌더.
- *   <li>owner_question: AI 가 준 완성 문구 {@code text} + {@code flag} 저장(재생성 불가).
- * </ul>
+ * <p>문구 생성은 FE 책임 — 서버는 화면에 보이는 텍스트({@link CardText})를 그대로 스냅샷 저장한다.
+ * 사용 언어를 바꿔도 저장된 카드는 변하지 않는다(재렌더 없음). 타입(order/ingredient_check/exclude)별로
+ * 텍스트 출처만 다를 뿐(재료확인=AI owner_card.question, 주문/빼고요청=FE 템플릿) 저장 형태는 동일하다.
  */
 @Entity
 @Table(name = "saved_cards")
@@ -45,54 +41,29 @@ public class SavedCard extends BaseEntity {
     @Column(name = "menu_name_ko", length = 255, nullable = false)
     private String menuNameKo;
 
-    // 템플릿형(order/ingredient_check/exclude) — hit 태그 코드. owner_question 에서는 null.
     @JdbcTypeCode(SqlTypes.JSON)
-    @Column(name = "ingredients", columnDefinition = "jsonb")
-    private List<String> ingredients;
-
-    // owner_question 전용 — AI 완성 문구. 템플릿형에서는 null.
-    @JdbcTypeCode(SqlTypes.JSON)
-    @Column(name = "text", columnDefinition = "jsonb")
+    @Column(name = "text", columnDefinition = "jsonb", nullable = false)
     private CardText text;
 
-    // owner_question 전용 — AI 애매 사유 플래그(예: has_unclear_broth).
-    @Column(name = "flag", length = 50)
-    private String flag;
-
     @Column(name = "scan_session_id", columnDefinition = "uuid")
     private UUID scanSessionId;
 
     @Column(name = "last_saved_at", nullable = false)
     private Instant lastSavedAt;
 
-    private SavedCard(
-            UUID userId,
-            CardType cardType,
-            String menuNameKo,
-            List<String> ingredients,
-            CardText text,
-            String flag,
-            UUID scanSessionId) {
+    private SavedCard(UUID userId, CardType cardType, String menuNameKo, CardText text, UUID scanSessionId) {
         this.id = UUID.randomUUID();
         this.userId = userId;
         this.cardType = cardType;
         this.menuNameKo = menuNameKo;
-        this.ingredients = ingredients;
         this.text = text;
-        this.flag = flag;
         this.scanSessionId = scanSessionId;
         this.lastSavedAt = Instant.now();
     }
 
     public static SavedCard create(
-            UUID userId,
-            CardType cardType,
-            String menuNameKo,
-            List<String> ingredients,
-            CardText text,
-            String flag,
-            UUID scanSessionId) {
-        return new SavedCard(userId, cardType, menuNameKo, ingredients, text, flag, scanSessionId);
+            UUID userId, CardType cardType, String menuNameKo, CardText text, UUID scanSessionId) {
+        return new SavedCard(userId, cardType, menuNameKo, text, scanSessionId);
     }
 
     /** 동일 카드 재저장 시 호출 — 목록 최상단으로 올라오도록 저장 시각만 갱신한다. */

src/main/java/com/hanspoon/backend_api/domain/card/service/CardService.java

@@ -8,7 +8,6 @@
 import com.hanspoon.backend_api.global.common.PageResponse;
 import com.hanspoon.backend_api.global.exception.BusinessException;
 import com.hanspoon.backend_api.global.exception.ErrorCode;
-import java.util.Objects;
 import java.util.UUID;
 import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
@@ -17,8 +16,8 @@
 /**
  * 자주 쓰는 카드 저장/목록/삭제. 문구·재료는 FE 가 구성하고 서버는 저장만 담당한다.
  *
- * <p>동일 카드 재저장 시 새 행을 만들지 않고 기존 카드의 저장 시각만 갱신해 목록 최상단으로 올린다.
- * 동일 판정은 타입별 페이로드(템플릿형=ingredients, owner_question=text)로 비교한다.
+ * <p>카드는 화면에 보이는 텍스트를 그대로 스냅샷 저장한다(언어 재렌더 없음). 동일 카드 재저장 시 새 행을 만들지 않고
+ * 기존 카드의 저장 시각만 갱신해 목록 최상단으로 올린다. 동일 판정은 (user, type, menuNameKo, text)로 한다.
  */
 @Service
 public class CardService {
@@ -39,22 +38,16 @@ public SavedCardResponse save(UUID userId, SaveCardRequest request) {
                 savedCardRepository
                         .findByUserIdAndCardTypeAndMenuNameKo(userId, request.type(), request.menuNameKo())
                         .stream()
-                        .filter(card -> isSamePayload(card, request))
+                        .filter(card -> card.getText().equals(request.text()))
                         .findFirst()
                         .orElse(null);
         if (duplicate != null) {
-            duplicate.resave(); // 중복은 새로 만들지 않고 최상단으로
+            duplicate.resave(); // 중복(동일 type/menuNameKo/text)은 새로 만들지 않고 최상단으로
             return SavedCardResponse.from(duplicate);
         }
 
-        SavedCard saved = savedCardRepository.save(SavedCard.create(
-                userId,
-                request.type(),
-                request.menuNameKo(),
-                request.ingredients(),
-                request.text(),
-                request.flag(),
-                scanId));
+        SavedCard saved = savedCardRepository.save(
+                SavedCard.create(userId, request.type(), request.menuNameKo(), request.text(), scanId));
         return SavedCardResponse.from(saved);
     }
 
@@ -72,15 +65,6 @@ public void delete(UUID userId, UUID cardId) {
         savedCardRepository.delete(card);
     }
 
-    /** 같은 (user, type, menuNameKo) 후보 중 타입별 페이로드까지 동일하면 중복으로 본다. */
-    private boolean isSamePayload(SavedCard card, SaveCardRequest request) {
-        return switch (request.type()) {
-            case ORDER -> true; // 키(user+type+menuNameKo)만으로 동일
-            case INGREDIENT_CHECK, EXCLUDE -> Objects.equals(card.getIngredients(), request.ingredients());
-            case OWNER_QUESTION -> Objects.equals(card.getText(), request.text());
-        };
-    }
-
     /** scanId 제공 시 본인 소유 스캔인지 검증하고 그대로 반환. 미제공이면 null. */
     private UUID resolveScanId(UUID userId, UUID scanId) {
         if (scanId == null) {

src/main/java/com/hanspoon/backend_api/global/config/DevDataSeeder.java

@@ -182,19 +182,21 @@ private void seedScans(UUID userId) {
     }
 
     private void seedCards(UUID userId) {
+        // 모든 카드는 화면에 보이는 text 를 그대로 저장(언어 스냅샷). 재료확인 text 는 AI owner_card.question 예시.
         savedCardRepository.saveAll(List.of(
-                // 템플릿형: ingredients(hit 코드)만 저장 → FE 가 언어별 문장 재렌더
-                SavedCard.create(userId, CardType.ORDER, "삼겹살", null, null, null, null),
-                SavedCard.create(userId, CardType.INGREDIENT_CHECK, "김치찌개", List.of("is_fish"), null, null, null),
-                SavedCard.create(userId, CardType.EXCLUDE, "비빔밥", List.of("is_egg"), null, null, null),
-                // owner_question: AI 완성 문구 text + flag 저장
                 SavedCard.create(
                         userId,
-                        CardType.OWNER_QUESTION,
+                        CardType.ORDER,
+                        "삼겹살",
+                        new CardText("삼겹살 하나 주세요.", "One pork belly, please.", null),
+                        null),
+                SavedCard.create(
+                        userId,
+                        CardType.INGREDIENT_CHECK,
                         "된장찌개",
-                        null,
-                        new CardText("멸치 육수를 사용하나요?", "Do you use anchovy broth?", null),
-                        "has_unclear_broth",
-                        null)));
+                        new CardText("이 메뉴에 멸치육수가 들어가나요?", "Does this dish contain fish broth?", null),
+                        null),
+                SavedCard.create(
+                        userId, CardType.EXCLUDE, "비빔밥", new CardText("계란 빼주세요.", "No egg, please.", null), null)));
     }
 }