問題
popup.js の toggleRefresh() でボタンのテキスト ('開始' か否か) でstart/stopアクションを判定している。UIの文言変更でバグになるリスクがある。
// 現状(脆弱)
const currentAction = toggleButton.textContent === '開始' ? 'start' : 'stop';
対応方針
状態変数 isRunning を popup.js 側でも持ち、テキストではなく変数でアクションを判定する。
// 改善案
const currentAction = isRunning ? 'stop' : 'start';
checkConnectionAndGetStatus() の戻り値で isRunning を更新し、UIとロジックを分離する。
優先度
🟡 中(保守性の改善)
問題
popup.jsのtoggleRefresh()でボタンのテキスト ('開始'か否か) でstart/stopアクションを判定している。UIの文言変更でバグになるリスクがある。対応方針
状態変数
isRunningをpopup.js側でも持ち、テキストではなく変数でアクションを判定する。checkConnectionAndGetStatus()の戻り値でisRunningを更新し、UIとロジックを分離する。優先度
🟡 中(保守性の改善)