Handle locked worktree force deletion#7963
Conversation
Co-authored-by: Orca <help@stably.ai>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis change updates the git worktree removal logic to pass the 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Validation
pnpm exec vitest run --config config/vitest.config.ts src/main/git/worktree-list-paths.test.ts --testNamePattern "force-removes a locked worktree"failed withcannot remove a locked working tree... use remove -f -fpnpm exec vitest run --config config/vitest.config.ts src/main/git/worktree-list-paths.test.tspnpm exec vitest run --config config/vitest.config.ts src/main/git/remove-worktree.test.ts src/relay/git-handler-worktree-ops.test.ts src/renderer/src/store/slices/store-cascades.test.tspnpm run typecheckoxlint, react-doctor lint,oxfmt --writeGit cleanup safety
--force --forcebehavior is only used after Orca receives explicit force-delete intent.Gaps
Made with Orca 🐋