🔀 :: [#737] - 애플리케이션 정보 응답에 초기화 스크립트 관련 필드 추가

[dolong2]

개요

• 애플리케이션 정보 응답에 초기화 스크립트 관련 필드를 추가합니다.

작업내용

• 애플리케이션 응답 dto에 initialScripts 필드 추가
• 애플리케이션 응답 객체에 initialScripts 필드 추가

체크리스트

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

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

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 애플리케이션 조회 시 초기 스크립트를 함께 제공하는 기능이 추가되었습니다. 모든 애플리케이션 목록 조회 및 개별 애플리케이션 조회 시 관련 초기 스크립트 정보가 응답에 포함됩니다.

• 개선 사항

  • 애플리케이션 응답 데이터 구조가 확장되어 초기 스크립트 정보를 더욱 효율적으로 제공합니다.

[coderabbitai]

워크스루

애플리케이션 DTO와 유스케이스에 초기 스크립트 지원을 추가합니다. Application DTO 확장을 업데이트하여 초기 스크립트를 수락하고 매핑하며, 응답 DTO에 initialScripts 필드를 추가하고, QueryApplicationInitialScriptPort 의존성을 유스케이스에 통합하고, 웹 어댑터에서 사용하지 않는 환경 유스케이스 의존성을 제거합니다.

변경사항

응집 / 파일	변경 요약
초기 스크립트 지원 src/main/kotlin/com/dcd/server/core/domain/application/dto/extenstion/ApplicationDtoExtension.kt, src/main/kotlin/com/dcd/server/core/domain/application/dto/response/ApplicationResDto.kt, src/main/kotlin/com/dcd/server/presentation/domain/application/data/exetension/ApplicationResponseDataExtension.kt, src/main/kotlin/com/dcd/server/presentation/domain/application/data/response/ApplicationResponse.kt	Application.toDto() 메서드 시그니처 업데이트하여 initialScriptList 파라미터 추가; ApplicationResDto 및 ApplicationResponse 데이터 클래스에 initialScripts: List<String> 필드 추가; 매핑 로직 확장
유스케이스 업데이트 src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCase.kt, src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCase.kt	QueryApplicationInitialScriptPort 의존성 추가; 생성자 시그니처 업데이트; 초기 스크립트 조회 및 toDto() 호출에서 initialScripts 파라미터 전달
웹 어댑터 정리 src/main/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapter.kt	putApplicationEnvUseCase 및 deleteApplicationEnvUseCase 생성자 주입 의존성 제거
테스트 업데이트 src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt, src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCaseTest.kt, src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt	toDto() 확장 함수 호출 업데이트하여 추가 파라미터 포함; ApplicationResDto 생성자 호출에 initialScripts 필드 추가

예상 코드 리뷰 노력

🎯 3 (Moderate) | ⏱️ ~20 minutes

• 주의 영역:
  • QueryApplicationInitialScriptPort의 올바른 의존성 주입 및 사용 검증 필요 (GetAllApplicationUseCase, GetOneApplicationUseCase)
  • ApplicationWebAdapter에서 제거된 의존성이 다른 곳에서도 정리되어야 하는지 확인
  • 모든 테스트가 새로운 toDto() 시그니처와 일치하는지 검증
  • DTO 매핑 체인 전체(ApplicationResDto → ApplicationResponse)에서 initialScripts 필드가 일관되게 전파되는지 확인

관련 가능성 있는 PR

• 🔀 :: [#732] - 애플리케이션 생성및 수정시 초기화 스크립트 스펙 추가 #735: 동일한 PR로서 "initialScripts" 기능을 애플리케이션에 구현하고 QueryApplicationInitialScriptPort를 추가/사용하며 동일한 도메인 및 관련 포트/DTO에 영향을 미침
• 🔀 :: [#686] - ApplicationEnvNotFoundException 패키지 위치 개선 #694: 동일 파일(ApplicationWebAdapter)의 생성자에서 PutApplicationEnvUseCase/DeleteApplicationEnvUseCase 제거 관련 — 환경 유스케이스 클래스 제거와 직접 대응
• 🔀 :: [#734] - 애플리케ㅣ션 배포시 초기화 스크립트 적용 #736: "초기 스크립트" 지원을 구현하고 QueryApplicationInitialScriptPort를 추가/사용하는 관련 PR (Dockerfile 생성/서비스에서 사용)

시

🐰 스크립트가 배치에 들어갔네, 우리의 앱도 한 단계 나아갔네
리스트와 리스트, 초기값이 흐르고 흘러
환경 의존성은 정리하고, 새로운 포트를 통해
DTO는 더 풍성해지고, 데이터는 더 완벽하게
초기 스크립트의 여정, 이제 시작되네 ✨

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-response-field

---

📜 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 edbfc7f and 9505b48.

📒 Files selected for processing (10)

• src/main/kotlin/com/dcd/server/core/domain/application/dto/extenstion/ApplicationDtoExtension.kt (3 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/dto/response/ApplicationResDto.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCase.kt (3 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCase.kt (1 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapter.kt (0 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/exetension/ApplicationResponseDataExtension.kt (1 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/application/data/response/ApplicationResponse.kt (1 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCaseTest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt (2 hunks)

💤 Files with no reviewable changes (1)

• src/main/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapter.kt

🧰 Additional context used

🧠 Learnings (8)

📓 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-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/GetOneApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt

📚 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/core/domain/application/usecase/GetOneApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.kt

📚 Learning: 2024-12-17T14:26:50.307Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 411
File: src/test/kotlin/com/dcd/server/core/domain/auth/usecase/SignOutUseCaseTest.kt:32-40
Timestamp: 2024-12-17T14:26:50.307Z
Learning: `SignOutUseCaseTest.kt`의 `afterContainer`에서 테스트 데이터 정리를 할 때 예외 처리를 하지 않아도 괜찮다.

Applied to files:

• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCaseTest.kt
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt

📚 Learning: 2025-07-14T15:00:05.921Z

Learnt from: dolong2
Repo: dolong2/dcd PR: 659
File: src/main/kotlin/com/dcd/server/core/domain/domain/usecase/ConnectDomainUseCase.kt:34-35
Timestamp: 2025-07-14T15:00:05.921Z
Learning: ConnectDomainUseCase에서 워크스페이스 불일치 시에도 ApplicationNotFoundException을 사용한다. 워크스페이스 컨텍스트에서 애플리케이션을 조회하는 관점에서 워크스페이스가 다르면 결과적으로 "존재하지 않는" 애플리케이션으로 처리하는 것이 dolong2의 설계 철학이다.

Applied to files:

• src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCaseTest.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/main/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCase.kt
• src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCase.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, all layers must be updated together: the port interface, persistence adapter implementation, repository layer, service callers, and tests. The user dolong2 demonstrated proper multi-layer refactoring by updating QueryApplicationEnvPort.findAllByLabelsIn to include workspace parameter and updating all dependent code in a single commit.

Applied to files:

• src/main/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCase.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/test/kotlin/com/dcd/server/presentation/domain/application/ApplicationWebAdapterTest.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 (13)

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

18-18: 초기화 스크립트 필드 추가 확인

initialScripts 필드가 올바르게 추가되었습니다. failureReason과 labels 사이에 배치되어 구조가 일관성 있게 유지됩니다.

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

21-21: DTO에서 Response로 필드 매핑 확인

initialScripts 필드가 ApplicationResDto에서 ApplicationResponse로 올바르게 매핑되고 있습니다.

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

18-18: Response 객체에 필드 추가 확인

ApplicationResponse에 initialScripts 필드가 올바르게 추가되어 DTO와 일관성을 유지합니다.

src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetOneApplicationUseCaseTest.kt (1)

39-39: 테스트 업데이트 확인

새로운 toDto 시그니처에 맞게 테스트가 올바르게 업데이트되었습니다. 빈 리스트를 사용하는 것이 테스트 컨텍스트에 적합합니다.

src/test/kotlin/com/dcd/server/core/domain/application/usecase/GetAllApplicationUseCaseTest.kt (1)

44-44: 테스트 시그니처 업데이트 확인

toDto 호출이 새로운 두 개의 파라미터 시그니처에 맞게 올바르게 업데이트되었습니다.

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

72-73: 웹 어댑터 테스트 데이터 업데이트 확인

테스트 데이터가 새로운 ApplicationResDto 구조에 맞게 올바르게 업데이트되었습니다. 두 테스트 케이스 모두 initialScripts = emptyList()를 일관성 있게 사용하고 있습니다.

Also applies to: 102-103

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

21-22: 초기화 스크립트 조회 및 전달 로직 확인

초기화 스크립트를 조회하고 DTO 매핑에 전달하는 로직이 올바르게 구현되었습니다. applicationEnvList와 동일한 패턴을 따르고 있어 일관성이 유지됩니다.

---

7-7: 포트 인터페이스의 테넌트 격리 패턴 불일치 발견

QueryApplicationInitialScriptPort가 QueryApplicationEnvPort와 달리 workspace 매개변수가 없어 테넌트 격리 패턴이 일관성 있게 구현되지 않았습니다.

• QueryApplicationEnvPort: workspace 파라미터 포함 (findAllByWorkspace, findAllByLabelsIn)
• QueryApplicationInitialScriptPort: workspace 파라미터 없음, Application 참조만 사용
• GetAllApplicationUseCase: WorkspaceInfo 주입
• GetOneApplicationUseCase: WorkspaceInfo 주입 없음

저장소 계층(ApplicationInitialScriptRepository)도 workspace 격리된 쿼리 메서드가 없습니다. 도메인 모델(ApplicationInitialScriptJpaEntity)은 Application과의 관계를 통해서만 workspace에 접근합니다.

다음을 확인해주세요:

1. 포트 메서드에 workspace 파라미터 추가 필요 여부
2. 레포지토리에 workspace 격리 쿼리 메서드 필요 여부
3. 모든 호출 레이어에서 workspace 컨텍스트 전달 필요 여부

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

8-8: 포트 의존성 추가 확인

GetAllApplicationUseCase에 QueryApplicationInitialScriptPort 의존성이 올바르게 추가되었습니다. GetOneApplicationUseCase와 일관된 패턴을 따르고 있습니다.

Also applies to: 17-18

---

29-30: 애플리케이션별 초기화 스크립트 조회 로직 확인

각 애플리케이션에 대해 초기화 스크립트를 조회하고 DTO 매핑에 전달하는 로직이 올바르게 구현되었습니다. map 블록 내에서 환경 변수와 초기화 스크립트를 순차적으로 조회하는 구조가 명확합니다.

src/main/kotlin/com/dcd/server/core/domain/application/dto/extenstion/ApplicationDtoExtension.kt (3)

7-7: LGTM!

필요한 임포트가 정확하게 추가되었습니다.

---

45-45: initialScripts 필드 정의 확인됨 - 구현이 정확합니다

ApplicationResDto에 initialScripts: List<String> 필드가 정의되어 있으며, 매핑 연산 initialScriptList.map { it.script }의 출력 타입과 정확히 일치합니다. 구현에는 문제가 없습니다.

---

27-27: API 호환성 확인 완료 - 변경사항 모두 반영됨

Application.toDto() 함수의 시그니처 변경(매개변수 1개 → 2개)이 코드베이스 전체에서 일관되게 적용되었습니다. 모든 호출 지점이 initialScriptList 매개변수를 포함하도록 업데이트되었습니다:

• GetOneApplicationUseCase.kt:22 - toDto(applicationEnvList, initialScripts) ✓
• GetAllApplicationUseCase.kt:30 - toDto(applicationEnvList, initialScripts) ✓
• GetOneApplicationUseCaseTest.kt:39 - toDto(listOf(), listOf()) ✓
• GetAllApplicationUseCaseTest.kt:44 - toDto(listOf(), listOf()) ✓

---

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