[Release] BE

[lepitaaar]

v 1.1.4

Revert webp 변환 5ed1bd4

Summary by CodeRabbit

릴리스 노트

• 리팩토링

  • 이미지 URL 변환 및 지속성 로직을 제거하여 시스템을 단순화했습니다.
  • 이미지가 저장된 URL에서 직접 제공되도록 변경되었습니다.
  • 불필요한 의존성 및 비동기 처리 로직을 제거했습니다.

• 테스트

  • 관련 테스트 케이스를 정리했습니다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
moadong	Ready	Preview, Comment	Feb 9, 2026 3:39am

[coderabbitai]

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)

Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"

⚙️ 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

Walkthrough

이미지 URL 해석 및 지속성 기능을 제거합니다. ImageDisplayUrlResolver 인터페이스, 그 구현체들, ClubImageUrlPersistenceService, 관련 비동기 설정을 삭제하고, ClubDetailedResult와 여러 서비스를 단순화합니다.

Changes

Cohort / File(s)	Summary
ImageDisplayUrlResolver 제거 backend/src/main/java/moadong/media/resolver/ImageDisplayUrlResolver.java, backend/src/main/java/moadong/media/resolver/NoOpImageDisplayUrlResolver.java, backend/src/main/java/moadong/media/resolver/PreferWebpImageDisplayUrlResolver.java	이미지 URL 해석 인터페이스 및 구현체 완전 삭제. NoOp 구현과 S3 기반 WebP 변환 로직 제거.
ClubDetailedResult 단순화 backend/src/main/java/moadong/club/payload/dto/ClubDetailedResult.java	resolver 인자를 받는 팩토리 메서드 오버로드 제거. logo, cover, feeds를 직접 clubRecruitmentInformation에서 초기화.
ClubImageUrlPersistenceService 제거 backend/src/main/java/moadong/club/service/ClubImageUrlPersistenceService.java	비동기 이미지 URL 지속성 로직 및 에러 처리 완전 삭제.
서비스 레이어 정리 backend/src/main/java/moadong/club/service/ClubProfileService.java, backend/src/main/java/moadong/club/service/ClubSearchService.java	ImageDisplayUrlResolver 및 ClubImageUrlPersistenceService 의존성 제거. 이미지 URL 변환 로직 삭제.
AsyncConfig 리팩토링 backend/src/main/java/moadong/global/config/AsyncConfig.java	fcmAsyncExecutor() → taskExecutor() 메서드명 변경. imageUrlAsync 빈 완전 제거.
테스트 정리 backend/src/test/java/moadong/club/service/ClubImageUrlPersistenceServiceTest.java, backend/src/test/java/moadong/media/resolver/PreferWebpImageDisplayUrlResolverTest.java, backend/src/test/java/moadong/club/service/ClubSearchServiceTest.java, backend/src/test/java/moadong/unit/club/ClubProfileServiceTest.java	제거된 기능 관련 테스트 삭제 및 의존성 모의 객체 정리.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• [BE] Release #1166: 이 PR이 제거한 ImageDisplayUrlResolver, ClubImageUrlPersistenceService, 비동기 설정을 추가한 PR로, 동일 클래스들을 다루는 정반대 변경입니다.
• feat: 이미지 로딩 시에 webp가 존재하면 로딩 아니면 원본 로딩 기능 추가 #1134: 이 PR의 모든 제거 대상(resolver 인터페이스, 구현체, persistence 서비스)을 추가했던 PR로, 본 PR이 그 기능을 완전히 되돌립니다.
• [Release] BE v1.0.6 배포 #675: ClubDetailedResult 팩토리 메서드 변경을 다루는 PR로, 본 PR은 resolver 오버로드를 제거하며 관련된 변경입니다.

Suggested reviewers

• Zepelown
• seongwon030

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	PR 제목 '[Release] BE'는 매우 모호하고 구체적인 정보를 제공하지 않습니다. 이 변경사항이 webp 이미지 로딩 기능 제거를 포함한 주요 아키텍처 변경임을 전혀 나타내지 않습니다.	PR 제목을 '[Release] BE - 이미지 URL 해석기 및 webp 변환 기능 제거' 또는 유사한 형태로 변경하여 주요 변경 사항을 명확히 표현하세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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 develop/be

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@backend/src/test/java/moadong/unit/club/ClubProfileServiceTest.java`:
- Around line 28-34: The test class ClubProfileServiceTest is missing mocks for
two dependencies of ClubProfileService; add Mock fields for ClubSearchRepository
and RecruitmentStateCalculator alongside the existing ClubRepository and Javers
so `@InjectMocks` can construct ClubProfileService correctly; specifically declare
`@Mock` private ClubSearchRepository clubSearchRepository; and `@Mock` private
RecruitmentStateCalculator recruitmentStateCalculator; in the test class so all
constructor/injected dependencies of ClubProfileService are provided.

🧹 Nitpick comments (2)

backend/src/test/java/moadong/unit/club/ClubProfileServiceTest.java (1)

33-34: javers mock의 접근 제한자가 다른 mock 필드와 일관되지 않습니다.

clubRepository는 private으로 선언되어 있지만 javers는 package-private입니다. 일관성을 위해 private을 추가하는 것이 좋습니다.

♻️ 접근 제한자 통일 제안

 `@Mock`
-Javers javers;
+private Javers javers;

backend/src/main/java/moadong/club/service/ClubProfileService.java (1)

32-32: ClubSearchRepository가 이 서비스에서 사용되지 않습니다.

clubSearchRepository 필드가 주입되고 있지만, 클래스 내 어떤 메서드에서도 참조되지 않습니다. 불필요한 의존성이므로 제거하는 것을 권장합니다.

[oesnuj]

다시 제거한거 좋습니다
보완을 더 해보죠

[lepitaaar]

다시 제거한거 좋습니다 보완을 더 해보죠

좋습니다... 예상하지못했던 문제였네요