From 5a05e696a7ad3f2e8c6dcf6e0e91f0e1fc7474f6 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:28:51 +0800
Subject: [PATCH 1/9] fix: restore double-click to copy OTP and hide account ID
 from row

---
 src/pages/AccountList.vue | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/pages/AccountList.vue b/src/pages/AccountList.vue
index ae192e2..4c2858b 100644
--- a/src/pages/AccountList.vue
+++ b/src/pages/AccountList.vue
@@ -313,6 +313,18 @@ function selectRow(a: ServiceAccount): void {
   account.selectedSid = a.sid
 }
 
+/**
+ * Double-click on a row → select + fetch OTP + copy to clipboard.
+ * Mirrors WPF `lstViewAccount_MouseDoubleClick` which called
+ * `btnGetOtp_Click` on the double-clicked row. Banned rows are
+ * ignored (same guard as {@link selectRow}).
+ */
+function handleRowDblClick(a: ServiceAccount): void {
+  if (!a.is_enable) return
+  account.selectedSid = a.sid
+  void handleGetOtp()
+}
+
 /* --------------- logout --------------- */
 
 async function handleLogout(): Promise<void> {
@@ -2243,6 +2255,7 @@ onBeforeUnmount(() => {
                 }"
                 :data-test="`account-row-${a.sid}`"
                 @click="selectRow(a)"
+                @dblclick="handleRowDblClick(a)"
               >
                 <span
                   class="account-list__row-grip"
@@ -2253,9 +2266,8 @@ onBeforeUnmount(() => {
                 <span class="account-list__row-num">{{ idx + 1 }}</span>
                 <div class="account-list__row-info">
                   <p class="account-list__row-name">{{ a.sname }}</p>
-                  <p class="account-list__row-sub">
-                    <template v-if="a.is_enable">ID: {{ a.sid }}</template>
-                    <template v-else>{{ t('accountList.statusBanned') }}</template>
+                  <p v-if="!a.is_enable" class="account-list__row-sub">
+                    {{ t('accountList.statusBanned') }}
                   </p>
                 </div>
                 <el-dropdown

From 27bad12ac96a7be50ab7ffd4cbebb4c444296c62 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:34:32 +0800
Subject: [PATCH 2/9] style: reduce account row vertical padding for tighter
 row height

---
 src/pages/AccountList.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/pages/AccountList.vue b/src/pages/AccountList.vue
index 4c2858b..4775163 100644
--- a/src/pages/AccountList.vue
+++ b/src/pages/AccountList.vue
@@ -2892,7 +2892,7 @@ onBeforeUnmount(() => {
   display: flex;
   align-items: center;
   gap: 0.625rem;
-  padding: 0.625rem 0.75rem;
+  padding: 0.4375rem 0.75rem;
   border-radius: var(--bf-radius-card);
   cursor: pointer;
   position: relative;

From 5be6b29419adcba303307ca6e2fc1fdf62d9059c Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:36:16 +0800
Subject: [PATCH 3/9] test(AccountList): drop sid assertion and cover dblclick
 OTP flow

---
 tests/unit/pages/AccountList.spec.ts | 49 ++++++++++++++++++++++++++--
 1 file changed, 46 insertions(+), 3 deletions(-)

diff --git a/tests/unit/pages/AccountList.spec.ts b/tests/unit/pages/AccountList.spec.ts
index c44f506..649d23c 100644
--- a/tests/unit/pages/AccountList.spec.ts
+++ b/tests/unit/pages/AccountList.spec.ts
@@ -853,12 +853,12 @@ describe('AccountList page', () => {
     const rows = wrapper.findAll('.account-list__row')
     expect(rows).toHaveLength(3)
     expect(rows[0].text()).toContain('Main Toon')
-    expect(rows[0].text()).toContain('sid-1')
     expect(rows[1].text()).toContain('Mule Account')
     expect(rows[2].text()).toContain('Suspended User')
     /*
-     * The banned row swaps the ID line for the localized
-     * "Disabled" copy — proves the conditional branch fires.
+     * The banned row shows the localized "Disabled" copy as a
+     * subtitle — proves the conditional branch fires. Enabled
+     * rows show only the display name (no ID subtitle).
      */
     expect(rows[2].text()).toContain(i18nMessages['zh-TW'].accountList.statusBanned)
     expect(rows[2].classes()).toContain('account-list__row--banned')
@@ -917,6 +917,49 @@ describe('AccountList page', () => {
     expect(account.selectedSid).toBe('sid-1')
   })
 
+  it('double-clicking an enabled row arms selection and fetches OTP (WPF parity, issue #239)', async () => {
+    /*
+     * WPF `lstViewAccount_MouseDoubleClick` selected the row and fired
+     * `btnGetOtp_Click`, which (with the default autoPaste=off) copies
+     * the OTP to the clipboard. Asserts the full chain so a regression
+     * dropping either side (selection vs. OTP IPC) trips a red test.
+     */
+    vi.mocked(commands.getAccounts).mockReturnValueOnce(ok(POPULATED_LIST))
+    vi.mocked(commands.getOtp).mockReturnValueOnce(ok('OTP-DBL'))
+    const clipboard = installClipboardMock()
+
+    const ctx = buildHarness()
+    const wrapper = await ctx.mountIt()
+    await flushPromises()
+
+    const account = useAccountStore()
+    expect(account.selectedSid).toBeNull()
+
+    await wrapper.get('[data-test="account-row-sid-2"]').trigger('dblclick')
+    await flushPromises()
+
+    expect(account.selectedSid).toBe('sid-2')
+    expect(commands.getOtp).toHaveBeenCalledTimes(1)
+    expect(commands.getOtp).toHaveBeenCalledWith(SECOND_SA)
+    expect(clipboard.writeText).toHaveBeenCalledWith('OTP-DBL')
+    expect(ElMessage.success).toHaveBeenCalledWith(i18nMessages['zh-TW'].GetOtpSuccessAndCopy)
+  })
+
+  it('double-clicking a banned row is a no-op (mirrors the single-click guard)', async () => {
+    vi.mocked(commands.getAccounts).mockReturnValueOnce(ok(POPULATED_LIST))
+
+    const ctx = buildHarness()
+    const wrapper = await ctx.mountIt()
+    await flushPromises()
+
+    const account = useAccountStore()
+    await wrapper.get('[data-test="account-row-sid-3"]').trigger('dblclick')
+    await flushPromises()
+
+    expect(account.selectedSid).toBeNull()
+    expect(commands.getOtp).not.toHaveBeenCalled()
+  })
+
   it('logout: confirm → auth.logout → account.clearSessionData → /login', async () => {
     vi.mocked(commands.getAccounts).mockReturnValueOnce(ok(POPULATED_LIST))
     vi.mocked(commands.logout).mockReturnValueOnce(ok(null))

From c7fcbdfc50084e9f00b22a0209c50160e431467d Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:40:47 +0800
Subject: [PATCH 4/9] style(settings): shrink settings window width to 680px

---
 src/router/index.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/router/index.ts b/src/router/index.ts
index 112d086..7cf5659 100644
--- a/src/router/index.ts
+++ b/src/router/index.ts
@@ -263,7 +263,7 @@ export const routes: RouteRecordRaw[] = [
     meta: {
       titleKey: 'titleBar.settings',
       titleIcon: 'settings',
-      windowWidth: 880,
+      windowWidth: 680,
       windowHeight: 680,
     },
     /*

From 5909dde58585f7486f7a65b04f5536ed76d16d21 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:44:59 +0800
Subject: [PATCH 5/9] chore(gitignore): ignore src-tauri/target-test build
 artifacts

---
 src-tauri/.gitignore | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src-tauri/.gitignore b/src-tauri/.gitignore
index b21bd68..61f45a0 100644
--- a/src-tauri/.gitignore
+++ b/src-tauri/.gitignore
@@ -2,6 +2,10 @@
 # will have compiled files and executables
 /target/
 
+# Local-only Cargo target dir used when running `cargo test --target-dir target-test`
+# to keep test artifacts separate from `cargo tauri dev/build` output.
+/target-test/
+
 # Generated by Tauri
 # will have schema files for capabilities auto-completion
 /gen/schemas

From aa1a7dd22686c0a99b9b2af301de7622dc89a205 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:45:03 +0800
Subject: [PATCH 6/9] chore: sync Cargo.lock with 5.9.4 version bump

---
 src-tauri/Cargo.lock | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src-tauri/Cargo.lock b/src-tauri/Cargo.lock
index 0b8fb98..4cd93f2 100644
--- a/src-tauri/Cargo.lock
+++ b/src-tauri/Cargo.lock
@@ -330,7 +330,7 @@ checksum = "72b3254f16251a8381aa12e40e3c4d2f0199f8c6508fbecb9d91f575e0fbb8c6"
 
 [[package]]
 name = "beanfun"
-version = "5.9.3"
+version = "5.9.4"
 dependencies = [
  "aes",
  "anyhow",

From 466697c6ce4ee6a5c2867cb8b8241ef75741f7b5 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:50:00 +0800
Subject: [PATCH 7/9] i18n(settings): drop GPU usage hint from hardware
 acceleration label

---
 src/locales/en-US.json | 2 +-
 src/locales/zh-CN.json | 2 +-
 src/locales/zh-TW.json | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/locales/en-US.json b/src/locales/en-US.json
index d76c4f8..1a68ee4 100644
--- a/src/locales/en-US.json
+++ b/src/locales/en-US.json
@@ -84,7 +84,7 @@
   "AutoCheckUpdate": "Notify when a new version is available",
   "RunAfterLogin": "Launch game after login",
   "MinimizeToTaskbar": "Minimize to system tray",
-  "DisableHardwareAcceleration": "Disable hardware acceleration (lower GPU usage)",
+  "DisableHardwareAcceleration": "Disable hardware acceleration",
   "MsgRestartForHardwareAccelTitle": "Restart required",
   "MsgRestartForHardwareAccel": "Hardware acceleration settings have changed. Fully exit Beanfun and start it again for the main window to use WPF software rendering.",
   "Game": "Game",
diff --git a/src/locales/zh-CN.json b/src/locales/zh-CN.json
index e39dc31..95514d8 100644
--- a/src/locales/zh-CN.json
+++ b/src/locales/zh-CN.json
@@ -73,7 +73,7 @@
   "AutoCheckUpdate": "有新版本时提示更新",
   "RunAfterLogin": "登录完成后运行游戏",
   "MinimizeToTaskbar": "最小化到通知中心",
-  "DisableHardwareAcceleration": "关闭硬件加速（降低 GPU 使用）",
+  "DisableHardwareAcceleration": "关闭硬件加速",
   "MsgRestartForHardwareAccelTitle": "需要重新启动",
   "MsgRestartForHardwareAccel": "已变更硬件加速相关设置。请完全关闭 Beanfun 后再打开，主窗口才会套用 WPF 软件渲染。",
   "Game": "游戏",
diff --git a/src/locales/zh-TW.json b/src/locales/zh-TW.json
index 23b4cc9..f0d9f41 100644
--- a/src/locales/zh-TW.json
+++ b/src/locales/zh-TW.json
@@ -84,7 +84,7 @@
   "AutoCheckUpdate": "有新版本時提示更新",
   "RunAfterLogin": "登入完成後開啟遊戲",
   "MinimizeToTaskbar": "最小化到通知中心",
-  "DisableHardwareAcceleration": "關閉硬體加速（降低 GPU 使用）",
+  "DisableHardwareAcceleration": "關閉硬體加速",
   "MsgRestartForHardwareAccelTitle": "需要重新啟動",
   "MsgRestartForHardwareAccel": "已變更硬體加速相關設定。請完全關閉 Beanfun 後再開啟，主視窗才會套用 WPF 軟體渲染。",
   "Game": "遊戲",

From 80a4bba40f4c93b09297bd6f99355df70992fb87 Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sat, 25 Apr 2026 11:50:53 +0800
Subject: [PATCH 8/9] style(settings): tweak settings window width from 680px
 to 660px

---
 src/router/index.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/router/index.ts b/src/router/index.ts
index 7cf5659..702be28 100644
--- a/src/router/index.ts
+++ b/src/router/index.ts
@@ -263,7 +263,7 @@ export const routes: RouteRecordRaw[] = [
     meta: {
       titleKey: 'titleBar.settings',
       titleIcon: 'settings',
-      windowWidth: 680,
+      windowWidth: 660,
       windowHeight: 680,
     },
     /*

From 207d077022dcc42845eaaabf89a49ac4ead767db Mon Sep 17 00:00:00 2001
From: YCC3741 <mo0307b1006@gmail.com>
Date: Sun, 26 Apr 2026 16:48:03 +0800
Subject: [PATCH 9/9] fix(AccountList): copy sid on double-click instead of OTP
 (review feedback)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Per @lshw54's review on #244: WPF `lstViewAccount_MouseDoubleClick`
only called `Clipboard.SetText(selected.sid)` -- it never armed the
selection or fetched an OTP. Replace the `handleGetOtp()` chain with
a direct `navigator.clipboard.writeText(a.sid)` + `CopyFinished` /
`CopyFailed` toast pair, and update the dblclick spec to assert the
clipboard write (and the absence of an OTP IPC) instead.

Banned-row guard is preserved; the i18n key `DoubleClickCopy`
("雙擊複製帳號" / "Double-click to copy account") already documents
this WPF parity.

Refs #239
---
 src/pages/AccountList.vue            | 44 +++++++++++++++++++++++-----
 tests/unit/pages/AccountList.spec.ts | 38 ++++++++++++++++--------
 2 files changed, 61 insertions(+), 21 deletions(-)

diff --git a/src/pages/AccountList.vue b/src/pages/AccountList.vue
index 4775163..6fb25c6 100644
--- a/src/pages/AccountList.vue
+++ b/src/pages/AccountList.vue
@@ -314,15 +314,43 @@ function selectRow(a: ServiceAccount): void {
 }
 
 /**
- * Double-click on a row → select + fetch OTP + copy to clipboard.
- * Mirrors WPF `lstViewAccount_MouseDoubleClick` which called
- * `btnGetOtp_Click` on the double-clicked row. Banned rows are
- * ignored (same guard as {@link selectRow}).
- */
-function handleRowDblClick(a: ServiceAccount): void {
+ * Double-click on a row → copy the row's service-account ID (`sid`)
+ * to the clipboard. Mirrors WPF `lstViewAccount_MouseDoubleClick`
+ * (`AccountList.xaml.cs`), whose only side effect was
+ * `Clipboard.SetText(selected.sid)` — paired with the WPF tooltip
+ * resource `DoubleClickCopy` ("雙擊複製帳號 / Double-click to copy
+ * account") which is still shipped via the `DoubleClickCopy` i18n
+ * key today. Banned rows fall through silently (same guard as
+ * {@link selectRow}) so the SPA mirrors WPF's "disabled rows ignore
+ * input" behaviour.
+ *
+ * Selection is intentionally **not** armed here: WPF treats the
+ * double-click as a one-shot copy, not a row-arm; mutating
+ * `selectedSid` would also flip the OTP / Start Game UI on top of
+ * the copy, which the WPF user never saw. Single-click + Enter is
+ * the ergonomic path for OTP fetch (issue #239 fixes the missing
+ * keyboard route too).
+ *
+ * The SPA bypasses the {@link clipboardWriteOtp} helper because that
+ * helper's success / failure toasts use the OTP-specific
+ * `GetOtpSuccessAndCopy` string. A generic `CopyFinished` /
+ * `CopyFailed` toast pair matches the WPF tooltip wording and keeps
+ * the OTP helper single-purpose.
+ */
+async function handleRowDblClick(a: ServiceAccount): Promise<void> {
   if (!a.is_enable) return
-  account.selectedSid = a.sid
-  void handleGetOtp()
+  try {
+    await navigator.clipboard.writeText(a.sid)
+    ElMessage.success(t('CopyFinished'))
+  } catch {
+    /*
+     * `navigator.clipboard.writeText` only rejects on permission /
+     * trust-boundary denial inside our Tauri webview — surface it
+     * so the user knows the buffer wasn't updated (otherwise
+     * they'd silently paste stale content into the launcher).
+     */
+    ElMessage.error(t('CopyFailed'))
+  }
 }
 
 /* --------------- logout --------------- */
diff --git a/tests/unit/pages/AccountList.spec.ts b/tests/unit/pages/AccountList.spec.ts
index 649d23c..98ef2ac 100644
--- a/tests/unit/pages/AccountList.spec.ts
+++ b/tests/unit/pages/AccountList.spec.ts
@@ -917,15 +917,17 @@ describe('AccountList page', () => {
     expect(account.selectedSid).toBe('sid-1')
   })
 
-  it('double-clicking an enabled row arms selection and fetches OTP (WPF parity, issue #239)', async () => {
+  it('double-clicking an enabled row copies sid to clipboard (WPF parity, issue #239)', async () => {
     /*
-     * WPF `lstViewAccount_MouseDoubleClick` selected the row and fired
-     * `btnGetOtp_Click`, which (with the default autoPaste=off) copies
-     * the OTP to the clipboard. Asserts the full chain so a regression
-     * dropping either side (selection vs. OTP IPC) trips a red test.
+     * WPF `lstViewAccount_MouseDoubleClick` only called
+     * `Clipboard.SetText(selected.sid)` — no row-arm, no OTP fetch.
+     * Mirror that here: the dblclick must hit `clipboard.writeText`
+     * with the row's sid and surface a generic `CopyFinished` toast,
+     * without touching `account.selectedSid` or `commands.getOtp`
+     * (those belong to the single-click + Enter path; auto-select
+     * from PR #245 is what arms the initial selection on mount).
      */
     vi.mocked(commands.getAccounts).mockReturnValueOnce(ok(POPULATED_LIST))
-    vi.mocked(commands.getOtp).mockReturnValueOnce(ok('OTP-DBL'))
     const clipboard = installClipboardMock()
 
     const ctx = buildHarness()
@@ -933,31 +935,41 @@ describe('AccountList page', () => {
     await flushPromises()
 
     const account = useAccountStore()
-    expect(account.selectedSid).toBeNull()
+    /*
+     * Snapshot whatever the auto-select from PR #245 armed on
+     * mount; the dblclick on sid-2 must NOT mutate it (otherwise
+     * the OTP / Start Game UI would silently flip to point at the
+     * just-copied row, which the WPF user never saw).
+     */
+    const selectedBeforeDblClick = account.selectedSid
 
     await wrapper.get('[data-test="account-row-sid-2"]').trigger('dblclick')
     await flushPromises()
 
-    expect(account.selectedSid).toBe('sid-2')
-    expect(commands.getOtp).toHaveBeenCalledTimes(1)
-    expect(commands.getOtp).toHaveBeenCalledWith(SECOND_SA)
-    expect(clipboard.writeText).toHaveBeenCalledWith('OTP-DBL')
-    expect(ElMessage.success).toHaveBeenCalledWith(i18nMessages['zh-TW'].GetOtpSuccessAndCopy)
+    expect(clipboard.writeText).toHaveBeenCalledTimes(1)
+    expect(clipboard.writeText).toHaveBeenCalledWith('sid-2')
+    expect(ElMessage.success).toHaveBeenCalledWith(i18nMessages['zh-TW'].CopyFinished)
+    expect(commands.getOtp).not.toHaveBeenCalled()
+    expect(account.selectedSid).toBe(selectedBeforeDblClick)
   })
 
   it('double-clicking a banned row is a no-op (mirrors the single-click guard)', async () => {
     vi.mocked(commands.getAccounts).mockReturnValueOnce(ok(POPULATED_LIST))
+    const clipboard = installClipboardMock()
 
     const ctx = buildHarness()
     const wrapper = await ctx.mountIt()
     await flushPromises()
 
     const account = useAccountStore()
+    const selectedBeforeDblClick = account.selectedSid
+
     await wrapper.get('[data-test="account-row-sid-3"]').trigger('dblclick')
     await flushPromises()
 
-    expect(account.selectedSid).toBeNull()
+    expect(clipboard.writeText).not.toHaveBeenCalled()
     expect(commands.getOtp).not.toHaveBeenCalled()
+    expect(account.selectedSid).toBe(selectedBeforeDblClick)
   })
 
   it('logout: confirm → auth.logout → account.clearSessionData → /login', async () => {