fix: 0.7.2 custom font glyph bleed 옵션 추가#13
Conversation
SignPainter, BlowBrush, Brush Script MT 같은 script/custom font에서 digit glyph가 column 경계 밖으로 그려질 때 잘릴 수 있는 문제를 선택적으로 완화합니다. - AnimateNumberText initializer에 glyphBleed 옵션을 추가했습니다. - 기본값은 EdgeInsets()라 일반 폰트의 기존 폭과 간격은 유지됩니다. - smooth/reel digit column 모두 GlyphBleedClipShape를 통해 clipping rect만 확장합니다. - 음수 glyphBleed 값은 0으로 보정합니다. - read-only initializer의 glyphBleed 컴파일 테스트를 추가했습니다. - README와 DocC 설치 버전 및 custom font clipping 예제를 0.7.2 기준으로 갱신했습니다. 검증: - swift test - DOCS_OUTPUT_PATH=.build/docc-site DOCC_ARCHIVE_PATH=.build/AnimateNumberText.doccarchive ./GeneratingDocumentationSite - git diff --check
|
Warning Review limit reached
More reviews will be available in 47 minutes and 48 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![codex-review-bot[bot]](https://avatars.githubusercontent.com/in/3409825?s=60&v=4){kind=small}
There was a problem hiding this comment.
변경 범위는 공개 API에 선택 옵션을 추가하고 기본값으로 기존 동작을 유지하는 방향이라 호환성 측면에서 큰 위험은 낮아 보입니다. 숫자 컬럼의 clipping 영역만 확장하도록 smooth/reel 경로에 모두 적용된 점도 의도와 맞습니다. 다만 새 동작은 시각적 회귀를 막는 기능인데 현재 테스트는 initializer 컴파일 확인에 그쳐, 실제 bleed 보정/음수 보정 로직의 회귀를 잡기 어렵습니다.
좋은 점
glyphBleed기본값을 zero로 둬 기존 사용자 레이아웃과 호출부 호환성을 유지한 점- smooth/reel 두 렌더링 경로 모두 같은
GlyphBleedClipShape를 사용해 옵션 적용 범위를 일관되게 맞춘 점 - 음수
EdgeInsets를 0으로 보정해 clipping rect가 의도치 않게 줄어드는 입력을 방어한 점
💡 권장 개선 사항
GlyphBleedClipShape.path(in:)또는EdgeInsets.sanitizedGlyphBleed의 계산 결과를 직접 검증할 수 있도록 최소 단위 테스트를 추가하면, 추후 리팩터링에서 bleed 확장/음수 보정이 깨지는 회귀를 더 빨리 잡을 수 있습니다.
기술 단위 코멘트 1건은 각 라인에 별도 표시됩니다.
리뷰 모델: gpt-5.5
![gemini-pr-review-bot[bot]](https://avatars.githubusercontent.com/in/3443580?s=60&v=4){kind=small}
There was a problem hiding this comment.
커스텀 폰트 사용 시 글리프가 텍스트 영역 밖으로 벗어나 잘리는 현상을 완화하기 위해 glyphBleed 옵션을 추가한 변경 사항입니다. 레이아웃 크기에 영향을 주지 않으면서 클리핑 영역만 안전하게 확장하도록 설계되었으며 이에 따른 문서 갱신과 컴파일 테스트도 꼼꼼히 반영되었습니다. 다만 음수 패딩 값의 정제(sanitize) 동작을 검증하는 엣지 케이스 테스트를 보완하고 애니메이션 순서 계산 로직을 개선하면 더욱 견고한 코드베이스가 될 것입니다.
좋은 점
- EdgeInsets의 sanitizedGlyphBleed 프로퍼티를 구현하여 음수 값이 입력되더라도 Swift.max 함수를 통해 0으로 보정함으로써 부적절한 레이아웃 연산을 사전에 방지하도록 설계한 점
- ReelDigitColumn 구조체에서 wrappedDigit 메서드를 정의하여 음수 영역의 포지션에 대해서도 0에서 9 사이의 인덱스로 안전하게 매핑되도록 처리해 역방향 애니메이션의 무결성을 유지한 점
개선할 점
- GeneratingDocumentationSite 스크립트 내 BUNDLE_VERSION 변수가 여전히 0.6.0으로 하드코딩되어 있습니다. 라이브러리의 버전 승격에 맞추어 스크립트의 기본값을 0.7.2로 갱신하거나, Package.swift 등 프로젝트 설정으로부터 동적으로 버전을 추출할 수 있도록 개선하는 것을 제안합니다.
기술 단위 코멘트 1건은 각 라인에 별도 표시됩니다.
(주: 다음 1개 코멘트는 PR diff 범위 밖이라 본문에 모았습니다.)
드롭된 라인 지적
Sources/AnimateNumberText/Public/AnimateNumberText.swift:270— [Minor] where 조건절에서 canAnimateDigitChange 검증을 거치므로 digitOrdinals[index]는 무조건 존재해야 합니다. 만약 이를 찾지 못해 문자열 인덱스인 index를 기본값으로 사용하게 되면, 소수점이나 천 단위 구분자가 포함된 텍스트에서 실제 숫자 위치와 문자열 위치가 어긋나 애니메이션 딜레이 타이밍이 일관되지 않게 동작할 수 있습니다.
리뷰 생성 모델: Gemini 3.5 Flash (High) #1
![mlx-review-bot[bot]](https://avatars.githubusercontent.com/in/3088706?s=60&v=4){kind=small}
There was a problem hiding this comment.
MLX 리뷰
이 PR은 AnimateNumberText 라이브러리의 버전을 0.7.1에서 0.7.2로 업데이트하고, 커스텀 폰트의 글자 클리핑을 위한 glyphBleed 매개변수를 추가하여 스크립트나 브러시 폰트에서 글자가 열려나는 문제를 해결합니다. 이 변경은 문서와 소스 코드에 반영되며, 테스트도 추가되어 기능의 안정성을 보장합니다.
라인 단위 코멘트
- 라인 단위로 남길 개선 사항은 발견되지 않았습니다.
사용된 모델: mlx-community/Qwen3-30B-A3B-Instruct-2507-8bit
![claudecode-pr-review-bot[bot]](https://avatars.githubusercontent.com/in/3920205?s=60&v=4){kind=small}
There was a problem hiding this comment.
custom/script 폰트의 glyph 잘림 문제를 glyphBleed 옵션으로 해결하는 PR입니다. 기본값이 zero EdgeInsets이므로 기존 사용자에게 영향이 없고, 음수 값 보정과 smooth/reel 양쪽 모드 일관 적용이 잘 되어 있습니다. 버그나 보안 이슈 없이 깔끔하게 구성되어 있으며, 테스트 보강 정도만 권장드립니다.
좋은 점
- 기본값
EdgeInsets()를 사용하여 기존 사용자의 레이아웃에 전혀 영향을 주지 않으면서 하위 호환성을 유지한 점이 좋습니다. sanitizedGlyphBleed계산 프로퍼티로 음수 입력을 0으로 보정하여 방어적으로 처리한 점이 좋습니다.GlyphBleedClipShape을 별도Shape로 분리하여 smooth 모드와 reel 모드 양쪽에 일관되게 적용한 설계가 적절합니다.- read-only 이니셜라이저가 designated 이니셜라이저에 위임하므로 sanitization 로직이 한 곳에서만 관리되는 점이 좋습니다.
💡 권장 개선 사항
sanitizedGlyphBleed가 음수 값을 0으로 보정하는 동작을 검증하는 단위 테스트를 추가해 주시면 좋겠습니다.EdgeInsetsextension이private이라 직접 테스트하기 어려우시다면, 내부 동작을 간접 검증하는 방식(예: 음수 bleed를 넘긴 뷰가 정상 생성되는지 확인)이라도 권장드립니다.GlyphBleedClipShape.path(in:)의 확장 rect 계산이 올바른지 검증하는 단위 테스트도 고려해 주시면 좋겠습니다.Shape는private이지만internal로 열어 직접path(in:)결과를 검증하면 향후 리팩터링 시 회귀를 방지할 수 있습니다.
기술 단위 코멘트 2건은 각 라인에 별도 표시됩니다.
리뷰 모델: Claude Opus 4.6 (Thinking) #2
- glyphBleed 음수 입력 보정 테스트를 추가해 clipping 영역이 줄어들지 않도록 회귀를 방지합니다. - GlyphBleedClipShape의 확장 rect 계산 테스트를 추가해 custom font glyph bleed 동작을 직접 검증합니다. - DocC 생성 스크립트의 fallback bundle version을 0.7.2로 갱신합니다. 검증: - swift test - DOCS_OUTPUT_PATH=.build/docc-site DOCC_ARCHIVE_PATH=.build/AnimateNumberText.doccarchive ./GeneratingDocumentationSite - git diff --check
There was a problem hiding this comment.
이번 변경은 glyphBleed 옵션을 공개 API에 추가하면서 기본값을 0으로 유지해 기존 호출부와 레이아웃 호환성을 잘 보존하고 있습니다. smooth/reel 렌더링 경로 모두 같은 클리핑 확장 로직을 사용하고, 음수 inset 보정과 clip rect 계산 테스트도 추가되어 이전 리뷰에서 지적된 핵심 회귀 위험이 해소됐습니다. 현재 변경 범위에서 머지를 막을 만한 버그나 예외 처리 누락은 확인되지 않았습니다.
좋은 점
glyphBleed기본값을EdgeInsets()로 둬 기존 사용자의 렌더링 폭과 호출부 호환성을 유지한 점- smooth/reel digit column 모두
GlyphBleedClipShape를 사용해 새 옵션 적용 범위를 일관되게 맞춘 점 - 음수 inset 보정과 clip rect 확장 계산을 직접 검증하는 테스트를 추가해 이전 테스트 공백을 보완한 점
리뷰 모델: gpt-5.5
There was a problem hiding this comment.
이번 PR은 커스텀 폰트 사용 시 글리프가 텍스트 컬럼 경계 밖으로 그려져 잘리는 현상을 완화하기 위해 glyphBleed 옵션을 추가하였습니다. 기존 레이아웃의 크기와 간격을 유지하면서 클리핑 영역만 안전하게 확장하도록 설계되었으며, 음수 보정 로직과 렌더링 경로 일관 적용 등 세세한 부분이 완성도 높게 반영되었습니다. 또한 추가된 기능에 대한 단위 테스트도 적절하게 작성되어 회귀 방지 대책이 마련되었습니다.
좋은 점
- EdgeInsets extension에
sanitizedGlyphBleed프로퍼티를 정의하여 음수 값이 입력되더라도Swift.max(0, ...)를 통해 0으로 보정함으로써 안전하게 방어한 점 - smooth 모드와 reel 모드 양쪽 모두에 일관되게
GlyphBleedClipShape를 활용하도록 설계하여 외부 레이아웃 레이아웃 크기에는 영향을 주지 않고 클리핑 영역만 깔끔하게 확장한 점 - 음수 보정 로직과 클리핑 영역 확장 계산을 검증하는 단위 테스트(
glyphBleedSanitizesNegativeInsets,glyphBleedClipShapeExpandsClipRect)를 충실히 구축한 점
개선할 점
- GeneratingDocumentationSite 스크립트 내
BUNDLE_VERSION이0.7.2로 하드코딩되어 있어 향후 버전 업데이트 시마다 수동 수정이 필요하므로,Package.swift나 Git 태그 등에서 동적으로 버전을 추출하도록 개선하는 것을 제안합니다.
리뷰 생성 모델: Gemini 3.5 Flash (High) #1
- glyphBleed가 zero일 때 GlyphBleedClipShape가 원본 rect를 유지하는지 검증합니다. - .clipShape 전환 이후 기본 동작 동등성을 테스트로 고정합니다. 검증: - swift test - git diff --check
|
수정 완료: 리뷰 본문에서 지적된 |
There was a problem hiding this comment.
커스텀 폰트 사용 시 글리프가 텍스트 컬럼 경계 밖으로 그려져 잘리는 문제를 완화하기 위해 glyphBleed 옵션을 추가한 변경 사항입니다. 입력값에 대한 음수 보정 처리가 철저하며, 이에 대응하는 다양한 테스트와 문서 업데이트가 누락 없이 깔끔하게 반영되었습니다.
좋은 점
- EdgeInsets의 sanitizedGlyphBleed 프로퍼티를 구현하여 음수 값이 입력되더라도 0으로 보정함으로써 안전하게 방어하도록 설계한 점
- smooth 모드와 reel 모드 양쪽 렌더링 경로 모두에 동일한 GlyphBleedClipShape를 일관되게 적용하여 외부 레이아웃 크기를 유지한 채 클리핑 영역만 안전하게 확장한 점
- 새로 도입된 음수 보정 계산 및 클리핑 영역 확장 Rect 계산의 무결성을 검증하기 위한 단위 테스트들을 꼼꼼하게 구축한 점
개선할 점
- EdgeInsets 구조체는 Equatable을 준수하지 않으므로, 향후 zero 여부를 조건부 렌더링 등에 활용할 수 있도록 isZero 같은 연산 프로퍼티나 Equatable 확장을 제공하는 구조적 개선을 제안합니다.
기술 단위 코멘트 1건은 각 라인에 별도 표시됩니다.
리뷰 생성 모델: Gemini 3.5 Flash (High) #1
There was a problem hiding this comment.
커스텀 폰트 glyph 잘림 문제를 glyphBleed 옵션으로 해결하는 변경이며, 기본값 zero 유지로 기존 호환성이 잘 보존되고 핵심 로직에 대한 단위 테스트도 충실하게 갖춰져 있습니다. 다만 README와 DocC에 새로 추가된 Custom Font Clipping 예제의 Swift 인자 순서가 이니셜라이저 선언 순서와 맞지 않아 복사·붙여넣기 시 컴파일 에러가 발생합니다. 이 문서 오류를 수정하면 머지해도 좋겠습니다.
좋은 점
glyphBleed기본값을EdgeInsets()로 두어 기존 사용자의 레이아웃과 호출부 호환성을 변경 없이 유지한 점sanitizedGlyphBleed로 음수 입력을 0으로 보정하고,GlyphBleedClipShape.path(in:)확장 계산과 zero bleed 항등성까지 단위 테스트로 직접 검증하여 회귀 방지 기반을 마련한 점- smooth/reel 두 렌더링 경로 모두 동일한
GlyphBleedClipShape를 적용해 옵션 동작의 일관성을 확보한 점
🔴 반드시 수정할 사항
- README.md(168행)와 DocC AnimateNumberText.md(179행)의 Custom Font Clipping 예제에서
value:,textColor:인자가font:인자보다 앞에 위치합니다. 이니셜라이저 선언 순서가font:, weight:, value:, textColor:, …, glyphBleed:이므로,font:를 생략하지 않고value:뒤에 배치하면 Swift 컴파일 에러('argument font must precede argument value')가 발생합니다.font:를value:앞으로 이동해 주세요.
기술 단위 코멘트 2건은 각 라인에 별도 표시됩니다.
리뷰 모델: Claude Opus 4.6 (Thinking) #2
- README의 custom font clipping 예제에서 font 인자를 value 앞에 배치해 실제 initializer 선언 순서와 맞춥니다. - DocC의 동일 예제도 같은 순서로 수정해 복사해도 컴파일되도록 정리합니다. 검증: - swift test - DOCS_OUTPUT_PATH=.build/docc-site DOCC_ARCHIVE_PATH=.build/AnimateNumberText.doccarchive ./GeneratingDocumentationSite - git diff --check
There was a problem hiding this comment.
변경사항을 기준으로 전체 코드 흐름, 문서 예제, 테스트 보강 상태를 확인했습니다. 이전 라운드에서 지적된 음수 glyphBleed 보정 테스트와 문서 예제 인자 순서 문제는 최신 코드에서 처리되어 있습니다. 현재 변경 범위에서 머지를 막을 만한 버그, 예외 처리 누락, 데이터 정합성 문제는 확인되지 않았습니다.
좋은 점
- glyphBleed 기본값을 0으로 유지해 기존 호출부와 레이아웃 동작을 보존한 점
- smooth/reel 양쪽 digit 렌더링 경로에 같은 GlyphBleedClipShape를 적용해 옵션 동작을 일관되게 만든 점
- 음수 EdgeInsets 보정과 clip rect 확장 계산을 단위 테스트로 직접 검증한 점
- README와 DocC 예제가 최신 initializer 인자 순서와 버전에 맞게 정리된 점
리뷰 모델: gpt-5.5
There was a problem hiding this comment.
이번 PR은 커스텀 및 스크립트 폰트 사용 시 글리프가 컬럼 경계 밖으로 그려져 잘리는 현상을 해결하기 위해 glyphBleed 옵션을 안전하게 추가했습니다. EdgeInsets의 음수 값을 보정하는 방어적 로직과 이를 검증하는 신규 테스트 코드가 충실히 반영되었습니다. 이전 리뷰에서 제안된 지적사항 및 문서 내 버전 오기도 누락 없이 잘 보완되었습니다.
좋은 점
- EdgeInsets extension 내에 sanitizedGlyphBleed를 도입하여 음수 입력값을 방어적으로 보정하고, 이를 기반으로 GlyphBleedClipShape가 렌더링 영역만 안전하게 확장하도록 설계한 점
- GlyphBleedClipShape를 구현하여 smooth와 reel 두 가지 자식 컬럼 뷰 모두에 일관된 클리핑 메커니즘을 공유하도록 구조화한 점
- 기존의 지정 이니셜라이저에 추가 매개변수 위임을 통해 내부 보정 로직을 단일화하고 하위 호환성을 완벽하게 유지한 점
개선할 점
- EdgeInsets 구조체에 대한 zero 여부를 판단하는 isZero 헬퍼 프로퍼티를 EdgeInsets extension에 추가하여, 뷰 드로잉 내부에서 .clipped() 최적화 분기문을 더 직관적으로 작성할 수 있도록 지원하는 설계 개선을 제안합니다.
기술 단위 코멘트 2건은 각 라인에 별도 표시됩니다.
리뷰 생성 모델: Gemini 3.5 Flash (High) #1
|
이 PR 은 자동 리뷰를 완료하지 못했습니다 (full 모드 실패 → diff-only 모드 재시도까지 실패).
조치 제안
|
There was a problem hiding this comment.
MLX 리뷰
이 PR은 AnimateNumberText 라이브러리의 버전을 0.7.1에서 0.7.2로 업데이트하고, 커스텀 폰트의 글자 클리핑을 위한 glyphBleed 옵션을 추가한다. 이로써 스크립트나 브러시 폰트에서 글자가 열 너비를 벗어나는 문제를 해결할 수 있으며, 문서 및 테스트도 함께 업데이트되었다.
반드시 수정할 사항
glyphBleed옵션을 사용하는 예제 코드에서font:인자가value:와textColor:뒤에 위치해 있다. Swift는 레이블 인자의 순서를 엄격히 따르므로, 이 순서로 코드를 복사하면 컴파일 에러가 발생한다.
라인 단위 코멘트
- 자동 리뷰에서 2개의 라인 단위 개선 사항을 남겼습니다.
사용된 모델: mlx-community/Qwen3-30B-A3B-Instruct-2507-8bit
|
수정 완료: MLX 리뷰 본문의 custom font clipping 예제 인자 순서 지적은 실제 문제였고, 최신 HEAD의 7af6c25에서 README와 DocC 모두 |
1 participant
작업 내용
Refs #2
custom/script font에서 digit glyph가 column 경계 밖으로 그려질 때
.clipped()경계에서 잘릴 수 있는 문제를 선택적으로 완화합니다.수정 사항:
AnimateNumberTextinitializer에glyphBleed: EdgeInsets = EdgeInsets()옵션을 추가했습니다.GlyphBleedClipShape를 사용해 외부 레이아웃 폭을 바꾸지 않고 clipping rect만 확장합니다.glyphBleed값은 0으로 보정합니다.glyphBleedinitializer 컴파일 테스트를 추가했습니다.사용 예:
검증
swift testDOCS_OUTPUT_PATH=.build/docc-site DOCC_ARCHIVE_PATH=.build/AnimateNumberText.doccarchive ./GeneratingDocumentationSitegit diff --check