BUG: Edit tool applies changes despite rejection error

The Edit tool returns an error message indicating the user rejected the change, but the changes are applied anyway.

Steps to reproduce:

1. Call Edit tool with old_string and new_string
2. User rejects the tool use with a comment
3. Edit returns error message: 'The user doesn't want to proceed with this tool use. The tool use was rejected'
4. Reading the file afterwards shows the changes WERE applied

Concrete example from this session:

• Edit was called to change 'line-height: 1' to 'line-height: 1.2' in CSS
• User rejected with a comment about what changed
• System returned rejection error
• File was read afterwards and showed line-height was already 1.2

This is a critical bug because it causes silent code modifications that the user explicitly rejected.

Expected: Changes should NOT be applied if user rejects
Actual: Changes are applied despite rejection error message