Skip to content

[FIX] practice-data 응답 midiData에 코드 진행(chords) 반영 - #141

Merged
p1001q merged 1 commit into
developfrom
fix/#133-practice-data-chords
Aug 5, 2026
Merged

[FIX] practice-data 응답 midiData에 코드 진행(chords) 반영#141
p1001q merged 1 commit into
developfrom
fix/#133-practice-data-chords

Conversation

@p1001q

@p1001q p1001q commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

피알 본문:

📍 개요

practice-data(1-6) 응답의 midiData에 코드 진행(chords) 정보 추가 — 문서/Swagger 예시 동기화

⛓️‍💥 관련 이슈


🛠️ 작업 내용

recording/options만으로는 화면에 코드 진행을 표시할 수 없어서,
midiData 안에 chords({bar, beat, symbol} 배열) 정보가 추가로 필요함을 확인했습니다.

  • midiDataString + @JsonRawValueplaying_example.midi_data(DB JSON 컬럼) 내용을
    가공 없이 그대로 응답하는 구조라, 백엔드 로직 변경은 없습니다.
  • 실제 코드 변경은 LearningPracticeDataResponseDTO@Schema(example=...) 예시 문자열에
    chords 반영한 것 하나뿐입니다(Swagger 문서 정확도 목적).
  • docs/학습·사용자관리 API 명세서.md 1-6 섹션(Success Response, 필드 설명 테이블, DB 값 연결)에도
    동일하게 반영해뒀습니다.

🔥 리뷰 요청 사항

  • Swagger 예시값 문구가 실제 프론트가 필요로 하는 chords 구조(bar/beat/symbol)와 맞는지

✅ 체크리스트

  • 코드 컨벤션을 준수했습니다.
  • 불필요한 코드 및 import를 제거했습니다.
  • 예외 처리를 적용했습니다.
  • 테스트를 완료했습니다.
  • 관련 Issue를 연결했습니다.

📎 참고 사항

  • 로직 변경이 없어 기존 테스트에 영향 없음(컴파일만 확인).

Summary by CodeRabbit

  • 문서
    • Swagger 예시 JSON에 코드 진행 정보를 나타내는 chords 배열을 추가했습니다.

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 419faf29-cc7b-451f-b415-72b3ebf5f846

📥 Commits

Reviewing files that changed from the base of the PR and between 3e4cec7 and a15984b.

📒 Files selected for processing (1)
  • src/main/java/com/mr/domain/learning/dto/res/LearningPracticeDataResponseDTO.java

📝 Walkthrough

Walkthrough

LearningPracticeDataResponseDTO의 Swagger midiData 예시 JSON에 코드 진행을 나타내는 chords 배열을 추가했습니다. 백엔드 처리 로직은 변경하지 않았습니다.

Changes

Practice data 응답 예시

Layer / File(s) Summary
Swagger midiData 예시 업데이트
src/main/java/com/mr/domain/learning/dto/res/LearningPracticeDataResponseDTO.java
Swagger 예시 JSON에 마디·박자별 코드 기호를 담은 chords 배열을 추가했습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • Musereview/BE#55: 동일 DTO의 midiData Swagger 예시 문서를 변경했습니다.
  • Musereview/BE#127: 동일 DTO의 midiData Swagger 예시 구조를 변경했습니다.

Poem

midiData에 코드가 모여
chords 배열 빛을 내요
로직은 그대로
예시는 더 선명하게
Swagger도 박자를 타요 🎵

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning DTO Swagger 예시에는 chords가 반영되었지만, 연결 이슈가 요구한 API 명세서와 노션 명세서 변경은 확인되지 않습니다. docs/학습·사용자관리 API 명세서.md와 노션 API 명세서에 chords 필드와 관련 설명을 추가하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 practice-data 응답의 midiData에 chords를 추가하는 주요 변경 내용을 명확하게 설명합니다.
Out of Scope Changes check ✅ Passed 변경 내용은 LearningPracticeDataResponseDTO의 Swagger 예시에 chords를 추가하는 작업으로, 연결 이슈의 범위와 일치합니다.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#133-practice-data-chords

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ownue ownue left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다~!

@kimyw1018 kimyw1018 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

바로 반영하셔도 될것 같습니다!

@p1001q
p1001q merged commit 76aa4d0 into develop Aug 5, 2026
2 checks passed
@p1001q
p1001q deleted the fix/#133-practice-data-chords branch August 5, 2026 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 BugFix - practice-data 응답 midiData에 코드 진행(chords) 정보 추가

3 participants