🔀 :: [#741] - Dockerfile 생성 메서드에 WORKDIR 지시어 추가

[dolong2]

개요

• Dockerfile 생성 메서드에 WORKDIR 지시어를 추가합니다.

작업내용

• 도커파일 내용에 workdir 추가

체크리스트

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

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

Summary by CodeRabbit

• Chores
  • 여러 애플리케이션 타입의 Docker 이미지 생성 시 작업 디렉토리 설정을 추가했습니다. 이를 통해 컨테이너 내 파일 시스템 구성이 개선되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Dockerfile 템플릿에서 여러 애플리케이션 타입(SPRING_BOOT, NEST_JS, MYSQL, MARIA_DB, Redis, H2_DB)의 FROM 지시자 이후에 WORKDIR /app 라인을 추가합니다. 이는 컨테이너 파일 시스템 컨텍스트에만 영향을 미치며 제어 흐름을 변경하지 않습니다.

Changes

Cohort / File(s)	변경 요약
Dockerfile 템플릿 업데이트 src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt	SPRING_BOOT, NEST_JS, MYSQL, MARIA_DB, Redis, H2_DB 애플리케이션 타입의 Dockerfile 템플릿 각각에 WORKDIR /app 지시자 추가

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3분

• 단일 파일에 대한 반복적이고 일관된 변경
• 모든 템플릿에 동일한 라인 추가로 패턴이 명확함
• 로직 변경 없이 Dockerfile 구성만 수정

Possibly related PRs

• 🔀 :: [#739] - nest 애플리케이션 도커 파일 내용 수정 #740: 동일한 파일(src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt)의 Dockerfile 템플릿을 수정하며, 특히 NEST_JS Dockerfile 콘텐츠와 관련 빌드 흐름을 리팩토링합니다.

Poem

🐰 WORKDIR를 추가한 채 컨테이너를 놓으니,
파일들은 /app에서 집을 지으리라~
경로 헷갈림은 이제 안녕,
깔끔한 구조로 앱들이 춤을 추네! 🎉

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의 주요 변경사항(Dockerfile 생성에 WORKDIR 지시어 추가)과 완벽하게 관련되어 있으며, 명확하게 변경 내용을 설명합니다.

✨ 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 refac/docker-file-content

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b8c5f04 and 1bdfac1.

📒 Files selected for processing (1)

• src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt (6 hunks)

🧰 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 (3)

src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt (3)

24-35: LGTM! Spring Boot Dockerfile에 WORKDIR 추가가 적절합니다.

WORKDIR /app 추가 후 모든 COPY, RUN, CMD 명령어가 상대 경로로 올바르게 동작합니다. Spring Boot 애플리케이션 컨테이너화 시 작업 디렉토리를 명시하는 것은 모범 사례입니다.

---

37-48: LGTM! NestJS Dockerfile에 WORKDIR 추가가 적절합니다.

WORKDIR /app 설정 후 package.json과 dist 디렉토리가 올바른 위치에 복사되고, npm 및 node 명령어가 정상적으로 실행됩니다. NestJS 애플리케이션 컨테이너화 시 작업 디렉토리를 명시하는 것은 모범 사례입니다.

---

50-57: WORKDIR /app이 데이터베이스 Dockerfile에서 문제를 발생시키지 않습니다.

initialScripts는 상대 경로가 아닌 셸 명령어("echo test" 등)이며, 공식 MySQL, MariaDB, Redis 이미지는 WORKDIR을 설정하지 않습니다. WORKDIR /app을 추가해도 데이터베이스 초기화 로직(runtime docker-entrypoint.d)에 영향을 주지 않으며, RUN 명령어는 설정된 WORKDIR에서 정상적으로 실행됩니다.

다만 데이터베이스 Dockerfile은 COPY 명령어가 없으므로 WORKDIR /app이 기술적으로 필요하지 않습니다. NestJS에서는 COPY 명령어 이후 상대 경로 참조를 위해 필요하지만, 데이터베이스의 경우 불필요한 중복입니다.

Likely an incorrect or invalid review comment.

---

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