Skip to content

[Hotfix/#85] fix(web): chromatic 배포 시 pandacss 미적용 이슈#86

Merged
seungdeok merged 1 commit into
mainfrom
hotfix/#85
Oct 24, 2025
Merged

[Hotfix/#85] fix(web): chromatic 배포 시 pandacss 미적용 이슈#86
seungdeok merged 1 commit into
mainfrom
hotfix/#85

Conversation

@seungdeok

@seungdeok seungdeok commented Oct 24, 2025

Copy link
Copy Markdown
Member

📝 PR 유형

  • 🚀 feature 기능 추가
  • 🐞 버그 발생
  • 🔨 리팩토링
  • 📋 문서작성
  • 🌍 빌드 설정 및 문제
  • ETC

📝 PR 설명

chromatic 배포 시 pandacss 미적용 이슈

관련된 이슈 넘버

close #85

✅ 작업 목록

  • package.json > postinstall 추가

MR하기 전에 확인해주세요

  • local code lint 검사를 진행하셨나요?
  • loca ci test를 진행하셨나요?

📚 논의사항

📚 ETC

Summary by CodeRabbit

  • Chores
    • 빌드 프로세스 구성 업데이트

@seungdeok seungdeok requested a review from widse October 24, 2025 14:33
@seungdeok seungdeok self-assigned this Oct 24, 2025
@seungdeok seungdeok added the bug Something isn't working label Oct 24, 2025
@coderabbitai

coderabbitai Bot commented Oct 24, 2025

Copy link
Copy Markdown

요약

apps/web/package.json에서 npm 스크립트 실행 시점을 변경하는 단순한 구성 수정이 이루어졌습니다. "prepare" 스크립트가 제거되고 "postinstall" 스크립트로 대체되었습니다.

변경사항

코호트 / 파일(들) 변경 요약
npm 스크립트 재구성
apps/web/package.json
"prepare" 스크립트 제거, "postinstall" 스크립트 추가 (모두 panda codegen 실행)

예상 코드 리뷰 난이도

🎯 1 (간단함) | ⏱️ ~3분

설명: 단순한 구성 변경으로, npm 스크립트 실행 시점을 변경하는 일회성 편집입니다. 로직 복잡도가 없고, 파일 범위가 1개로 제한적이며, 변경 내용이 명확합니다.

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 (3 passed)
Check name Status Explanation
Title Check ✅ Passed PR 제목 "[Hotfix/#85] fix(web): chromatic 배포 시 pandacss 미적용 이슈"는 변경사항의 주요 내용을 명확하게 반영하고 있습니다. 제목은 Chromatic 배포 중 pandacss가 미적용되는 문제를 해결하려는 의도를 명확하게 나타내며, 이는 package.json에서 prepare 스크립트를 postinstall 스크립트로 변경하는 실제 코드 변경과 일치합니다. 제목은 구체적이고 읽기 쉬우며, 팀원이 커밋 히스토리를 스캔할 때 주요 변경사항을 즉시 이해할 수 있습니다.
Out of Scope Changes Check ✅ Passed 변경사항은 단 하나의 파일(apps/web/package.json)에만 영향을 미치며, prepare 스크립트를 postinstall 스크립트로 교체하는 단일하고 명확한 변경입니다. 이 변경은 linked issue #85에서 명시된 요구사항과 정확하게 일치하며, Chromatic 배포 중 pandacss 미적용 문제 해결이라는 객체와도 완벽하게 부합합니다. 범위를 벗어나는 변경사항이 없으며, 모든 코드 변경이 진술된 목표에 직접적으로 관련되어 있습니다.
Description Check ✅ Passed PR 설명은 제공된 템플릿을 잘 따르고 있으며, 필수 섹션들이 완성되어 있습니다. PR 유형에서 버그(🐞)가 체크되었고, PR 설명에 문제점이 명시되었으며, 관련 이슈 번호(close #85)가 지정되었고, 작업 목록에 "package.json > postinstall 추가"가 기록되어 있습니다. 또한 로컬 린트 검사와 CI 테스트 완료 여부도 모두 확인(체크)되어 있습니다. 비어있는 논의사항과 ETC 섹션은 선택사항이므로 완성도에 영향을 주지 않습니다.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/#85

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 77853de and 4d4a1da.

📒 Files selected for processing (1)
  • apps/web/package.json (1 hunks)
🔇 Additional comments (1)
apps/web/package.json (1)

13-13: postinstall 스크립트로 변경하여 Chromatic CD 환경에서 pandacss 코드 생성 실행 보장.

npm 라이프사이클 스크립트 동작 방식:

  • prepare: CI 환경에서 실행이 보장되지 않을 수 있음
  • postinstall: npm install 완료 후 항상 실행되어 더 신뢰할 수 있음

이 변경으로 Chromatic 배포 시 pandacss codegen이 안정적으로 실행될 것으로 예상됩니다. 로컬 개발과 CI 파이프라인에서 모두 동작하는지 확인하시기 바랍니다.

이 변경만으로 문제가 완전히 해결되는지 확인하세요:

  • Chromatic 배포 후 스타일이 정상적으로 적용되는지 확인
  • 로컬 개발 워크플로우(npm installnext dev)에서 변화 없는지 확인

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

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

@seungdeok seungdeok merged commit a03dc57 into main Oct 24, 2025
2 of 3 checks passed
@seungdeok seungdeok deleted the hotfix/#85 branch October 24, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 [Bug] chromatic 배포 시 pandacss 미적용 이슈

1 participant