Skip to content

Conversation

@franekp
Copy link
Contributor

@franekp franekp commented Mar 15, 2025

Context

Related issue: #1656

We need this in roospawn to enable resuming tasks which have ended with a question for the user.

Implementation

We use the same initialization as startNewTask, but then use ClineProvider.initClineWithHistoryItem().
One thing which is not clear is whether we should also send { type: "invoke", invoke: "newChat" } to the webview, or some other message?

Screenshots

None.

How to Test

Didn't test yet.

Get in Touch

Discord handle: franciszekpiszcz_15867


Important

Add resumeTask and isTaskInHistory methods to API class and RooCodeAPI interface for task resumption and history checking.

  • New Methods in API class (api.ts):
    • resumeTask(taskId: string): Initializes a task from history and sends a chat button click action to the webview.
    • isTaskInHistory(taskId: string): Checks if a task exists in history, returning a boolean.
  • Interface Updates in roo-code.d.ts:
    • Added resumeTask and isTaskInHistory to RooCodeAPI interface with appropriate documentation.

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

@changeset-bot
Copy link

changeset-bot bot commented Mar 15, 2025

⚠️ No Changeset found

Latest commit: 91376a2

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

@hannesrudolph hannesrudolph linked an issue Mar 17, 2025 that may be closed by this pull request
@hannesrudolph hannesrudolph moved this from New to PR [Pre Approval Review] in Roo Code Roadmap Mar 17, 2025
@wkordalski wkordalski force-pushed the resume-task branch 2 times, most recently from d1828e8 to dd7556c Compare March 19, 2025 15:29
@franekp franekp marked this pull request as ready for review April 1, 2025 15:25
@franekp franekp requested review from cte and mrubens as code owners April 1, 2025 15:25
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. enhancement New feature or request labels Apr 1, 2025
@hannesrudolph
Copy link
Collaborator

@cte Can you please review this when you get a chance?

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 7, 2025
@cte cte merged commit cf74568 into RooCodeInc:main Apr 7, 2025
26 checks passed
@github-project-automation github-project-automation bot moved this from PR [Pre Approval Review] to Done in Roo Code Roadmap Apr 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:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RooCodeAPI is missing a few crucial events and functions

4 participants