Crone : ci cd pipline - #16

.gitattributes (4)

15-19: *중복된 .sh 규칙 제거 및 주석 정정 제안

.gitattributes로 실행 권한을 설정할 수 없습니다. 현재 주석은 오해의 소지가 있고, *.sh 규칙이 중복됩니다. 아래처럼 주석을 명확히 하고 중복/불필요 항목을 정리하는 편이 좋습니다.

-# Ensure that shell scripts are executable
-*.sh text eol=lf
-deploy.sh text eol=lf
-deploy-dev.sh text eol=lf
+# Shell scripts: LF
+# NOTE: 실행 권한은 .gitattributes로 설정되지 않습니다.
+# 필요한 경우 다음 명령으로 권한을 부여하고 커밋하세요:
+# git update-index --chmod=+x <file>

---

24-26: Dockerfile 변형명 매칭 보강

Dockerfile.dev, Dockerfile.prod 같은 관례적 네이밍을 추가로 커버하세요. 현재 *.dockerfile은 Dockerfile.dev를 매칭하지 않습니다.

 Dockerfile text eol=lf
+Dockerfile.* text eol=lf
 *.dockerfile text eol=lf
 docker-compose*.yml text eol=lf

---

21-21: PowerShell 파생 파일도 포함

PowerShell 모듈/데이터 파일(.psm1, .psd1)도 CRLF로 고정하는 것이 일관성에 도움 됩니다.

 *.ps1 text eol=crlf
+*.psm1 text eol=crlf
+*.psd1 text eol=crlf

---

32-32: 파일 말미 개행(Newline at EOF) 추가 권장

일부 도구가 최종 개행을 요구합니다. 마지막 줄에 개행을 추가해 주세요.

 *.json text eol=lf
+

.env.example (3)

6-8: DB 호스트 지침 추가 제안

개발/로컬은 host.docker.internal이 편리하지만, prod Compose(동일 호스트 내 네트워크)에서는 Server=projectvg-db 사용이 일반적입니다. 주석으로 환경별 권장 값을 안내해 혼선을 줄이세요.

---

7-7: dotenv-linter 경고(인용부호) 처리

ValueWithoutQuotes 경고 해소가 필요하다면 값을 따옴표로 감쌀 수 있습니다. Docker Compose는 .env의 따옴표를 제거해 주입하므로 안전합니다.

-DB_CONNECTION_STRING=Server=host.docker.internal,1433;Database=ProjectVG;User Id=sa;Password=YOUR_DB_PASSWORD;TrustServerCertificate=true;MultipleActiveResultSets=true
+DB_CONNECTION_STRING="Server=host.docker.internal,1433;Database=ProjectVG;User Id=sa;Password=YOUR_DB_PASSWORD;TrustServerCertificate=true;MultipleActiveResultSets=true"

---

39-39: 파일 끝 개행 추가

POSIX 스타일 및 린터 정합성을 위해 EOF 개행을 추가하세요.

.github/workflows/develop.yml (3)

20-21: steps 들여쓰기 및 트레일링 스페이스 정리

YAMLlint 경고가 다수 발생합니다. 기능에는 영향 없으나 린팅 실패를 막기 위해 트레일링 스페이스 제거, EOF 개행 추가, 들여쓰기(라인 20 경고)만 정리하세요.

---

54-69: TRX 업로드/보존 추가 제안

dorny/test-reporter 사용은 적절합니다. 필요 시 원본 TRX를 아티팩트로 업로드하면 추적성이 좋아집니다.

 - name: Publish Test Results
 uses: dorny/test-reporter@v1
 if: success() || failure()
 with:
 name: Test Results
 path: coverage/*.trx
 reporter: dotnet-trx
+
+ - name: Upload TRX Artifacts
+ if: always()
+ uses: actions/upload-artifact@v4
+ with:
+ name: trx-results
+ path: coverage/*.trx

---

14-17: 동시 실행 취소로 러너 낭비 방지

PR 업데이트가 잦다면 이전 실행 자동 취소를 권장합니다.

 jobs:
 build-and-test:
 name: Build & Test
 runs-on: ubuntu-latest
+ concurrency:
+ group: develop-${{ github.ref }}
+ cancel-in-progress: true

docker-compose.yml (3)

5-7: 미사용 외부 네트워크 정의 제거 또는 사용 여부 명시

external-db-network가 어떤 서비스에도 연결되지 않습니다. 제거하거나 서비스에 연결해 목적을 명확히 하세요.

- external-db-network:
- external: true
- name: projectvg-external-db

---

23-41: 환경변수 주입 방식 간소화 제안

변수가 많으므로 env_file를 병행하면 유지보수가 쉬워집니다. 현 방식도 동작하므로 선택사항입니다.

- environment:
- - ASPNETCORE_ENVIRONMENT=${ASPNETCORE_ENVIRONMENT:-Production}
- - LLM_BASE_URL=${LLM_BASE_URL}
- - MEMORY_BASE_URL=${MEMORY_BASE_URL}
- - TTS_BASE_URL=${TTS_BASE_URL}
- - TTS_API_KEY=${TTS_API_KEY}
- - DB_CONNECTION_STRING=${DB_CONNECTION_STRING}
- - REDIS_CONNECTION_STRING=${REDIS_CONNECTION_STRING}
- - JWT_SECRET_KEY=${JWT_SECRET_KEY}
- - JWT_ACCESS_TOKEN_LIFETIME_MINUTES=${JWT_ACCESS_TOKEN_LIFETIME_MINUTES}
- - JWT_REFRESH_TOKEN_LIFETIME_DAYS=${JWT_REFRESH_TOKEN_LIFETIME_DAYS}
- - OAUTH2_ENABLED=${OAUTH2_ENABLED}
- - GOOGLE_OAUTH_ENABLED=${GOOGLE_OAUTH_ENABLED}
- - GOOGLE_OAUTH_CLIENT_ID=${GOOGLE_OAUTH_CLIENT_ID}
- - GOOGLE_OAUTH_CLIENT_SECRET=${GOOGLE_OAUTH_CLIENT_SECRET}
- - GOOGLE_OAUTH_REDIRECT_URI=${GOOGLE_OAUTH_REDIRECT_URI}
- - GOOGLE_OAUTH_AUTO_CREATE_USER=${GOOGLE_OAUTH_AUTO_CREATE_USER}
- - GOOGLE_OAUTH_DEFAULT_ROLE=${GOOGLE_OAUTH_DEFAULT_ROLE}
+ env_file:
+ - .env
+ environment:
+ ASPNETCORE_ENVIRONMENT: ${ASPNETCORE_ENVIRONMENT:-Production}

---

18-22: 리소스 제한 값 유형 고정화(선택)

cpus/mem_limit/memswap_limit는 수치형으로 두는 편이 도구 호환성에 유리합니다. 큰 문제는 아니므로 선택입니다.

- cpus: '${API_CPU_LIMIT:-1.0}'
- mem_limit: ${API_MEMORY_LIMIT:-1g}
- memswap_limit: ${API_MEMORY_LIMIT:-1g}
+ cpus: ${API_CPU_LIMIT:-1.0}
+ mem_limit: ${API_MEMORY_LIMIT:-1g}
+ memswap_limit: ${API_MEMORY_LIMIT:-1g}

deploy/DEPLOYMENT.md (1)

117-119: Bare URL → Markdown 링크

markdownlint 경고(MD034) 제거를 위해 링크로 감싸세요.

-- **API**: http://localhost:7910
-- **헬스체크**: http://localhost:7910/health
-- **Swagger**: http://localhost:7910/swagger (개발 환경)
+- **API**: <http://localhost:7910>
+- **헬스체크**: <http://localhost:7910/health>
+- **Swagger**: <http://localhost:7910/swagger> (개발 환경)

deploy/docker-compose.prod.yml (1)

71-79: EOF 개행 추가

뉴라인이 없어 린터 경고가 납니다. 개행을 추가하세요.

.github/workflows/release.yml (1)

1-121: 트레일링 스페이스/EOF 개행

여러 라인의 트레일링 스페이스와 EOF 개행 누락으로 린터 경고가 납니다. 정리 권장.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro