Harden Windows installer lock recovery#7
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis change adds a bounded retry mechanism to Windows installer logic for handling file-locked binaries. Documentation and PowerShell installation script are updated to clarify that when the target Tempyr binary is temporarily locked, the installer waits and retries multiple times before failing, rather than failing immediately. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
feat: journal stats dashboard (v2 backlog #7)
Summary
cargo installpathtempyr.exeprocess stop logic, but also retry when a brief external file lock causes a lock-related install failuredocs/install.mdVerification
install.ps1with a clean custom install rootinstall.ps1while the installedtempyr.exe --mcpprocess was running and verified the installer stopped the matching process and retried successfullyinstall.ps1while the target binary was briefly locked by another process and verified the installer waited and retried successfullyinstall.ps1with the PowerShell parser and rangit diff --checkSummary by CodeRabbit
Bug Fixes
Documentation