Fix #5173: Save checkpoints before tool execution instead of after

[roomote]

Summary

Fixes issue #5173 where checkpoints functionality disappeared in Roo Code version 3.20.1.

Problem

The checkpoint system was saving checkpoints AFTER tool execution instead of BEFORE, which meant:

• No initial checkpoint was created before the first modification
• Each checkpoint captured the state AFTER changes, not before
• Users couldn't revert to the state before changes were made

Solution

• Moved checkpoint saving logic to occur before file-modifying tools are executed (lines 411-414)
• Removed old logic that saved checkpoints after tool execution (lines 524-530)
• Identified file-modifying tools: write_to_file, apply_diff, insert_content, search_and_replace

Changes

• Modified src/core/assistant-message/presentAssistantMessage.ts
• Added checkpoint saving before tool execution for file-modifying tools
• Removed checkpoint saving after tool execution

Testing

• Existing checkpoint tests continue to pass
• The fix ensures checkpoints capture workspace state before modifications
• Restores the expected checkpoint functionality for version comparison and rollback

Impact

This restores the checkpoint functionality that users relied on in version 3.19.6, allowing them to:

• Compare versions before and after changes
• Roll back changes when needed
• Have confidence that their work is properly tracked

Closes #5173

---

Important

Move checkpoint saving to occur before file-modifying tools in presentAssistantMessage, restoring expected functionality for version comparison and rollback.

• Behavior:
  • Move checkpoint saving logic to occur before file-modifying tools are executed in presentAssistantMessage.
  • Remove old logic that saved checkpoints after tool execution.
• Tools:
  • Identified file-modifying tools: write_to_file, apply_diff, insert_content, search_and_replace.
• Testing:
  • Existing checkpoint tests continue to pass, ensuring checkpoints capture workspace state before modifications.

^{This description was created by for 5a9c9f0. You can customize this summary. It will automatically update as commits are pushed.}

[delve-auditor]

✅ No security or compliance issues detected. Reviewed everything up to 5a9c9f0.

Security Overview

• 🔎 Scanned files: 1 changed file(s)

Detected Code Changes

The diff is too large to display a summary of code changes.

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

[ellipsis-dev]

Checkpoint is now saved before executing file-modifying tools. Consider wrapping 'await checkpointSave(cline)' in a try/catch block to ensure that a failure in saving a checkpoint doesn't halt tool execution.