Skip to content

Conversation

@GulSauce
Copy link
Member

@GulSauce GulSauce commented Dec 17, 2025

Summary by CodeRabbit

  • Documentation

    • Updated prompt terminology and formatting guidelines with refined structure requirements.
    • Adjusted content length constraints to 25-55 characters.
    • Clarified title formatting rules and mandatory prefix standards.
  • Refactor

    • Updated and reorganized core dependencies including HTTP clients, AI model libraries, and web server components for enhanced stability.

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

* [ICC-25] 구성 완료

* [ICC-29] 구현 완료

* [ICC-29] 응답 구조 수정

* [ICC-30] 작성 완료

* [ICC-31] 레디스 구독 패턴 구현 완료

* [ICC-31] 레디스 클라이언트를 각 생성마다 생성되게함

* [ICC-31] 필요 없는 의존성 제거

* [ICC-31] requirements.txt 등록

* [ICC-31] 베드락에 요청하는 코드 분리

* [ICC-31] 타입아웃 설정

* [ICC-32] 첫번째 시도

* [ICC-32] 두번째 시도

* [ICC-32] 세번째 시도

* [ICC-32] 네번째 시도

* [ICC-32] 다섯번째 시도

* [ICC-32] 성공

* [ICC-32] 리모트환경에서는 바로 SQS를 보게함

* [ICC-32] 임시 수정

* [ICC-32] boto3 의존성 추가

* [ICC-32] 리전 추가

* [ICC-32] 반응 브랜치 변경

* [ICC-39] 구현 완료

* [ICC-47] quiz parsing (#13)

* [ICC-35] Create 5 Quiz

* ICC-38 json 객체 테스트

* [ICC-47] Redis 주석 처리 해제

---------

Co-authored-by: Oh YoungJe <pine7420@naver.com>

* [ICC-49] Dependency 해결

* [ICC-50] API서버 연결 간 문제 해결 (#15)

* [ICC-50] generation으로 엔드포인트 변경

* [ICC-50] api와 연결간 오류 해결

* [ICC-53] 구현 완료

* [ICC-53] 로거, 전역 예외처리기 추가

* [ICC-53] 일부 수정

* [ICC-57] 구현 완료

* [ICC-67] 응답 구조 수정 완료

* [ICC-63] Webb's DOK

# Conflicts:
#	app/service/generate_service.py

* [ICC-63] 추가 코드 작성

* [ICC-70] api 수정

* [ICC-48] MCP 기능 구현  완료

* [ICC-48] MCP 리서치

* [ICC-48] 엔드포인트 분리

* [ICC-74] 요청 dto 변경

* [ICC-74] 페이지 일부만 가져오기 구현 완료

* [ICC-74] 파워포인트도 적용 완료

* [ICC-74] 불필요한 로그출력 제거

* [ICC-74] 페이지 선택 방식 변경

* [ICC-76] 발생한 문제 해결

* [ICC-76] 코드 수정

* [ICC-80] 프롬프트 추가

* [ICC-80] 프롬프트 추가

* [ICC-80] 롤백

* [ICC-80] 롤백

* [ICC-87] 최대 청크 개수 증가

* [ICC-87] 기본 구조 작성

* [ICC-87] 레디스 분산락 구현

* [ICC-87] 필요 없는  레디스 함수 제거:

* [ICC-87] 한번에 데이터를 넣게 함

* [ICC-99] generate dto 변경

* [ICC-99] 구현 완료

* [ICC-99] generate dto 변경

* [ICC-99] 구현 완료

* [ICC-94] specific explanation 서비스 추가

* [ICC-94] 구현완료

* ICC-112 OX 프롬프트 수정

* [ICC-112] 리팩터링 + BLANK 타입 추가

* [ICC-134] 필요한 프롬프트 추가

* [ICC-164] 작성 완료

* [ICC-167] 프롬프트 수정

* [ICC-167] 프롬프트 변경

* [ICC-167] 프롬프트 변경

* [ICC-167] 프롬프트 변경

* [ICC-167] 프롬프트 변경

* [ICC-167] 프롬프트 변경

* [ICC-167] 프롬프트 수정

* [ICC-167] 프롬프트 수정

* [ICC-167] 프롬프트 수정

* [ICC-167] 잘못된 프롬프트 반환 수정

* [ICC-167] 잘못된 프롬프트 반환 수정

* [ICC-167] 모델 변경

* [ICC-167] 리팩터링 완료

* [ICC-167] 코드 포맷팅

* [ICC-167] 제약사항 반영

* [ICC-167] 프롬프트 수정

* [ICC-167] 리팩터링

* [ICC-175] 도커 컴포즈 제거

* [ICC-175] 개발 깃허브 액션 스크립터 제거

* [ICC-180] 셔플 로직 추가

* [ICC-180] CD 브랜치명 올바르게 변경

* [ICC-189] 타임아웃시 전체 실패가 아닌 일부라도 반환하게함

* [ICC-192] 로직변경완료

* [ICC-194] 선지를 밝히지 말라는 문구 추가

* [ICC-196] 마이그레이션 완료

* [ICC-196] 프롬프트 롤백

* [ICC-196] min quiz size todo에 추가

* [ICC-196] 개행 문구 추가

* [ICC-200] 타입에 따라 셔플하도록 함

* Refactor quiz generation and explanation services; remove unused Bedrock adapter and integrate GPT for specific explanations. Update router to utilize new explanation service.

* [ICC-212] 커서를 통한 1차 리팩터링 완료

* [ICC-212] 어댑터 단순화

* [ICC-212] 블랙 포맷팅 적용

* [ICC-212] 기능 동작하게 수정 완료

* [ICC-212] 개행 프롬프트 추가

* [ICC-212] 구성 완료

* [ICC-215] 수정 완료

---------

Co-authored-by: 이찬우 <105221020+ChanJinYeon@users.noreply.github.com>
Co-authored-by: Adansonia <lcw061145@gmail.com>
Co-authored-by: lhoju0158 <lhoju0158@gmail.com>
@coderabbitai
Copy link

coderabbitai bot commented Dec 17, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Terminology updated from "지문" to "전제" across prompt templates in blank.py and multiple.py, with structural enhancements to the multiple.py template including mandatory title-format rules and refined length constraints. Dependencies in requirements.txt substantially revised with version updates and additions across core libraries.

Changes

Cohort / File(s) Summary
Prompt Template Terminology Update
app/prompt/core/blank.py
Replaced "지문" with "전제" across prompt template titles, descriptors, and examples; purely textual renaming within existing structure
Prompt Template Restructuring
app/prompt/core/multiple.py
Reorganized sections from "지문" to "전제" terminology; introduced mandatory title-format rule block specifying two-block structure, "[전제]" prefix, "[질문]" label placement, and formatting constraints; updated length guidance from 2545자 to 2555자; refined readability and content guidelines for 전제 block
Dependency Management
requirements.txt
Substantial version updates across core libraries (HTTP: httpx, aiohttp, httpcore; ML tooling: langchain suite, openai; serialization: orjson, jiter; document processing: PyMuPDF, python-pptx, docx2txt; others: pydantic, urllib3, uvicorn); added ecosystem packages (aiohappyeyeballs, aiosignal, marshmallow, build, distro); downgraded/removed several conflicting versions

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Requirements.txt dependency version changes should be verified for compatibility and potential breaking changes across the updated packages
  • Prompt template changes are mechanical terminology replacements; ensure consistency across all references

Possibly related PRs

Poem

🐰 From 지문 to 전제, words dance and sway,
New rules guide the prompts in their structured way,
Dependencies shimmer with versions anew,
A terminology bloom in the rabbit's grand brew! 🌱✨

✨ 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 main

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ba4647 and 44f5bd9.

📒 Files selected for processing (3)
  • app/prompt/core/blank.py (4 hunks)
  • app/prompt/core/multiple.py (6 hunks)
  • requirements.txt (1 hunks)

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

@GulSauce GulSauce merged commit 02e6602 into develop Dec 17, 2025
2 of 3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Dec 20, 2025
1 task
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