Merge remote-tracking branch 'origin/feature/ISSUE-51' into feature/ISSUE-51

# Conflicts:
#	app/src/main/java/com/laseroptek/raman/ui/screens/main/MainViewModel.kt

app/src/main/java/com/laseroptek/raman/const/HzTable_10_10.kt

@@ -9,7 +9,7 @@ package com.laseroptek.raman.const
 
 val HzTable_10_10 = mapOf(
     // Pulse Width = 0.5f
-    Pair(0.5f, 0.5f) to KEY_GREEN,
+    Pair(0.5f, 0.5f) to KEY_BLUE,
     Pair(0.5f, 0.6f) to KEY_BLUE,
     Pair(0.5f, 0.7f) to KEY_BLUE,
     Pair(0.5f, 0.8f) to KEY_BLUE,

app/src/main/java/com/laseroptek/raman/const/HzTable_5_5.kt

@@ -24,7 +24,7 @@ val HzTable_5_5 = mapOf(
     Pair(0.5f,3.4f) to KEY_BLUE,
     Pair(0.5f,3.6f) to KEY_BLUE,
     Pair(0.5f,3.8f) to KEY_BLUE,
-    Pair(0.5f,4.0f) to KEY_BLUE,
+    Pair(0.5f,4.0f) to KEY_YELLOW,
     Pair(0.5f,4.2f) to KEY_YELLOW,
     Pair(0.5f,4.4f) to KEY_YELLOW,
     Pair(0.5f,4.6f) to KEY_YELLOW,

app/src/main/java/com/laseroptek/raman/const/HzTable_7_7.kt

@@ -108,7 +108,7 @@ val HzTable_7_7 = mapOf(
     Pair(1.5f, 8.0f) to KEY_GRAY,
 
     // Pulse Width = 3f
-    Pair(3f, 1.1f) to KEY_GRAY,
+    Pair(3f, 1.1f) to KEY_RED,
     Pair(3f, 1.2f) to KEY_GRAY,
     Pair(3f, 1.3f) to KEY_GRAY,
     Pair(3f, 1.4f) to KEY_GRAY,

app/src/main/java/com/laseroptek/raman/data/model/serial/HandPiece.kt

@@ -1,5 +1,5 @@
 package com.laseroptek.raman.data.model.serial
 
 data class HandPiece(
-    val type:Int = 0  // ascii (00 ~ 99)
+    val type:Int = 1  // ascii (00 ~ 99)
 )

app/src/main/java/com/laseroptek/raman/ui/screens/home/HomeScreen.kt

@@ -304,7 +304,7 @@ fun HomeScreen(
                     pulseType = pulseType,
                     handPieceType = handPiece.type,
                     type = LaserStatusType.REPETITION,
-                    angle = if (pulseType == 0 || repetitionList.isEmpty()) 0f else repetitionAngle,
+                    angle = if (pulseType == 0 || repetitionList.size < 2) 0f else repetitionAngle,
                     onChange = { angle ->
                         if (handPiece.type == 0) {
                             Toast.makeText(
@@ -315,7 +315,7 @@ fun HomeScreen(
                             return@LaserControlView
                         }
 
-                        if (pulseType == 0 || repetitionList.isEmpty()) {
+                        if (pulseType == 0 || repetitionList.size < 2) {
                             //Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show()
                         } else {
                             mainViewModel.onChangeRepetition(angle)
@@ -332,7 +332,7 @@ fun HomeScreen(
                             return@LaserControlView
                         }
 
-                        if (pulseType == 0 || repetitionList.isEmpty()) {
+                        if (pulseType == 0 || repetitionList.size < 2) {
                             //Toast.makeText(context, "Single pulse type (0 Hz)", Toast.LENGTH_SHORT).show()
                         } else {
                             //mainViewModel.onClickRepetition(state)

app/src/main/java/com/laseroptek/raman/ui/screens/home/preset/PresetLoadPopup.kt

@@ -780,25 +780,21 @@ fun PresetLoadPopup(
                                 .noRippleClickable(onClick = {
                                     Timber.d("onClick - Preset Load")
 
-                                    val selectedPreset = presetViewModel.getPreset(selectedPresetIndex)
+                                    val selectedPreset =
-                                    if (selectedPreset == null) {
+                                        presetViewModel.getPreset(selectedPresetIndex)
-                                        Timber.w("onClick - Preset Load: selectedPreset is null. index=$selectedPresetIndex")
+                                    val priority =
-                                        return@noRippleClickable
+                                        selectedPreset?.priority ?: 0
-                                    }
-                                    val priority = selectedPreset.priority
 
                                     Timber.d("onClick - Preset Load ($priority)")
 
-                                    if (priority > 0) {
+//                                    if (priority > 0) {   // TODO : 검증 필요
                                         mainViewModel.setSelectedPresetIndex( priority )
                                         mainViewModel.applyPreset(priority)
-                                    } else {
+                                        presetViewModel.clearPreset()
-                                        // 우선순위가 NONE인 경우, 인덱스 설정 X
+                                        onClick.invoke(false)
-                                        mainViewModel.setSelectedPresetIndex(0)
+//                                    } else {
-                                        mainViewModel.applyPreset(selectedPreset)
+//                                        Timber.d("SKIP - Preset Load ($priority)")
-                                    }
+//                                    }
-                                    presetViewModel.clearPreset()
-                                    onClick.invoke(false)
                                 })
                                 .size(40.px.dp)
                                 .background(Color.Transparent)

app/src/main/java/com/laseroptek/raman/ui/screens/home/preset/PresetViewModel.kt

@@ -28,7 +28,6 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.launch
 import timber.log.Timber
-import kotlin.math.abs
 import javax.inject.Inject
 
 @HiltViewModel
@@ -81,49 +80,6 @@ class PresetViewModel @Inject constructor(
         _repetitionList.value = r
     }
 
-    private data class ResolvedRepetitionContext(
-        val fluence: Float,
-        val repetitionOptions: List<Float>,
-    )
-
-    private fun resolveRepetitionContext(
-        pulseWidth: Float,
-        fluence: Float,
-    ): ResolvedRepetitionContext {
-        val availableFluenceList = energyTable.value.getKey2ListForKey1(pulseWidth)
-        val resolvedFluence = availableFluenceList.minByOrNull { abs(it - fluence) }
-            ?: availableFluenceList.firstOrNull()
-            ?: fluence
-
-        val hzType = hzTable.value.getValue(pulseWidth, resolvedFluence)
-        val repetitionOptions = RepetitionsByColorKey[hzType] ?: RepetitionsByColorKey[KEY_YELLOW].orEmpty()
-
-        if (hzType == null) {
-            Timber.w(
-                "No hzType found for pulseWidth=%s, fluence=%s. Falling back to default repetition list.",
-                pulseWidth,
-                resolvedFluence
-            )
-        }
-
-        setRepetitionList(repetitionOptions)
-        return ResolvedRepetitionContext(
-            fluence = resolvedFluence,
-            repetitionOptions = repetitionOptions,
-        )
-    }
-
-    private fun resolveNearestLowerOrEqualRepetition(
-        currentRepetition: Float,
-        repetitionOptions: List<Float>,
-    ): Float {
-        return repetitionOptions
-            .filter { it <= currentRepetition }
-            .maxOrNull()
-            ?: repetitionOptions.minOrNull()
-            ?: currentRepetition
-    }
-
     fun loadHzTable(handPieceType: Int) {
         Timber.d("loadHzTable($handPieceType)")
 
@@ -295,17 +251,21 @@ class PresetViewModel @Inject constructor(
 
             // 4. fluence update (energyTable)
             val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth)
-            val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
+            val newFluence = newFluenceList.get(0)
 
             // 5. repetition update (hzTable)
-            val repetitionContext = resolveRepetitionContext(newPulseWidth, newFluence)
+            val hzType = hzTable.value.getValue(newPulseWidth, newFluence)
-            Timber.d("Resolved repetition options for handPieceType update: ${repetitionContext.repetitionOptions}")
+            Timber.d("hzType: ${hzType}")
-            val newRepetition = repetitionContext.repetitionOptions.firstOrNull() ?: currentList[index].repetition
+            val list = RepetitionsByColorKey.get(hzType)!!
+            setRepetitionList( list )
+
+            val newRepetitionIndex = 0
+            val newRepetition = list[newRepetitionIndex]
 
             val updatedPreset = currentList[index].copy(
                 handPieceType = newHandPieceType,
                 pulseWidth = newPulseWidth,
-                fluence = repetitionContext.fluence,
+                fluence = newFluence,
                 repetition = newRepetition
             )
             currentList[index] = updatedPreset
@@ -351,18 +311,20 @@ class PresetViewModel @Inject constructor(
 
         // 3. update fluence
         val newFluenceList = energyTable.value.getKey2ListForKey1(newPulseWidth)
-        val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
+        val newFluenceIndex = 0
+        val newFluence = newFluenceList.get(newFluenceIndex)
         
         // 4. update repetition
-        val repetitionContext = resolveRepetitionContext(newPulseWidth, newFluence)
+        val hzType = hzTable.value.getValue(newPulseWidth, newFluence)
-        val newRepetition = resolveNearestLowerOrEqualRepetition(
+        val list = RepetitionsByColorKey[hzType]!!
-            currentRepetition = currentList[index].repetition,
+        setRepetitionList( list )
-            repetitionOptions = repetitionContext.repetitionOptions,
+
-        )
+        val newRepetitionIndex = 0
+        val newRepetition = repetitionList.value[newRepetitionIndex]
 
         val updatedPreset = currentList[index].copy(
             pulseWidth = newPulseWidth,
-            fluence = repetitionContext.fluence,
+            fluence = newFluence,
             repetition = newRepetition
         )
         currentList[index] = updatedPreset
@@ -392,16 +354,17 @@ class PresetViewModel @Inject constructor(
         }
 
         // 3. update repetition (0)
-        val repetitionContext = resolveRepetitionContext(pulseWidth, newFluence)
+        val hzType = hzTable.value.getValue(pulseWidth, newFluence)
-        val newRepetition = resolveNearestLowerOrEqualRepetition(
+        val list = RepetitionsByColorKey[hzType]!!
-            currentRepetition = currentList[index].repetition,
+        setRepetitionList( list )
-            repetitionOptions = repetitionContext.repetitionOptions,
+
-        )
+        val newRepetitionIndex = 0
+        val newRepetition = repetitionList.value[newRepetitionIndex]
 
         // The original code here had a bug: it was updating `pulseWidth = newFluence`
         // It should be `fluence = newFluence`
         val updatedPreset = currentList[index].copy(
-            fluence = repetitionContext.fluence,
+            fluence = newFluence,
             repetition = newRepetition
         )
         currentList[index] = updatedPreset
@@ -422,28 +385,20 @@ class PresetViewModel @Inject constructor(
         loadHzTable(handPieceType)
 
         // update repetition
-        val repetitionContext = resolveRepetitionContext(pulseWidth, fluence)
+        val hzType = hzTable.value.getValue(pulseWidth, fluence)
-        val currentRepetitionList = repetitionContext.repetitionOptions
+        val list = RepetitionsByColorKey[hzType]!!
+        setRepetitionList( list )
 
-        val newRepetitionIndex = currentRepetitionList.indexOf(repetition).takeIf { it != -1 } ?: 0
+        val newRepetitionIndex = repetitionList.value.indexOf(repetition).takeIf { it != -1 } ?: 0
         val newRepetition = if (isUp == UpDownState.Up) {
-            if (newRepetitionIndex < currentRepetitionList.size - 1) {
+            // Original logic: repetitionIndex < PulseDurations.size -1 (Likely a typo, should be Repetitions.size)
-                currentRepetitionList[newRepetitionIndex + 1]
+            if (newRepetitionIndex < repetitionList.value.size - 1) repetitionList.value[newRepetitionIndex + 1] else repetition
-            } else {
-                repetition
-            }
         } else {
-            if (newRepetitionIndex > 0) {
+            // Original logic: repetitionIndex > 1
-                currentRepetitionList[newRepetitionIndex - 1]
+            if (newRepetitionIndex > 0) repetitionList.value[newRepetitionIndex - 1] else repetition
-            } else {
-                repetition
-            }
         }
 
-        val updatedPreset = currentList[index].copy(
+        val updatedPreset = currentList[index].copy(repetition = newRepetition)
-            fluence = repetitionContext.fluence,
-            repetition = newRepetition
-        )
         currentList[index] = updatedPreset
         //_presetList.value = currentList
         setPresetList(currentList)
@@ -494,4 +449,4 @@ class PresetViewModel @Inject constructor(
     init {
         Timber.d("PresetViewModel init")
     }
-}
+}

app/src/main/java/com/laseroptek/raman/ui/screens/main/MainViewModel.kt

@@ -134,7 +134,6 @@ import kotlinx.coroutines.joinAll
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 import timber.log.Timber
-import kotlin.math.abs
 import javax.inject.Inject
 import kotlin.experimental.or
 import kotlin.time.Duration.Companion.milliseconds
@@ -843,9 +842,8 @@ class MainViewModel @Inject constructor(
             // tx DCD_GAS Write (DEFAULT VALUE)
             txPacket(READ_WRITE.WRITE, CMD.DCD_GAS, dcdGas.value.copy(status = 0x50))
 
-            // tx SPRAY_DCD Write (DEFAULT VALUE)
+        // tx SPRAY_DCD Write (DEFAULT VALUE)
-            txPacket(READ_WRITE.WRITE, CMD.SPRAY_DCD, sprayDcd.value.copy(status = 0x41))
+        txPacket(READ_WRITE.WRITE, CMD.SPRAY_DCD, sprayDcd.value.copy(status = 0x41))
-        }
     }
 
     private suspend fun waitUntilSerialReady(
@@ -2082,58 +2080,28 @@ class MainViewModel @Inject constructor(
         Timber.d("preset: ${preset}")
 
         val newPreset = if (preset == null) {
-            val defaultPulseWidth = PulseDurations.first()
             Preset(
                 handPieceType = handPiece.value.type,
                 priority = priority,
-                fluence = energyTable.value.getKey2ListForKey1(defaultPulseWidth).firstOrNull() ?: 0f,
+                fluence = energyTable.value.getKey2ListForKey1(0.5f).first(),
                 repetition = repetitionList.value.first(),
-                pulseWidth = defaultPulseWidth,
+                pulseWidth = PulseDurations.first(),
             )
         } else {
             preset
         }
-        applyPreset(newPreset)
-    }
 
-    fun applyPreset(preset: Preset) {
+        val fluenceList = energyTable.value.getKey2ListForKey1(newPreset.pulseWidth)
-        val newPreset = preset.copy()
 
         val pulseStep = PulseDurations.indexOf(newPreset.pulseWidth).takeIf {it != -1} ?: 0
-        val resolvedPulseWidth = PulseDurations[pulseStep]
+        val fluenceStep = fluenceList.indexOf(newPreset.fluence).takeIf {it != -1} ?: 0
-
+        val repetitionStep = repetitionList.value.indexOf(newPreset.repetition).takeIf {it != -1} ?: 0
-        // 프리셋의 pulseWidth 기준으로 실제 사용 가능한 fluence 목록을 먼저 동기화한다.
-        val newFluenceList = energyTable.value.getKey2ListForKey1(resolvedPulseWidth)
-        if (newFluenceList != fluenceList.value) {
-            setFluenceList(newFluenceList)
-        }
-
-        // 프리셋 fluence가 정확히 없을 수 있으므로, 가장 가까운 값으로 보정한다.
-        val resolvedFluence = newFluenceList.minByOrNull { abs(it - newPreset.fluence) }
-            ?: newFluenceList.firstOrNull()
-            ?: 0f
-        val fluenceStep = newFluenceList.indexOf(resolvedFluence).takeIf { it != -1 } ?: 0
-
-        // 보정된 (pulseWidth, fluence) 조합으로 repetition 테이블을 다시 계산한다.
-        val hzType = hzTable.value.getValue(resolvedPulseWidth, resolvedFluence)
-        val newRepetitionList = RepetitionsByColorKey[hzType] ?: RepetitionsByColorKey[KEY_YELLOW]!!
-        if (newRepetitionList != repetitionList.value) {
-            setRepetitionList(newRepetitionList)
-        }
-        val repetitionStep = newRepetitionList.indexOf(newPreset.repetition).takeIf { it != -1 } ?: 0
 
         Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
 
         val pulseAngle = pulseStep.stepToDegree(totalSteps = PulseDurations.size)
-        val fluenceAngle = if (newFluenceList.isNotEmpty()) {
+        val fluenceAngle = fluenceStep.stepToDegree(totalSteps = fluenceList.size)
-            fluenceStep.stepToDegree(totalSteps = newFluenceList.size)
+        val repetitionAngle = repetitionStep.stepToDegree(totalSteps = repetitionList.value.size)
-        } else {
-            0f
-        }
-        val repetitionAngle = repetitionIndexToAngle(
-            index = repetitionStep,
-            totalSteps = newRepetitionList.size,
-        )
 
         Timber.d("pulseStep: ${pulseStep} fluenceStep: ${fluenceStep} repetitionStep: ${repetitionStep}")
 
@@ -2163,35 +2131,30 @@ class MainViewModel @Inject constructor(
             setFluenceList(newFluenceList)
         }
 
-        // 2-1. Resolve and apply fluence by value (not by old angle/index).
-        val resolvedFluence = newFluenceList.minByOrNull { abs(it - newFluence) } ?: 0f
-        val resolvedFluenceIndex = newFluenceList.indexOf(resolvedFluence).takeIf { it >= 0 } ?: 0
-        if (newFluenceList.isNotEmpty()) {
-            setFluenceAngle(resolvedFluenceIndex.stepToDegree(totalSteps = newFluenceList.size))
-        }
-
         // 3. Safely Update Repetition List (Prevents NullPointerException)
-        val newHzType = hzTable.value.getValue(newPulseDuration, resolvedFluence)
+        val newHzType = hzTable.value.getValue(newPulseDuration, newFluence)
         val newRepetitionList = RepetitionsByColorKey[newHzType] ?: RepetitionsByColorKey[KEY_YELLOW]!!
         if (newRepetitionList != repetitionList.value) {
             setRepetitionList(newRepetitionList)
         }
 
-        // 4. Resolve and apply repetition by value (not by old angle/index).
+        // Smartly preserve Repetition Angle
-        val targetRepetition = currentRepetitionValue ?: 0f
+        // 4. Check if the old repetition value exists in the new list.
-        val resolvedRepetition = newRepetitionList.minByOrNull { abs(it - targetRepetition) } ?: 0f
+        val oldRepetitionIndex = if (currentRepetitionValue != null) {
-        val resolvedRepetitionIndex = newRepetitionList.indexOf(resolvedRepetition).takeIf { it >= 0 } ?: 0
+            newRepetitionList.indexOf(currentRepetitionValue)
-
-        if (newRepetitionList.isNotEmpty()) {
-            val resolvedRepetitionAngle = repetitionIndexToAngle(
-                index = resolvedRepetitionIndex,
-                totalSteps = newRepetitionList.size,
-            )
-            setRepetitionAngle(resolvedRepetitionAngle)
-            Timber.d("Resolved repetition by value: old=$currentRepetitionValue, new=$resolvedRepetition, angle=$resolvedRepetitionAngle")
         } else {
+            -1
+        }
+
+        if (oldRepetitionIndex != -1) {
+            // If the old value exists, set the slider to that position.
+            val preservedAngle = oldRepetitionIndex.stepToDegree(totalSteps = newRepetitionList.size)
+            setRepetitionAngle(preservedAngle)
+            Timber.d("Repetition value $currentRepetitionValue preserved at new angle $preservedAngle.")
+        } else {
+            // If it doesn't exist, THEN reset the slider to the beginning.
             setRepetitionAngle(0f)
-            Timber.d("Repetition list is empty. Resetting repetition angle to 0f.")
+            Timber.d("Repetition value $currentRepetitionValue not supported in new list. Resetting angle.")
         }
 
         // 5. Conditionally reset the fluence angle slider if needed
@@ -2204,7 +2167,7 @@ class MainViewModel @Inject constructor(
         // 6. Any change invalidates the current preset selection.
         setSelectedPresetIndex(0)
 
-        Timber.d("Updated Laser Parameters: pulse=$newPulseDuration, fluence=$resolvedFluence -> newRepListSize=${newRepetitionList.size}")
+        Timber.d("Updated Laser Parameters: pulse=$newPulseDuration, fluence=$newFluence -> newRepListSize=${newRepetitionList.size}")
     }
 
     /**
@@ -2224,16 +2187,13 @@ class MainViewModel @Inject constructor(
         setPulseAngle(newPulseStep.stepToDegree(totalSteps = PulseDurations.size))
 
         val newPulseDuration = PulseDurations[newPulseStep]
-        // Keep the current fluence value when pulse duration changes via slider.
+        // When pulse duration changes via slider, we use the first available fluence for the new list.
-        val currentFluenceStep = fluenceAngle.value.degreeToStep(totalSteps = fluenceList.value.size)
+        val firstFluence = energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull() ?: 0f
-        val currentFluence = fluenceList.value.getOrNull(currentFluenceStep)
-            ?: energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull()
-            ?: 0f
 
-        // Call the centralized helper using the preserved fluence value.
+        // Call the centralized helper, resetting the fluence slider.
         updateLaserParameters(
             newPulseDuration = newPulseDuration,
-            newFluence = currentFluence,
+            newFluence = firstFluence,
         )
     }
 
@@ -2245,17 +2205,12 @@ class MainViewModel @Inject constructor(
             setPulseAngle(newStep.stepToDegree(totalSteps = PulseDurations.size))
 
             val newPulseDuration = PulseDurations[newStep]
-            // Keep the current fluence value when pulse duration changes via slider.
+            val firstFluence = energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull() ?: 0f
-            val currentFluenceStep = fluenceAngle.value.degreeToStep(totalSteps = fluenceList.value.size)
-            val currentFluence = fluenceList.value.getOrNull(currentFluenceStep)
-                ?: energyTable.value.getKey2ListForKey1(newPulseDuration).firstOrNull()
-                ?: 0f
-
 
             // Call the centralized helper, resetting the fluence slider.
             updateLaserParameters(
                 newPulseDuration = newPulseDuration,
-                newFluence = currentFluence,
+                newFluence = firstFluence,
             )
         }
     }
@@ -2298,12 +2253,8 @@ class MainViewModel @Inject constructor(
 
     fun onChangeRepetition(angle: Float) {
         // This logic is simple and has no complex dependencies, so it can remain as is.
-        val normalizedAngle = repetitionIndexToAngle(
+        if (angle != repetitionAngle.value) {
-            index = angle.degreeToStep(totalSteps = repetitionList.value.size),
+            setRepetitionAngle(angle)
-            totalSteps = repetitionList.value.size,
-        )
-        if (normalizedAngle != repetitionAngle.value) {
-            setRepetitionAngle(normalizedAngle)
         }
         setSelectedPresetIndex(0)
     }
@@ -2314,23 +2265,12 @@ class MainViewModel @Inject constructor(
         val newStep = if (state == UpDownState.Up) currentStep + 1 else currentStep - 1
 
         if (newStep in repetitionList.value.indices) {
-            val newRepetitionAngle = repetitionIndexToAngle(
+            val newRepetitionAngle = newStep.stepToDegree(totalSteps = repetitionList.value.size)
-                index = newStep,
-                totalSteps = repetitionList.value.size,
-            )
             setRepetitionAngle(newRepetitionAngle)
             setSelectedPresetIndex(0)
         }
     }
 
-    private fun repetitionIndexToAngle(index: Int, totalSteps: Int): Float {
-        return when {
-            totalSteps <= 0 -> 0f
-            totalSteps == 1 -> 270f
-            else -> index.stepToDegree(totalSteps = totalSteps)
-        }
-    }
-
     /**
      * Perform all heavy I/O in a single background block.
      * This prevents the "Skipped frames" caused by 30+ sequential bridge calls

docs/change_summary_2026-03-10.md

@@ -1,237 +0,0 @@
-# 변경 소스 설명 (2026-03-10)
-
-## 대상
-
-현재 워크스페이스에서 수정된 항목은 아래와 같습니다.
-
-- `app/src/main/java/com/laseroptek/raman/ui/screens/home/preset/PresetViewModel.kt`
-- `app/build.gradle.kts`
-
-실제 로직 변경은 `PresetViewModel.kt`에 있습니다.
-`app/build.gradle.kts`는 `git status`상 수정으로 보이지만, 확인한 텍스트 diff 기준으로는 의미 있는 코드 변경이 보이지 않았습니다. 줄바꿈(CRLF/LF) 같은 형식 변화일 가능성이 큽니다.
-
-## 이 파일이 하는 일
-
-`PresetViewModel.kt`는 프리셋 화면의 상태를 관리하는 코드입니다.
-
-쉽게 말하면 사용자가 화면에서 아래 값을 올리거나 내릴 때:
-
-- 핸드피스 종류
-- 펄스폭 (`pulseWidth`)
-- 에너지 단계 (`fluence`)
-- 반복수 (`repetition`)
-
-그 조합에 맞는 다음 값들을 다시 계산해서 화면에 반영하는 역할입니다.
-
-## 이번 변경의 핵심
-
-핵심은 반복수(`repetition`) 계산을 더 안전하고 더 자연스럽게 만든 것입니다.
-
-이전 방식은 다음 문제가 생길 수 있었습니다.
-
-- `pulseWidth`와 `fluence` 조합이 테이블에 정확히 없으면 적절한 반복수 목록을 못 찾을 수 있음
-- 못 찾았을 때 강제 `!!` 사용으로 앱이 비정상 종료될 가능성이 있음
-- `fluence` 변경 시 다른 필드를 잘못 갱신하는 버그가 있었음
-
-이번 수정에서는 공통 계산 함수가 추가되어, 값이 정확히 일치하지 않아도 가장 가까운 `fluence`를 찾아서 처리하도록 바뀌었습니다.
-
-또한 `pulse duration`이나 `fluence`를 바꿀 때 repetition이 무조건 목록의 첫 값(`1Hz` 등)으로 초기화되지 않도록 규칙이 바뀌었습니다.
-
-새 규칙은 아래와 같습니다.
-
-- 현재 repetition 값 이하인 후보들 중
-- 가장 큰 값을 선택
-- 만약 그런 값이 하나도 없으면 새 목록의 최소값 사용
-
-예를 들어 현재 repetition이 `10Hz`이고, 새 조건에서 가능한 repetition 목록이 `1, 3, 5, 8`이라면 결과는 `8Hz`가 됩니다.
-즉, "현재 값에 최대한 가깝게 유지하되, 새 조건에서 허용되는 값으로 내린다"는 방식입니다.
-
-## 새로 추가된 구조
-
-### 1. `ResolvedRepetitionContext`
-
-```kotlin
-private data class ResolvedRepetitionContext(
-    val fluence: Float,
-    val repetitionOptions: List<Float>,
-)
-```
-
-이 구조는 반복수 계산 결과를 한 번에 묶어서 전달하기 위한 작은 데이터 상자입니다.
-
-- `fluence`: 실제로 확정된 에너지 값
-- `repetitionOptions`: 그 조건에서 선택 가능한 반복수 목록
-
-### 2. `resolveRepetitionContext()`
-
-이 함수가 이번 수정의 핵심입니다.
-
-동작 순서는 아래와 같습니다.
-
-1. 현재 `pulseWidth`에 대해 사용 가능한 `fluence` 목록을 가져옵니다.
-2. 사용자가 가지고 있던 `fluence`와 가장 가까운 실제 값을 찾습니다.
-3. 그 조합으로 `hzType`을 찾습니다.
-4. `hzType`에 맞는 반복수 목록을 가져옵니다.
-5. 만약 `hzType`을 찾지 못하면 기본값(`KEY_YELLOW`) 목록으로 대체합니다.
-6. 최종 반복수 목록을 `setRepetitionList()`로 화면 상태에 반영합니다.
-
-즉, 예전보다 훨씬 덜 깨지도록 방어 로직이 들어갔습니다.
-
-### 3. `resolveNearestLowerOrEqualRepetition()`
-
-이 함수는 repetition 선택 규칙을 담당합니다.
-
-동작은 단순합니다.
-
-1. 새 repetition 목록에서 현재 repetition 이하인 값만 고릅니다.
-2. 그중 가장 큰 값을 선택합니다.
-3. 없다면 새 목록의 최소값을 사용합니다.
-
-이 함수가 추가되면서 `pulse duration` 또는 `fluence` 변경 시 repetition이 불필요하게 `1Hz`로 떨어지지 않게 되었습니다.
-
-## 함수별로 무엇이 바뀌었는가
-
-### 1. `updateHandPieceType()`
-
-핸드피스 종류를 바꾸면:
-
-- 해당 핸드피스에 맞는 테이블을 다시 불러오고
-- 기본 `pulseWidth`를 고른 뒤
-- 가능한 `fluence`를 정하고
-- 그에 맞는 `repetition` 목록을 계산합니다
-
-변경 전에는 반복수 목록을 직접 찾다가 실패 가능성이 있었고,
-변경 후에는 `resolveRepetitionContext()`를 통해 안전하게 계산합니다.
-
-또한 새 반복수 값도 목록의 첫 번째 값을 안전하게 가져오도록 바뀌었습니다.
-
-### 2. `updatePulseWidth()`
-
-펄스폭을 올리거나 내리면:
-
-- 새 `pulseWidth`에 맞는 `fluence`를 다시 정하고
-- 그 조합에 맞는 `repetition` 목록을 다시 계산합니다
-
-여기도 동일하게 공통 함수 사용으로 바뀌어,
-정확히 일치하는 값이 없을 때 현재 값 또는 가장 가까운 값으로 안전하게 처리합니다.
-
-추가로 repetition 선택 방식도 바뀌었습니다.
-
-- 예전: 새 repetition 목록의 첫 번째 값으로 사실상 초기화
-- 현재: 기존 repetition 이하에서 가장 가까운 최대값 선택
-
-예:
-
-- 기존 repetition = `10Hz`
-- 새 목록 = `1, 3, 5, 8`
-- 결과 = `8Hz`
-
-### 3. `updateFluence()`
-
-이 부분은 버그 수정이 포함되어 있어 중요합니다.
-
-기존 코드에는 주석으로도 남아 있듯이,
-원래 `fluence`를 업데이트해야 하는 상황에서 잘못된 필드를 갱신하던 문제가 있었습니다.
-
-수정 후에는:
-
-- `pulseWidth`에 맞는 `fluence` 목록에서 다음/이전 값을 고르고
-- 그 값으로 반복수 목록을 다시 계산한 뒤
-- repetition도 기존 값 이하에서 가장 가까운 최대값으로 다시 선택하고
-- 실제 프리셋에는 `fluence = repetitionContext.fluence`를 저장합니다
-
-즉, 사용자가 에너지 단계를 바꿨을 때 실제 프리셋 데이터도 올바르게 바뀌도록 고친 것입니다.
-이제 `fluence` 변경 시에도 repetition이 불필요하게 최소값으로 초기화되지 않습니다.
-
-### 4. `updateRepetition()`
-
-반복수만 변경할 때도 사실은 현재 `pulseWidth`와 `fluence`에 맞는 반복수 목록이 먼저 계산되어야 합니다.
-
-이번 수정 후에는:
-
-- 현재 조건에 맞는 반복수 목록을 공통 함수로 얻고
-- 그 목록 안에서 위/아래 이동을 수행하고
-- 필요하면 `fluence`도 보정된 값으로 같이 저장합니다
-
-이전 코드보다 현재 조건과 실제 목록이 더 일관되게 맞춰집니다.
-
-## 왜 이 수정이 필요한가
-
-이 코드는 여러 테이블을 조합해서 동작합니다.
-
-- `energyTable`: `(pulseWidth, fluence)` 조합에 대한 에너지 정보
-- `hzTable`: `(pulseWidth, fluence)` 조합에 대한 `hzType`
-- `RepetitionsByColorKey`: `hzType`에 따라 선택 가능한 반복수 목록
-
-문제는 사용자가 가진 값이 테이블의 키와 딱 맞지 않을 수 있다는 점입니다.
-
-예를 들어:
-
-- 현재 `pulseWidth`는 존재하지만
-- `fluence`가 소수점 차이로 정확히 일치하지 않거나
-- 핸드피스 변경 후 이전 값이 새 테이블에 없는 경우
-
-예전 코드는 이런 경우에 취약했습니다.
-이번 수정은 이런 경계 상황에서도 앱이 가능한 범위 안에서 계속 동작하도록 만든 것입니다.
-
-## 코틀린을 모르는 사람 기준으로 문법 설명
-
-### `fun`
-
-함수를 만든다는 뜻입니다.
-
-```kotlin
-fun updateFluence(...)
-```
-
-### `val`
-
-한 번 정하면 다시 바꾸지 않는 값입니다.
-
-```kotlin
-val newFluence = ...
-```
-
-### `data class`
-
-관련 있는 값 몇 개를 묶는 작은 자료형입니다.
-
-```kotlin
-data class ResolvedRepetitionContext(...)
-```
-
-### `copy(...)`
-
-기존 객체를 복사하면서 일부 값만 바꾸는 방식입니다.
-
-```kotlin
-currentList[index].copy(
-    fluence = repetitionContext.fluence,
-    repetition = newRepetition
-)
-```
-
-이 코드는 "기존 프리셋을 복사하되 `fluence`와 `repetition`만 새 값으로 바꾼다"는 뜻입니다.
-
-### `?:`
-
-왼쪽 값이 없으면 오른쪽 값을 대신 사용하라는 뜻입니다.
-
-```kotlin
-val newFluence = newFluenceList.firstOrNull() ?: currentList[index].fluence
-```
-
-즉:
-
-- 목록 첫 값이 있으면 그 값을 쓰고
-- 없으면 현재 저장된 `fluence`를 그대로 씁니다
-
-### `firstOrNull()`
-
-목록의 첫 번째 값을 가져오되, 비어 있으면 오류 대신 `null`을 돌려줍니다.
-
-안전한 코드에서 자주 쓰입니다.
-
-## 한 줄 요약
-
-이번 수정은 프리셋 편집 시 `pulseWidth`, `fluence`, `repetition`의 연결 계산을 더 안전하게 만들고, `pulse duration` 및 `fluence` 변경 시 repetition을 현재 값에 가장 가깝게 유지하도록 개선했으며, `fluence` 변경 시 잘못된 필드가 갱신되던 버그도 함께 바로잡은 변경입니다.