Skip to content

test(#2523 후속): composeImageFilter 효과별 CSS filter 문자열 pin - #2664

Closed
planet6897 wants to merge 1 commit into
edwardkim:develfrom
planet6897:task2523-filter-pins
Closed

planet6897 wants to merge 1 commit into
edwardkim:develfrom
planet6897:task2523-filter-pins

Conversation

@planet6897

Copy link
Copy Markdown
Contributor

대상

PR #2523 검토(pr_2523_review.md)의 후속 권고 대응:

현행 flow-image-clip.test.ts는 clip 계보만 검사하고 새 composeImageFilter()의 효과별 기대 문자열은 직접 pin하지 않는다. … 회색조·흑백·brightness/contrast·baked watermark 회귀 단위 테스트를 후속 보강하는 것을 권고한다.

내용

rhwp-studio/tests/flow-image-clip.test.tscomposeImageFilter() 효과별 CSS filter 문자열 pin 3케이스(단정 13건) 추가:

  • 효과 단독: realPic/기본=null, grayScale·pattern8x8=grayscale(100%), blackWhite=grayscale(100%) contrast(1000%)
  • 밝기/명암 스케일: ±값의 brightness(1.2000)/contrast(0.7000) 등 소수 4자리 고정 + grayScale+밝기+명암 조합 순서
  • 경계: bakedWatermark(효과 기적용 픽셀)=null, 비유한(NaN/Infinity)·비문자 effect 입력 정규화

pin 문자열은 web_canvas.rs::compose_image_filter 구현({:.4} 포맷)과 대조해 동일 산출 계약을 가드한다 — 문자열이 갈라지면 WASM canvas 경로와 DOM flow-image 경로의 그림 효과가 달라진다.

검증

  • node --test tests/flow-image-clip.test.ts 5/5 GREEN (기존 2 + 신규 3)
  • Rust 쪽 compose_image_filter 소스 대조로 기대 문자열 동일성 확인

🤖 Generated with Claude Code

PR edwardkim#2523 검토 권고 대응 — 회색조/pattern8x8/흑백(contrast 1000%)/밝기·명암
스케일/조합/baked watermark 제외/비유한 입력 정규화를 직접 pin.
web_canvas.rs::compose_image_filter 와의 동일 산출 계약 가드.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@edwardkim
edwardkim requested a review from jangster77 July 21, 2026 07:31
@edwardkim edwardkim added this to the v1.0.0 milestone Jul 21, 2026
@edwardkim edwardkim added the enhancement New feature or request label Jul 21, 2026
@jangster77
jangster77 requested a review from edwardkim July 21, 2026 11:29
@jangster77

jangster77 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

@planet6897 감사합니다. 이 변경은 #2732에 체리픽 통합되어 merge됐습니다.

  • merge commit: 32367b8
  • 검증: node --test tests/flow-image-clip.test.ts 5/5, 전체 release-test integration, clippy --all-targets -D warnings 성공
  • 검토 기록: pr_2664_review.md

원 DOM flow-image 계약을 직접 고정하는 회귀 테스트로 수용했습니다. 이 원 PR은 통합 PR로 대체되어 close합니다.

@jangster77 jangster77 closed this Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants