Skip to content

Revert "[Feat] voice to text by stt"#8

Merged
H4nnhoi merged 1 commit intomainfrom
revert-6-5-feat-voice-to-text-by-stt
Oct 26, 2025
Merged

Revert "[Feat] voice to text by stt"#8
H4nnhoi merged 1 commit intomainfrom
revert-6-5-feat-voice-to-text-by-stt

Conversation

@H4nnhoi
Copy link
Contributor

@H4nnhoi H4nnhoi commented Oct 26, 2025

Reverts #6

Summary by CodeRabbit

변경 사항

  • 제거된 기능

    • 음성 파일 업로드 및 관리 기능
    • 감정 분석 엔드포인트
    • 음성 인식(STT) 엔드포인트
  • 리팩토링

    • 애플리케이션 구조 및 의존성 정리
    • 헬스 체크 엔드포인트만 유지
    • 프로젝트 설명서 업데이트

@H4nnhoi H4nnhoi merged commit 67cce91 into main Oct 26, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 26, 2025

Caution

Review failed

The pull request is closed.

개요

애플리케이션의 핵심 기능들을 제거하는 대규모 리팩토링으로, S3 업로드, 감정 분석, 음성 인식 관련 엔드포인트 및 서비스 모듈을 삭제하고 헬스 체크 엔드포인트만 남겼습니다. 환경 변수 로딩 및 의존성도 제거되었습니다.

변경 사항

응집 / 파일 변경 요약
문서 업데이트
README.md
uvicorn 실행 명령어를 python -m app.main 형식으로 변경. API 문서 URL, 환경 변수 설정, AWS S3, Google Cloud Speech-to-Text 구성 섹션 제거. main.py 설명을 "메인 엔트리 포인트"로 수정.
환경 및 설정 정리
app/__init__.py, app/constants.py
dotenv 로드 로직 제거. VOICE_BASE_PREFIX, DEFAULT_UPLOAD_FOLDER 상수 제거.
서비스 모듈 전체 삭제
app/emotion_service.py, app/stt_service.py, app/s3_service.py
EmotionAnalyzer 클래스 및 analyze_voice_emotion 함수 제거. GoogleSTTService 클래스 및 transcribe_voice 함수 제거. S3 클라이언트 유틸리티(get_s3_client, upload_fileobj, list_bucket_objects) 제거.
API 엔드포인트 대폭 축소
app/main.py
음성 업로드, 목록 조회, 감정 분석, 음성 인식 관련 6개 엔드포인트 제거. 헬스 체크 엔드포인트만 유지. S3, 감정 분석, STT 관련 모든 임포트 및 비즈니스 로직 제거.
의존성 정리
requirements.txt
구체적인 패키지 목록(fastapi, uvicorn, boto3, python-dotenv, transformers, torch, librosa, scipy, google-cloud-speech 등) 제거. 한글 코멘트와 예시 패키지로 템플릿 형식으로 변경.

예상 코드 리뷰 난이도

🎯 4 (복잡) | ⏱️ ~45분

  • 주의 필요 영역:
    • app/main.py: 6개 엔드포인트 및 관련 로직의 완전한 제거 검증 필요
    • app/emotion_service.py, app/stt_service.py, app/s3_service.py: 삭제된 모듈이 다른 곳에서 호출되지 않는지 확인 필요
    • app/constants.py: 제거된 상수들의 의존성 검토
    • requirements.txt: 템플릿 형식 변경으로 인한 설치 프로세스 영향 확인

연관된 PR들

  • [Feat] voice to text by stt #6: 이 PR에서 제거되는 정확한 모듈들(s3_service, stt_service, emotion_service, constants)과 API 엔드포인트들을 이전에 추가했던 PR

🐰 기능들을 정리하고 버리니,
헬스 체크만 남겨두고
의존성 꾸러미도 비워냈지,
깨끗한 캔버스에서 다시 시작하네! ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-6-5-feat-voice-to-text-by-stt

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e0f0bb6 and dbdcbec.

📒 Files selected for processing (8)
  • README.md (1 hunks)
  • app/__init__.py (0 hunks)
  • app/constants.py (0 hunks)
  • app/emotion_service.py (0 hunks)
  • app/main.py (1 hunks)
  • app/s3_service.py (0 hunks)
  • app/stt_service.py (0 hunks)
  • requirements.txt (1 hunks)

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

@H4nnhoi H4nnhoi deleted the revert-6-5-feat-voice-to-text-by-stt branch October 26, 2025 06:16
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.

1 participant