Conversation
* fix: DB 백업 S3 버킷 변경 * chore: 도메인 이전에 따른 파일 수정
ci: 운영 소셜 로그인 환경 변수 전달
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6ee6e36288
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| fun classify(relevance: MessageRelevance) { | ||
| this.relevance = relevance | ||
| includedInResult = relevance == MessageRelevance.RETROSPECTIVE |
There was a problem hiding this comment.
Exclude irrelevant V2 messages from summary input
When a V2 message is classified as OFF_TOPIC, BRIDGEABLE, or SERVICE_HELP, this flag is set to false, but the result-generation path still calls Retrospective.getAllAnswers(), which includes every non-skipped user message without checking includedInResult. Consequently, calling the existing /api/v1/retrospectives/{id}/complete endpoint after finishing a V2 conversation feeds unrelated requests into the final AI summary, despite the new relevance classification explicitly excluding them.
Useful? React with 👍 / 👎.
| ConversationTurnStatus.FAILED -> { | ||
| existing.retry() | ||
| turnRepository.save(existing) | ||
| metrics.incrementConversationRetry() | ||
| return@execute createPreparation(retrospective, existing, userMessage) |
There was a problem hiding this comment.
Check for another processing turn before retrying
When a failed clientMessageId is retried, this branch immediately changes it to PROCESSING and returns before the conflict check below. If another message was submitted after the first failure and its AI call is still running, retrying the failed message therefore launches a second AI call for the same retrospective; both calls use independently captured context and may save assistant messages and analysis updates out of conversational order. Apply the same existing-processing-turn guard to failed retries before changing their status.
Useful? React with 👍 / 👎.
chore: main 운영 이력 develop 동기화
변경 사항
변경 이유
개발 환경에서 검증한 최신 기능과 신규 도메인·소셜 로그인 복구 구성을 운영 환경에 동일한 순서로 반영하기 위한 릴리스입니다.
develop전체를 배포해 Flyway 버전 순서와 브랜치 이력을 일관되게 유지합니다.영향
OAUTH_APPLE_ENABLED=false로 유지됩니다.검증
배포 전 확인
(provider, provider_id)중복 점검 결과 0행 확인PROD_OAUTH_*GitHub Secrets 등록 확인PROD_CORS_ALLOWED_ORIGINS에https://app.didit.io.kr포함 확인PROD_OAUTH_KAKAO_REDIRECT_URI=https://app.didit.io.kr/auth/kakao/callback확인배포 후 확인
https://api.didit.io.kr/actuator/health가UP인지 확인