fix: 修复QGA资源泄漏、strip CSI逻辑缺陷、atomic race等8项问题

[TonyTsang81]

修复内容

BLOCKING

• QGA resize try-finally 确保 agent release（资源泄漏）
• stripEchoedCsi partial match 修复（重复扫描）

MAJOR

• 统一 CSI strip 逻辑，删除 JS 重复代码
• atomic TSFN load 一次消除 race window
• port mapping filter 明确 null/undefined 检查

MINOR

• socket unlink 加 hilog.warn
• VncPage pendingSurface 初始化改 -1

验证

• Build: ✅ BUILD SUCCESSFUL
• 设备运行: ✅ Ubuntu 26.04 LTS 正常启动

🤖 Generated with Claude Code

[TonyTsang81]

PR #167 最终审查报告

PR: harmoninux/HiSH#167
规模: 44 files, +2706 / −459
审查日期: 2026-06-26（最终轮）

---

上一轮 3 个 BLOCKING — 全部修复 ✅

问题	修复方式
CSI partial match	新增 csi_partial_buf + csi_partial_len 静态缓冲区，拼接跨 poll 边界的残缺序列
TSFN 双重 load	auto cb = on_shutdown_callback.load(...) 单次 load
background 硬编码	改为 this.customBg

---

🆕 本轮新增功能

LauncherPage（启动固件选择页）

pages/LauncherPage.ets — 336 行，全新页面

流程：

打开 App → EntryAbility 检查 showLauncher
  ├── true  → 加载 LauncherPage → 显示 VM 卡片列表 → 3 秒倒计时自动启动上次的
  │             ├── 点其他 VM → 取消倒计时，立即启动
  │             └── 点"跳过" → 手动选择
  └── false → 直接加载 Index → 启动默认 VM（和原来一样）

亮点：

• main_pages.json 注册了新路由 ✅
• 设置页有 Toggle 开关控制 showLauncher ✅
• lastEmulatorId 记录上次启动，倒计时默认选它 ✅
• doStartVm 逻辑复用了 EntryAbility 的 VM 启动代码（包括性能预设、VNC 端口预检）✅

小建议：

• emulators.length <= 1 时直接跳过 Launcher 是对的，但如果用户只有 1 个 VM 但想看看，可以改为 <= 0 才跳过。不过这个属于偏好，可改可不改。

---

SnippetsPanel（命令面板）

components/SnippetsPanel.ets — 309 行，全新组件

功能：

• 半屏面板 + 下拉把手
• 搜索过滤 (label + command 双重匹配)
• 收藏 (starred) 排序
• 添加/编辑/删除命令（弹窗编辑）
• 长按删除、点一下发送

数据模型：

interface Snippet {
  id: string
  label: string      // "更新软件包"
  command: string    // "apt update -y && apt upgrade -y\n"
  starred: boolean
  sortOrder: number
}

存储在 Emulator.snippets 里，默认 6 条预设命令。

集成：

• WebTerminal 右下角浮动 ⚡ 按钮 → 弹起 SnippetsPanel
• 点击命令 → this.sendInput(cmd) → 面板自动收起
• 所有 CRUD 操作直接写回 AppStorage.emulators

小建议：

• onToggleStar 回调传入了但 SnippetsPanel 内部没有对应的 UI 触发点。如果要支持点击 ⭐ 切换收藏，需要在列表项的 Row 里加一个可点击的 Star 图标。目前只能通过编辑来变 starred。（如果暂时不需要这个功能，回调可以删掉避免死代码）

---

耗电优化

startVm.ets 两行改动：

参数	改前	改后
tb-size	2048	512
poll-max-ns	2000000	500000

---

Preferences 批量写入 + debounce

appOption.ets 重构：

• 类名 appOption → AppOption（大写），所有字段加 readonly
• 新增 showLauncher、lastEmulatorId 等键
• savePreference 改为 Map 收集 + 50ms debounce + 批量 flush
• Preferences 实例缓存，避免重复 getPreferences

---

⚠️ 新发现

1. LauncherPage 与 EntryAbility 的 VM 启动逻辑重复

LauncherPage.doStartVm() 和 EntryAbility.startDefaultEmulator() 是两套几乎一样的代码（VNC 端口预检、性能预设、端口过滤）。如果未来改 VM 启动参数，需要改两个地方。

建议：把 doStartVm 抽到 lib/startVm.ets 里的一个共享函数，LauncherPage 和 EntryAbility 都调它。

2. Snippets 编辑时 starred 不可改

编辑弹窗只有 label 和 command 两个输入框，没办法切换 starred 状态。如果要让用户能收藏/取消收藏，需要在编辑弹窗加一个 Checkbox。

---

✅ 做得好的地方

改动	评价
3 个 BLOCKING 全部修复	质量高，尤其 CSI buffer 拼接方案
LauncherPage 路由设计	EntryAbility 判断 showLauncher 决定首屏，干净
Snippets 数据跟 Emulator 走	不引入新的持久化层，架构简洁
耗电两行参数	影响小风险低，先试了再说
Preferences debounce	50ms 批量写入避免高频 IO
DEFAULT_SNIPPETS 预设 6 条	开箱即用

---

总评

类别	数量	状态
🔴 BLOCKING	0	✅ 全部修复
🟡 可改进	2	代码重复 + star UI
🟢 MINOR	0	—

结论：PR 可以提请上游 review 了。核心并发修复正确，新增功能（启动页 + Snippets + 耗电优化）设计合理，代码量和侵入度匹配。代码重复和 star UI 这两个可以在后续迭代中处理。

[hackeris]

master也支持了“字体”相关的功能，目前有冲突。另外，这个PR能否拆成多个，每个只解决一项具体的问题。

现在这个PR有点大，内容很多，而且有冲突，review和合并都非常困难

[TonyTsang81]

明白的，我这边也是在使用过程中发现的问题和痛点，然后就顺手让claude改一下，没有想太多。这些都是在一天内改的，现在拆可能有点麻烦。