Skip to content

chore: CI 패리티 훅 정렬 및 워크플로우 동기화#418

Merged
manNomi merged 1 commit intomainfrom
parser
Feb 14, 2026
Merged

chore: CI 패리티 훅 정렬 및 워크플로우 동기화#418
manNomi merged 1 commit intomainfrom
parser

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 14, 2026

요약

  • Husky pre-commit/pre-push를 GitHub CI의 변경 감지 기준(apps/web, apps/admin, 루트 핵심 파일)과 동일하게 정렬했습니다.
  • 변경 파일이 CI 대상이 아닐 때는 훅을 스킵하고, 대상일 때만 web/admin 검사 및 빌드를 실행하도록 조정했습니다.
  • docs/development-workflow.md에 새 훅 동작(검사/빌드 범위 및 실행 명령)을 반영했습니다.

CI 실패 원인 (기존)

  • 로컬 훅과 GitHub Actions가 서로 다른 범위/명령을 사용해, 로컬에서는 통과했지만 CI에서 실패하는 불일치가 발생했습니다.
  • 특히 변경 영향이 없는 상황에서도 과도한 검사를 돌리거나, 반대로 CI에서만 실행되는 경로가 있어 재현성이 낮았습니다.

해결 방식

  • pre-commit: staged 파일 기준으로 대상 판별 후
    • web 대상: pnpm --filter @solid-connect/web run ci:check
    • admin 대상: pnpm --filter @solid-connect/admin run lint + pnpm --filter @solid-connect/admin run format
  • pre-push: upstream(or merge-base) 대비 변경 파일 기준으로 대상 판별 후
    • web 대상: NODE_ENV=production pnpm --filter @solid-connect/web run build
    • admin 대상: NODE_ENV=production pnpm --filter @solid-connect/admin run build
  • 대상이 없으면 명확한 skip 메시지를 출력해 개발자 경험과 실행 시간을 개선했습니다.

검증

  • sh .husky/pre-commit
  • sh .husky/pre-push
  • pnpm --filter @solid-connect/admin run test (현재 테스트 파일 부재로 No test files found 확인)

리스크/참고

  • 저장소 내 기존 lint/format 이슈가 남아 있는 경우, 훅/CI 정렬 이후 더 일관되게 조기 탐지됩니다.
  • 이번 변경은 훅 실행 정책/문서 정렬에 집중했으며, 애플리케이션 로직 변경은 없습니다.

@vercel
Copy link

vercel bot commented Feb 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Error Error Feb 14, 2026 10:29am
solid-connection-web Error Error Feb 14, 2026 10:29am

@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Warning

Rate limit exceeded

@manNomi has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 34 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch parser

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant