Skip to content

refactor: 로거 테스트 대역 공용화 및 HttpTranscriber logger 필수화 - #42

Merged
yuchem2 merged 1 commit into
mainfrom
CNV-70
Oct 9, 2026
Merged

yuchem2 merged 1 commit into
mainfrom
CNV-70

Conversation

@yuchem2

@yuchem2 yuchem2 commented Oct 9, 2026

Copy link
Copy Markdown
Member
  • spec 13곳에 복사돼 있던 noopLogger/silentLogger를 shared-kernel/testing/noop-logger.ts 하나로 통합
  • HttpTranscriber의 fetchFn·logger 기본값 제거: 실제 배선은 항상 둘 다 넘기므로, 누락 시 재시도 경고 로그가 조용히 사라지지 않게 타입으로 막는다

- spec 13곳에 복사돼 있던 noopLogger/silentLogger를 shared-kernel/testing/noop-logger.ts 하나로 통합
- HttpTranscriber의 fetchFn·logger 기본값 제거: 실제 배선은 항상 둘 다 넘기므로, 누락 시 재시도 경고 로그가 조용히 사라지지 않게 타입으로 막는다

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@notion-workspace

Copy link
Copy Markdown

@yuchem2 yuchem2 self-assigned this Oct 9, 2026
@yuchem2
yuchem2 merged commit 5cc116f into main Oct 9, 2026
2 checks passed
@yuchem2
yuchem2 deleted the CNV-70 branch October 9, 2026 10:20
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