Repository navigation
클래스 확정 디자인과 TTS 참여 흐름 반영 - #78
Conversation
클래스 투표·청취·의견·결과 화면과 라우팅을 확정 시안 및 Android 흐름에 맞춘다. Figma 노드 대조 플러그인과 클래스 및 App 흐름 테스트를 보강한다.
UUID 테스트 의존성을 명시하고 현재 라우팅 및 결과 응답 계약에 맞춰 상태 검증을 보완한다.
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: fb2acdaf48
ℹ️ 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".
리포트 표시값을 상태에서 계산하고 공통 색상 토큰을 적용합니다. 좋아요 요청 취소 후 재진입 시 서버 상태를 다시 조회하며 회귀 테스트를 추가합니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: be84c7f1ee
ℹ️ 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".
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
이 PR은 클래스 흐름(사전 투표, TTS 청취, 의견, 결과 화면)을 Figma 시안 및 Android 개발 흐름에 맞춰 대폭 개선하고 있습니다. 주요 변경 사항은 다음과 같습니다:
- 아키텍처 개선:
ClassInterface를ClassRoomInterface,ClassResultInterface,ClassDebateInterface등 세분화된 프로토콜로 분리하여 모듈 간 의존성을 더욱 명확히 하고, 단일 책임 원칙을 강화했습니다. - 내비게이션 및 UI 흐름 재구조화:
ClassCoordinator와ClassBattleFlowView의 내비게이션 로직을 대대적으로 개편하여, 각 클래스 진행 단계에 맞는 명확하고 일관된 사용자 경험을 제공합니다. 탭바 가시성 로직도 여러 화면에서 일관성 있게 업데이트되었습니다. - TTS 및 의견 기능 강화: 오디오 재생 시 TTS 스크립트를 표시하고, 의견 작성 및 조회를 위한 UI/UX를 개선했습니다. AI 인사이트 및 개인 피드백 통합도 강화되었습니다.
- 테스트 코드 정합성 확보: 변경된 비즈니스 로직과 UI 흐름에 맞춰 기존 테스트를 업데이트하고, 새로운 테스트 케이스를 추가하여 코드 변경에 대한 신뢰도를 높였습니다.
전반적으로 구조 개선과 기능 추가가 잘 이루어졌습니다. 특히 UseCase 분리는 모듈 아키텍처 관점에서 매우 긍정적입니다. 몇 가지 스타일 및 가독성 관련 개선 의견이 있습니다.
투표 결과 라벨과 의견 제출 오류 표시를 수정하고 회귀 검사를 추가합니다. Gemini 리뷰의 변경 줄 검증, 중복 게시 방지, API 오류 후 저장 확인을 보강합니다.
There was a problem hiding this comment.
클래스 참여 흐름, 투표, TTS 청취, 의견, 결과 화면 라우팅 및 디자인을 Figma 시안에 맞춰 크게 변경한 PR입니다. ClassCoordinator의 내비게이션 로직이 대폭 수정되었고, 각 화면 컴포넌트(ClassBattleFlowView, ClassReportView 등)들이 여러 step에 따라 다른 UI를 렌더링하도록 재구성되었습니다. 특히 ClassBattleFlowView는 투표, 청취, 의견 작성 단계별로 UI를 분리하고 ClassBattleTranscriptLine을 통해 TTS 자막을 실시간으로 표시하는 기능을 추가했습니다. 새로운 ClassResultInterface를 중심으로 UseCase 의존성이 classResultUseCase, classRoomUseCase, classDebateUseCase 등으로 세분화되었으며, 이에 맞춰 테스트 코드의 의존성 주입 및 스텁 구현도 업데이트되었습니다. 전반적으로 코드 품질은 양호하며, 복잡한 플로우 변화에 대한 테스트 커버리지도 확보되어 있습니다. 한 가지 잠재적 상태 불일치 문제가 확인되었습니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 407b0a4834
ℹ️ 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".
제출 중 화면 이탈 후 서버 진행 상태를 다시 조회하도록 수정합니다. Gemini 리뷰 댓글을 리뷰별 API로 확인하고 조회 지연을 처리합니다.
리뷰 대응
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3b04129157
ℹ️ 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".
클래스 결과의 서버 집계 표시값을 State에서 계산하고 Figma 감사 파일 다운로드를 바로잡습니다. Gemini 리뷰 응답을 검증한 뒤 게시하며 잘린 응답은 한 번 재시도합니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4a1863bd98
ℹ️ 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".
사전 투표 문구의 계산을 State로 모으고 기존 표시 조건과 날짜 형식을 검증합니다.
리뷰 생성 요청에서 429와 일시적 5xx를 제한된 횟수로 재시도합니다.
There was a problem hiding this comment.
클래스 참여 흐름과 결과 화면을 확정 시안에 맞춰 리팩토링하고 신규 기능을 통합한 변경 사항입니다. ClassCoordinator 및 ClassBattleFlowFeature에서 복잡한 내비게이션 및 상태 전환 로직을 개선하고, TTS 청취 화면 및 의견 작성/조회 화면을 새로 도입했습니다. 도메인 유스케이스를 세분화하여 관심사를 분리하고, 관련 테스트 코드를 업데이트하여 변경 사항을 검증한 점이 좋습니다. 코드 컨벤션과 아키텍처 가이드라인을 잘 준수하고 있습니다. 심각한 결함(P1) 또는 주요 결함(P2)은 발견되지 않았습니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3b603b45e4
ℹ️ 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".
좋아요 응답과 목록 재조회 순서가 달라도 서버 상태를 유지합니다. 투표 비율 표시와 잘못된 좋아요 응답 처리를 보강합니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8214ed4365
ℹ️ 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".
수동 스와이프를 네이티브 뒤로가기로 교체하고 화면 제거 시 요청과 오디오를 정리합니다. 재생 및 제출 중 이탈 테스트를 추가합니다.
There was a problem hiding this comment.
클래스 도메인 인터페이스를 리팩토링하고, 복잡한 배틀 플로우 및 리포트 화면을 도입했습니다. 오디오 재생 및 네트워크 요청 취소와 같은 생명주기 관리와 관련된 중요한 수정 사항들이 포함되어 있습니다.
추가 확인 사항 (미검증 항목은 인라인 미게시):
- Projects/App/Sources/Navigation/Class/ClassCoordinator.swift:456 [미검증] 🟠 [P2] Major
ClassBattleFlowFeature에서 사용자의 의견 제출과 같은 네트워크 요청이 진행 중일 때, 사용자가 화면을 이탈하면 해당 요청이 취소되지 않고 서버로 전송될 수 있습니다. 이로 인해 사용자 의도와 다른 서버 상태 변경이나 예기치 않은 오류가 발생할 수 있습니다.ClassCoordinator에서removedBattleFlows처리 시 `CancelID.submit(id
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ed24f6fba8
ℹ️ 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".
의견 수 표시를 State의 행 모델로 옮기고 결과 화면 뒤로 가기를 복구합니다. 화면 이탈 시 대기 중인 재생을 취소한 뒤 일시정지하며 회귀 테스트를 추가합니다.
There was a problem hiding this comment.
클래스 배틀 플로우의 내비게이션, 오디오 재생, 의견 작성 및 결과 조회 기능이 전반적으로 개선되었습니다. Coordinator와 View에서 탭바 표시 여부와 뒤로가기 제스처가 디자인 확정 시안에 따라 수정되었고, 배틀 플로우 내에서 TTS 스크립트 생성, 의견 필터링 및 정렬, 좋아요 기능이 추가되었습니다. 특히 TCA cancellation 및 상태 동기화 로직이 복잡한 상황(예: 좋아요 요청과 목록 재조회 간의 순서 불일치)을 포함하여 견고하게 구현되었음을 테스트 코드를 통해 확인했습니다. ClassReportView에서는 사용자 역할에 따른 결과 페이지 분리 및 AI 인사이트 표시가 추가되었으며, 전반적으로 아키텍처 원칙에 부합하는 리팩토링 및 테스트 개선이 이루어졌습니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ad021e1fd2
ℹ️ 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".
Reducer에서 시나리오 대본 구성 로직을 분리합니다. 화자와 문장 시각, 노드 순서를 고정하는 회귀 테스트를 추가합니다.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cb697f0cbf
ℹ️ 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".
번호 제목과 인용문 대체값을 State에서 구성합니다. 행 순서와 표시값 회귀 테스트를 추가합니다.
There was a problem hiding this comment.
전반적으로 UI 로직 분리, 코드 리팩토링, TTS 기능 확장 및 테스트 커버리지 강화 등 긍정적인 변경 사항이 많습니다. 특히 오디오 재생 및 제출 관련 라이프사이클 관리와 테스트 추가는 중요합니다. 다만, Coordinator의 탭바 가시성 로직과 뷰의 toolbar(.hidden) 적용 방식 간의 불일치, ClassReportView의 불필요한 onClose 제거, ClassBattleFlow의 의견 작성 로직 변경, ClassReportFeature의 인사이트 로딩 메시지, ClassJoinView의 ignoresSafeArea 제거로 인한 레이아웃 변경은 잠재적인 P2 이슈입니다.
추가 확인 사항 (미검증 항목은 인라인 미게시):
- Projects/Feature/Class/Sources/ClassBattleFlow/Reducer/ClassBattleFlowFeature.swift:134 [미검증] 🟠 [P2] Major
shouldShowComposer의 로직이session?.participationCompleted == false && session?.commentRequired == true에서canParticipate && hasPublishedPerspective == false && !hasSubmittedInCurrentSession로 변경되었습니다. 특히 `hasSubmittedInC
다음 대사가 있는 노드의 시작 시각을 기준으로 대본 노출 시간을 계산하고 역전 시각을 정렬합니다. 대본 및 투표 아바타 표시값을 상태 행 모델로 옮기고 회귀 테스트를 추가합니다.
변경 내용
13337:56744의 40pt에 맞췄습니다.종료로 표시합니다.확인된 검증
build-for-testing: 통과.class-api.md및 서버dev와 정적 대조 완료.git diff --check및 Claude 계획·코드 교차 점검: 통과.남은 검증
cgWindowNotFound로 실패해 수정 후 화면의 확정 Figma 시안 대비 픽셀 검증은 완료되지 않았습니다.위 두 항목은 리뷰 중에도 미검증 상태로 취급해 주세요.