[Refactor] analysis async composition 전환 (#271)

[shinae1023]

✨ 어떤 이유로 PR를 하셨나요?

• feature 병합
• 버그 수정(아래에 issue #를 남겨주세요)
• 코드 개선
• 코드 수정
• 배포
• 기타(아래에 자세한 내용 기입해주세요)

📋 세부 내용 - 왜 해당 PR이 필요한지 작업 내용을 자세하게 설명해주세요

analysis async 계층에서 service가 infrastructure/usecase를 상속하고 있어 역할 경계와 트랜잭션 경계가 클래스 선언만으로는 잘 드러나지 않는 문제가 있었습니다.
이번 PR에서는 상속 대신 composition으로 전환해서 외부에 노출하는 service는 필요한 public 메서드만 가지도록 정리했습니다.

변경 내용:

• AnalysisAsyncFacadeService가 AnalysisAsyncUseCase를 상속하지 않고 submit/getTask/cancel만 delegate 하도록 변경
• AnalysisAsyncProcessor가 AnalysisAsyncQueueProcessor를 내부 협력자로 가지도록 변경
• AnalysisWorkerBridgeService가 AnalysisAsyncWorkerBridge를 상속하지 않고 delegate 하도록 변경
• AnalysisWorkerBridgeService의 delegating method에 트랜잭션 경계를 명시
• AnalysisAsyncSweepService가 AnalysisAsyncTaskSweepCoordinator를 상속하지 않고 sweepTimedOutTasks()만 위임하도록 변경

기대 효과:

• service / usecase / infrastructure 역할 구분이 더 선명해짐
• 상속으로 퍼져 있던 public surface를 줄여서 이후 리팩터링과 테스트가 쉬워짐
• 트랜잭션 경계가 service 레이어에서 바로 보이도록 정리됨

📸 작업 화면 스크린샷

• 없음

⚠️ PR하기 전에 확인해주세요

• 로컬테스트를 진행하셨나요?
• 머지할 브랜치를 확인하셨나요?
• 관련 label을 선택하셨나요?

🚨 관련 이슈 번호 [ #271 ]

Summary by CodeRabbit

• 리팩토링
  • 비동기 분석 서비스의 상속 구조를 명시적인 위임 구조로 변경했습니다.
  • 작업 제출, 상태 조회, 취소, 처리, 시간 초과 작업 정리 기능을 기존과 동일하게 제공합니다.
  • 워커 상태 변경과 결과 저장·조회 기능을 명시적으로 노출했습니다.
  • 주요 상태 변경 및 결과 조회 작업에 트랜잭션 처리를 적용했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: fa1c2e21-8855-4c92-ac8a-fb931cc5d6d1

📥 Commits

Reviewing files that changed from the base of the PR and between 86c30e6 and dbf7a01.

📒 Files selected for processing (4)

• src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncFacadeService.java
• src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncProcessor.java
• src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncSweepService.java
• src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisWorkerBridgeService.java

---

📝 Walkthrough

Walkthrough

비동기 분석 관련 서비스 4개가 상속 구조를 제거하고 내부 유스케이스, 프로세서, 코디네이터, 브리지에 호출을 위임합니다. AnalysisWorkerBridgeService는 작업 상태와 결과 처리 메서드에 트랜잭션 경계를 명시합니다.

Changes

비동기 분석 서비스 위임 전환

Layer / File(s)	Summary
분석 서비스와 작업 오케스트레이션 위임 src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncFacadeService.java, src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncProcessor.java, src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisAsyncSweepService.java	서비스가 상속 대신 내부 객체를 보유합니다. 분석 접수·조회·취소, 큐 처리, 타임아웃 작업 정리 호출을 내부 구현에 위임합니다.
워커 브리지 상태 처리 위임 src/main/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisWorkerBridgeService.java	워커 실행·재시도·실패·완료와 컨텍스트 및 결과 처리를 명시적 메서드로 노출합니다. 상태 변경과 저장에는 일반 트랜잭션을 적용하고, 저장 결과 조회에는 읽기 전용 트랜잭션을 적용합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• JobDri-Developer/BackEnd#126: 동일한 비동기 분석 서비스 클래스의 도입 및 확장과 직접 연결됩니다.
• JobDri-Developer/BackEnd#269: 동일한 서비스들의 상속 기반 구현을 위임 기반 구현으로 전환한 변경과 직접 관련됩니다.
• JobDri-Developer/BackEnd#272: AnalysisAsyncSweepService의 생성자와 동일한 서비스 연결 지점을 변경합니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	상속에서 컴포지션으로 전환한 분석 비동기 계층의 핵심 변경을 간결하게 설명합니다.
Description check	✅ Passed	필수 섹션과 변경 목적, 세부 내용, 테스트 상태 및 이슈 번호를 포함해 설명이 충분합니다.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#271-analysis-organize

---

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.}

[shinae1023]

@coderabbitai review korean

[coderabbitai]

@shinae1023 한국어로 PR #273을 검토하겠습니다. 상속 제거 후 위임 구조, 공개 API 범위, 트랜잭션 경계를 중점적으로 확인하겠습니다.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.