Skip to content

Conversation

@klmhyeonwoo
Copy link
Member

@klmhyeonwoo klmhyeonwoo commented Feb 4, 2026

I've improved TypeScript compatibility.
I've fixed a moduleResolution compatibility issue when importing packages in a TypeScript environment.

Copilot AI review requested due to automatic review settings February 4, 2026 05:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@klmhyeonwoo klmhyeonwoo requested a review from prgmr99 February 4, 2026 05:43
Copy link
Member

@prgmr99 prgmr99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거는 무슨 설정을 변경하신건가요??

더 좋은 효과가 있나요??

@klmhyeonwoo
Copy link
Member Author

이거는 무슨 설정을 변경하신건가요??

더 좋은 효과가 있나요??

일단 moduleResolution 설정에 대한 내용은 https://developerhia.tistory.com/72 레퍼런스를 참고했습니다!
사내 개발 환경을 타입스크립트로 변환하면서 타입스크립트 환경에서 kr-corekit 를 사용하려고 하니까 moduleResolution 호환성 에러가 발생해서, module : ESNext랑 자연스럽게 호환되는 bundler로 변경했습니다.

무언가 성능 차이보다는 사용처를 명확하게 했다는 느낌으로 이해하여 이렇게 수정을 해보았어요!
Node 환경(CLI 같은)에서도 동작이 되어야하는거라면 NodeNext (package.jsontype별 확장자 및 경로 반영 필요)로 하는 것도 방법이지만, 브라우저에서만 사용되기 때문에 bundler 설정으로 변경해주었습니다!

추가 레퍼런스 : https://sangmin802.github.io/Study/Think/moduleresolution/

@prgmr99
Copy link
Member

prgmr99 commented Feb 4, 2026

이거는 무슨 설정을 변경하신건가요??
더 좋은 효과가 있나요??

일단 moduleResolution 설정에 대한 내용은 https://developerhia.tistory.com/72 레퍼런스를 참고했습니다! 사내 개발 환경을 타입스크립트로 변환하면서 타입스크립트 환경에서 kr-corekit 를 사용하려고 하니까 moduleResolution 호환성 에러가 발생해서, module : ESNext랑 자연스럽게 호환되는 bundler로 변경했습니다.

무언가 성능 차이보다는 사용처를 명확하게 했다는 느낌으로 이해하여 이렇게 수정을 해보았어요! Node 환경(CLI 같은)에서도 동작이 되어야하는거라면 NodeNext (package.jsontype별 확장자 및 경로 반영 필요)로 하는 것도 방법이지만, 브라우저에서만 사용되기 때문에 bundler 설정으로 변경해주었습니다!

추가 레퍼런스 : https://sangmin802.github.io/Study/Think/moduleresolution/

이런 개념이...
새로운 사실을 알게되었네요 👍

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.

2 participants