Skip to content

Fix/chat input and remove cron#26

Merged
SanghunYun95 merged 66 commits intomainfrom
fix/chat-input-and-remove-cron
Mar 26, 2026
Merged

Fix/chat input and remove cron#26
SanghunYun95 merged 66 commits intomainfrom
fix/chat-input-and-remove-cron

Conversation

@SanghunYun95
Copy link
Copy Markdown
Owner

@SanghunYun95 SanghunYun95 commented Mar 26, 2026

Summary by CodeRabbit

릴리스 노트

  • 버그 수정

    • 채팅 입력창의 Enter 키 송신 동작을 개선했습니다.
  • 기타

    • GitHub Actions 정기 헬스 체크 워크플로우를 제거했습니다.

…ifespan, and move asyncio import to stdlib group
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 26, 2026

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

Project Deployment Actions Updated (UTC)
philo-rag Ready Ready Preview, Comment Mar 26, 2026 1:02am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 430ae3bc-c5f1-4644-8d3b-b3f5c01ee4bd

📥 Commits

Reviewing files that changed from the base of the PR and between be7dc7a and c507e59.

📒 Files selected for processing (2)
  • .github/workflows/keep-alive.yml
  • frontend/components/chat/FloatingInput.tsx
💤 Files with no reviewable changes (2)
  • frontend/components/chat/FloatingInput.tsx
  • .github/workflows/keep-alive.yml

📝 Walkthrough

Walkthrough

GitHub Actions 키프-얼라이브 워크플로우 파일이 제거되었으며, 채팅 입력 컴포넌트에서 IME 조합 입력(composition) 타이밍 검사 로직이 삭제되었습니다. 두 변경사항 모두 이전에 있던 기능성을 제거하는 것입니다.

Changes

Cohort / File(s) Summary
GitHub Actions 워크플로우 삭제
.github/workflows/keep-alive.yml
정기적으로 실행되는 헬스 체크 엔드포인트에 대한 HTTP HEAD 요청을 보내던 전체 워크플로우 제거(cron 스케줄 및 수동 실행 포함).
입력 컴포넌트 수정
frontend/components/chat/FloatingInput.tsx
Enter 키 전송 시 IME/네이티브 조합 입력 상태 검사(isComposing, 조합 종료 후 50ms 타이밍 제한) 제거.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Feat/add keep alive action #24: 동일한 keep-alive 워크플로우 파일을 수정하는 PR—이 PR에서 삭제하는 워크플로우를 변경함.
  • Feat/add keep alive action #22: 이 PR에서 삭제하는 .github/workflows/keep-alive.yml 워크플로우를 추가한 PR과 직접 관련.
  • Feat/add keep alive action #25: 동일한 keep-alive 워크플로우를 건드리는 PR—이 PR에서 삭제하는 워크플로우의 curl/헬스 체크 로직을 수정함.

Poem

🐰 설정 파일을 지우고, 조합 검사도 빼네요,
단순함의 미학, 깔끔한 흐름 찾아요!
불필요한 가드 없이 Enter가 춤춘답니다,
워크플로우는 작별, 입력은 자유로워져요. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 두 가지 주요 변경사항을 정확하게 반영하고 있습니다: 채팅 입력 수정 및 크론 워크플로우 제거.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/chat-input-and-remove-cron
⚔️ Resolve merge conflicts
  • Resolve merge conflict in branch fix/chat-input-and-remove-cron

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

@SanghunYun95 SanghunYun95 merged commit a5b609e into main Mar 26, 2026
3 checks passed
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