PrezelChip 구현

[HamBeomJoon]

📌 작업 내용

PrezelChip 구현

분기처리

• Feedback = BAD일 때 -> 붉은색
• Feedback = DEFAULT일때 -> Interaction으로 DEFAULT, ACTIVE, DISABLED 분기처리

🧩 관련 이슈

• close PrezelChip 구현 #27

📸 스크린샷

• PrezelChip

• IconChip

• Custom Chip

📢 논의하고 싶은 내용

1. 우선, 라이트테마에 맞게 설정했습니다. (피그마에 역시 안맞는 색상들이 좀 있어서,,,)
2. 전반적으로 PrezelButton 코드랑 비슷하게 했기때문에 읽기 쉬울거라 예상합니다. 오버로딩쪽봐주세요

Summary by CodeRabbit

• 신규 기능

  • 확장된 Chip 컴포넌트 추가: 타입(채움/외곽), 크기, 상호작용 및 피드백 상태 지원
  • 텍스트·아이콘·아이콘 전용 구성 가능(텍스트/아이콘 최소 하나 요구) 및 전용 레이아웃/간격 처리
  • 컨테이너·콘텐츠 색상 오버라이드(커스텀 색상) 및 스타일 설정 지원
  • 간편 아이콘 전용 컴포넌트(PrezelIconChip) 추가

• 디자인/미리보기

  • 다양한 변형·크기·상호작용·피드백을 시연하는 다수의 미리보기 화면 추가

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

이 PR은 Prezel 디자인 시스템에 칩 컴포넌트를 추가합니다. 공개 컴포넌트 PrezelChip(텍스트·아이콘 병행 또는 아이콘 전용, 색상 오버라이드 가능)과 PrezelIconChip을 도입하고, PrezelChipType/Interaction/Feedback/Size 열거형 및 PrezelChipStyle·PrezelChipColors를 추가했습니다. 내부적으로 모양 선택, 경계선 계산, 텍스트 스타일, 컨테이너·콘텐츠 색상 결정, 패딩·아이콘 크기 유틸리티와 관련 프리뷰들이 구현되었습니다.

Possibly related PRs

• PR 18: 디자인 토큰(모양, 여백, 스트로크 등)과 PrezelTheme 로컬 값들을 도입하여 칩 스타일 함수들이 해당 프리미티브를 참조합니다.
• PR 25: IconSource 및 DrawableIcon 아이콘 추상화 도입과 관련되며, 새 칩 컴포넌트들이 동일한 아이콘 처리 방식을 사용합니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목 '구현'이 명확하게 PrezelChip 구현이라는 주요 변경사항을 직결적으로 요약하고 있습니다.
Description check	✅ Passed	PR 설명이 작업 내용, 관련 이슈, 스크린샷, 논의사항을 포함하여 템플릿 구조를 충족하고 있습니다.
Linked Issues check	✅ Passed	PrezelChip과 PrezelIconChip 구현 요구사항이 모두 충족되었으며, Feedback/Interaction 상태별 스타일링도 구현되었습니다.
Out of Scope Changes check	✅ Passed	추가된 모든 파일이 PrezelChip 관련 컴포넌트, 스타일, 미리보기로 이슈 #27의 범위 내에 있습니다.

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

---

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

[moondev03]

린트도 확인해주세요 :)

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.kt`:
- Around line 144-145: PrezelTheme이 중첩 호출되어 두 번 래핑되고 있으니 중복된 호출 중 하나를 제거하여 단일
PrezelTheme 블록만 남기세요; PrezelChip.kt 내의 PrezelTheme { PrezelTheme { ... } } 구조를
찾아 바깥 또는 안쪽 하나를 삭제해 중복을 없애고 레이아웃/스코프 영향이 없도록 남은 PrezelTheme 블록이 자식 컴포저블을 모두
포함하도록 정리하세요.

In
`@Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChipStyle.kt`:
- Around line 178-184: The passed-in containerColor/contentColor parameters are
never consumed because LocalPrezelChipColors is provided but never read; update
the PrezelChip implementation so the provided colors are actually used — either
read LocalPrezelChipColors.current where colors are needed (replace direct calls
to style.containerColor() and style.contentColor() with reads that prefer the
provided values) or alter the style usage to accept overrides from the
parameters; specifically ensure the composition provider of
LocalPrezelChipColors(PrezelChipColors(containerColor, contentColor)) is paired
with a consumer (e.g., when computing final colors replace calls to
style.containerColor() and style.contentColor() with logic that first checks
LocalPrezelChipColors.current and falls back to style.* methods) so the
parameters are not ignored.

♻️ Duplicate comments (1)

Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.kt (1)

27-67: 두 번째 오버로드와의 중복 코드

첫 번째 오버로드(Lines 27-67)와 두 번째 오버로드(Lines 69-124) 간에 상당한 중복 코드가 존재합니다. 공통 로직을 추출하여 내부 컴포저블로 분리하거나, 첫 번째 오버로드가 두 번째를 호출하도록 리팩터링하면 유지보수성이 향상됩니다.

🧹 Nitpick comments (2)

Prezel/core/designsystem/src/main/java/com/team/prezel/core/designsystem/component/chip/PrezelChip.kt (2)

38-38: Destructuring 대신 직접 프로퍼티 접근 권장

Line 38에서 val (_, chipSize, _, _) = style로 destructuring을 사용하고 있지만, 두 번째 오버로드(Line 112)에서는 style.size로 직접 접근합니다. 일관성을 위해 직접 프로퍼티 접근 방식으로 통일하는 것이 좋습니다.

♻️ 제안된 수정

-    val (_, chipSize, _, _) = style
...
-                    val spacing = if (chipSize == PrezelChipSize.REGULAR) PrezelTheme.spacing.V4 else PrezelTheme.spacing.V2
+                    val spacing = if (style.size == PrezelChipSize.REGULAR) PrezelTheme.spacing.V4 else PrezelTheme.spacing.V2

---

110-119: 제어 흐름 패턴 불일치

첫 번째 오버로드(Lines 57-63)에서는 조기 반환(if (!hasText) return@Row) 패턴을 사용하고, 두 번째 오버로드에서는 중첩 조건문 패턴을 사용합니다. 코드 일관성을 위해 동일한 패턴으로 통일하는 것을 권장합니다.