Skip to content

[refactor/MAT-914] drawing import 패키지 전환 + scrap/utils/skia 삭제#355

Open
b0nsu wants to merge 1 commit into
developfrom
feat/mat-914-use-pkg-drawing
Open

[refactor/MAT-914] drawing import 패키지 전환 + scrap/utils/skia 삭제#355
b0nsu wants to merge 1 commit into
developfrom
feat/mat-914-use-pkg-drawing

Conversation

@b0nsu
Copy link
Copy Markdown
Collaborator

@b0nsu b0nsu commented May 26, 2026

Summary

scrap/utils/skia/ 의 drawing.tsx / smoothing.ts 는 `@repo/pointer-native-drawing` 도입 전 위치이며 develop 시점 패키지와 코드 동일. caller(ProblemScreen / ScrapDetailScreen / useHandwritingManager / handwritingEncoder) 4곳의 import 를 패키지로 전환하고 옛 위치 삭제.

Linear

Changes

  • `ProblemScreen.tsx` — `@repo/pointer-native-drawing` import
  • `ScrapDetailScreen.tsx` — default → named import + 패키지 경로
  • `useHandwritingManager.ts` — `DrawingCanvasRef` type 패키지 import
  • `handwritingEncoder.ts` — `Stroke` / `TextItem` type 패키지 import
  • `scrap/utils/skia/` 디렉토리 삭제 (drawing.tsx + smoothing.ts + index.ts)

Testing

  • `pnpm --filter native exec tsc --noEmit` — exit 0
  • `pnpm --filter native exec eslint <변경 파일>` — 0 errors (warnings 는 사전 존재 패턴)

Risk / Impact

  • 코드 동일성 확인 — diff 결과 develop 시점 패키지 DrawingCanvas.tsx 와 scrap/utils/skia/drawing.tsx 가 완전 동일. import 변경만으로 동작 회귀 0
  • 영향 범위: ProblemScreen / Scrap 화면. 스크랩 임시 숨김 상태라 사용자 영향 사실상 ProblemScreen 만

…-914)

기존 apps/native/src/features/student/scrap/utils/skia/ 의 drawing.tsx / smoothing.ts 는 @repo/pointer-native-drawing 패키지 도입 전 위치. develop 시점 패키지와 코드 동일.

- ProblemScreen / ScrapDetailScreen / useHandwritingManager / handwritingEncoder 의 import 를 @repo/pointer-native-drawing 으로 전환
- scrap/utils/skia/ 디렉토리 삭제 (drawing.tsx + smoothing.ts + index.ts)

향후 MAT-897 우산의 drawing 5단계 PR(MAT-898~902) 변경이 ProblemScreen / Scrap 양쪽에 자동 반영됨.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear Bot commented May 26, 2026

MAT-914

@b0nsu b0nsu requested a review from sterdsterd May 26, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant