Skip to content

fix(react-zpl): zpl-core를 번들하지 않고 external 의존성으로 유지 - #27

Merged
Bori-github merged 1 commit into
mainfrom
fix/react-zpl-external-zpl-core
Jul 18, 2026
Merged

fix(react-zpl): zpl-core를 번들하지 않고 external 의존성으로 유지#27
Bori-github merged 1 commit into
mainfrom
fix/react-zpl-external-zpl-core

Conversation

@Bori-github

Copy link
Copy Markdown
Owner

문제

apps/react-zpl의 빌드가 두 도구로 나뉘는데(런타임 rolldown, 타입 tsc), external 처리가 어긋나 있었습니다.

  • 런타임 JS: rolldown external에 react/react-dom만 있어 @zpl-kit/zpl-coredist/index.js번들됐습니다.
  • 타입 .d.ts: @zpl-kit/zpl-coreexternal로 import.
  • package.json: @zpl-kit/zpl-core를 런타임 의존성으로 선언.

결과적으로 zpl-core 코드가 이중 포함되고, 런타임·타입·의존성 선언 셋이 불일치했습니다.

변경

  • rolldown.config.ts — external에 /^@zpl-kit\// 추가 → 런타임도 zpl-core를 import로 남김
  • package.jsonworkspace:*workspace:^ (발행 시 정확 버전 대신 ^0.1.0 범위 핀)

검증

  • 재빌드 후 dist/index.js가 zpl-core 심볼을 인라인하지 않고 import { renderLabel } from "@zpl-kit/zpl-core" 하는 것 확인
  • pnpm pack으로 발행될 dependencies"@zpl-kit/zpl-core": "^0.1.0"인 것 확인
  • 42개 테스트 통과, type-check·format:check·lint(경고 0) 통과

rolldown external에 react/react-dom만 있어 @zpl-kit/zpl-core가 런타임 JS에
번들됐다. 반면 .d.ts와 package.json은 zpl-core를 external로 참조 → 코드가
이중 포함되고 런타임·타입·의존성 선언이 어긋났다.

- rolldown external에 /^@zpl-kit\// 추가 → 런타임도 zpl-core를 import로 남김
- workspace:* → workspace:^ (발행 시 ^0.1.0 범위 핀, 정확 버전 고정 해소)

빌드 후 dist/index.js가 zpl-core 심볼을 인라인하지 않고 import하는 것과
42개 테스트 통과를 확인.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Bori-github Bori-github added the fix Bug fix — corrects broken behavior label Jul 18, 2026
@vercel

vercel Bot commented Jul 18, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
zpl-viewer Ignored Ignored Jul 18, 2026 4:54pm

@Bori-github Bori-github self-assigned this Jul 18, 2026
@Bori-github
Bori-github merged commit ab9f0ca into main Jul 18, 2026
4 checks passed
@Bori-github
Bori-github deleted the fix/react-zpl-external-zpl-core branch July 18, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fix — corrects broken behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant