Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Dec 30, 2025

Summary

Removes the Human Relay provider and all related UI/extension plumbing as this provider does not operate with native tool calling.

Changes

  • Remove the human-relay API handler and provider registration
  • Remove the Human Relay webview dialog and settings copy
  • Remove Human Relay message types/commands and related tests/translations

Important

Remove Human Relay provider and all related components, including API handlers, UI elements, message types, and translations.

  • Behavior:
    • Remove Human Relay API handler and provider registration from index.ts.
    • Remove Human Relay webview dialog and settings from App.tsx.
    • Remove Human Relay message types/commands from extension.test.ts and vscode.ts.
  • Configuration:
    • Remove Human Relay from provider-settings.ts and related schemas.
    • Remove Human Relay from ProfileValidator in ProfileValidator.ts.
  • Translations:
    • Remove Human Relay translations from multiple locale files, including locales/ca/humanRelay.json and locales/de/humanRelay.json.

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

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 30, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 30, 2025

Oroocle Clock   See task on Roo Cloud

Review updated. All previously flagged items look resolved in the latest commit.

  • Update e2e expected command list to remove Human Relay commands (test currently expects showHumanRelayDialog, etc.)
Previous reviews

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 Dec 30, 2025
"unregisterHumanRelayCallback",
"handleHumanRelayResponse",

"newTask",
Copy link
Contributor

Choose a reason for hiding this comment

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

apps/vscode-e2e/src/suite/extension.test.ts still expects the Human Relay commands (showHumanRelayDialog, etc). Since those commands were removed from commandIds here, the e2e test will likely fail until the expected command list is updated.

Fix it with Roo Code or mention @roomote and request a fix.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@roomote fix it

Copy link
Contributor

@roomote roomote bot Dec 30, 2025

Choose a reason for hiding this comment

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

Fixaroo Clock   See task on Roo Cloud

Fixed the reported issues. All local checks passed.

View commit | Revert commit

Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

Looks good, as long as you confirmed the behavior for what happens to people who have the human relay provider selected when it goes away (just want to make sure the extension doesn’t crash the way it used to)

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 30, 2025
@hannesrudolph hannesrudolph merged commit c37aa02 into main Dec 30, 2025
13 checks passed
@hannesrudolph hannesrudolph deleted the remove-human-relay branch December 30, 2025 22:38
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Dec 30, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Dec 30, 2025
@roomote roomote bot mentioned this pull request Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants