Conversation
Copilot 코드리뷰 6건 검토 후 반영(4건 반영·1건 부분·1건 반려): - 발주 작성: 새 발주 ID를 기존 목록의 최댓값+1로 생성해 시드/재제출 간 ID 충돌 방지 (Order.kt) — 리뷰의 "시드와 충돌" 지적은 범위상 사실이 아니나 재제출 시 중복 가능성은 실재하여 고유화로 반영 - 스캔 폼: 입고/출고 수량 상한 999 클램프로 비정상 증가 방지 (Scan.kt) - 로그인: 빈 비밀번호 입력 차단 + 오류 필드(사번/비밀번호) 구분 하이라이트 (Login.kt) - 공통: 스크림 탭이 공유하던 단일 MutableInteractionSource 제거, 스크림별 분리(no-indication) (Components.kt, Overlays.kt) - 미사용 import 정리 (Home/Worklog/Scan/Order.kt) - 작업이력 칩 카운트는 30일 요약 기준(목록은 샘플)임을 주석으로 명시 — 디자인 핸드오프 명세를 따르는 의도된 동작이라 로직 변경은 반려 (Worklog.kt) 빌드(:app:assembleDebug) 통과 및 에뮬레이터에서 로그인 비밀번호 게이트 양방향 동작 확인. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…플래그) - data/remote: Net(OkHttp+Retrofit+Gson, Bearer 인터셉터), UiState, SalesApi, DTO - data/repo: SalesOrderRepository (arrivals/branchOrders/receivedByMe/receive) - build: BuildConfig BASE_URL/USE_API(gradle property), INTERNET 권한 - 화면 미배선(USE_API 기본 false) → 앱 동작 무변경. 모델 매핑은 후속 PR.
- ui/state/AsyncStates: LoadingRows/ErrorState(재시도)/EmptyState (디자인 토큰) - Order 화면: USE_API=true면 sales GET sales-orders 로드(주문 단위 카드)+로딩/에러/빈, false면 기존 목업 그대로 - 모델 매핑은 주문 단위로 정직하게(요약 응답에 라인 없음). 작성·라인상세·인증은 후속 - 로컬 assembleDebug GREEN(AS JBR 21)
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
- build.gradle: BBD_USE_API → boolean 리터럴 정규화(임의 문자열 codegen 깨짐 방지) - Net: Retrofit BASE_URL 트레일링 슬래시 정규화(런타임 IllegalArgumentException 방지) - Manifest: cleartext(http) 허용을 디버그 빌드 전용으로 분리(src/debug), 릴리스는 https만 - Repository: 전체 페이지 수집(첫 페이지 누락 방지, 20p 안전캡) + CancellationException 재던짐(structured concurrency 보존) - (Copilot Overlays interactionSource=null 지적은 tapNoRipple 사용+Compose 2026 nullable 오버로드라 무효) - 로컬 assembleDebug GREEN
- 발주 ID: 'PR-2026-' 하드코딩 → 현재 연도 prefix, 해당 prefix 범위에서만 max+1 채번 - 신규 발주 날짜/시각: 고정 '2026-05-22 15:20' → LocalDate/LocalTime.now() - assembleDebug GREEN
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
통합 PR (closes #2, #3, #4)
충돌 해결을 위해
fix/review-reflect(#2)를feat/4에 머지 → #2 · #3 · #4 단일 통합본. base=main.포함
충돌 해결
텍스트 자동 머지는 깨끗했으나 의미 충돌 1건: #2의 import 정리가 지운
Pretendard를 #4OrderScreenApi가 사용 → 미복구 시 통합 main 컴파일 실패. 복구 후assembleDebugGREEN(AS JBR 21)으로 검증.