Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Aug 18, 2025

Important

Add abortTask() method to TaskLike interface and Task class, update version to 1.53.0, and add tests for new functionality.

  • Behavior:
    • Add abortTask() method to TaskLike interface in task.ts.
    • Implement abortTask() in Task class in Task.ts, setting abort flag and emitting TaskAborted event.
    • Handle errors during disposal in abortTask() gracefully.
  • Version:
    • Update version to 1.53.0 in package.metadata.json.
  • Tests:
    • Add tests for abortTask() in Task.spec.ts to verify abort flag, event emission, and error handling.

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

@mrubens mrubens requested review from cte and jr as code owners August 18, 2025 06:23
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Aug 18, 2025
Copy link
Contributor

@roomote roomote bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution! This PR improves type safety by adding the abortTask() method to the TaskLike interface and provides comprehensive test coverage. The changes look good to me.

approveAsk(options?: { text?: string; images?: string[] }): void
denyAsk(options?: { text?: string; images?: string[] }): void
submitUserMessage(text: string, images?: string[]): void
abortTask(): void
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good addition! This formalizes the abortTask() method in the interface, improving type safety for consumers of the TaskLike interface.

})
})
})

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent test coverage! These tests thoroughly verify the abort functionality including error handling, interface compliance, and event emission. The test for graceful error handling during disposal is particularly valuable.

@mrubens mrubens merged commit a8aea14 into main Aug 18, 2025
15 checks passed
@mrubens mrubens deleted the roo-code-types-v1.53.0 branch August 18, 2025 06:30
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Aug 18, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants