improvement: 메시지 저장/검증 계약 통일 (#222) #223

seonkyu.kim · 2026-02-25T05:07:24Z

seonkyu.kim commented

2026-02-25 05:07:24 +00:00

📋 작업 요약

MessageValidateRequestDto에 JsonPropertyName 추가하여 save/validate API 간 snake_case 필드명 통일
Data 타입을 string? → object?로 변경하여 save/validate 간 타입 일치
MessageValidationService.ValidateData 파라미터를 object?로 변경하여 JsonElement/string 모두 처리
Swagger Description에 snake_case 필드 규약 및 검증 오류 구조 명시

Closes #222

🛠️ 작업 내용 (Changes)

SPMS.Application/DTOs/Message/MessageValidateRequestDto.cs — JsonPropertyName 추가, Data 타입 object?로 변경
SPMS.Application/Services/MessageValidationService.cs — ValidateData 파라미터 string? → object? 변경, JsonElement/string 분기 처리
SPMS.API/Controllers/MessageController.cs — save/validate 엔드포인트 Swagger Description 보강

📢 리뷰어 참고 사항 (To Reviewers)

ValidateData는 JsonElement(API 호출 시)와 string(내부 호출 시) 모두 수용하도록 분기 처리됨
DB 변경 없음, Migration 불필요
save API와 동일한 JSON 페이로드로 validate 호출 가능

✅ 체크리스트 (Self Checklist)

빌드(Build)가 성공적으로 수행되었는가?
모든 단위 테스트(Unit Test)를 통과하였는가?
불필요한 로그나 주석을 제거하였는가?
컨벤션(Clean Architecture, Naming)을 준수하였는가?
기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가?

📸 스크린샷 / 테스트 로그 (Screenshots/Logs)

빌드했습니다.
    경고 0개
    오류 0개

## 📋 작업 요약 - `MessageValidateRequestDto`에 `JsonPropertyName` 추가하여 save/validate API 간 snake_case 필드명 통일 - `Data` 타입을 `string?` → `object?`로 변경하여 save/validate 간 타입 일치 - `MessageValidationService.ValidateData` 파라미터를 `object?`로 변경하여 JsonElement/string 모두 처리 - Swagger Description에 snake_case 필드 규약 및 검증 오류 구조 명시 ## 🔗 관련 이슈 (Related Issues) Closes #222 ## 🛠️ 작업 내용 (Changes) - [x] `SPMS.Application/DTOs/Message/MessageValidateRequestDto.cs` — JsonPropertyName 추가, Data 타입 object?로 변경 - [x] `SPMS.Application/Services/MessageValidationService.cs` — ValidateData 파라미터 string? → object? 변경, JsonElement/string 분기 처리 - [x] `SPMS.API/Controllers/MessageController.cs` — save/validate 엔드포인트 Swagger Description 보강 ## 📢 리뷰어 참고 사항 (To Reviewers) - `ValidateData`는 `JsonElement`(API 호출 시)와 `string`(내부 호출 시) 모두 수용하도록 분기 처리됨 - DB 변경 없음, Migration 불필요 - save API와 동일한 JSON 페이로드로 validate 호출 가능 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) ``` 빌드했습니다. 경고 0개 오류 0개 ```

seonkyu.kim added 1 commit 2026-02-25 05:07:25 +00:00

improvement: 메시지 저장/검증 계약 통일 (#222 ) b373d59710

- MessageValidateRequestDto에 JsonPropertyName 추가 (snake_case 통일)
- MessageValidateRequestDto.Data 타입 string? → object? 변경
- MessageValidationService.ValidateData 파라미터 타입 변경
- Swagger Description 업데이트 (save/validate 엔드포인트)

Closes #222

seonkyu.kim added the

labels 2026-02-25 05:07:36 +00:00

seonkyu.kim self-assigned this 2026-02-25 05:07:42 +00:00

seonkyu.kim added this to the Phase 2: 메시지 + 통계 milestone 2026-02-25 05:07:45 +00:00

seonkyu.kim requested review from Owners 2026-02-25 05:07:51 +00:00

seonkyu.kim merged commit b9b3fa2fc0 into develop