refactor: P2 清理 - 闭环 check_updates 结果回传 + SDK 文档修正 + DeviceResponse 错误留痕

1. service_manager.rs check_plugin_updates: 返回 Vec<(id, cur, new)>,
   调用处通过 Custom { kind: "plugin_updates" } 广播结果,闭环 HTTP
   调用方获取检查结果的链路 (CLAUDE.md:163 第3项, 王浩然风险3 建议)

2. plugin-sdk/src/lib.rs broadcast 文档: 替换误导性的 WifiProvisioned
   示例为 StateChanged (当前真正闭环的消息),并新增 Phase 2 占位消息
   说明,列出 WifiProvisioned/DeviceEvent/CursorVisibility 三个当前无
   生产者的变体,提醒开发者不要为其编写业务逻辑 (CLAUDE.md:163 第1项)

3. src/plugins/device/mod.rs handle_message: DeviceResponse::Error
   转换时补 eprintln 留痕,避免设备层故障完全静默 (CLAUDE.md:163 第4项,
   王浩然风险3 建议: 至少在一处记录 Error 响应)

未处理项:
- 单实例保护 (锁文件/socket 互斥): 需目标机实测锁行为,留给设备端工程师
- unclutter 僵尸 reap: 核查发现 shutdown() 已做 kill+wait+pkill 兜底,
  仅 SIGKILL 极端场景残留,不值得加 SIGCHLD handler (过度工程)
- CursorVisibility 变体: 删除会破坏示例插件和测试,倾向 Phase 2 统一清理

注: Windows 开发环境缺 dbus/pkg-config, 无法本地 cargo check;
改动经 rust-analyzer 类型检查零诊断, 待目标机验证.
This commit is contained in:
2026-07-04 12:27:35 +08:00
parent ab1dd9160c
commit 38dcd21352
3 changed files with 55 additions and 13 deletions

View File

@@ -121,7 +121,10 @@ impl Plugin for DevicePlugin {
let response = match self.backend.handle_command(cmd) {
Ok(resp) => resp,
Err(e) => {
// 将错误转换为 DeviceResponse::Error
// 将错误转换为 DeviceResponse::Error,并记录日志
// (核心静态插件目前无人消费 DeviceResponse故至少在此留痕
// 避免设备层故障完全静默——王浩然风险3 建议)
eprintln!("[DevicePlugin] 命令处理失败: {e}");
crate::core::message::DeviceResponse::Error(e.to_string())
}
};