Skip to content

feat(frontend): support @aioia/core 2.x#85

Merged
ywkim merged 2 commits into
mainfrom
feat/support-aioia-core-2x
Feb 5, 2026
Merged

feat(frontend): support @aioia/core 2.x#85
ywkim merged 2 commits into
mainfrom
feat/support-aioia-core-2x

Conversation

@ywkim

@ywkim ywkim commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

요약

trade-safety가 @aioia/core 2.x를 지원하여 buppy와의 의존성 호환성을 확보합니다.

목적

@aioia/core 2.0.0이 buppy에 설치되었으나, trade-safety는 아직 1.x만 지원하고 있습니다. 2.x를 지원함으로써:

  • buppy와 trade-safety 간 의존성 호환성 확보
  • PR #45에서 추가된 httpOnly 쿠키 및 credentials 자동 포함 기능 활용
  • 향후 @aioia/core의 최신 기능 활용 가능

주요 변경 사항

  • package.json: @aioia/core peerDependencies를 >=2.0.0 <3.0.0으로 업그레이드
  • 자동 마이그레이션: @aioia/core 2.x에서 credentials: 'include'가 자동으로 포함되므로 trade-safety 코드 수정 불필요

테스트 체크리스트

  • 라이브러리 빌드 성공 (ESM/CJS 듀얼 빌드)
  • TypeScript 타입 검사 통과
  • ESLint, Prettier 포맷팅 검사 통과
  • 기존 기능 호환성 유지 (API 변경 없음)

참고

🤖 Generated with Claude Code

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

이 PR은 @aioia/core 의존성을 2.x 버전으로 업데이트하여 buppy와의 호환성을 확보하는 것을 목표로 합니다. package.jsonpackage-lock.json의 변경 사항은 이 목표를 정확하게 반영하고 있으며, 코드 변경은 없어 보입니다. 전반적으로 PR의 목적에 부합하는 좋은 변경입니다.

한 가지 제안 사항이 있습니다. 레포지토리 스타일 가이드(라인 81)에 따르면 PR 제목은 <type>(<scope>): <한국어 제목> 형식으로 한국어로 작성해야 합니다. 현재 제목인 feat(frontend): support @aioia/core 2.xfeat(frontend): @aioia/core 2.x 지원과 같이 수정하는 것을 고려해 주세요. 이는 레포지토리의 일관성을 유지하는 데 도움이 됩니다.

@ywkim ywkim merged commit 799fed9 into main Feb 5, 2026
3 of 4 checks passed
@ywkim ywkim deleted the feat/support-aioia-core-2x branch February 5, 2026 05:43
@github-actions

github-actions Bot commented Feb 5, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant