Skip to content

Feature/voice records - #16

Open
dlrudwns0912 wants to merge 9 commits into
mainfrom
feature/voice-records
Open

dlrudwns0912 wants to merge 9 commits into
mainfrom
feature/voice-records

Conversation

@dlrudwns0912

@dlrudwns0912 dlrudwns0912 commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

🔀 Pull Request

1. 작업 내용

음성 기록 기능의 초기 구조 추가

주요 변경 사항

  • 음성 기반 육아 기록 기능을 위한 domain.voice 패키지 구조를 추가했습니다.
  • STT 연동을 위한 Client, 설정, DTO, 예외 클래스의 초기 뼈대를 추가했습니다.
  • 향후 STT 변환 결과 검토 및 voice_inputs 저장 기능을 구현할 기반을 구성했습니다.

2. 구현 상세

구현 방식

  • 음성 기록 기능을 아래 계층으로 분리했습니다.

  • domain/voice
    ├── client # 외부 STT API 연동
    ├── controller # 음성 기록 API 요청 처리
    ├── dto # 요청/응답 DTO
    ├── entity # voice_inputs 엔티티
    ├── exception # 음성 파일·STT 관련 예외
    ├── repository # voice_inputs 저장소
    └── service # 음성 기록 비즈니스 로직

설계 및 기술적 결정

  • 외부 STT 연동 코드는 client 패키지로 분리해 서비스 로직과 HTTP 통신 구현을 분리했습니다.
  • 현재 ERD의 voice_inputs 테이블을 음성 입력 및 확정 텍스트 저장 대상으로 사용합니다.
  • STT 결과는 사용자 검토 후 확정 저장하는 흐름으로 구현할 예정입니다.

3. API 변경 내역

해당 없음

4. 데이터 및 인프라 영향

  • Database: 스키마 변경 없음. 향후 voice_inputs 테이블과 매핑 예정입니다.
  • Configuration: STT 연동을 위한 설정 클래스 뼈대를 추가했습니다.
  • Dependencies: 라이브러리 의존성 변경 없음.
  • Compatibility: 기존 API 및 데이터에 영향 없음.

5. 테스트 결과

미실행

6. 리뷰 포인트

domain.voice 패키지 및 클래스 분리 방식이 적절한지 검토 부탁드립니다.
voice_inputs를 최종 확정된 STT 텍스트 저장소로 사용하는 방향이 적절한지 검토 부탁드립니다.
STT API 설정 및 Client 구조가 팀의 외부 API 연동 방식과 일치하는지 확인 부탁드립니다.

7. 배포 참고 사항

  • 배포 전 확인: 실제 STT 서버 URL, 인증 방식, 응답 형식 확정 필요
  • 배포 후 확인: 해당 없음
  • 롤백 고려사항: 신규 구조 추가만 포함되어 기존 기능 영향 없음

8. 관련 이슈

  • TS-34 음성 기반 자동 육아 기록
  • TS-77 음성 입력 기능 구현
  • TS-78 STT API 연동
  • TS-101 육아 기록 저장 기능 구현

✅ Merge Checklist

  • 요구사항에 맞게 구현했습니다.
  • 예외 처리 및 입력값 검증을 확인했습니다.
  • 기존 기능에 미치는 영향을 검토했습니다.
  • 인증·인가 및 민감정보 노출 여부를 확인했습니다.
  • 테스트 및 빌드 결과를 확인했습니다.
  • 불필요한 코드와 디버깅 로그를 제거했습니다.

@claude

claude Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Claude encountered an error after 0s —— View job


I'll analyze this and get back to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants