From 88094afe061d2d4ecaeaecdf613276c03dcd961d Mon Sep 17 00:00:00 2001
From: root <root@debian5825u>
Date: Tue, 19 May 2026 05:39:23 +0900
Subject: [PATCH] fix: auto-select firmware after upload, guard flash button
 without selection

- Auto-select uploaded firmware immediately after upload completes
- Auto-select single firmware on list load (no manual click needed)
- Hide flash button and show warning when no firmware is selected
- Re-apply manifest on list reload to avoid stale state
- Restore warning banner on firmware delete

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 frontend/index.html | 43 +++++++++++++++++++++++++------------------
 frontend/js/app.js  | 22 ++++++++++++++++++----
 2 files changed, 43 insertions(+), 22 deletions(-)

diff --git a/frontend/index.html b/frontend/index.html
index bb365c6..be2f845 100644
--- a/frontend/index.html
+++ b/frontend/index.html
@@ -262,26 +262,33 @@
       <div style="border-top:1px solid var(--border);padding-top:16px;margin-top:4px;">
         <div class="status-row" style="margin-bottom:12px;">
           <span>💾</span>
-          <span id="selected-fw-info" style="font-size:14px;">선택된 펌웨어 없음 — 위 목록에서 선택하세요</span>
+          <span id="selected-fw-info" style="font-size:14px;">선택된 펌웨어 없음 — 위 목록에서 클릭하여 선택하세요</span>
         </div>
 
-        <!--
-          esp-web-install-button 웹 컴포넌트
-          manifest 속성에 백엔드 manifest URL을 동적으로 설정
-          펌웨어 선택 시 selectFirmware() 함수가 manifest를 업데이트
-        -->
-        <esp-web-install-button id="esp-install-btn">
-          <button
-            slot="activate"
-            class="btn btn-primary needs-serial"
-            style="width:100%;justify-content:center;padding:12px;"
-          >
-            ⚡ 플래시 실행
-          </button>
-          <span slot="unsupported" style="color:var(--danger);font-size:13px;">
-            ⚠ 이 브라우저는 Web Serial을 지원하지 않습니다 (Chrome/Edge 필요)
-          </span>
-        </esp-web-install-button>
+        <!-- 펌웨어 미선택 경고 -->
+        <div id="no-fw-warning" class="alert alert-warn" style="margin-bottom:12px;">
+          <span>⚠</span>
+          <div style="font-size:13px;">
+            <strong>펌웨어를 먼저 선택하세요.</strong>
+            위 목록에서 플래시할 펌웨어를 클릭하면 이 버튼이 활성화됩니다.
+          </div>
+        </div>
+
+        <!-- 플래시 버튼 (펌웨어 선택 후 표시) -->
+        <div id="flash-btn-wrap" style="display:none;">
+          <esp-web-install-button id="esp-install-btn">
+            <button
+              slot="activate"
+              class="btn btn-primary needs-serial"
+              style="width:100%;justify-content:center;padding:12px;"
+            >
+              ⚡ 플래시 실행
+            </button>
+            <span slot="unsupported" style="color:var(--danger);font-size:13px;">
+              ⚠ 이 브라우저는 Web Serial을 지원하지 않습니다 (Chrome/Edge 필요)
+            </span>
+          </esp-web-install-button>
+        </div>
 
         <div class="alert alert-warn" style="margin-top:14px;">
           <span>⚠</span>
diff --git a/frontend/js/app.js b/frontend/js/app.js
index 26bd558..2ee2312 100644
--- a/frontend/js/app.js
+++ b/frontend/js/app.js
@@ -288,6 +288,7 @@ uploadForm.addEventListener('submit', async e => {
     });
 
     await loadFirmwareList();
+    selectFirmware(data.id);   // 업로드 직후 자동 선택
     switchTab('tab-flash');
   } catch (err) {
     progressPct.textContent = `✗ ${err.message}`;
@@ -324,6 +325,8 @@ async function loadFirmwareList() {
     const resp = await fetch(`${API}/api/firmware`);
     state.firmwareList = await resp.json();
     renderFwList();
+    // 목록 로드 후 자동 선택 적용 (선택된 것이 있으면 manifest 재설정)
+    if (state.selectedFwId) selectFirmware(state.selectedFwId);
   } catch (err) {
     log(`✗ 펌웨어 목록 로드 실패: ${err.message}`, 'err');
   }
@@ -340,6 +343,11 @@ function renderFwList() {
     return;
   }
 
+  // 선택된 펌웨어가 없고 하나만 있으면 자동 선택
+  if (!state.selectedFwId && state.firmwareList.length === 1) {
+    state.selectedFwId = state.firmwareList[0].id;
+  }
+
   fwListEl.innerHTML = '';
   state.firmwareList.forEach(fw => {
     const el = document.createElement('div');
@@ -379,10 +387,14 @@ function selectFirmware(id) {
 
   const manifestUrl = `${location.origin}/api/firmware/${id}/manifest`;
   installBtn.setAttribute('manifest', manifestUrl);
-  $('#selected-fw-info').textContent = `선택됨: ${state.firmwareList.find(f=>f.id===id)?.name}`;
 
-  log(`✓ 플래시 대상 선택: ${state.firmwareList.find(f=>f.id===id)?.name}`, 'ok');
-  log(`  Manifest: ${manifestUrl}`, 'info');
+  const fw = state.firmwareList.find(f => f.id === id);
+  $('#selected-fw-info').textContent = `선택됨: ${fw?.name}`;
+  $('#flash-btn-wrap').style.display = 'block';
+  $('#no-fw-warning').style.display  = 'none';
+
+  log(`✓ 플래시 대상 선택: ${fw?.name}`, 'ok');
+  log(`  Manifest URL: ${manifestUrl}`, 'info');
 }
 
 async function deleteFirmware(id) {
@@ -392,7 +404,9 @@ async function deleteFirmware(id) {
     if (state.selectedFwId === id) {
       state.selectedFwId = null;
       installBtn.removeAttribute('manifest');
-      $('#selected-fw-info').textContent = '선택된 펌웨어 없음';
+      $('#selected-fw-info').textContent = '선택된 펌웨어 없음 — 위 목록에서 클릭하여 선택하세요';
+      $('#flash-btn-wrap').style.display  = 'none';
+      $('#no-fw-warning').style.display   = 'block';
     }
     await loadFirmwareList();
     log(`✓ 펌웨어 삭제 완료`, 'ok');