Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Mar 6, 2025

Context

Implementation

Screenshots

before after

How to Test

Get in Touch


Important

This PR updates ClineProvider to choose the correct checkpoint storage strategy using ShadowCheckpointService, and adds methods for task and branch deletion with corresponding tests.

  • Behavior:
    • ClineProvider now determines checkpoint storage using ShadowCheckpointService.getTaskStorage().
    • Deletes associated shadow repository or branch using ShadowCheckpointService.deleteTask().
  • Services:
    • ShadowCheckpointService:
      • Adds getTaskStorage() to determine storage type (task or workspace).
      • Adds deleteTask() to remove task storage.
      • Adds deleteBranch() to handle branch deletion.
  • Tests:
    • Adds tests for getTaskStorage() and deleteTask() in ShadowCheckpointService.test.ts.

This description was created by Ellipsis for f685932. It will automatically update as commits are pushed.

@cte cte requested a review from mrubens as a code owner March 6, 2025 19:18
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 6, 2025
@changeset-bot
Copy link

changeset-bot bot commented Mar 6, 2025

⚠️ No Changeset found

Latest commit: f685932

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the Enhancement New feature or request label Mar 6, 2025
@cte cte force-pushed the cte/resume-task-checkpoints-config branch from 13641e4 to 1901a0d Compare March 6, 2025 19:23
@hannesrudolph hannesrudolph moved this from New to PR [Greenlit] in Roo Code Roadmap Mar 6, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 6, 2025
@cte cte merged commit 0a19fb5 into main Mar 7, 2025
11 checks passed
@cte cte deleted the cte/resume-task-checkpoints-config branch March 7, 2025 05:31
@github-project-automation github-project-automation bot moved this from PR [Greenlit] to Done in Roo Code Roadmap Mar 7, 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 lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants