Skip to content

Conversation

@can019
Copy link
Collaborator

@can019 can019 commented Sep 29, 2025

No description provided.

thkim7 and others added 30 commits September 26, 2025 13:48
1. insert-workflow.sql 수정
2. BlogRagBodyBuilder 수정
3. ImageOcrBodyBuilder 추가
OCR 단계가 추가되면서 bodybuilder 작성 및 sql 수정
1. Job/Task 서비스 통합
   - JobService, TaskService 삭제
   - WorkflowService에 Job/Task CRUD 메서드 통합
   - JobController, TaskController에서 WorkflowService 사용하도록 변경

2. Task 데이터 구조 확장 (9개 → 10개)
   - 새로운 Task 추가: "이미지 OCR 태스크" (ID: 8)
   - 기존 Task ID 변경: RAG 생성 (8→9), 블로그 발행 (9→10)
   - H2 테스트 데이터(03-insert-workflow-h2.sql) 업데이트

3. Integration 테스트 리팩토링
   - JobApiIntegrationTest, TaskApiIntegrationTest에서 WorkflowService 사용
   - Task 10개 구조에 맞춘 테스트 케이스 추가
   - 새로운 OCR Task 및 업데이트된 블로그 발행 Task 테스트 추가

4. 코드 일관성 및 응집도 개선
   - 워크플로우 관련 모든 CRUD 기능을 단일 서비스로 집중
   - Job-Task 연결 구조 업데이트: Job1(Task 1-8), Job2(Task 9-10)
1. h2.sql task 9개 -> 10개
2. JobApiIntegrationTest는 모두 통과
3. TaskApiIntegrationTest는 자꾸 Json 직렬화 때문에 통과를 못함
* fix: RDB loging execution_type TASK로 통일

* refactor: 사용하지 않는 endpoint 삭제

* refactor: 에러 상태코드  변경
- HTTP_422_UNPROCESSABLE_ENTITY -> HTTP_422_UNPROCESSABLE_CONTENT 변경

* refactor: 사용하지않는 코드 삭제

* test: 테스트코드 최신화

* refactor: 사용하지않는 endpoint 주석

* style: FAST API 코드 포멧팅

---------

Co-authored-by: kakusia <[email protected]>
- verify 호출 시 ArgumentMatchers.eq() 사용하여 InvalidUseOfMatchersException 해결
- workflowId 매개변수에 Matcher 적용으로 통합 테스트 안정화
…am-icebang into feature/task

# Conflicts:
#	apps/user-service/src/test/java/site/icebang/integration/tests/workflow/WorkflowRunApiIntegrationTest.java
* chore: fix branch에서 deploy 비활성화

* fix: Log4j2 classpath가 아닌 외부 파일 mount

* chore: Bootjar에서 log4j 설정 제거
* feat: TaskIoData model 생성

* feat: TaskIoData관련 Mapper 작성

* feat: TaskRun관련 Mapper 추가

* feat: 워크플로우 컨텍스트 조회 서비스 추가

- 퍼싸드 패턴 도입

* feat: 인메모리 컨텍스트 방식에서 영속적 컨텍스트 방식으로 변경

* feat: 인터페이스 변경에 따른 구현체 변경 및 컨텍스트 조회 서비스 사용

* refactor: Code Formatting

* refactor: WorkflowContext가 영속적 컨텍스트를 기반으로 동작하도록 리팩토링

* refactor: s3 태스크 관련 로직은 임시로 service에 둠

- 추후에 리팩토링 필요
* fix: blogger post_url 추가

* sytle: 코드 포맷팅
`@Async`와 `@Transactinon` 동시에 사용했던 문제
* chore: gitingore에 key, blogger 설정 추가

* feat: Task io data api
fix: rag_create 문제 생긴거 수정
@can019 can019 merged commit bfe8d84 into main Sep 29, 2025
10 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.

6 participants