Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Nov 18, 2025

This PR attempts to address Issue #9334. Feedback and guidance are welcome.

Summary

This enhancement automatically creates checkpoints before any context compression (manual or automatic) to prevent users from losing recent messages if they need to revert the compression.

Changes

  • Add checkpoint save before manual context compression in condenseContext method
  • Add checkpoint save before automatic context compression in attemptApiRequest method
  • Use force=true to ensure checkpoint is created even without file changes
  • Use suppressMessage=true for cleaner UI experience

Benefits

  • Users can now revert context compression if needed
  • No data loss when context is compressed
  • Better user experience with ability to undo compression operations
  • Minimal UI disruption with suppressed checkpoint messages

Testing

  • All existing tests pass
  • Checkpoint tests specifically verified to ensure no regression

Fixes #9334


Important

Adds automatic checkpoint creation before context compression in Task.ts to prevent data loss and allow reversion, addressing issue #9334.

  • Behavior:
    • Adds checkpoint creation before manual context compression in condenseContext() in Task.ts.
    • Adds checkpoint creation before automatic context compression in attemptApiRequest() in Task.ts.
    • Uses force=true to ensure checkpoint creation even without file changes.
    • Uses suppressMessage=true for a cleaner UI.
  • Benefits:
    • Allows users to revert context compression if needed.
    • Prevents data loss during context compression.
    • Enhances user experience with the ability to undo compression operations.
  • Testing:
    • All existing tests pass.
    • Checkpoint tests verified to ensure no regression.

This description was created by Ellipsis for bd7175a. You can customize this summary. It will automatically update as commits are pushed.

- Add checkpoint save before manual context compression in condenseContext method
- Add checkpoint save before automatic context compression in attemptApiRequest method
- Use force=true to ensure checkpoint even without file changes
- Use suppressMessage=true for cleaner UI experience

Fixes #9334
@roomote roomote bot requested review from cte, jr and mrubens as code owners November 18, 2025 05:11
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. enhancement New feature or request labels Nov 18, 2025
@roomote
Copy link
Contributor Author

roomote bot commented Nov 18, 2025

Rooviewer Clock   See task on Roo Cloud

Review completed. No issues found.

The implementation correctly addresses issue #9334 by creating checkpoints before context compression operations. The changes are well-integrated with the existing checkpoint system and follow the codebase's patterns.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] Create checkpoints before compressing the context

3 participants