fix: make ffmpeg audio repair verifiable and recoverable - #127
fix: make ffmpeg audio repair verifiable and recoverable#127PristineStream wants to merge 4 commits into
Conversation
74cc473 to
915ef63
Compare
|
感谢提交,这次重构增加了备份、哈希检查和
另外建议处理:
修复后我再重新 review,非常感谢。 |
|
感谢提交,之前反馈的三个问题已经修复,新增的 11 个测试也都通过了。 不过 Windows 当前还有一个阻塞问题:新版 VS Code 的实际程序文件可能位于安装目录下的版本子目录。例如我的 VS Code 1.130.0: C:\Users\ylwgg\AppData\Local\Programs\Microsoft VS Code\1b6a188127\resources\app\package.json 当前 find_installation() 返回 Microsoft VS Code 根目录,package_path() 随后固定查找根目录下的 resources/app/package.json,实际运行 Error: VS Code package.json was not found at ...\Microsoft VS Code\resources\app\package.json 旧脚本原本包含扫描安装目录下一级子目录的兼容逻辑。请恢复该逻辑,并确保找到版本目录后,package.json 和 ffmpeg.dll 都基于同一个版本目录处理。建议增加一个 Windows 版本子目录布局测试。 |
变更内容
reaplace-ffmpeg.py的替换流程:--check只读诊断模式VSCODE_INSTALLATION指定非标准安装路径code可能指向其他编辑器根因
实际复现环境为 macOS arm64、VS Code 1.130.0、Electron 42.6.0、TouchFish 17.9.0。用户曾运行旧脚本,但 VS Code 更新后重新写入了自带的精简 FFmpeg。旧脚本只输出
replace done,没有哈希校验、备份或更新覆盖提示,因此用户无法判断当前安装是否仍然有效。匹配 Electron 42.6.0 的媒体库安装并重新签名后,TouchFish 视频声音恢复正常。
用户影响
用户现在可以在修改编辑器前安全检查当前 FFmpeg 是否匹配;替换失败时不会丢失原媒体库。编辑器升级导致问题复发时,README 也提供了明确的判断和恢复步骤。
验证
--help命令通过--check检出不匹配且不修改文件git diff --check通过注意
macOS 的 ad-hoc 签名会替换编辑器的官方应用签名,README 已明确提示风险。重新安装官方编辑器可恢复原始文件和签名。