Skip to content

fix(repo): CI에 prisma generate 단계 추가 - #14

Merged
ehlung merged 1 commit into
developfrom
fix/ci-prisma-generate
Jun 8, 2026
Merged

fix(repo): CI에 prisma generate 단계 추가#14
ehlung merged 1 commit into
developfrom
fix/ci-prisma-generate

Conversation

@meteorqz6

@meteorqz6 meteorqz6 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

개요

CI에서 prisma generate 없이 tsc를 실행해 @prisma/client 타입을 찾지 못하는 문제를 수정합니다.


변경 사항

  • .github/workflows/ci.ymlGenerate Prisma client 단계 추가
  • Type check — sharedType check — database 사이에 위치

변경 유형

  • fix — 버그 수정

영향 범위

  • repo — 루트 설정·CI/CD·문서

원인

pnpm install@prisma/client 패키지만 설치하고, PrismaClient·Prisma 등 스키마 기반 타입 생성은 prisma generate를 별도로 실행해야 합니다. CI에 이 단계가 없어 GitHub Actions 캐시 히트 여부에 따라 성공/실패가 갈리는 불안정한 상태였습니다.

테스트 방법

  1. 이 PR의 CI 결과에서 Type check — database 가 통과하는지 확인

체크리스트

  • CI Type check — database 통과 확인

database build 전 prisma generate를 실행하지 않아
@prisma/client 타입을 찾지 못하는 문제 수정.
캐시 히트 여부에 따라 성공/실패가 갈리는 불안정한 상태를 해결한다.
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

CI 워크플로우에 Prisma 클라이언트 생성 단계가 추가되었습니다. 공유 모듈 타입 체크 후 데이터베이스 모듈의 Prisma 클라이언트를 생성한 후, 데이터베이스 모듈 타입 체크가 진행되도록 단계 순서가 변경되었습니다.

Changes

CI 워크플로우 Prisma 클라이언트 생성 단계 추가

Layer / File(s) Summary
CI 워크플로우에 Prisma 클라이언트 생성 단계 추가
.github/workflows/ci.yml
Generate Prisma client 단계가 추가되어 공유 모듈 타입 체크와 데이터베이스 모듈 타입 체크 사이에 pnpm --filter @ilgam/database db:generate 명령이 실행되도록 변경.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 프리즈마 클라이언트 생성을 CI에 담아
타입 체크 순서를 정렬하니
데이터베이스 빌드가 이제 안전하네
자동화된 흐름 속에서
우리의 파이프라인 한층 튼튼해졌어 ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 요약하고 있습니다. CI 워크플로우에 Prisma 생성 단계를 추가한다는 주요 변경 내용을 간결하고 구체적으로 전달합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed PR 설명이 요구되는 대부분의 섹션을 포함하고 있으며, 변경 사항, 변경 유형, 영향 범위, 테스트 방법이 명확하게 작성되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-prisma-generate

Comment @coderabbitai help to get the list of available commands and usage tips.

@meteorqz6 meteorqz6 self-assigned this Jun 8, 2026
@ehlung
ehlung merged commit 6d1b3ab into develop Jun 8, 2026
2 checks passed
@ehlung
ehlung deleted the fix/ci-prisma-generate branch June 8, 2026 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants