Skip to content

[#574] SearchView에 TCA를 적용한다#592

Merged
opficdev merged 7 commits into
developfrom
refactor/#574-SearchView-TCA
Jun 12, 2026
Merged

[#574] SearchView에 TCA를 적용한다#592
opficdev merged 7 commits into
developfrom
refactor/#574-SearchView-TCA

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • SearchView의 ViewModel 기반 상태 관리를 TCA Store 기반으로 전환
  • 기존 검색 상태 변화와 비동기 검색 흐름을 TestStore로 검증하기 위한 변경

📝 작업 내용

📌 요약

  • SearchViewModel 제거 및 SearchFeature 추가
  • SearchView와 HomeViewCoordinator의 Search Store 연결
  • searchable 입력과 표시 상태를 BindingAction으로 처리
  • SearchFeature 로딩 상태를 LoadingFeature 기반으로 관리
  • LoginFeature 로딩 상태와 TCA helper 스타일 통일
  • SearchFeature TestStore 기반 테스트 추가

🔍 상세

  • searchQuery/isSearching 바인딩 변경 시 표시 범위 초기화, 빈 검색어 결과 초기화, 디바운스 검색 요청 취소/재시작 흐름 유지
  • Todo/WebPage 통합 검색, 해시태그 Todo-only 검색, 최근 검색어 추가/삭제/전체 삭제 상태 관리 검증
  • 검색 실패 시 공통 에러 alert 표시 및 로딩 종료 흐름 검증
  • LoadingFeature immediate 모드로 Search/Login의 기존 즉시 로딩 표시 동작 유지
  • alertState/alertType/settings 등 Feature 전용 순수 helper를 private extension 내부 static helper로 통일
  • 검증: swiftlint 변경 파일 0 violations, git diff --check 통과, 관련 feature 테스트 60개 통과, DevLogPresentation build-for-testing 성공

📸 영상 / 이미지 (Optional)

  • 없음

@opficdev opficdev self-assigned this Jun 12, 2026

@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

이번 풀리퀘스트는 기존의 SearchViewModel을 Composable Architecture(TCA) 기반의 SearchFeature로 마이그레이션하고, 관련된 뷰 및 테스트 코드를 업데이트하는 변경사항을 포함하고 있습니다. 또한 여러 피처에서 얼럿 생성 등의 헬퍼 메서드들을 static으로 변경하여 리팩토링을 진행했습니다. 리뷰어 피드백 중 LoginFeature에서 로그인 성공 시 로딩 상태가 해제되지 않아 UI가 멈출 수 있는 버그를 지적한 코멘트는 유효하므로 유지합니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Application/DevLogPresentation/Sources/Login/LoginFeature.swift
@opficdev opficdev merged commit 04c7b24 into develop Jun 12, 2026
5 checks passed
@opficdev opficdev deleted the refactor/#574-SearchView-TCA branch June 12, 2026 04:24
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.

SearchView에 TCA를 적용한다

1 participant