fix(build): enable sourcemaps, preserve JSDoc, remove dead vite config and .npmignore (#9 #11 #13 #19) - #27
Merged
Merged
Conversation
…lizer, ArrayPlanner, ArrayRebaser (#16) Resolves #16 - ValueHelper: get/set nested object/array paths, null/undefined/missing handling, intermediate container creation, values reconstruction from flat fields. - FormPath: pathToKey, keyToPath round-trip, pathInputToKey, toFieldPath, root key handling. - FormStateInitializer: primitive/nested/array/mixed null-undefined default values, arrayKeys generation, default submit state. - FormArrayMutationPlanner: push, insert, remove, move, swap, replace with index mapping verification. - FormArrayRebaser: preserve non-array fields, move child metadata, drop removed item metadata, update arrayKeys, preserve submit state.
…g and .npmignore Resolves #9 (sourcemaps): tsconfig.json sourceMap: true Resolves #11 (JSDoc): tsconfig.json removeComments: false Resolves #13 (vite dead code): removed build.lib block, vite.config.ts is now vitest-only with explanatory comment Resolves #19 (.npmignore): removed — package.json files field already controls published files Verification: - pnpm typecheck passes - pnpm test passes (101 tests) - pnpm build passes, dist/ now includes .js.map and .d.ts with JSDoc - npm pack --dry-run confirms LICENSE/README included without .npmignore
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
4개 P1/P2 build 이슈를 단일 PR로 처리 (모두 build config 파일 공유).
Resolves #9
Resolves #11
Resolves #13
Resolves #19
Changes
#9 소스맵 미생성
tsconfig.json:sourceMap: true추가dist/에.js.map파일 생성됨 → 컨슈머 디버깅 시 원본 소스 확인 가능#11 removeComments가 .d.ts에서 한국어 JSDoc 제거
tsconfig.json:removeComments: false로 변경dist/**/*.d.ts에 한국어 JSDoc 보존 → IDE tooltip에 설명 표시#13 vite.config.ts의 lib 블록이 데드 코드
vite.config.ts의build.lib블록 제거 (production 빌드는tsc만 호출, Vite 미호출)#19 .npmignore가 package.json files 필드와 중복
.npmignore제거 —package.jsonfiles: ["dist", "README.md", "README.ko.md", "LICENSE"]가 이미 게시 파일 제한npm pack --dry-run으로 LICENSE/README 포함 확인Verification
pnpm typecheck통과pnpm test통과 — 10 files / 101 testspnpm build통과 —dist/index.js.map,dist/index.d.ts생성 확인.d.ts에 한국어 JSDoc 보존 확인 (grep -c 'field의' dist/core/types.d.ts→ 4)npm pack --dry-run에서 LICENSE, README.md, README.ko.md 포함,.npmignore없이도 정상Notes
tsconfig.json/vite.config.ts/package.json/.npmignore공유