🔀 :: [#732] - 애플리케이션 생성및 수정시 초기화 스크립트 스펙 추가

[dolong2]

개요

• 애플리케이션 생성및 수정시 초기화 스크립트 스펙을 추가합니다.

작업내용

• 애플리케이션에 초기화 스크립트 작성 서비스 추가
• 애플리케이션 생성및 수정 스펙에 초기화 스크립트 필드 추가
• 변경사항 관련 테스트 케이스 추가

체크리스트

탬플릿외에 필요한 항목이 있으면 추가해주세요.

• 로컬에서 빌드가 성공하나요?
• 추가(수정)한 코드가 정상적으로 동작하나요?
• pr 타켓 브랜치가 맞게 설정되어 있나요?
• pr에서 작업할 내용만 작업됐나요?
• 기존 API와 호환되지 않는 사항이 있나요?

Summary by CodeRabbit

릴리스 노트

• New Features

  • 애플리케이션 생성 및 업데이트 시 초기 스크립트를 지정·저장 가능
  • 애플리케이션 업데이트 요청에서 포트 설정 지원

• Bug Fixes / Data

  • 관련 데이터베이스 외래키 제약 강화

• Tests

  • 초기 스크립트 저장 및 검증을 위한 테스트 시나리오 추가

[coderabbitai]

Walkthrough

애플리케이션 생성·업데이트 요청에 initialScripts: List<String> 필드를 추가하고, 이를 저장·갱신하는 InitialScriptService와 구현체를 도입하여 유스케이스에서 초기 스크립트를 삭제 후 재저장하도록 호출하도록 변경했습니다. 일부 DTO에 port 필드가 추가·재정렬되었습니다.

Changes

코호트 / 파일(s)	변경 요약
프레젠테이션 요청 DTO src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/CreateApplicationRequest.kt, src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/UpdateApplicationRequest.kt	initialScripts: List<String> 필드 추가 (Create: 기본값 빈 리스트). Update에는 port: Int 추가 및 일부 필드 순서 조정
도메인 요청 DTO src/main/kotlin/com/dcd/server/core/domain/application/dto/request/CreateApplicationReqDto.kt, src/main/kotlin/com/dcd/server/core/domain/application/dto/request/UpdateApplicationReqDto.kt	initialScripts: List<String> 필드 추가; UpdateApplicationReqDto에 port: Int 추가 (생성자 시그니처 변경)
InitialScriptService (인터페이스 + 구현) src/main/kotlin/com/dcd/server/core/domain/application/service/InitialScriptService.kt, src/main/kotlin/com/dcd/server/core/domain/application/service/impl/InitialScriptServiceImpl.kt	새 인터페이스 추가 (write(application, initialScripts)) 및 구현체 추가. 구현은 기존 스크립트 삭제 후 요청된 스크립트들을 UUID로 매핑하여 saveAll 호출
유스케이스 통합 src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt, src/main/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCase.kt	생성자에 InitialScriptService 주입 추가 및 생성/업데이트 흐름에서 initialScriptService.write(application, req.initialScripts) 호출 삽입
요청 DTO 매핑 확장 src/main/kotlin/com/dcd/server/presentation/domain/application/data/exetension/ApplicationRequestDataExtension.kt	프레젠테이션 요청 객체에서 도메인 DTO로 변환할 때 initialScripts 필드 매핑 추가
테스트 및 데이터베이스 src/test/kotlin/.../usecase/CreateApplicationUseCaseTest.kt, src/test/kotlin/.../usecase/UpdateApplicationUseCaseTest.kt, src/test/kotlin/.../ApplicationWebAdapterTest.kt, src/test/resources/data.sql	테스트 요청 객체에 initialScripts 포함, 초기 스크립트 저장/조회 검증 추가, 테스트용 SQL에 외래키 제약 추가

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Client
    participant UseCase as CreateApplicationUseCase
    participant Env as EnvAutoMatchService
    participant ScriptSvc as InitialScriptService
    participant CmdPort as CommandApplicationInitialScriptPort
    participant Repo as Persistence

    Client->>UseCase: execute(createRequest)
    UseCase->>Env: match(application)
    Env-->>UseCase: matched
    UseCase->>ScriptSvc: write(application, initialScripts)
    ScriptSvc->>CmdPort: deleteByApplication(application)
    CmdPort->>Repo: DELETE existing scripts
    Repo-->>CmdPort: deleted
    ScriptSvc->>ScriptSvc: map strings -> ApplicationInitialScript (UUID 생성)
    ScriptSvc->>CmdPort: saveAll(newScripts)
    CmdPort->>Repo: INSERT scripts
    Repo-->>CmdPort: saved
    CmdPort-->>ScriptSvc: saved
    ScriptSvc-->>UseCase: done
    UseCase-->>Client: createdApplication

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 분

• 주의할 점:
  • InitialScriptServiceImpl.write()의 삭제-저장 로직이 트랜잭션/동시성 측면에서 안전한지 검토
  • DTO 생성자/매핑 변경으로 인한 호출부(테스트·컨트롤러 등) 누락 여부 확인
  • UpdateApplicationUseCase에 추가된 port 인자 순서가 DI/생성자 호출과 일치하는지 검증
  • 테스트들이 초기 스크립트의 경계조건(빈 리스트, 중복 등)을 충분히 커버하는지 확인

Possibly related PRs

• 🔀 :: [#689] - 애플리케이션 생성시 matcher를 생성하는 로직 추가 #692 — CreateApplicationUseCase에 동기적 사전 호출 패턴(envAutoMatchService.match)을 추가한 PR로, 본 PR과 같은 실행 흐름상 위치에 새로운 동작(initialScriptService.write)을 추가한 연관성 있음.
• 🔀 :: [#731] - 초기화 스크립트 엔티티 추가 #733 — ApplicationInitialScript 도메인·포트·퍼시스턴스 관련 변경을 도입한 PR로, 본 PR의 초기 스크립트 저장/조회 경로와 직접적으로 연결됨.

시

🐰 새 코드에 깡충, 스크립트 한 줄,
삭제하고 다시 적어두니 반짝이네.
생성과 갱신에 함께 뛰어,
빈 리스트도 안심하고 맡겨둬요. ✨

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 애플리케이션 생성 및 수정 시 초기화 스크립트 스펙 추가라는 주요 변경사항을 명확하게 요약하고 있으며, 변경 사항의 핵심 목적과 일치합니다.

✨ 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 feature/initial-script-to-application-spec

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 922e42d and f44b815.

📒 Files selected for processing (2)

• src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/CreateApplicationRequest.kt (1 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/UpdateApplicationRequest.kt (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/UpdateApplicationRequest.kt

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: dolong2
Repo: dolong2/dcd PR: 703
File: src/main/kotlin/com/dcd/server/core/domain/volume/model/Volume.kt:1-12
Timestamp: 2025-09-01T14:52:42.625Z
Learning: dolong2 prefers to implement validation logic (such as path validation for Volume.physicalPath) in the business logic layer rather than in domain model init blocks, keeping domain models as simple data containers.

Learnt from: dolong2
Repo: dolong2/dcd PR: 703
File: src/test/resources/data.sql:28-30
Timestamp: 2025-09-01T14:57:07.656Z
Learning: dolong2 prefers to implement deletion validation in business logic rather than using database CASCADE constraints. For volume deletion, they want to validate if any mounts exist before allowing deletion, preventing accidental data loss while maintaining explicit control over the deletion process.

Learnt from: dolong2
Repo: dolong2/dcd PR: 655
File: src/main/kotlin/com/dcd/server/persistence/workspace/entity/WorkspaceJpaEntity.kt:12-14
Timestamp: 2025-07-12T14:05:23.333Z
Learning: dolong2는 JPA의 GeneratedValue(strategy = GenerationType.UUID) 대신 생성자에서 UUID.randomUUID()로 직접 초기화하는 방식을 선호한다. 더 간단하고 예측 가능한 동작을 위해서다.

Learnt from: dolong2
Repo: dolong2/dcd PR: 659
File: src/main/kotlin/com/dcd/server/core/domain/domain/usecase/ConnectDomainUseCase.kt:40-41
Timestamp: 2025-07-14T15:01:47.531Z
Learning: dolong2는 기능적으로 문제가 없는 코드 개선사항의 경우, 실제로 필요할 때까지 수정을 미루는 실용적인 접근법을 선호한다. 예를 들어 의미론적 정확성을 위한 변경은 해당 기능이 실제로 필요할 때 수정하는 방식을 택한다.

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build and Test

🔇 Additional comments (1)

src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/CreateApplicationRequest.kt (1)

16-16: 이전 리뷰 피드백이 반영되었습니다.

initialScripts 필드에 기본값 = listOf()가 추가되어 이전 리뷰에서 지적된 API 하위 호환성 문제가 해결되었습니다. 이제 기존 클라이언트가 이 필드를 생략해도 요청이 정상적으로 처리됩니다.

---

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: 2

🧹 Nitpick comments (2)

src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/UpdateApplicationRequest.kt (1)

14-14: port 필드 검증 확인이 필요합니다.

port 필드에 유효 범위 검증(1-65535)이 있는지 비즈니스 로직에서 확인하세요.

src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt (1)

48-48: 초기화 스크립트 저장 위치를 확인하세요.

initialScriptService.write가 코루틴 외부에서 동기적으로 호출되어 요청 스레드를 블로킹할 수 있습니다. 이 작업이 I/O 작업(데이터베이스 저장 등)을 포함하는 경우, 성능에 영향을 줄 수 있습니다.

작업이 빠르게 완료되거나 트랜잭션 컨텍스트 내에서 실행되어야 하는 이유가 있는지 확인하세요. 그렇지 않다면, 코루틴 내부로 이동하는 것을 고려하세요.

Based on learnings

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1642ed6 and 922e42d.

📒 Files selected for processing (13)

• src/main/kotlin/com/dcd/server/core/domain/application/dto/request/CreateApplicationReqDto.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/dto/request/UpdateApplicationReqDto.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/service/InitialScriptService.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/service/impl/InitialScriptServiceImpl.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt (2 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCase.kt (3 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/exetension/ApplicationRequestDataExtension.kt (2 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/CreateApplicationRequest.kt (1 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/request/UpdateApplicationRequest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt (8 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCaseTest.kt (3 hunks)
• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt (2 hunks)
• src/test/resources/data.sql (1 hunks)

🧰 Additional context used

🧠 Learnings (10)

📓 Common learnings

Learnt from: dolong2
Repo: dolong2/dcd PR: 703
File: src/main/kotlin/com/dcd/server/core/domain/volume/model/Volume.kt:1-12
Timestamp: 2025-09-01T14:52:42.625Z
Learning: dolong2 prefers to implement validation logic (such as path validation for Volume.physicalPath) in the business logic layer rather than in domain model init blocks, keeping domain models as simple data containers.

Learnt from: dolong2
Repo: dolong2/dcd PR: 703
File: src/test/resources/data.sql:28-30
Timestamp: 2025-09-01T14:57:07.656Z
Learning: dolong2 prefers to implement deletion validation in business logic rather than using database CASCADE constraints. For volume deletion, they want to validate if any mounts exist before allowing deletion, preventing accidental data loss while maintaining explicit control over the deletion process.

Learnt from: dolong2
Repo: dolong2/dcd PR: 655
File: src/main/kotlin/com/dcd/server/persistence/workspace/entity/WorkspaceJpaEntity.kt:12-14
Timestamp: 2025-07-12T14:05:23.333Z
Learning: dolong2는 JPA의 GeneratedValue(strategy = GenerationType.UUID) 대신 생성자에서 UUID.randomUUID()로 직접 초기화하는 방식을 선호한다. 더 간단하고 예측 가능한 동작을 위해서다.

Learnt from: dolong2
Repo: dolong2/dcd PR: 659
File: src/main/kotlin/com/dcd/server/core/domain/domain/usecase/ConnectDomainUseCase.kt:40-41
Timestamp: 2025-07-14T15:01:47.531Z
Learning: dolong2는 기능적으로 문제가 없는 코드 개선사항의 경우, 실제로 필요할 때까지 수정을 미루는 실용적인 접근법을 선호한다. 예를 들어 의미론적 정확성을 위한 변경은 해당 기능이 실제로 필요할 때 수정하는 방식을 택한다.

📚 Learning: 2024-11-01T15:14:45.123Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 363
File: src/main/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationEnvUseCase.kt:35-52
Timestamp: 2024-11-01T15:14:45.123Z
Learning: 개발자는 `UpdateApplicationEnvUseCase.kt` 파일의 `execute` 함수에서 `updatedApplicationList`가 비어 있는지 확인하거나, `envKey`가 없는 애플리케이션에 대한 로그를 추가하는 것을 원치 않습니다.

Applied to files:

• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCaseTest.kt

📚 Learning: 2025-08-10T14:46:38.244Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 688
File: src/main/kotlin/com/dcd/server/core/domain/env/usecase/PutApplicationEnvUseCase.kt:60-71
Timestamp: 2025-08-10T14:46:38.244Z
Learning: DCD 프로젝트의 PutApplicationEnvUseCase.execute(id, putApplicationEnvReqDto) 메서드에서 환경변수 업데이트 시 details는 기존 값과 병합하지 않고 요청 데이터로 완전히 덮어씌우는 방식으로 동작해야 한다.

Applied to files:

• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt

📚 Learning: 2025-08-08T15:07:46.340Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 683
File: src/main/kotlin/com/dcd/server/presentation/domain/env/ApplicationEnvWebAdapter.kt:30-0
Timestamp: 2025-08-08T15:07:46.340Z
Learning: DCD 프로젝트: ApplicationEnvWebAdapter(파일: src/main/kotlin/com/dcd/server/presentation/domain/env/ApplicationEnvWebAdapter.kt)의 DELETE /{workspaceId}/env/{envId} 성공 응답은 팀 컨벤션상 204가 아닌 200 OK를 유지한다.

Applied to files:

• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt

📚 Learning: 2025-08-08T15:09:11.874Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 683
File: src/main/kotlin/com/dcd/server/core/domain/env/usecase/DeleteApplicationEnvUseCase.kt:14-18
Timestamp: 2025-08-08T15:09:11.874Z
Learning: DCD 프로젝트: DeleteApplicationEnvUseCase는 workspaceId를 받지 않으며, 워크스페이스 경계 검증은 ApplicationEnvWebAdapter의 WorkspaceOwnerVerification(AOP)에서 처리한다. 불일치 시 NotFound로 응답하는 것은 프레젠테이션 레이어에서 보장하는 것이 선호된다.

Applied to files:

• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt

📚 Learning: 2024-11-09T09:51:02.007Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 373
File: src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt:55-56
Timestamp: 2024-11-09T09:51:02.007Z
Learning: `CreateApplicationUseCase`의 코루틴 내에서 호출되는 서비스들은 실패 시 애플리케이션의 상태를 자동으로 FAILURE로 변경합니다.

Applied to files:

• src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt

📚 Learning: 2025-01-30T10:42:29.969Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 483
File: src/main/kotlin/com/dcd/server/core/domain/workspace/usecase/DeleteWorkspaceUseCase.kt:23-24
Timestamp: 2025-01-30T10:42:29.969Z
Learning: In this Spring application, service objects are called within transaction context, so adding explicit transaction handling in UseCase classes is not necessary as it's already managed at a higher level.

Applied to files:

• src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt

📚 Learning: 2024-10-26T02:32:19.442Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 359
File: src/main/kotlin/com/dcd/server/core/domain/application/usecase/StopApplicationUseCase.kt:44-45
Timestamp: 2024-10-26T02:32:19.442Z
Learning: `StopApplicationUseCase.kt`에서 `this`가 이미 `CoroutineScope(Dispatchers.IO)`를 상속하고 있는 경우, 새로운 `CoroutineScope`를 생성할 때 `this + job`을 사용하는 것이 코드 일관성을 유지하는 데 도움이 됩니다.

Applied to files:

• src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt

📚 Learning: 2025-08-12T14:59:22.963Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 692
File: src/main/kotlin/com/dcd/server/core/domain/env/spi/QueryApplicationEnvPort.kt:0-0
Timestamp: 2025-08-12T14:59:22.963Z
Learning: In the dcd codebase, when updating port interface method signatures to include workspace parameters for tenant isolation, all layers must be updated together: the port interface, service interface, service implementation, persistence adapter, repository layer, and use case callers. The user dolong2 demonstrated proper multi-layer refactoring by updating QueryApplicationEnvPort.findAllByLabelsIn to include workspace parameter and systematically updating all dependent code in a single commit, including updating repository method name to findAllByWorkspaceAndLabelsIn.

Applied to files:

• src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCaseTest.kt

📚 Learning: 2024-10-23T14:48:00.144Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 357
File: src/main/kotlin/com/dcd/server/core/domain/application/usecase/RunApplicationUseCase.kt:37-68
Timestamp: 2024-10-23T14:48:00.144Z
Learning: RunApplicationUseCase.kt의 `execute` 함수에서는 애플리케이션을 비동기로 실행하고 상태를 `PENDING`으로 변경한 후 즉시 반환하는 것이 목적이다.

Applied to files:

• src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCaseTest.kt

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build and Test

🔇 Additional comments (19)

src/test/resources/data.sql (1)

39-44: 외래 키 제약조건 추가가 적절합니다.

애플리케이션 관련 엔티티들에 대한 참조 무결성 제약조건이 올바르게 추가되었습니다. 새로 추가된 application_initial_script_entity에 대한 FK 제약조건도 포함되어 있습니다.

src/main/kotlin/com/dcd/server/core/domain/application/dto/request/UpdateApplicationReqDto.kt (1)

9-12: DTO 필드 추가가 적절합니다.

port와 initialScripts 필드가 올바르게 추가되었습니다. 비즈니스 로직 레이어에서 port 범위 검증(1-65535) 및 initialScripts 내용 검증(빈 문자열, 길이 제한 등)이 수행되고 있는지 확인하세요.

src/main/kotlin/com/dcd/server/core/domain/application/service/InitialScriptService.kt (1)

5-7: 인터페이스 설계가 명확하고 간결합니다.

InitialScriptService 인터페이스가 단일 책임 원칙을 잘 따르고 있으며, 시그니처가 명확합니다.

src/main/kotlin/com/dcd/server/core/domain/application/dto/request/CreateApplicationReqDto.kt (1)

12-12: DTO 필드 추가가 적절합니다.

initialScripts 필드가 올바르게 추가되었습니다. 프레젠테이션 레이어에서 이 필드로의 매핑이 적절히 처리되고 있습니다.

src/main/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCase.kt (1)

31-31: 의존성 추가가 적절합니다.

InitialScriptService 의존성이 올바르게 추가되었습니다.

src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt (2)

42-42: 테스트가 올바르게 업데이트되었습니다.

새로운 initialScripts 필드가 테스트 데이터에 적절히 포함되었습니다.

---

161-161: 테스트가 올바르게 업데이트되었습니다.

port와 initialScripts 필드가 업데이트 요청 테스트에 적절히 포함되었습니다.

src/main/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCase.kt (2)

12-12: 의존성 주입 방식이 적절합니다.

InitialScriptService를 생성자 주입 방식으로 추가한 것은 기존 코드베이스의 패턴과 일치합니다.

Also applies to: 27-27

---

56-56: 리뷰 코멘트는 부정확합니다.

@UseCase 어노테이션이 @Transactional을 메타-어노테이션으로 포함하고 있어서, UpdateApplicationUseCase.execute() 메서드는 이미 트랜잭션으로 감싸져 있습니다. commandApplicationPort.save()와 initialScriptService.write() 호출은 모두 동일한 트랜잭션 범위 내에서 실행되며, write() 메서드 내부의 deleteByApplication()과 saveAll()도 같은 트랜잭션에 참여합니다. 따라서 write() 호출이 실패하면 전체 트랜잭션이 롤백되어 데이터 불일치 상태가 발생하지 않습니다. 원본 코드는 올바른 트랜잭션 처리를 이미 갖추고 있습니다.

Likely an incorrect or invalid review comment.

src/main/kotlin/com/dcd/server/presentation/domain/application/data/exetension/ApplicationRequestDataExtension.kt (2)

22-22: LGTM!

CreateApplicationRequest에서 initialScripts 필드를 DTO로 정확하게 매핑하고 있습니다.

---

33-34: LGTM!

UpdateApplicationRequest에서 port와 initialScripts 필드를 DTO로 정확하게 매핑하고 있습니다.

src/test/kotlin/com/dcd/server/core/domain/application/usecase/UpdateApplicationUseCaseTest.kt (3)

10-10: LGTM!

QueryApplicationInitialScriptPort 의존성을 적절하게 추가하여 초기화 스크립트 검증을 지원합니다.

Also applies to: 34-34

---

41-41: LGTM!

테스트 데이터에 port와 initialScripts 필드를 추가하여 실제 사용 시나리오를 반영하고 있습니다.

---

66-71: LGTM!

초기화 스크립트의 개수와 내용을 모두 검증하는 명확한 테스트 케이스입니다. 검증 로직이 적절합니다.

src/main/kotlin/com/dcd/server/core/domain/application/service/impl/InitialScriptServiceImpl.kt (2)

22-22: LGTM!

UUID.randomUUID()를 사용한 ID 생성 방식은 프로젝트의 기존 패턴과 일치합니다. Based on learnings

---

14-28: 검증 완료: 의도된 동작입니다

initialScripts가 빈 리스트인 경우의 동작을 확인했습니다:

1. 구현 분석: ApplicationInitialScriptPersistenceAdapter.saveAll()은 Spring Data JPA에 위임하며, 빈 리스트는 저장 작업 없음 (no-op)

2. 실제 동작: deleteByApplication() 실행 → saveAll([]) 실행 → 모든 기존 스크립트 삭제, 새 스크립트 없음

3. 테스트 증거:

   • CreateApplicationUseCaseTest (lines 62, 89, 112, 151)
   • UpdateApplicationUseCaseTest (line 41+)
   • ApplicationWebAdapterTest (lines 42, 161)

   위 테스트들에서 emptyList()를 사용하여 정상 동작 검증됨

4. 설계 패턴: "삭제 후 저장" 패턴으로 기존 스크립트를 완전히 교체하는 의도된 설계입니다.

src/test/kotlin/com/dcd/server/core/domain/application/usecase/CreateApplicationUseCaseTest.kt (3)

8-8: LGTM!

QueryApplicationInitialScriptPort 의존성을 적절하게 추가하여 초기화 스크립트 검증을 지원합니다.

Also applies to: 38-38

---

62-62: LGTM!

기존 테스트 케이스들에 빈 initialScripts 리스트를 추가하여 하위 호환성을 유지하고 있습니다.

Also applies to: 89-89, 112-112, 151-151

---

175-197: LGTM!

초기화 스크립트가 있는 경우를 명시적으로 테스트하는 케이스가 잘 작성되었습니다. 스크립트의 개수와 내용을 모두 검증하고 있어 테스트 커버리지가 충분합니다.