docs(content): 전체 탭 spacing 검증을 기록한다

This commit is contained in:
2026-06-25 12:23:29 +09:00
parent 7436ac77d9
commit 2e86e21cb7

View File

@@ -564,6 +564,14 @@
- PASS: `./gradlew :app:ktlintCheck` - PASS: `./gradlew :app:ktlintCheck`
- PASS: `git diff --check` - PASS: `git diff --check`
- 참고: `./gradlew :app:mergeDebugResources`는 최초 일반 실행에서 Gradle wrapper lock 파일 접근이 샌드박스에서 차단되어 실패했고, 승인 실행으로 재검증해 성공했다. - 참고: `./gradlew :app:mergeDebugResources`는 최초 일반 실행에서 Gradle wrapper lock 파일 접근이 샌드박스에서 차단되어 실패했고, 승인 실행으로 재검증해 성공했다.
- 2026-06-25 Phase 5 리뷰 finding 3열 grid spacing 수정 및 재검증:
- 수정 완료: `ContentRecyclerItemLayoutParams``ContentGridItemDecoration``isLeftColumn` 기반 2열 전제 계산에서 `columnIndex`/`spanCount` 기반 offset 계산으로 변경했다. 2열은 기존 인접 gap `8dp`를 유지하고, 3열은 1-2열/2-3열 사이 gap이 모두 `8dp`가 되도록 보정했다.
- 회귀 방지: `ContentMainFragmentSourceTest`에 grid spacing이 `spanCount` 기반으로 계산되고 `isLeftColumn`을 사용하지 않는다는 source 계약을 추가했다.
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.main.content.ContentMainFragmentSourceTest"`
- PASS: `./gradlew :app:compileDebugKotlin`
- PASS: `./gradlew :app:ktlintCheck`
- PASS: `git diff --check`
- 참고: `ktlintCheck``.editorconfig disabled_rules` deprecation warning과 Gradle deprecation warning은 기존 경고로 이번 변경과 무관하다.
- 확인: Phase 6 작업(`Task 6.1`, `Task 6.2`, `Task 6.3`)은 모두 `[ ]` 상태를 유지한다. - 확인: Phase 6 작업(`Task 6.1`, `Task 6.2`, `Task 6.3`)은 모두 `[ ]` 상태를 유지한다.
- 2026-06-25 Phase 4~5 day-filter/sort-bar Figma 수정 및 재검증: - 2026-06-25 Phase 4~5 day-filter/sort-bar Figma 수정 및 재검증:
- 수정 완료: `SERIES` 요일 필터를 Figma 기준의 36dp rounded pill 컨테이너와 선택 요일 흰 배경/검은 텍스트 구조로 조정했다. - 수정 완료: `SERIES` 요일 필터를 Figma 기준의 36dp rounded pill 컨테이너와 선택 요일 흰 배경/검은 텍스트 구조로 조정했다.
@@ -608,3 +616,24 @@
- PASS: 디바이스 `2cec640c34017ece`에서 debug 앱 실행 후 콘텐츠 `추천`, `랭킹`, `전체 > 오디오`, `전체 > 시리즈`, sort popup을 실제 화면으로 확인했다. - PASS: 디바이스 `2cec640c34017ece`에서 debug 앱 실행 후 콘텐츠 `추천`, `랭킹`, `전체 > 오디오`, `전체 > 시리즈`, sort popup을 실제 화면으로 확인했다.
- PARTIAL: 수동 검증 환경에서 `GET /api/v2/audio/contents``HTTP 404`를 반환해 실제 API 데이터 기반 3열 카드, 페이징 append, 실데이터 성인 배지 표시는 확인하지 못했다. 관련 로직은 Phase 6.1 단위 테스트와 Phase 6.2 컴파일/리소스 검증으로 대체 확인했다. - PARTIAL: 수동 검증 환경에서 `GET /api/v2/audio/contents``HTTP 404`를 반환해 실제 API 데이터 기반 3열 카드, 페이징 append, 실데이터 성인 배지 표시는 확인하지 못했다. 관련 로직은 Phase 6.1 단위 테스트와 Phase 6.2 컴파일/리소스 검증으로 대체 확인했다.
- 참고: Kotlin LSP는 로컬에 `kotlin-lsp`가 설치되어 있지 않아 이번 Phase 6에서도 `lsp_diagnostics`를 실행하지 못했고, Gradle compile/tests로 대체 검증했다. - 참고: Kotlin LSP는 로컬에 `kotlin-lsp`가 설치되어 있지 않아 이번 Phase 6에서도 `lsp_diagnostics`를 실행하지 못했고, Gradle compile/tests로 대체 검증했다.
- 2026-06-25 Phase 1~5 코드 리뷰 및 점검:
- 리뷰 결과: Phase 1~4의 기존 구조 경계, API/DTO/Repository, 요일/타입 mapper, ViewModel paging/type/sort/day 상태 처리는 PRD와 계획/TASK 문서 기준의 추가 차단 이슈를 발견하지 않았다.
- 리뷰 결과: Phase 5에서 `rvContentAllItems``CONTENT_ALL_GRID_SPAN_COUNT = 3`을 전달하지만, `ContentGridItemDecoration`의 offset 계산이 left column 여부만 사용해 3열의 2번째-3번째 열 사이 간격이 1번째-2번째 열 사이보다 좁아질 수 있는 이슈를 발견했다.
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.main.content.*"`
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.widget.SeriesContentCardViewTest"`
- PASS: `./gradlew :app:mergeDebugResources`
- PASS: `./gradlew :app:compileDebugKotlin`
- PASS: `./gradlew :app:ktlintCheck`
- PASS: `git diff --check`
- 참고: `./gradlew :app:mergeDebugResources`는 최초 일반 실행에서 Gradle wrapper lock 파일 접근이 샌드박스에서 차단되어 실패했고, 승인 실행으로 재검증해 성공했다.
- 2026-06-25 Phase 1~5 spacing 수정 후 코드 리뷰 및 재점검:
- 리뷰 결과: `ContentGridItemDecoration``columnIndex``spanCount` 기반 offset 계산으로 변경되어 2열 기본 grid와 전체 탭 3열 grid 모두 인접 열 간격이 `GRID_ITEM_GAP_DP`로 유지됨을 확인했다.
- 리뷰 결과: 회귀 방지를 위해 `ContentMainFragmentSourceTest``spanCount` 기반 spacing source 계약이 추가되었고, Phase 1~5 범위에서 추가 차단 이슈는 발견하지 않았다.
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.main.content.ContentMainFragmentSourceTest"`
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.main.content.*"`
- PASS: `./gradlew :app:testDebugUnitTest --tests "kr.co.vividnext.sodalive.v2.widget.SeriesContentCardViewTest"`
- PASS: `./gradlew :app:mergeDebugResources`
- PASS: `./gradlew :app:compileDebugKotlin`
- PASS: `./gradlew :app:ktlintCheck`
- PASS: `git diff --check`
- 참고: `./gradlew :app:mergeDebugResources`는 최초 일반 실행에서 Gradle wrapper lock 파일 접근이 샌드박스에서 차단되어 실패했고, 승인 실행으로 재검증해 성공했다.