Skip to content

feat: boolean 상태 관리 훅 추가 #59

Merged
D5ng merged 5 commits intodevelopfrom
feature/58-boolean-hook
May 5, 2026
Merged

feat: boolean 상태 관리 훅 추가 #59
D5ng merged 5 commits intodevelopfrom
feature/58-boolean-hook

Conversation

@D5ng
Copy link
Copy Markdown
Member

@D5ng D5ng commented May 3, 2026

📝 개요

  • 전반적으로 사용할 수 있는 useBooleanState 공통 훅 추가
  • useCallback 메모이제이션을 적용하여 리렌더링마다 함수가 재생성되는 것을 방지하고, 참조 동일성을 보장
  • jsdoc 작성하여, IDE 툴팁으로 의도와 사용 맥락을 쉽게 파악할 수 있도록 개선

🔗 관련 이슈

🛠️ 변경 사항 (Checklist)

  • ✨ Feature: 새로운 기능 추가
  • 🚀 Enhancement: 기존 기능 개선/성능 향상
  • 🐞 Bug: 버그 수정
  • ♻️ Refactor: 코드 구조 개선 (기능 변화 없음)
  • 🏗️ Chore: 빌드/패키지 설정/단순 잡일
  • 🎨 Design: UI/UX 스타일 수정
  • 📚 Documentation: 문서 수정

✅ 아래 내용을 한 번 더 점검해 주세요

1. 의도와 가독성 (Naming & Readability)

  • 의도 중심 네이밍: 변수명에서 '역할'이, 함수명에서 '행위+대상'이 명확히 드러나나요?
  • 선언적 코드: '어떻게'가 아닌 '무엇을' 하는지 코드만 보고도 알 수 있나요? (복잡한 로직은 내부 메서드로 숨겼나요?)
  • 주석: 코드만으로 설명이 어려운 '특정 로직'에만 주석을 달았나요?

2. 타입과 논리 (Type Safety & Logic)

  • 타입 안전성: any 사용을 지양하고, 모든 함수의 반환 타입을 명시했나요?
  • 엣지 케이스: 데이터가 없거나(null/undefined), 에러가 발생할 경우를 처리했나요?
  • 하드코딩 방지: API 주소나 설정값들이 환경 변수나 상수로 분리되었나요?

3. 코드 다이어트 (Clean-up)

  • 찌꺼기 제거: 디버깅용 console.log나 사용하지 않는 import를 모두 지웠나요?
  • 불필요한 코드: "나중에 쓰겠지" 하고 남겨둔 죽은 코드(Dead Code)는 없나요?
  • Linter: 린트 에러나 워닝이 남아있지 않나요?

4. 지속 가능성 (Sustainability)

  • 테스트: 수동으로든 코드로든 정상 작동을 확인했나요? (특히 기존 기능이 망가지지 않았나요?)
  • 문서화: 새로운 환경 변수나 라이브러리가 추가되어 README 업데이트가 필요한가요?

💭 회고 (Optional)

@D5ng D5ng self-assigned this May 3, 2026
@D5ng D5ng added the ✨ Feature 아예 없던 새로운 기능을 추가 label May 3, 2026
Copy link
Copy Markdown
Contributor

@MintPansy MintPansy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다! as const를 활용해 타입 추론까지 세심하게 신경 써주셨네요. 덕분에 모달이나 드롭다운 같은 상태 관리가 훨씬 깔끔해질 것 같습니다! 승인할게요! 👍

@D5ng D5ng merged commit 82c91b1 into develop May 5, 2026
1 check passed
@D5ng D5ng deleted the feature/58-boolean-hook branch May 5, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 아예 없던 새로운 기능을 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] boolean 상태 관리 훅 추가

2 participants