Skip to content

refactor!: rename role_provider to user_info_provider#40

Merged
ywkim merged 3 commits into
mainfrom
refactor/unify-user-info-provider
Dec 28, 2025
Merged

refactor!: rename role_provider to user_info_provider#40
ywkim merged 3 commits into
mainfrom
refactor/unify-user-info-provider

Conversation

@ywkim

@ywkim ywkim commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

Summary

BaseCrudRouter의 파라미터명을 role_provider에서 user_info_provider로 변경하여 프로토콜명과 일관성을 유지합니다.

배경

PR #39에서 UserRoleProviderUserInfoProvider 프로토콜로 변경했으나, BaseCrudRouter의 파라미터명은 role_provider로 유지되어 의미 불일치가 발생했습니다.

변경사항

  • 파라미터명: role_provideruser_info_provider
  • 영향 범위: BaseCrudRouter 생성자 파라미터
  • docstring: "role lookup" → "user information lookup"으로 명확화

Breaking Changes

Consumer 프로젝트는 다음과 같이 수정 필요:

# Before
router = BaseCrudRouter(
    ...,
    role_provider=my_provider,
    ...
)

# After
router = BaseCrudRouter(
    ...,
    user_info_provider=my_provider,
    ...
)

테스트

  • ✅ 전체 테스트 통과 (23개)
  • ✅ Lint: 10.00/10
  • ✅ Type Check: 0 errors

🤖 Generated with Claude Code


ywkim and others added 3 commits December 28, 2025 15:20
TDD Red 단계: 테스트를 먼저 변경하여 실패 상태로 만듦

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
파라미터명과 프로토콜명의 의미 불일치 개선:
- 변수명: role_provider → user_info_provider
- docstring: "role lookup"에서 "user information lookup"으로 변경
- 주석: "No role provider"에서 "No user info provider"로 변경

TDD Green 단계: 모든 테스트가 통과하도록 구현 수정

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
누락된 테스트 파일 파라미터명 업데이트

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist 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.

Code Review

안녕하세요. role_provideruser_info_provider로 변경하여 명칭의 일관성을 높이는 리팩토링 작업을 검토했습니다. PR 설명에 Breaking Change에 대한 안내를 명확하게 포함해주셔서 다른 개발자들이 변경 사항을 이해하고 적용하는 데 큰 도움이 될 것입니다. 전체적으로 깔끔하고 목적이 뚜렷한 변경입니다.

한 가지 테스트 코드 개선에 대한 제안을 리뷰 코멘트로 남겼으니 확인 부탁드립니다. 감사합니다.

Comment thread python/tests/unit/test_base_crud_router.py

Copilot AI 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.

Pull request overview

이 PR은 BaseCrudRouter의 파라미터명을 role_provider에서 user_info_provider로 변경하여 프로토콜명(UserInfoProvider)과의 일관성을 확보합니다. PR #39에서 프로토콜명이 변경되었으나 파라미터명이 업데이트되지 않아 발생한 불일치를 해결하는 Breaking Change입니다.

  • BaseCrudRouter 생성자 파라미터 및 인스턴스 속성명 변경
  • 관련 docstring 및 주석을 "user information lookup"으로 명확화
  • 모든 테스트 코드에서 일관되게 파라미터명 업데이트

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
python/aioia_core/fastapi/base_crud_router.py role_provider 파라미터를 user_info_provider로 변경, 속성명과 모든 참조 업데이트, docstring 개선
python/tests/unit/test_base_crud_router.py 테스트 코드의 변수명과 파라미터명을 user_info_provider로 일관되게 변경 (4곳)
python/tests/unit/test_deprecation.py deprecation 테스트에서 파라미터명을 user_info_provider로 업데이트 (2곳)

@ywkim ywkim merged commit ac96e36 into main Dec 28, 2025
8 checks passed
@ywkim ywkim deleted the refactor/unify-user-info-provider branch December 28, 2025 08:52
@github-actions

github-actions Bot commented Feb 4, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 2.0.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.

2 participants