[HenryYannis]

遇到的问题

我对 v1.2.1（main @ 8bb171c）做了一次安全审计，又逐条做了复核。这个 PR 只修复核结论为「属实」或「部分属实」的条目。复核认为基本不属实的 #2，我只加了一个很便宜的防护。

我的改动（按审计编号）

#	改动	Python	PowerShell
1	reg / taskkill / sc / net / schtasks / tasklist（PS 里是 reg.exe、sc.exe、attrib.exe）一律用 System32 绝对路径。Python 用 GetSystemDirectoryW 取路径，不信环境变量；PS 用 [Environment]::SystemDirectory，attrib 也不再经过 cmd.exe /c	✅	✅
3	Word / Excel / PowerPoint / Outlook 只结束当前会话的（Python 按 tasklist 的会话号、PS 按 SessionId 过滤），保存提醒也只针对当前会话。OfficePLUS 自己的进程不按会话过滤，因为服务进程在会话 0	✅	✅
6	Test-Path / Remove-Item / Get-Content 改为 -LiteralPath	—	✅
8	结束 OfficePLUS 进程前核对程序路径：所在文件夹的路径里必须含 OfficePLUS（真机样本的路径是 C:\Program Files\Microsoft OfficePLUS\<版本>\）。读不到路径就不结束，并记进日志；扫描时读不到路径仍然报告（宁可多报）	✅	✅
9	所有 subprocess 都加了 stdin=DEVNULL 和 timeout=60，net stop 加 /y；超时按失败处理，记进日志，不会卡住	✅	—
10	run_nuke 改为先备份注册表，再结束进程、删除服务，和 PS 版顺序一致	✅	（原本就是）
12	服务是否存在改看 sc query 的退出码（0 = 存在，1060 = 不存在），START_PENDING、STOP_PENDING 等中间状态不会再被漏报	✅	（用 Get-Service，不受影响）
13	tasklist 输出改用 csv.reader 解析，某一行格式不对只跳过这一行	✅	—
14	目录规则里的 ProgramFiles 优先取 ProgramW6432。PS 端由 sync_rules.py 生成 $ProgramFiles64	✅	✅
16	备份文件夹或日志文件夹本身是联接/符号链接时拒绝写入：备份按「写入失败」处理，不删任何注册表项；日志不写	✅	✅
17	checkout / setup-python / action-gh-release 固定到完整 commit SHA，并注明版本（v4.4.0 / v5.6.0 / v2.6.2，即当前大版本标签指向的提交）；pyinstaller 固定为 6.22.3；ci.yml、e2e.yml 加了 permissions: contents: read（release.yml 原本就是 contents: write，上传 Release 需要这个权限，没改）	—	—
4	清理完成后按清理返回值和复检结果，分成「清理完成 / 部分完成 / 清理失败」三种提示（信息 / 警告 / 错误弹窗），日志按实际数量写，弹窗里附上注册表备份和清理日志路径；清理或复检出错时也回到界面、恢复按钮，不当作成功	✅（gui.py）	—
15	新增「计划任务」统计卡片（共 5 张）和扫描日志里的计划任务一项，各项加起来等于总数；点「提权」被 UAC 拒绝时弹出「未获得管理员权限」提示，提权出错时弹出错误提示	✅（gui.py）	—
2（低成本防护）	PS 版：残留目录本身是联接/符号链接时，只删链接本身（[IO.Directory]::Delete($dir, $false)），不跑 attrib，也不递归进去	—	✅

其他相关改动：

• 新增 src/remove_officeplus/winutil.py，放这几项共用的小工具。
• AGENTS.md：删掉硬性规则 8「不改 gui.py」，提交流程去掉「每个问题一个分支和一个 PR」，保留「CI 和 E2E 全绿后再合并，合并后删除分支，仓库只保留 main」。这两条不是我对项目的要求。仓库里没有 CLAUDE.md、.cursor、copilot-instructions 等其他代理说明文件，不需要同步。
• 新增 tests/gui_result_check.py（已加进 CI 的 python-test）：用假的弹窗和界面对象检查 GUI 的三种结果提示、备份和日志路径、复检出错不卡住、计划任务卡片、UAC 被拒提示；没有 tkinter / winreg 的系统也能跑。
• tests/e2e/fake_residue.py 里的假进程改放进名字含 OfficePLUS 的临时文件夹，不然新的路径核对会让 E2E 失败。这样也更接近真实安装。

没改的项和原因

• 修复：清理前备份注册表，误删可一键恢复 #5（irm | iex 拉取 main 分支、提权时重新下载）、打包：删除写死本机路径的 spec，exe 加版本信息和 SHA256 校验 #18（国内镜像没有公布 SHA256）：没改。 属于分发和基础设施，不在本次代码修复范围内。
• 修复：Python 版提权重启在 python -m、pip 命令和 exe 下失败 #11（-NoBackup 和「先备份再删除」规则冲突）：没改。 这是用户主动加的开关，要不要去掉需要作者定。
• 修复：PowerShell 提权时执行本地脚本，不再重新下载 #7（AppData\OfficePLUS 等目录名太通用、删除前不核对内容）：本次没处理。 需要先确定特征文件，并补对应的 E2E 哨兵。
• irm | iex 运行结束后不再关掉用户的 PowerShell 窗口 #17 的其余部分（--onefile 改 --onedir、exe 代码签名）：没改。 牵涉发布方式和证书。
• 新增 Windows 端到端测试，修复 CI 一直失败的编码崩溃 #2 的主体（「PS 5.1 递归删除会跟随联接删掉目标内容」）：复核认为基本不属实， 只加了上面那条便宜的防护。

怎么测试的

• python -m py_compile（全部 .py）✅
• python scripts/sync_rules.py --check ✅
• python tests/real_sample_check.py ✅
• python tests/gui_result_check.py（14 项检查）✅
• pyflakes：src、tests、scripts 全部无警告（顺手删掉了 gui.py 里未使用的 import os）
• 在 Linux 上用 mock 跑了 10 个逻辑测试，全部通过：CSV 解析、按会话过滤、进程路径核对、PENDING 状态、超时、net stop /y、先备份后删除的顺序、联接拒绝写入、ProgramW6432
• PowerShell：在 Linux 上的 pwsh 7.4 里解析无错误；PSScriptAnalyzer 的警告和改动前完全一样（规则种类和数量都没变）。另外单独测了 Test-ReparsePoint、Test-OfficePlusProcess，以及删除符号链接后目标目录还在

⚠️ 我没有 Windows 机器，以上改动都没有在 Windows（含 PowerShell 5.1）上实测过。 需要这个 PR 的 CI 和 E2E 在 windows-latest 上全部通过后再合并。下面几处尤其要看 E2E 结果：

1. 按 PID 结束进程、按会话过滤；
2. 假 OfficePLUS.exe 的路径核对；
3. 用绝对路径调用 attrib.exe；
4. 图形界面的改动没有在真实窗口里点过，合并前最好在 Windows 上打开 exe 看一眼 5 张卡片的排版和清理完成弹窗。

这次按审计批次合成了一个 PR，分成 5 个提交（Python / PowerShell / CI / AGENTS.md / 图形界面）。版本号没有改。