Skip to content

Feat: OpenAI fallback 관측성 강화 - #14

Merged
Heyaaz merged 1 commit into
mainfrom
feature/openai-fallback-observability
May 17, 2026
Merged

Feat: OpenAI fallback 관측성 강화#14
Heyaaz merged 1 commit into
mainfrom
feature/openai-fallback-observability

Conversation

@Heyaaz

@Heyaaz Heyaaz commented May 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • OpenAI fallback 관측성 강화

Changes

  • LLM refinement 시도 결과 모델 추가
  • OpenAI 실패 원인 분류 및 deterministic fallback 처리
  • LLM 성공/fallback 구조화 로그 추가
  • Micrometer call/latency 지표 추가
  • 일정 생성/조회 LLM metadata 응답 및 저장 반영
  • LLM 응답 슬롯 전체성/후보 유효성 검증 추가
  • 공개 공유 일정 응답 LLM 운영 metadata 제외
  • fallback/parse/metadata/public share 회귀 테스트 추가

Verification

  • Not run
  • Build
  • Test: ./gradlew clean test --no-daemon --no-build-cache
  • Manual

Notes

  • public share 전용 mapper 완전 분리는 후속 가능

@Heyaaz Heyaaz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Nogada PR Review

Verdict

  • 승인 가능: blocking 이슈 없음

Blocking issues

  • 없음

Non-blocking notes

  • ./gradlew clean test --no-daemon --no-build-cache 성공 후 Testcontainers/Hikari PostgreSQL 종료 경고가 로그에 남았지만, Gradle exit 0 및 XML 기준 실패 없음.

Checked scope

  • 전체 PR 변경 파일 10개 검토
  • OpenAI fallback reason 분류, deterministic fallback, Micrometer call/latency 지표, LLM metadata 저장/응답 매핑 확인
  • 공개 공유 일정 응답에서 LLM 운영 metadata 제외 확인
  • LLM 응답 슬롯 전체성/후보 유효성 검증 및 회귀 테스트 확인
  • git diff --check origin/main...pr-14 통과
  • /tmp/tripsync_server-pr-14 worktree에서 ./gradlew clean test --no-daemon --no-build-cache 실행: 33 tests, 0 failures, 0 errors, 0 skipped

@Heyaaz
Heyaaz merged commit c56895f into main May 17, 2026
1 check passed
@Heyaaz
Heyaaz deleted the feature/openai-fallback-observability branch May 17, 2026 08:58
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