🔀 :: [#717] - 볼륨 마운트 API 추가

[dolong2]

개요

• 볼륨을 마운트하는 API를 추가합니다.

작업내용

• 애플리케이션 컨테이너 생성시 마운트된 볼륨을 반영해서 생성하는 로직 추가
• 볼륨 마운트 정보 세이브 메서드 추가
• 볼륨 마운트 요청 dto 추가
• 볼륨 마운트 유스케이스 추가
• 볼륨 마운트 요청 객체 추가
• 볼륨 마운트 엔드포인트 추가
• 볼륨 마운트관련 테스트 추가

체크리스트

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

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

Summary by CodeRabbit

• 신기능

  • 애플리케이션에 볼륨을 마운트하는 신규 엔드포인트 추가(POST /{workspaceId}/volume/{volumeId}/mount). mountPath 및 readOnly 파라미터 지원.
  • 볼륨 마운트 적용 시 애플리케이션이 자동 재배포되어 변경사항이 즉시 반영됩니다.
  • 컨테이너 생성 시 마운트 설정이 자동으로 포함됩니다.

• 보안

  • 볼륨 마운트 엔드포인트에 인증 요구 추가.

• 문서

  • 새 엔드포인트 및 요청 파라미터 사용 안내 반영.

• 테스트

  • 마운트 동작과 오류 경로를 검증하는 테스트 추가.

[coderabbitai]

Walkthrough

컨테이너 생성 명령에 애플리케이션의 볼륨 마운트 플래그(-v ...)를 조립해 추가하고, 볼륨을 애플리케이션에 마운트하는 새로운 유스케이스·엔드포인트(POST /{workspaceId}/volume/{volumeId}/mount)와 saveMount 영속화 경로를 도입했다.

Changes

Cohort / File(s)	Change Summary
컨테이너 생성: 볼륨 마운트 통합 src/main/kotlin/com/dcd/server/core/domain/application/service/impl/CreateContainerServiceImpl.kt, src/test/kotlin/com/dcd/server/core/domain/application/service/CreateContainerServiceImplTest.kt	CreateContainerServiceImpl에 QueryVolumePort 주입 추가. 애플리케이션의 마운트 목록을 조회해 docker -v 플래그를 생성·삽입. 테스트는 QueryVolumePort 목킹으로 빈 목록 반환을 검증.
볼륨 마운트 유스케이스 및 테스트 src/main/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCase.kt, src/test/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCaseTest.kt	MountVolumeUseCase 추가: 워크스페이스/소유 검증 후 VolumeMount 생성·저장 및 DeployApplicationEvent 발행. 성공 및 오류 경로에 대한 통합/단위 테스트 추가.
프레젠테이션 DTO 및 매핑 src/main/kotlin/com/dcd/server/core/domain/volume/dto/request/MountVolumeReqDto.kt, src/main/kotlin/com/dcd/server/presentation/domain/volume/data/request/MountVolumeRequest.kt, src/main/kotlin/com/dcd/server/presentation/domain/volume/data/extension/VolumeRequestExtension.kt	MountVolumeReqDto와 MountVolumeRequest 추가 및 MountVolumeRequest.toDto() 확장 함수 도입(검증 어노테이션 포함).
웹 어댑터·보안·테스트 보강 src/main/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapter.kt, src/main/kotlin/com/dcd/server/infrastructure/global/config/SecurityConfig.kt, src/test/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapterTest.kt	POST /{workspaceId}/volume/{volumeId}/mount 엔드포인트 추가, SecurityConfig에 해당 매처 등록, VolumeWebAdapter에 MountVolumeUseCase 주입 및 테스트 추가.
영속성 포트·어댑터 확장 src/main/kotlin/com/dcd/server/core/domain/volume/spi/CommandVolumePort.kt, src/main/kotlin/com/dcd/server/persistence/volume/VolumePersistenceAdapter.kt	CommandVolumePort에 saveMount(VolumeMount) 메서드 추가 및 VolumePersistenceAdapter에서 해당 메서드 구현하여 VolumeMount 영속화 지원.
배포 흐름 테스트 조정 src/test/kotlin/com/dcd/server/core/domain/application/usecase/DeployApplicationUseCaseTest.kt	CreateContainerService 인터페이스 추가 및 테스트에서 CreateContainerService 호출 검증으로 기존 docker-create 문자열 검사 대체.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User as 클라이언트
  participant Web as VolumeWebAdapter
  participant UC as MountVolumeUseCase
  participant QV as QueryVolumePort
  participant QA as QueryApplicationPort
  participant CV as CommandVolumePort
  participant EP as ApplicationEventPublisher

  User->>Web: POST /{wsId}/volume/{volumeId}/mount\n?applicationId=...\nbody: {mountPath, readOnly}
  Web->>UC: execute(volumeId, applicationId, reqDto)
  UC->>UC: workspace 확인
  UC->>QV: findById(volumeId)
  UC->>QA: findById(applicationId)
  UC->>CV: saveMount(VolumeMount)
  UC-->>EP: publish(DeployApplicationEvent([applicationId]))
  UC-->>Web: 완료
  Web-->>User: 200 OK

Loading

sequenceDiagram
  autonumber
  participant Deploy as DeployApplicationUseCase
  participant CCS as CreateContainerService
  participant QV as QueryVolumePort
  participant Docker as Docker CLI

  Deploy->>CCS: createContainer(application, externalPort)
  activate CCS
  CCS->>QV: findAllMountByApplication(application)
  QV-->>CCS: [VolumeMount...]
  CCS->>Docker: docker create ... -v volumeName:mountPath[:ro] ... -p ...
  CCS->>Docker: docker network connect ...
  deactivate CCS

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• 🔀 :: [#702] - 볼륨 엔티티 추가 #703: 볼륨·볼륨마운트 모델 및 포트 도입과 관련되어 Query/Command 포트 및 영속화 연계 관점에서 직접 관련.
• 🔀 :: [#486] - 리소스 이름을 반환하는 필드 추가 #487: CreateContainerServiceImpl 관련 변경 이력과 겹치며 컨테이너 생성 명령 구성(네트워크/이름 등)과 연계되어 있음.
• 🔀 :: [#705] - 볼륨 테이블 not null 제약 조건 추가 #706: 볼륨/volume_mount DB 스키마·제약 변경과 연계되어, 새 영속화(saveMount) 및 마운트 처리와 상호 영향 가능.

Poem

깡총, 깡총—난 토끼 개발자! 🐰
볼륨을 붙여 컨테이너에 찰싹,
-v 깃발 펄럭, 읽기전용도 척척.
저장하고 퐁—재배포 콜 한 번,
초원에선 배포가 또 춤춘다. 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	제목 "🔀 :: [#717] - 볼륨 마운트 API 추가"는 변경셋의 핵심인 '볼륨 마운트 API 추가'를 명확하게 요약하고 있어 PR 목적과 일치합니다; 다만 선행의 이모지와 PR 번호 표기는 불필요한 노이즈로 보일 수 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/volume-mount-api

---

📜 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 9663644 and 3b12c5d.

📒 Files selected for processing (1)

• src/main/kotlin/com/dcd/server/presentation/domain/volume/data/request/MountVolumeRequest.kt (1 hunks)

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

• src/main/kotlin/com/dcd/server/presentation/domain/volume/data/request/MountVolumeRequest.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

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/test/kotlin/com/dcd/server/core/domain/application/service/CreateContainerServiceImplTest.kt (1)

24-37: 목 세팅 순서 오류 및 네트워크 문자열 불일치로 인한 취약/플레이키 테스트

• Stubbing이 호출 이후에 이루어집니다(Line 26). 호출 전에 세팅해야 합니다.
• 본 서비스는 workspace.networkName을 사용하지만 기대 문자열은 title.replace(' ', '_')로 검증합니다(Line 31). 실제 구현과 불일치합니다.

아래와 같이 수정해 주세요.

-        `when`("service를 실행할때") {
-            createContainerService.createContainer(application, application.externalPort)
-            every { queryVolumePort.findAllMountByApplication(application) } returns listOf()
+        `when`("service를 실행할때") {
+            every { queryVolumePort.findAllMountByApplication(application) } returns listOf()
+            createContainerService.createContainer(application, application.externalPort)
             then("컨테이너를 실행하는 명령을 실행해야함") {
                 verify {
                     commandPort.executeShellCommand(
-                        "docker create --network ${application.workspace.title.replace(' ', '_')} " +
+                        "docker create --network ${application.workspace.networkName} " +
                         "--name ${application.containerName} " +
                         "-p ${application.externalPort}:${application.port} ${application.containerName}:latest"
                     )
                 }

🧹 Nitpick comments (14)

src/test/kotlin/com/dcd/server/core/domain/application/usecase/DeployApplicationUseCaseTest.kt (2)

37-39: MockkBean 주입은 적절. 다만 단계 순서와 실패 경로도 함께 보장하세요

• 제안: 배포 단계의 호출 순서를 명시적으로 검증해 회귀를 방지하세요.
• 제안: 빌드 실패 시 createContainerService가 호출되지 않음을 검증하는 네거티브 케이스를 추가하세요.

예시:

coVerifyOrder {
    commandPort.executeShellCommand("docker rm ${result.containerName}")
    commandPort.executeShellCommand("docker rmi ${result.containerName}")
    commandPort.executeShellCommand("git clone ${result.githubUrl} '${result.name}'")
    commandPort.executeShellCommand("cd ./'${result.name}' && docker build -t ${result.containerName}:latest .")
    createContainerService.createContainer(result, result.externalPort)
    commandPort.executeShellCommand("rm -rf '${result.name}'")
}

// 빌드 실패 네거티브 케이스(개략)
coEvery {
    commandPort.executeShellCommand("cd ./'${result.name}' && docker build -t ${result.containerName}:latest .")
} throws RuntimeException("build failed")

shouldThrow<Exception> {
    deployApplicationUseCase.execute(targetApplicationId)
}
coVerify(exactly = 0) { createContainerService.createContainer(any(), any()) }

원하시면 위 시나리오를 포함한 테스트 추가 패치를 드리겠습니다.

---

71-71: 위임 호출을 단 1회로 제한해 중복 생성 회귀 방지

정확도 향상을 위해 호출 횟수를 명시하세요.

다음과 같이 수정 권장:

-                coVerify { createContainerService.createContainer(result, result.externalPort) }
+                coVerify(exactly = 1) { createContainerService.createContainer(result, result.externalPort) }

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

5-9: 입력단 최소 패턴 검증 추가 제안

비즈니스 레이어에서 최종 검증하더라도, 프리밸리데이션으로 경로 패턴을 제한하면 UX와 보안을 동시에 개선할 수 있습니다.

적용 예시:

 import jakarta.validation.constraints.NotBlank
+import jakarta.validation.constraints.Pattern

 data class MountVolumeRequest(
     @field:NotBlank
-    val mountPath: String,
+    @field:Pattern(regexp = "^(?:/[^\\s:]+)+$", message = "Invalid mount path")
+    val mountPath: String,
     val readOnly: Boolean,
 )

src/main/kotlin/com/dcd/server/persistence/volume/VolumePersistenceAdapter.kt (1)

31-33: 중복 마운트 및 무결성 보장 방안 필요

단순 save는 (vol, app, mountPath) 중복 삽입을 허용할 수 있습니다. DB 레벨 Unique 제약 및/또는 upsert(idempotent) 처리를 권장합니다. 트랜잭션 경계에서 처리되는지도 확인해 주세요.

예:

• UNIQUE(volume_id, application_id, mount_path)
• 존재 시 무시/갱신 정책 정의

src/test/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCaseTest.kt (4)

31-33: 불필요한 MockkBean 제거 제안

@MockkBean CommandPort는 본 테스트에서 사용되지 않습니다. 불필요한 빈 주입은 컨텍스트 로딩 시간을 늘립니다. 제거를 권장합니다.

-    @MockkBean(relaxed = true)
-    private val commandPort: CommandPort,

---

53-57: 테스트 간 데이터 오염 방지

findAll().size shouldBe 1 단언은 이전 테스트 잔여 데이터에 민감합니다. 각 컨테이너/테스트 전에 volumeMountRepository.deleteAll()로 정리하세요.

         beforeContainer {
+            volumeMountRepository.deleteAll()
             val targetWorkspace = queryWorkspacePort.findById("d57b42f5-5cc4-440b-8dce-b4fc2e372eff")!!
             workspaceInfo.workspace = targetWorkspace
         }

Also applies to: 64-67

---

71-72: ID 타입 일관성 검토

first.application.id.toString() 비교는 타입 불일치의 신호입니다. 애플리케이션 ID를 전 구간(UUID ↔︎ String)에서 일관되게 다루도록 정렬을 검토해주세요(예: 웹 레이어는 UUID 받고 유스케이스는 UUID 사용).

---

89-94: 오탈자 수정

"에러가 발생행해야함" → "에러가 발생해야함".

-            then("에러가 발생행해야함") {
+            then("에러가 발생해야함") {

src/test/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapterTest.kt (2)

56-56: MockK verifier에서 불필요한 캐스팅 제거

any() as Type 대신 제네릭 any<Type>() 사용이 간결하고 안전합니다.

-                verify { createVolumeUseCase.execute(any() as CreateVolumeReqDto) }
+                verify { createVolumeUseCase.execute(any<CreateVolumeReqDto>()) }
...
-                verify { updateVolumeUseCase.execute(testVolumeId, any() as UpdateVolumeReqDto) }
+                verify { updateVolumeUseCase.execute(testVolumeId, any<UpdateVolumeReqDto>()) }
...
-                verify { mountVolumeUseCase.execute(testVolumeId, testApplicationId, any() as MountVolumeReqDto) }
+                verify { mountVolumeUseCase.execute(testVolumeId, testApplicationId, any<MountVolumeReqDto>()) }

Also applies to: 91-91, 144-145

---

51-53: 테스트 설명 오탈자 다듬기

• "응답되어여함" → "응답되어야함"
• "레핑되서" → "래핑되어"

-            then("상태코드가 OK가 응답되어여함") {
+            then("상태코드가 OK가 응답되어야함") {
...
-            then("유스케이스의 응답이 레핑되서 반환되어야함") {
+            then("유스케이스의 응답이 래핑되어 반환되어야함") {
...
-            then("유스케이스의 응답이 레핑되서 반환되어야함") {
+            then("유스케이스의 응답이 래핑되어 반환되어야함") {

Also applies to: 106-108, 125-127

src/main/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapter.kt (2)

86-87: applicationId 유효성 강화

쿼리 파라미터에 대한 최소 유효성 보장을 추가하세요. (예: 공백 금지, UUID 형태 권장) 현재는 빈 문자열도 통과합니다.

+import jakarta.validation.constraints.NotBlank
...
-        @RequestParam applicationId: String,
+        @RequestParam @NotBlank applicationId: String,

추가로, 가능하다면 applicationId: UUID로 받고 내부에서 .toString() 변환하도록 일관성 확보를 제안합니다.

---

88-90: 생성성 작업의 HTTP 응답 코드 재검토

마운트 생성은 자원 생성 성격입니다. 200 OK 대신 204 No Content 또는 201 Created(+ Location 헤더)를 고려하세요. 기존 API 일관성 유지가 우선이라면 그대로 둬도 무방합니다.

-        mountVolumeUseCase.execute(volumeId, applicationId, mountVolumeRequest.toDto())
-            .run { ResponseEntity.ok().build() }
+        mountVolumeUseCase.execute(volumeId, applicationId, mountVolumeRequest.toDto())
+            .run { ResponseEntity.noContent().build() }

src/main/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCase.kt (2)

25-47: 마운트 경로 검증 및 중복 방지(멱등성) 추가 제안

• 입력 검증: 비어있지 않고, 절대 경로(/ 시작), 루트(/) 금지, .. 금지 등 최소 규칙을 유스케이스 레이어에서 강제하세요. (도메인 init 대신 비즈니스 레이어 선호 — 과거 러닝과 일치)
• 중복 방지: 동일 애플리케이션의 동일 mountPath 중복 생성 방지(선점 체크 + DB 유니크 키).

 class MountVolumeUseCase(
@@
 ) {
     fun execute(volumeId: UUID, applicationId: String, mountVolumeReqDto: MountVolumeReqDto) {
+        require(mountVolumeReqDto.mountPath.isNotBlank()) { "mountPath must not be blank" }
+        require(mountVolumeReqDto.mountPath.startsWith("/") && mountVolumeReqDto.mountPath.length > 1) {
+            "mountPath must be an absolute, non-root path"
+        }
+        require(!mountVolumeReqDto.mountPath.contains("..")) { "mountPath must not contain '..'" }
@@
-        val volumeMount = VolumeMount(
+        // TODO: 중복 존재 시 멱등 처리(무시 또는 AlreadyExists 예외)
+        val volumeMount = VolumeMount(
             id = UUID.randomUUID(),
             application = application,
             volume = volume,
             mountPath = mountVolumeReqDto.mountPath,
             readOnly = mountVolumeReqDto.readOnly
         )

DB 측면에서는 (application_id, mount_path) 유니크 인덱스를 권장합니다.

---

49-51: 이벤트 발행 타이밍(트랜잭션 커밋 후) 확인

현재 즉시 발행입니다. 리스너가 @TransactionalEventListener(phase = AFTER_COMMIT)가 아니라면, 커밋 실패 시 불일치가 생길 수 있습니다. 커밋 이후 발행을 보장하도록 리스너 설정을 확인하거나, after-commit 훅을 사용하세요.

예상 조치:

• 리스너에 @TransactionalEventListener(phase = TransactionPhase.AFTER_COMMIT) 적용 여부 확인
• 또는 TransactionSynchronizationManager.registerSynchronization(...afterCommit { publish }) 사용

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9e623ff and 9663644.

📒 Files selected for processing (13)

• src/main/kotlin/com/dcd/server/core/domain/application/service/impl/CreateContainerServiceImpl.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/volume/dto/request/MountVolumeReqDto.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/volume/spi/CommandVolumePort.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCase.kt (1 hunks)
• src/main/kotlin/com/dcd/server/infrastructure/global/config/SecurityConfig.kt (1 hunks)
• src/main/kotlin/com/dcd/server/persistence/volume/VolumePersistenceAdapter.kt (1 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapter.kt (4 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/volume/data/extension/VolumeRequestExtension.kt (2 hunks)
• src/main/kotlin/com/dcd/server/presentation/domain/volume/data/request/MountVolumeRequest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/service/CreateContainerServiceImplTest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/core/domain/application/usecase/DeployApplicationUseCaseTest.kt (4 hunks)
• src/test/kotlin/com/dcd/server/core/domain/volume/usecase/MountVolumeUseCaseTest.kt (1 hunks)
• src/test/kotlin/com/dcd/server/presentation/domain/volume/VolumeWebAdapterTest.kt (4 hunks)

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: dolong2
PR: dolong2/dcd#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.

📚 Learning: 2025-09-04T14:59:15.319Z

Learnt from: dolong2
PR: dolong2/dcd#710
File: src/main/kotlin/com/dcd/server/core/domain/volume/service/impl/DeleteDockerVolumeServiceImpl.kt:13-18
Timestamp: 2025-09-04T14:59:15.319Z
Learning: dolong2 addressed command injection vulnerability in volume deletion by adding validation in the volume creation service (CreateDockerVolumeServiceImpl) using NAME_SAFE_REGEX pattern, preferring to validate at the source rather than in the deletion service for better security and fail-fast behavior.

Applied to files:

• src/main/kotlin/com/dcd/server/core/domain/application/service/impl/CreateContainerServiceImpl.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 (6)

src/test/kotlin/com/dcd/server/core/domain/application/usecase/DeployApplicationUseCaseTest.kt (2)

7-7: CreateContainerService 의존성 임포트 적절

유스케이스 위임을 테스트로 보장하는 방향이 맞습니다.

---

22-22: 해당 변경에 대한 코멘트 생략

정적 분석에서 경고가 없다면 추가 조치 불필요합니다.

src/main/kotlin/com/dcd/server/infrastructure/global/config/SecurityConfig.kt (1)

107-107: 마운트 엔드포인트 인증 추가 적절 (기존 정책과 일관). 권한 모델은 추후 점검 권장

다른 볼륨 API와 동일하게 authenticated로 맞춘 점 OK. 워크스페이스 멤버십/ROLE 기반 세분 권한이 필요하다면 추후 인가 규칙 확장 고려해 주세요.

현재는 인증만 요구합니다. 워크스페이스 리소스 접근 권한 체크가 별도 필터/어드바이저로 이미 적용되는지 확인 부탁드립니다.

src/main/kotlin/com/dcd/server/core/domain/volume/spi/CommandVolumePort.kt (2)

4-4: 새 API 추가(LGTM) — 도메인 모델 의존성 적절

VolumeMount를 포트에 노출하는 결정은 합리적입니다. 도메인 중심 흐름과 일관됩니다.

---

11-11: 모든 구현체 동기화 확인 — saveMount 구현 반영 여부
rg 실행 결과 saveMount를 오버라이드한 구현체는 src/main/kotlin/com/dcd/server/persistence/volume/VolumePersistenceAdapter.kt:31 한 곳뿐입니다. 다른 어댑터(타 모듈 포함)가 있다면 동일 시그니처로 구현을 추가하세요.

src/main/kotlin/com/dcd/server/presentation/domain/volume/data/extension/VolumeRequestExtension.kt (1)

22-26: 매핑 함수 적절 (변환 누락 없음)

필드 1:1 매핑이 정확합니다.