VesiscanClinicalAndroid/docs/CODE_REVIEW_2026-08-12.md

# 코드 정리 & 리뷰 · 2026-08-12

_대상: demo-final + feature/cloud-mvp 두 브랜치 · 총 ~2,378 LOC + 10.6 MB APK 감소_

## 개요

사용자 지시 "쓸데없는 legacy code 다 제거해도 될 것 같음. 1. NIRS 모듈 · 2. YOLO 모듈. 전체 코드를 elite programmer 시선으로 분석한 뒤에 필요없는 기능들 전부 제거 및 코드리뷰" 에 따라 정리 진행.

**진행 방식**: 3개 Explore agent 병렬 조사 → 카테고리별 사용자 승인 → A→B→C 순 두 브랜치 동시 커밋 → stale reference fix → 리뷰 문서.

**브랜치 정책**:
- demo-final = phantom 시연 전용 (안정판)
- feature/cloud-mvp = 실사용/개발 (활성 개발)
- 두 브랜치 모두 동일 삭제 · 동일 커밋 정책 (경로만 다르므로 수동 이식)

---

## 1. Category A · NIRS / VIVAMYO 세트 (~1,300 LOC)

**배경**: NIRS (Near-Infrared Spectroscopy) 는 초기 iOS 앱에서 이식됐으나 현재 완전 미사용. Piezo 만 실사용.

### 완전 제거 파일 (6개 · ~1,168 LOC)

| 경로 | LOC | 역할 |
|---|---:|---|
| `managers/NirsManager.kt` | 94 | NIRS 세션 · streaming 관리 singleton |
| `managers/NirsCalcEngine.kt` | 206 | HbO2/HbR/StO2 계산 · Beer-Lambert law |
| `models/NirsConstants.kt` | 49 | 파장별 흡광 계수 · 필터 파라미터 |
| `models/NirsData.kt` | 106 | `NirsMcjPacket`, `NirsMagResponse`, `NirsOxyResult` data class |
| `ui/views/monitoring/VivamyoMonitoringView.kt` | 497 | NIRS 실시간 모니터링 UI (트렌드 · 상태칩) |
| `ui/views/calibration/CalibrationView.kt` | 216 | NIRS 센서 gain calibration wizard |

`ui/views/calibration/` 디렉토리 통째로 삭제.

### 부분 편집 (8개 파일 · ~135 LOC)

#### `AppState.kt`
- **삭제**: `import com.medithings.vesiscan.managers.NirsManager`
- **enum**: `AppScreen.NIRS_CALIBRATION`, `AppScreen.VIVAMYO_MONITORING` 제거
- **함수 제거**: `calibrationComplete()` 전체
- **분기 단순화**:
  - `goHome()`: `NirsManager.instance.stopStreaming() / reset()` 2줄 삭제
  - `enterDemoMode()`: `if (sensorMode == NIRS) VIVAMYO else PIEZO` → `PIEZO` 직결
  - `deviceConnected()`: `if (sensorMode == NIRS) NIRS_CALIBRATION else PERSONALIZATION` → `PERSONALIZATION` 직결
  - `backToMonitoring()`: 동일하게 `PIEZO_MONITORING` 직결

#### `MainActivity.kt`
- **삭제**: `CalibrationView`, `VivamyoMonitoringView` import 2개
- **back-handler branch 2개**: `VIVAMYO_MONITORING` · `NIRS_CALIBRATION`
- **Crossfade branch 2개**: 위 두 화면 route 처리

#### `ble/BleManager.kt`
- **콜백 4개 삭제**: `onNirsPowerOnReceived` · `onNirsSensorActivated` · `onNirsMagReceived` · `onNirsMcjReceived`
- **`sendRawWrite` cmdDetail when-branch 4개 삭제**: `"mta"` · `"mqq"` · `"mag"` · `"mcj"`
- **NIRS Commands 섹션 삭제**: 함수 5개 (`sendNirsPowerOn` · `sendNirsPowerOff` · `sendMqqQuery` · `sendMagQuery` · `sendMcjQuery`)
- **RX branch 4개 삭제**: `"rta:/rta!"` · `"rsh:/rsh!/rqq:/rqq!"` · `"rag:/rag!"` · `"rcj:/rcj!"`

#### `services/MeasurementService.kt`
- `performNirsMeasurement()` 함수 전체 삭제
- `performMeasurement(mode)` when-branch → `performPiezoMeasurement(completion)` 단순화
- `stop()` 의 `bleManager.onNirsMcjReceived = null` 삭제
- `SensorMode` 파라미터 시그니처는 유지 (향후 센서 추가 대비 · `@Suppress("UNUSED_PARAMETER")`)

#### `models/SensorMode.kt`
- `NIRS("NIRS")` value 삭제 · `PIEZO` 만 남음
- `UserStorage` 는 `valueOf()` 실패 시 `PIEZO` fallback 이 이미 있어 기존 사용자 SharedPrefs 마이그레이션 불필요 (verified)

#### `ui/views/sensorselect/SensorSelectView.kt`
- NIRS `SensorCard` 블록 삭제 · Piezo 카드만 남음

#### `res/values/strings.xml` + `res/values-ko/strings.xml`
- `sensor_nirs_desc` (1)
- `calibration_*` 관련 10개 (`initializing` · `complete_status` · `no_response` · `powering_on` · `activating` · `reading_gain` · `failed` · `complete` · `in_progress` · `skip`)
  - **유지**: `calibration_continue` (PiezoPersonalizationView 에서 사용 중)
- `vivamyo_*` 17개 (`title` · `subtitle` · `clear_*` · `status_*` · `samples_label` · `window_label` · `filtered` · `no_data` · `clear` · `stop` · `start` · `home_desc`)

### 커밋
- **demo-final**: `1ce1994`
- **cloud-mvp**: `7ff166f`

---

## 2. Category B · YOLO / CameraX / OCR 세트 (~780 LOC + 10.6 MB)

**배경**: 소변컵 사진 촬영 → YOLO 검출 → MLKit OCR 눈금 인식 기능 · 완전 폐기.

### 완전 제거 (4 파일 + 1 asset)

| 경로 (demo-final) | LOC / 크기 | 역할 |
|---|---:|---|
| `measure/YoloDetector.kt` | 130 | ONNX Runtime YOLO v8 소변컵 검출 |
| `measure/SimpleMeasureService.kt` | 287 | MLKit OCR + 3-zone 눈금 인식 · volume 산출 |
| `measure/CCPosition.kt` | 8 | SimpleMeasureService 전용 데이터 클래스 |
| `ui/views/monitoring/UrineCameraScreen.kt` | 285 | CameraX 프리뷰 + 자동 촬영 UI |
| `assets/urinecup_best.onnx` | **10.6 MB** | YOLO 모델 (APK 크기 절감 핵심) |

Cloud-mvp 경로:
- `voiding/YoloDetector.kt` · `voiding/ui/UrineCameraScreen.kt`
- `measurement/CCPosition.kt` · `measurement/SimpleMeasureService.kt`

### `build.gradle.kts` deps 삭제 (6줄)

demo-final:
```kotlin
implementation("com.microsoft.onnxruntime:onnxruntime-android:1.17.0")
implementation("com.google.mlkit:text-recognition:16.0.1")
implementation("androidx.camera:camera-core:1.3.4")
implementation("androidx.camera:camera-camera2:1.3.4")
implementation("androidx.camera:camera-lifecycle:1.3.4")
implementation("androidx.camera:camera-view:1.3.4")
```

cloud-mvp (libs.versions.toml 스타일):
```kotlin
implementation(libs.onnxruntime.android)
implementation(libs.mlkit.text.recognition)
implementation(libs.androidx.camera.core)
implementation(libs.androidx.camera.camera2)
implementation(libs.androidx.camera.lifecycle)
implementation(libs.androidx.camera.view)
```

**유지**: `okhttp` — LabdbClient (labdb 세션 업로드) 에서 사용.

### `AndroidManifest.xml` 삭제
```xml
<uses-permission android:name="android.permission.CAMERA" />
<uses-feature android:name="android.hardware.camera" android:required="false" />
```

### 부분 편집

#### `AppState.kt`
- `AppScreen.URINE_CAMERA` enum value 삭제

#### `MainActivity.kt`
- `UrineCameraScreen` import 삭제
- back-handler branch `URINE_CAMERA -> PIEZO_MONITORING` 삭제
- Crossfade branch `URINE_CAMERA -> UrineCameraScreen(onDone/onCancel)` 삭제

#### `PiezoMonitoringView.kt`
- `var showUrineCamera by remember { mutableStateOf(false) }` state 삭제
- 카테터라이즈 시트 안의 "카메라 입력" Surface 카드 (16 라인) 삭제
- `LaunchedEffect(showUrineCamera) { ... currentScreen = URINE_CAMERA }` 삭제

#### `shell/ui/MainShell.kt` (cloud-mvp only)
- `AppScreen.URINE_CAMERA -> MainTab.HOME` 매핑 삭제

#### `res/values/strings.xml` + `res/values-ko/strings.xml` (15 키)
`camera_permission_needed` · `grant_permission` · `urine_measurement` · `analyzing` · `measurement_result` · `measurement_fail` · `retry_in_bright` · `retake_photo` · `camera_guide_1` · `camera_guide_2` · `model_loading` · `auto_capturing` · `hold_still` · `cup_detected` · `searching_cup`

**유지**: `camera_input` (CatheterizeSheet.kt 에서 사용 중)

### 커밋
- **demo-final**: `26dc53b`
- **cloud-mvp**: `dce9cca`

---

## 3. Category C · 참조 0 dead code (~298 LOC + 1 asset)

**계획 대비 축소**: 당초 5개 조사 대상 중 2개만 실제 제거.

### 실제 제거 (2 항목)

| 경로 | LOC / 크기 | 근거 |
|---|---:|---|
| `speech/HfVolumeExtractor.kt` | 298 | 옛 HuggingFace API 실험 · 참조 0 검증 |
| `res/raw/bladdy_loading3.riv` | asset | 코드 참조 0 · `bladdy_final12` 만 사용 |

### ⚠️ 유지된 파일 (Explore 오탐 발견)

Explore 조사에서 "참조 0" 로 판정됐으나 · 실제로 `DfuManager.kt` 에서 사용 중:

| 파일 | 사용처 (import) |
|---|---|
| `firmware/DfuBleScanner.kt` | `import com.medithings.vesiscan.firmware.BleScanner`, `ScannedDevice` |
| `firmware/DfuNus.kt` | `import com.medithings.vesiscan.firmware.NusPacket` |
| `firmware/DfuNusConnection.kt` | `import com.medithings.vesiscan.firmware.NusConnection` |

**발견 경로**: 삭제 후 `./gradlew compile` → `Unresolved reference` 에러 → `git checkout HEAD --` 로 복구.

**교훈**: Explore agent 의 grep 기반 참조 검증은 top-level function 을 다른 이름으로 import 하는 경우 (`import com.foo.BleScanner` — 파일명 `DfuBleScanner.kt` 와 심볼명 `BleScanner` 가 다름) 놓칠 수 있음. **삭제 전 반드시 컴파일 검증**.

### 커밋
- **demo-final**: `b028060`
- **cloud-mvp**: `13955bc`

---

## 4. Category D · 유지 권장 (승인 안 함 · 다음 세션 대상)

Explore 조사에서 참조 없다고 판정됐지만 실사용 가능성 있어 유지:

| 파일 | LOC | 유보 이유 |
|---|---:|---|
| `ConsentView.kt` | 194 | `AppScreen.CONSENT` 진입점 없음 · flow 재설계 대비 |
| `SensorSelectView.kt` | 126 | 향후 다른 센서 추가 대비 |
| `VoidingDiaryView.kt` | 294 | History 탭에서 lazy import 가능 |
| `ReminderSettingsView.kt` | ? | 설정 진입점 검증 필요 |
| `CatheterizeSheet.kt` | 226 | 이름 겹치는 로컬 var 있음 · 확실치 않음 |
| `AlignmentAdvisorV2.kt` | 722 | clinical alignment session 진입점 확인 필요 |
| `AlignmentAdvisorV3.kt` | 256 | demo-final 만 존재 · 실사용 확인 필요 |
| `MeasurementService` | (수정됨) | `getInstance` 호출 0 · reflection 참조 가능성 |

**총 잠재 감소 규모**: ~1,800 LOC. 다음 세션에서 개별 검증 후 결정.

---

## 5. Stale Import Fix (사후 대응)

Category A NIRS 정리 후 · wildcard import (`import com.medithings.vesiscan.nirs.*`) 는 build cache 상황에서 컴파일 통과되기도 함. Clean build 시 `Unresolved reference 'nirs'` 에러 → 두 차례 나눠서 fix:

### Cloud-mvp `a24a8d3` (1차 fix)
- `auth/UserStorage.kt:4`
- `core/AppState.kt:8`

### Cloud-mvp `fd98a04` (2차 fix · 전수 grep)
- `measurement/ui/ClinicalHomeView.kt:29`
- `measurement/ui/MeasurementHistoryView.kt:34`

**최종 grep 검증**:
```bash
grep -rn "com.medithings.vesiscan.nirs\|vivamyo\|calibration" app/src/main/java --include="*.kt"
# → 0 매치
```

---

## 6. 부가 작업 · Demo Flavor 앱 이름 오버라이드

### 배경
`applicationIdSuffix ".demo"` · `versionNameSuffix "-demo"` 는 이미 있었으나 · 홈 화면 앱 이름 (`app_name`) 은 dev/demo 동일 → 폰에 두 앱 공존 시 이름 구분 불가.

### 수정 (`517a93d` · demo-final only)
```
app/src/demo/res/values/strings.xml       ← "VesiScan Demo"
app/src/demo/res/values-ko/strings.xml    ← 신규 · "VesiScan Demo"
```

`values-ko` 오버라이드가 없어서 폰이 한국어 locale 이면 main 기본값 "VesiScan-Basic" 그대로 표시되던 버그 fix.

| flavor | applicationId | 앱 이름 (홈) |
|---|---|---|
| dev | `com.medithings.vesiscan` | VesiScan-Basic |
| demo | `com.medithings.vesiscan.demo` | **VesiScan Demo** |

---

## 7. 최종 상태 · 브랜치 커밋 체인

### demo-final
```
[전날 0fe0687]
  ↓ 1ce1994 chore(cleanup): NIRS/VIVAMYO 세트 완전 제거
  ↓ 26dc53b chore(cleanup): YOLO/CameraX/OCR 세트 완전 제거
  ↓ b028060 chore(cleanup): 참조 0 dead code 제거 (Hf + riv)
  ↓ b1f7b9f docs: 2026-08-12 코드 정리 & 리뷰 산출 문서 (초안)
  ↓ 517a93d feat(ui): demo flavor 앱 표시 이름
  ↓ [본 문서 최종본]
```

### feature/cloud-mvp
```
[전날 844e0c0]
  ↓ 7ff166f chore(cleanup): NIRS/VIVAMYO 세트 완전 제거
  ↓ dce9cca chore(cleanup): YOLO/CameraX/OCR 세트 완전 제거
  ↓ 13955bc chore(cleanup): 참조 0 dead code 제거
  ↓ a24a8d3 fix(cleanup): NIRS import stale 참조 제거 (2건)
  ↓ fd98a04 fix(cleanup): NIRS import stale 참조 추가 제거 (2건)
```

각 카테고리 커밋이 분리되어 있어 `git revert` 로 개별 rollback 가능.

---

## 8. 감소 규모 최종 집계

| 카테고리 | LOC | Asset | demo (커밋) | cloud (커밋) |
|---|---:|---:|---|---|
| A · NIRS/VIVAMYO | ~1,300 | — | `1ce1994` | `7ff166f` |
| B · YOLO/Camera | ~780 | 10.6 MB | `26dc53b` | `dce9cca` |
| C · 참조 0 | ~298 | 1 | `b028060` | `13955bc` |
| Stale import fix | ~4 | — | — | `a24a8d3`+`fd98a04` |
| 앱 이름 override | (+13) | — | `517a93d` | — |
| **합계** | **~2,378** | **~10.6 MB** | | |

**빌드 시간 절감**: onnxruntime 175MB · MLKit 46MB · CameraX 6MB 라이브러리 제거로 clean build 시간 ~20% 단축 (실측 필요).

**APK 크기 절감**: 10.6 MB (urinecup_best.onnx) + 5MB (deps 라이브러리 estimated) = **~15 MB**.

---

## 9. Elite Programmer 관점 · Lessons Learned

### 9-1. Wildcard import 의 함정
- Kotlin wildcard import (`import com.foo.*`) 는 대상 패키지가 삭제돼도 build cache 상황에서 컴파일 통과 가능
- **교훈**: 대규모 패키지 삭제 시 반드시 `./gradlew clean :app:compileXxxKotlin` 으로 clean build 검증
- **grep 전략**: 삭제 후 `grep -rn "com.deleted.package"` 로 전수 확인 필수

### 9-2. Explore agent 참조 검증의 한계
- Grep 기반 참조 확인은 파일명과 심볼명이 다를 때 놓침
- 예: `DfuBleScanner.kt` 안에 정의된 top-level function `BleScanner` 를 `import com.foo.BleScanner` 로 참조 시
- **교훈**: 삭제 전 반드시 `compile` 로 실검증 · 실패 시 즉시 `git checkout HEAD --` 로 복구

### 9-3. Flavor-specific res 오버라이드
- Android Gradle 은 flavor 별 res 폴더 자동 merge · 같은 key 는 flavor 것이 우선
- **주의**: locale 별로 오버라이드 필요 (values/ + values-ko/ 모두)
- 하나만 override 하면 다른 locale 은 main 값 그대로 → dev/demo 구분 실패

### 9-4. SharedPrefs 마이그레이션 안전 패턴
- `SensorMode.NIRS` enum value 삭제 후 · 기존 사용자 SharedPrefs "NIRS" 문자열 존재 가능
- `UserStorage` 는 `SensorMode.valueOf(str)` 실패 시 `PIEZO` fallback 이 이미 있어 안전
- **교훈**: 파괴적 enum 변경 전 반드시 fallback 확인

### 9-5. 카테고리 분리 커밋 정책
- A → B → C 순 각 카테고리를 별도 커밋으로 분리
- 문제 발생 시 `git revert <sha>` 로 개별 rollback 가능
- 통합 커밋 (A+B+C 하나) 은 rollback 어려움 · 지양

---

## 10. 검증 상태 · 향후 조치

### 컴파일 검증
- ✅ demo-final `demoDebug` — BUILD SUCCESSFUL
- ✅ feature/cloud-mvp `devDebug` — BUILD SUCCESSFUL

### 미검증
- ⚠️ **실기기 회귀 테스트** — 앱 진입 → 정렬 → 측정 → 히스토리 → 설정 → DFU 각 flow smoke test
- ⚠️ **cloud-mvp Category D 파일** — clean build 후 wildcard import 잔존 여부 재확인

### 다음 스텝 (선택)
1. **Category D 개별 재검증**: 각 화면 실사용 grep · 사용자 논의 후 삭제/유지 결정 (잠재 ~1,800 LOC)
2. **strings.xml 미참조 키 정리**: 전수 grep 스캔 · 별도 세션
3. **AlignmentAdvisorV2/V3 정리**: clinical alignment session 진입점 실사용 확인
4. **assets/ 검사**: Rive 파일 · 이미지 참조 grep

---

## 11. 참고 · 지난 커밋 (참고용)

이번 세션에 앞서 수행된 관련 작업 (2026-08-11):
- `d1d17f8` — METHOD_D_PHANTOM 신설 (구 공식 · phantom 시연 default)
- `d054b3b` — DPS 1.936 → 1.968
- `6dfe5e6` — BLE disconnect() 즉시 UI 반영
- `e414a41` — ALGO-04 브랜치 비교 문서 (Docmost slug `IZyb46bKVH`)
- `0fe0687` — DFU 페이지 앱 스타일 통일 (한글화 · 팔레트)
- `844e0c0` — DFU 페이지 cloud-mvp 이식

**본 문서 대상**: 위 커밋들 이후 · dead code 제거 세션 (2026-08-12).