fix: 온보딩 뒤로가기와 PR 커버리지 누락 수정 - #100
Conversation
테스트와 화면 설정을 체이닝 API로 통일하고 단계 뒤로가기를 공통 액션으로 전달한다 Confidence: medium Scope-risk: narrow Tested: 최종 Stage 시뮬레이터 빌드와 변경사항 공백 검사 통과 이름 뒤로가기 액션 회귀 테스트 통과 Not-tested: 실제 버튼 탭과 전체 테스트 임시 화면 판별 검증은 두 번 실패하여 되돌림
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7200f162c4
ℹ️ 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".
| .scaledToFit() | ||
| .frame(width: 12, height: 20) | ||
| .foregroundStyle(.gray400) | ||
| .frame(width: 44, height: 44) |
There was a problem hiding this comment.
On every onboarding step, this wider leading view participates in the HStack layout, but there is no matching 44-point trailing view. Consequently, the progress strip moves another 16 points to the right compared with the previous 12-point image instead of remaining centered; use a symmetric trailing placeholder or expand the hit target without changing the leading view's layout footprint.
Useful? React with 👍 / 👎.
🛠️ Tuist Run Report 🛠️Tests 🧪
Builds 🔨
Bundles 🧰
|
✅ 테스트 통과840 passed · 25m 0s · 총 840개 iPhone 18 Pro (iOS Simulator 27.0) · Tuist 테스트 실행 · Tuist 빌드 실행 · 📦 Bundle 크기
📊 커버리지 50.2% (14,661 / 29,217 라인)🟡 모듈별 커버리지
|
Tuist의 Xcode 27 경로 매핑이 자사 모듈 커버리지를 0으로 기록하므로 커버리지 빌드에서만 매핑을 끈다 Constraint: 샤드 실행과 컴파일 캐시를 유지한다 Confidence: high Scope-risk: narrow Tested: 재현 프로젝트 두 테스트에서 매핑 활성화 시 영 라인 및 수정 후 프레임워크별 실행 라인 복구 검증 그리고 보고서 테스트 18개 통과 Not-tested: 수정 커밋의 전체 CI 커버리지 보고서
1 participant
What
온보딩 뒤로가기 버튼을 누르기 쉽게 바꾸고 체이닝 API로 통일했습니다. PR 보고서에서 자사 모듈 커버리지가 누락되는 CI 설정도 수정했습니다.
Why
기존 뒤로가기는 12×20 이미지에만 탭 제스처가 연결돼 누르기 어려웠습니다. Xcode 27에서는 Tuist 캐시가 켠 소스 경로 매핑 때문에 테스트가 통과해도 자사 모듈의 커버리지가
0 / 0으로 기록됐습니다.How
StepNavigationBar를 44×44 터치 영역의 버튼으로 바꾸고 뒤로가기 접근성 레이블을 적용했습니다.public init()을 두고 온보딩 네 화면 및 디자인 시스템 테스트에서 체이닝 API를 사용합니다.backAction으로 전달합니다. 이름 화면에서 뒤로가면 부모 로그인 라우트를 유지하면서 초대 코드 화면만 남도록 회귀 테스트를 추가했습니다.Test
7200f16의 실제 CI: 테스트 840개 통과, Bundle 크기 수집 성공. 자사 커버리지는 누락됐습니다.0 / 0, 수정 적용 시 각각1 / 1로 커버리지가 복구됐습니다.git diff --check통과.d4a38e40의 전체 CI 성공: 테스트 840개 통과, 두 샤드 모두 병합.Risk
커버리지 빌드의 절대 소스 경로를 유지하므로 서로 다른 체크아웃 경로 간 컴파일 캐시 재사용률은 낮아질 수 있습니다.