Skip to content

[feat/MAT-365] 스크랩 화면 전환#307

Open
b0nsu wants to merge 2 commits intorefactor/mat-364-canvasfrom
refactor/mat-365-scrap
Open

[feat/MAT-365] 스크랩 화면 전환#307
b0nsu wants to merge 2 commits intorefactor/mat-364-canvasfrom
refactor/mat-365-scrap

Conversation

@b0nsu
Copy link
Copy Markdown
Collaborator

@b0nsu b0nsu commented Apr 30, 2026

Summary

ScrapDetailScreen을 @repo/pointer-native-drawing 패키지 import로 전환하고,
기존 scrap/utils/skia/drawing.tsx + smoothing.ts를 삭제합니다.

Stacked PR 10/10 — base: refactor/mat-364-canvas

Linear

Changes

  • ScrapDetailScreen → @repo/pointer-native-drawing import 전환
  • ProblemScreen import 경로 수정
  • 기존 scrap/utils/skia/drawing.tsx, smoothing.ts, index.ts 삭제 (-1323줄)
  • handwritingEncoder, useDrawingState, useHandwritingManager import 수정
  • 패키지 내부 lint/prettier 정리

Testing

  • pnpm typecheck 통과
  • pnpm lint 통과
  • 기존 데이터 호환성: 별도 실기기 테스트 필요

Risk / Impact

  • 영향 범위: ScrapDetailScreen, ProblemScreen import 경로
  • 확인이 필요한 부분: 기존 필기/텍스트 데이터 로드+렌더링 호환성
  • 배포 시 유의사항: 실기기 테스트 후 배포 권장

b0nsu and others added 2 commits April 30, 2026 20:48
- ScrapDetailScreen → @repo/pointer-native-drawing import
  - textMode/eraserMode → activeTool enum
  - onHistoryChange → onUndoStateChange
- ProblemScreen → @repo/pointer-native-drawing import
  - onHistoryChange → onUndoStateChange
- useHandwritingManager, handwritingEncoder → 패키지 import
- useDrawingState: setHistoryState 시그니처 → object style
- 기존 scrap/utils/skia/ (drawing.tsx, smoothing.ts, index.ts) 삭제 (-1366줄)

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

linear Bot commented Apr 30, 2026

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pointer-admin Ready Ready Preview, Comment Apr 30, 2026 11:51am

@b0nsu
Copy link
Copy Markdown
Collaborator Author

b0nsu commented May 1, 2026

⚠️ 선행 이슈: canvas ref 타이밍 문제 해소 필요

PR #288 디버깅 중 기존 useHandwritingManager의 canvas ref 타이밍 이슈가 확인되었습니다.

문제

현재 useHandwritingManager의 데이터 로드 useEffect가 canvasRef를 deps에 포함하지만, ref 객체는 참조가 불변이라 .current 변경 시 effect가 재실행되지 않음. 데이터가 캔버스보다 먼저 도착하면 렌더링 누락 발생.

이 PR에서 해소해야 할 것

useHandwritingManager 상태머신 재작성 시 canvas ref 직접 의존을 제거하고, idle → loading → ready → dirty → saving → ready 흐름에서 캔버스 준비 상태를 명시적으로 추적하여 타이밍 이슈를 구조적으로 해소.

관련: PR #288 코멘트 참조

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