Skip to content

[#52] NXRequestBuilder의 단일 send 경로를 구성한다 - #53

Merged
opficdev merged 12 commits into
developfrom
refactor/#52
Aug 22, 2026
Merged

opficdev merged 12 commits into
developfrom
refactor/#52

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

📝 작업 내용

📌 요약

  • NXRequestBuilder 중심의 단일 send 요청 경로 구성
  • 중복 raw() 공개 API 제거 및 raw 응답 경로 단일화
  • 기존 typed 요청·Endpoint 호환 경로 보존
  • 공개 문서와 요청 아키텍처 다이어그램 갱신
  • 로컬 Spec·Plan 파일의 Git 추적 제외

🔍 상세

  • NXRequestBuilder에 다음 전송 API 추가
    • send() -> NXRawResponse
    • 반환 타입 추론 기반 send() -> Response
    • send(as:) -> Response
  • NXRequestExecutor의 요청 조립, interceptor, transport, validation, decoding 흐름 재사용
  • NXRequestBuilder.raw()NXTypedRequestBuilder.raw() 제거
  • as(_:)와 typed HTTP method overload의 사용 중단 안내 유지
  • NXEndpoint.configure(_:), client.request(_:), client.send(_:) 계약 유지
  • retry, 인증, interceptor, cache, transport 대체 경로의 정식 send API 테스트 전환
  • README.md, README.ko.md, DocC의 사용 예시와 전환 안내 갱신
  • 요청 흐름을 책임 계층 기반 Mermaid architecture flowchart로 교체
  • .agents/specs/, .agents/plans/은 로컬 디버깅 용도로 유지하고 Git 추적만 제외

📸 영상 / 이미지 (Optional)

  • 없음

@opficdev opficdev self-assigned this Aug 22, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2aa1857172

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Nexa/Public/NXTypedRequestBuilder.swift Outdated
Comment thread .gitignore
Comment thread Sources/Nexa/Public/NXAPIClient.swift Outdated
@opficdev
opficdev merged commit b5fedd0 into develop Aug 22, 2026
4 checks passed
@opficdev
opficdev deleted the refactor/#52 branch August 22, 2026 07:53
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.

NXRequestBuilder의 단일 send 경로를 구성한다

1 participant