🔀 :: [#739] - nest 애플리케이션 도커 파일 내용 수정

[dolong2]

개요

• 이미지 빌드 과정에서 npm install및 build를 진행하는 대신 빌드된 파일을 올릴 수 있도록 리펙토링합니다.

작업내용

• nest 애플리케이션 도커파일에서 npm install, npm run build를 수행하지 않도록 리펙토링
• nest 애플리케이션 빌드시 npm run build를 수행하도록 리펙토링

체크리스트

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

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

Summary by CodeRabbit

릴리스 노트

• Chores
  • NestJS 애플리케이션의 Docker 이미지 빌드 프로세스를 최적화했습니다.
  • NestJS 유형에 대해 이미지 생성 전 별도의 빌드 단계를 추가해 사전 빌드된 아티팩트를 패키징하도록 변경했습니다.
  • 컨테이너 시작 방식을 프로덕션에 맞게 간소화하고 배포 효율을 개선했습니다.

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

[coderabbitai]

Walkthrough

NestJS용 Dockerfile 템플릿을 개발 흐름에서 프로덕션 중심의 미리 빌드된 아티팩트 방식으로 변경하고, 이미지 빌드 서비스에 ApplicationType.NEST_JS 분기를 추가해 Docker 빌드 전에 npm run build를 실행하도록 제어 흐름을 수정함.

Changes

Cohort / File(s)	Change Summary
NestJS Dockerfile 템플릿 최적화 src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt	Dockerfile 템플릿을 개발용 전체 워크스페이스 복사/빌드에서, package*.json과 dist만 복사하고 npm ci --production=true로 설치한 뒤 빌드된 dist를 실행하도록(시계열 TZ 설정 포함) 변경.
NestJS 빌드 단계 추가 src/main/kotlin/com/dcd/server/core/domain/application/service/impl/BuildDockerImageServiceImpl.kt	ApplicationType.NEST_JS 분기 추가. Docker 이미지 빌드 전에 프로젝트에 대해 npm run build를 실행하도록 제어 흐름을 확장함. 기존 SPRING_BOOT 경로는 변경 없음.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• FileContent.kt: Dockerfile 문자열 변경(복사 대상, 설치 명령, CMD) 정확성 검증 필요(경로·파일명·포트 등).
• BuildDockerImageServiceImpl.kt: 빌드 명령 실행 시 작업 디렉터리, 환경(예: Node 버전, package script 존재) 및 실패 시 처리 흐름 확인 필요.

Possibly related PRs

• 🔀 :: [#672] - 애플리케이션 배포시 유효하지 않은 도커파일 생성 현상 수정 #673 — 같은 getNestJsDockerFileContent 변경을 다루며 Dockerfile 생성 로직과 직접 연관.
• 🔀 :: [#734] - 애플리케ㅣ션 배포시 초기화 스크립트 적용 #736 — FileContent.kt 내 NestJS Dockerfile 관련 리팩터 및 getApplicationDockerFileContent 도입과 겹침.

Suggested labels

:sparkles: Feature

Poem

🐰 먼저 땅을 파서 빌드했지,
가벼운 가방엔 오직 package와 dist만,
서울 시간에 맞춰 한걸음 뛴다,
이미지 작고 마음은 가볍게,
나는 당근 대신 배포 로그를 씹는다 🥕✨

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 제목이 주요 변경사항인 NestJS 애플리케이션의 Docker 파일 내용 수정을 정확하게 반영하고 있습니다.

✨ 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/nest-application-docker-file

---

📜 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 0a2bb8a and 50b3ed6.

📒 Files selected for processing (1)

• src/main/kotlin/com/dcd/server/core/domain/application/service/impl/BuildDockerImageServiceImpl.kt (1 hunks)

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

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

---

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

Caution

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

⚠️ Outside diff range comments (1)

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

46-66: buildImageByApplicationId 메서드에도 NEST_JS 분기 추가를 고려하세요.

buildImageByApplication 메서드에는 NEST_JS 분기가 추가되었지만, buildImageByApplicationId 메서드(lines 26-39)에는 NEST_JS 분기가 없습니다. 두 메서드 간 일관성을 위해 buildImageByApplicationId에도 동일한 로직을 추가하는 것이 좋습니다.

buildImageByApplicationId 메서드에도 다음과 같이 NEST_JS 분기를 추가하세요:

val exitValue = when (application.applicationType) {
    ApplicationType.SPRING_BOOT -> {
        commandPort.executeShellCommand("cd ./$directoryName && ./gradlew clean build")
            .run {
                if (this == 0)
                    commandPort.executeShellCommand("cd ./$directoryName && docker build -t ${application.containerName}:latest .")
                else this
            }
    }
    ApplicationType.NEST_JS -> {
        commandPort.executeShellCommand("cd ./$directoryName && npm run build")
            .run {
                if (this == 0)
                    commandPort.executeShellCommand("cd ./$directoryName && docker build -t ${application.containerName}:latest .")
                else this
            }
    }
    else -> {
        commandPort.executeShellCommand("cd ./$directoryName && docker build -t ${application.containerName}:latest .")
    }
}

🧹 Nitpick comments (1)

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

45-45: 타임존 하드코딩 확인 필요

CMD에서 타임존이 Asia/Seoul로 하드코딩되어 있습니다. 프로젝트 요구사항에 따라 의도된 것일 수 있으나, 환경 변수로 설정하는 것이 더 유연한 방식입니다.

환경 변수로 관리하려면 다음과 같이 수정할 수 있습니다:

-    CMD ["sh", "-c", "TZ=Asia/Seoul node dist/main.js"]
+    ENV TZ=Asia/Seoul
+    CMD ["node", "dist/main.js"]

또는 타임존을 env 파라미터로 전달받아 ${getEnvString(env)}에 포함시킬 수 있습니다.

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ad6d95f and 0a2bb8a.

📒 Files selected for processing (2)

• src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt (1 hunks)
• src/main/kotlin/com/dcd/server/core/domain/application/service/impl/BuildDockerImageServiceImpl.kt (1 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