Skip to content

Comments

운영환경 배포 - V1.5.3#83

Merged
2ghrms merged 14 commits intomainfrom
develop
Feb 23, 2026
Merged

운영환경 배포 - V1.5.3#83
2ghrms merged 14 commits intomainfrom
develop

Conversation

@2ghrms
Copy link
Member

@2ghrms 2ghrms commented Feb 23, 2026

🚀 Why - 해결하려는 문제가 무엇인가요?

  • 어떤 문제를 해결하고자 하나요?

  • 어떤 배경이 있었나요?

✅ What - 무엇이 변경됐나요?

  • 구현한 기능 요약

  • 주요 변경사항

🛠️ How - 어떻게 해결했나요?

  • 핵심 로직 설명

  • 예외 사항, 고민 포인트 등

🖼️ Attachment

  • 화면 이미지, 결과 캡처 등 첨부

💬 기타 코멘트

  • 리뷰어에게 전하고 싶은 말, 테스트 방법, 주의할 점 등

2ghrms added 14 commits February 7, 2026 21:46
- 변경된 prompt의 gradingListScores에 따른 파서 수정
- 파싱 실패시 재시도 로직 추가
…into SRLT-128-pdf-ai-리포트-고도화-및-이벤트-처리
- PDF: 공유 계층 예외 도입 (PdfDownloadException, PdfDownloadErrorType)
  - PdfDownloadClient가 aireport OcrException 의존 제거
- PDF URL 저장 시 검증
  - 커스텀 Validation @ValidPdfUrl 추가 (ValidPdfUrlValidator)
  - BusinessPlan/AiReport 요청 DTO에 @ValidPdfUrl 적용
  - 저장 전 downloadFromUrl로 접근 가능 여부 검증 (INVALID_PDF_URL)
- 전문가 신청: businessPlan에 pdfUrl 있으면 해당 URL에서 PDF 다운로드 후 전송
- AI 리포트: SpringAiSectionGradeAgent 재시도 지수 백오프, 마지막 시도 실패 메시지로 통일
- 이미지/PDF 인프라: PresignedUrlProviderPort 분리, NcpPresignedUrlProvider 정리
- ValidImageFileName/Validator를 adapter.shared.webapi.validation으로 이동
- BusinessPlanErrorType에 INVALID_PDF_URL 추가
- BusinessPlanImageController 등 관련 테스트·API Doc 반영
- 기존의 @url 방식의 검증을 ValidPdfUrlValidator에 통일
  - @notblank와 중첩되지 않도록 함
- PdfDownloadClient의 에러 메시지를 넘기도록 수정
- 기존의 AiReportResponseParser의 DTO/Entity 변환 로직은 AiReportResult로 로직 이동
- AiReportResponseParser를 adapter/aireport/report/parser로 패키지 이동
- BusinessPlanContentExtractor를 application/aireport/util로 이동
- 기존의 AiReportResponseParser의 DTO/Entity 변환 로직은 AiReportResult로 로직 이동
…트-처리

[SRLT-126] PDF AI 리포트 관련 에러 수정 및 구조 리팩토링
@2ghrms 2ghrms self-assigned this Feb 23, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'tools'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link

Test Results

246 tests   246 ✅  9s ⏱️
 45 suites    0 💤
 45 files      0 ❌

Results for commit b3b6140.

@2ghrms 2ghrms merged commit fabaaac into main Feb 23, 2026
5 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