Skip to content

fix(desktop): Windows 卸载前关闭正在运行的 Cindy - #3753

Draft
Stella-xixi wants to merge 1 commit into
makecindy:mainfrom
Stella-xixi:codex/fix-windows-uninstall
Draft

fix(desktop): Windows 卸载前关闭正在运行的 Cindy#3753
Stella-xixi wants to merge 1 commit into
makecindy:mainfrom
Stella-xixi:codex/fix-windows-uninstall

Conversation

@Stella-xixi

Copy link
Copy Markdown
Contributor

这次改了什么

摘要

修复 Windows 上 Cindy 仍在运行时卸载不干净的问题。安装与卸载现在复用同一套进程检测和终止循环;卸载器会在删除应用文件前关闭 Cindy,并确认进程已经退出。静默卸载默认继续关闭进程,不会被新增确认框卡住。

变更类型

  • feat 新功能
  • fix 缺陷修复
  • refactor / perf 重构或性能优化
  • docs / test / chore 文档、测试或工程维护
  • 其他:

范围

  • 关联 Issue / 需求:Fixes Windows 卸载时未先关闭 Cindy,导致卸载不干净 #3748
  • 本 PR 包含:Windows NSIS 安装/卸载进程停止宏,以及对应契约测试
  • 明确不包含:Electron 应用内退出协议、用户数据清理、其他平台安装器改动
  • 用户可见变化:运行中卸载时会先提示并关闭 Cindy,再继续删除应用文件
  • 是否存在 breaking change:无

UI 变化

  • 引用的设计规范:docs/design-rules/DESIGN.md §11 Voice & Content;涉及 Windows NSIS 原生确认框文案,沿用现有简洁、直接并给出下一步动作的口径,不涉及应用内布局、主题或组件

怎么验证的

自动验证

pnpm test:unit:related
结果:通过;Desktop related tests passed

pnpm --filter desktop run --if-present typecheck
结果:通过

pnpm --filter desktop exec eslint src/main/__tests__/installerNsh.test.ts
结果:通过

pnpm check:dco -- --base origin/main
结果:通过;1 commit signed off

手工验证

未执行完整 Windows 安装包的真实卸载流程。

未执行的验证

尚未生成完整 Windows 安装包。合并前需要验证:窗口打开时卸载、托盘中卸载、未运行时卸载、取消提示、静默卸载 /S,以及卸载后无 Cindy 相关进程和文件残留。

风险

风险分类

  • 无已知风险
  • SQLite / migration
  • system prompt
  • 协议兼容
  • 权限 / 安全 / 用户数据
  • 存量插件兼容(批准状态 / 指纹 / manifest 校验 / 安装布局 / 包格式)
  • 原生层 / fingerprint / OTA
  • 跨平台差异
  • 其他:Windows NSIS 卸载生命周期

影响与回滚

  • 影响范围:仅 Windows NSIS 安装器和卸载器;macOS、Linux、应用内退出链及用户数据不受影响
  • 回滚 / 降级方式:回退本 PR,恢复原有 installer.nsh 行为

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名
  • UI 改动已在「UI 变化」注明引用的设计规范章节
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档(本次无需新增文档)
  • 已确认测试结果或说明未执行原因

Signed-off-by: Stella-xixi <2559689615@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows 卸载时未先关闭 Cindy,导致卸载不干净

1 participant