Skip to content

Annotation Hand 책임을 정본 모듈로 수렴한다 - #589

Open
developer-1px wants to merge 3 commits into
mainfrom
issue-588-annotation-canonical-hand
Open

Annotation Hand 책임을 정본 모듈로 수렴한다#589
developer-1px wants to merge 3 commits into
mainfrom
issue-588-annotation-canonical-hand

Conversation

@developer-1px

Copy link
Copy Markdown
Owner

Goal Anchor

  • Outcome: Annotation을 생태계 정본 API만으로 구성하고 Demo에는 fixture·copy·layout·제품 정책·조합만 남깁니다.
  • Done: Editing geometry API와 단일 @interactive-os/json-document-annotation 공개 패키지, owner 문서·Usage·source registration·테스트를 갖추고 production source LOC를 줄입니다.
  • Don't: 기존 6개 도구, comment/reaction, move/resize, undo/cancel, raster와 minimalist 시각 결과를 바꾸지 않으며 json-document-react에 Annotation 책임을 넣지 않습니다.

변경

  • Editing에 transformAnnotationSelector, annotationSelectorBounds, annotationResizeHandle 정본 API를 추가했습니다.
  • AnnotationHand가 도구 descriptor, gesture→Intent, SVG projection, transient preview, resize handle, comment UI를 소유합니다.
  • Demo route는 editor·fixture·ID·문구·스타일·배치 주입만 남겼습니다.
  • /docs/api/annotation, package README, Usage source registration, release/CI/workspace 등록을 닫았습니다.
  • Demo에 displaced behavior가 재도입되면 실패하는 canonical closure guard를 추가했습니다.
  • 영향 범위 production source는 1,168 LOC에서 550 LOC로 618 LOC 감소했습니다.

검증

  • npm run verify -w @interactive-os/json-document-editing — 16 files, 73 tests 통과
  • npm run verify -w @interactive-os/json-document-annotation — typecheck, 2 tests, build 통과
  • npm test -w @interactive-os/json-document-site — 13 files, 47 tests 통과
  • npm run typecheck -w @interactive-os/json-document-site — canonical module/design/icon/primitives guard 포함 통과
  • npm run build -w @interactive-os/json-document-site — 통과
  • PLAYWRIGHT_PORT=18820 npm run browser:test -- site/tests/browser/annotation-demo.spec.ts --workers=1 — 10/10 통과
  • npm run docs:api:check && npm run docs:evaluate && npm run workspace:check — 통과
  • node --test scripts/ci-plan.test.mjs scripts/external-kit-plan.test.mjs scripts/release-package.test.mjs — 17/17 통과
  • pre-push npm run build — 통과

Closes #588

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.

Annotation Hand 책임을 정본 모듈로 수렴한다

1 participant