Skip to content

Conversation

@wantkdd
Copy link
Member

@wantkdd wantkdd commented Dec 1, 2025

✨ 작업 개요

챗봇 오늘의 용어 단어 영어로 가져오기

📌 관련 이슈

✅ 작업 내용

챗봇 오늘의 용어 단어 영어로 가져오기(기존 한글)

📷 UI 스크린샷 (해당 시)

image

Summary by CodeRabbit

  • 리팩토링
    • 빠른 작업 기능에서 영어 용어를 우선으로 사용하고, 영어가 없을 때는 한국어 용어를 사용하도록 개선했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@wantkdd wantkdd self-assigned this Dec 1, 2025
@wantkdd wantkdd requested a review from a team as a code owner December 1, 2025 07:54
@wantkdd wantkdd added the 🧹 Refactor 코드 리팩토링 label Dec 1, 2025
@vercel
Copy link

vercel bot commented Dec 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
gotit Ready Ready Preview Comment Dec 1, 2025 7:54am

@coderabbitai
Copy link

coderabbitai bot commented Dec 1, 2025

Walkthrough

getTodaysTermQuery 함수가 영어 용어를 우선적으로 선택하도록 변경되었습니다. 영어 용어가 없으면 한국어 용어로 폴백하며, 용어가 없는 경우의 기존 동작(TypeScript 반환)은 유지됩니다. 공개 API 변경은 없습니다.

Changes

코호트 / 파일 변경 사항
챗봇 유틸리티 함수 리팩터링
src/app/chatbot/utils/quick-actions.ts
getTodaysTermQuery 함수 로직 수정: 영어 용어 우선 선택, 한국어 용어로 폴백 처리

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5-10분

  • 단일 파일 변경으로 범위가 제한적
  • 조건부 로직 변경만 포함되며 공개 인터페이스는 유지
  • 기존 폴백 동작(todayTerm 누락 시)에 대한 영향 확인 필요

Possibly related PRs

Poem

🐰 영어도 좋지만, 한글도 남겨두고,
오늘의 용어는 이제 영어로 먼저 나타나요!
폴백은 한글로, 부드럽게 내려앉고,
TypeScript는 여전히 우리의 기본값,
챗봇은 더욱 국제적이 되었다네! ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 챗봇의 오늘의 용어를 영어로 가져오는 작업을 명확하게 설명하며, 변경사항의 주요 목표를 정확히 반영합니다.
Linked Issues check ✅ Passed 코드 변경사항(getTodaysTermQuery 함수가 영어 용어를 우선하도록 변경)이 이슈 #102의 요구사항(챗봇 오늘의 용어를 한글에서 영어로 변경)을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 getTodaysTermQuery 함수 내 영어 용어 우선순위 설정에 집중되어 있으며, 요구사항 범위 내에 있습니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ 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 refactor/chatbot

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 and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
src/app/chatbot/utils/quick-actions.ts (1)

25-31: 영문 우선 + 한글 폴백 로직이 요구사항에 잘 맞습니다

todayTerm.termEn || todayTerm.termKo로 영어가 있으면 영어, 없으면 한국어로 폴백하도록 구현되어 있어서 PR 목표를 충족합니다. 또한 termEn이 빈 문자열일 때도 한글로 폴백되므로, "실제 값이 없으면 한글"이라는 의도라면 현재 구현이 적절합니다.

다만, 빈 문자열은 유효한 값으로 보고 null/undefined일 때만 폴백하고 싶다면 다음처럼 ??를 고려해볼 수도 있습니다:

return todayTerm.termEn ?? todayTerm.termKo;
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ba8046c and 103bc68.

📒 Files selected for processing (1)
  • src/app/chatbot/utils/quick-actions.ts (1 hunks)

@wantkdd wantkdd merged commit 6199f6d into develop Dec 1, 2025
4 checks passed
@wantkdd wantkdd deleted the refactor/chatbot branch December 1, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧹 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] 챗봇 오늘의 용어 영어로 가져오기

2 participants