Skip to content

release: develop 운영 반영 - #302

Merged
hyoj-dev merged 19 commits into
mainfrom
develop
Aug 24, 2026
Merged

release: develop 운영 반영#302
hyoj-dev merged 19 commits into
mainfrom
develop

Conversation

@hyoj-dev

Copy link
Copy Markdown
Collaborator

변경 사항

  • 소셜 로그인 계정 복구 및 서버 판정 흐름을 운영에 반영합니다.
  • Google·Kakao 토큰 검증과 운영 OAuth 환경변수 전달을 추가합니다.
  • 회고 대화형 V2 API와 관련 저장 구조를 반영합니다.
  • STT 오류 처리와 OpenAI 연동을 개선합니다.
  • 관리자 마케팅 푸시 발송 API와 마케팅 수신 동의 정책을 반영합니다.
  • 신규 도메인 기준 Nginx 설정과 DB 백업 S3 경로 변경을 반영합니다.
  • Flyway V46, V47 마이그레이션을 순서대로 적용합니다.

변경 이유

개발 환경에서 검증한 최신 기능과 신규 도메인·소셜 로그인 복구 구성을 운영 환경에 동일한 순서로 반영하기 위한 릴리스입니다. develop 전체를 배포해 Flyway 버전 순서와 브랜치 이력을 일관되게 유지합니다.

영향

  • 운영 배포 시 Flyway V46과 V47이 적용됩니다.
  • 기존 Google·Kakao 식별자는 그대로 로그인하고, 새로운 식별자는 이메일 OTP를 거쳐 기존 계정 연결 또는 신규 가입 여부를 서버가 판정합니다.
  • 회고 V2 API는 배포되지만 프런트에서 호출하지 않으면 사용자 화면에는 노출되지 않습니다.
  • 기존 STT 처리와 관리자 이메일 대상 정책 일부가 변경됩니다.
  • Apple 로그인은 OAUTH_APPLE_ENABLED=false로 유지됩니다.

검증

  • develop 전체 테스트 통과
  • pre-push 전체 테스트 통과
  • GitHub Actions build, docker-build, develop deploy 통과
  • 개발 환경 Google·Kakao 로그인 확인
  • 개발 환경 기존 서비스 기능 확인

배포 전 확인

  • 최신 운영 DB 백업 및 S3 업로드 확인
  • 활성 회원 (provider, provider_id) 중복 점검 결과 0행 확인
  • PROD_OAUTH_* GitHub Secrets 등록 확인
  • PROD_CORS_ALLOWED_ORIGINShttps://app.didit.io.kr 포함 확인
  • PROD_OAUTH_KAKAO_REDIRECT_URI=https://app.didit.io.kr/auth/kakao/callback 확인

배포 후 확인

  • https://api.didit.io.kr/actuator/healthUP인지 확인
  • Flyway V46·V47 적용 성공 확인
  • 기존 API와 Google·Kakao 로그인 스모크 테스트
  • 백엔드 확인 후 프런트 Production 배포 진행

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment on lines +153 to +155
fun classify(relevance: MessageRelevance) {
this.relevance = relevance
includedInResult = relevance == MessageRelevance.RETROSPECTIVE

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 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 👍 / 👎.

Comment on lines +212 to +216
ConversationTurnStatus.FAILED -> {
existing.retry()
turnRepository.save(existing)
metrics.incrementConversationRetry()
return@execute createPreparation(retrospective, existing, userMessage)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

@hyoj-dev
hyoj-dev merged commit dda782c into main Aug 24, 2026
6 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.

2 participants