Skip to content

fix(jin-frame): v5.0.1 — multipart PUT/PATCH, OAuth2 cleanup, Axios ref removal, naming convention docs#194

Merged
imjuni merged 3 commits into
masterfrom
develop
Apr 27, 2026
Merged

fix(jin-frame): v5.0.1 — multipart PUT/PATCH, OAuth2 cleanup, Axios ref removal, naming convention docs#194
imjuni merged 3 commits into
masterfrom
develop

Conversation

@imjuni

@imjuni imjuni commented Apr 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • multipart/form-data: PUT, PATCH 메서드 지원 추가 (기존 POST만 지원)
  • AuthorizationData: OAuth2Provider 제거에 따른 잔재 refreshToken 필드 삭제
  • 소스 주석: DebugInfo, JinFrameRequestConfig, RequestDedupeManager, getAuthorization, applySecurityProviders 의 Axios 레퍼런스를 fetch 기반으로 수정
  • docs (EN/KO): Axios 레퍼런스 전면 제거 — form.md, what-is-jin-frame.md, getting-to-start.md(Axios 버전 테이블 삭제), inheritance.md(AxiosResponseJinResp, _postHook 오타 수정), validation.md(AxiosResponseJinPassResp), usage-method.md(transformRequest 옵션 제거)
  • docs (EN/KO): 네이밍 컨벤션 페이지 신규 추가 (# / _ / 접두사 없음 세 규칙) 및 사이드바 등록
  • version: 5.0.05.0.1

Test plan

  • 전체 테스트 472개 통과 (pnpm --filter jin-frame test)
  • multipart PUT 테스트 추가 및 통과
  • multipart PATCH 테스트 추가 및 통과
  • 브랜치 커버리지 99.83% 유지

🤖 Generated with Claude Code

imjuni and others added 3 commits April 27, 2026 23:44
… up Axios references

- Remove refreshToken from AuthorizationData (OAuth2Provider was removed in v5)
- Support multipart/form-data for PUT and PATCH in addition to POST
- Add tests for multipart PUT and PATCH (472 tests total)
- Replace Axios references with fetch-based descriptions in source comments
  (DebugInfo, JinFrameRequestConfig, RequestDedupeManager, getAuthorization,
  applySecurityProviders)
- Update docs (EN/KO) to remove all Axios mentions:
  - index.md: "Axios Ecosystem" → "Standards-Based HTTP (fetch)"
  - what-is-jin-frame.md: built on fetch, not Axios
  - getting-to-start.md: remove Axios version compatibility table
  - method/form.md: URLSearchParams and FormData instead of transformRequest/form-data
  - method/inheritance.md: AxiosResponse → JinResp, $_postHook → _postHook
  - method/validation.md: AxiosResponse → JinPassResp, Validator → BaseValidator
  - usage-method.md: remove transformRequest option, fix body/status descriptions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add docs/method/naming-convention.md (EN) explaining the three rules:
  # prefix for private state, _ prefix for all instance methods/hooks,
  no prefix for static methods — with motivation and summary table
- Add docs/ko/method/naming-convention.md (KO) Korean translation
- Register page as first item under Method section in VitePress sidebar
- Update inheritance.md (EN/KO) to link to the new naming convention page

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- multipart/form-data now supports PUT and PATCH in addition to POST

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov

codecov Bot commented Apr 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (5b87d1e) to head (bce5d84).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #194   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          107       107           
  Lines         1083      1088    +5     
  Branches       284       289    +5     
=========================================
+ Hits          1083      1088    +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@imjuni imjuni merged commit ed33067 into master Apr 27, 2026
5 checks passed
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.

1 participant