Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Jan 15, 2026

Reverts #10720


Important

Reverts E2E test setup by removing README and modifying test configurations and logic across multiple test files.

  • Revert E2E Test Setup:
    • Removes README.md from apps/vscode-e2e.
    • Modifies test setup and teardown in index.ts to remove multi-model testing logic.
    • Adjusts test logic in subtasks.test.ts, apply-diff.test.ts, execute-command.test.ts, list-files.test.ts, read-file.test.ts, search-files.test.ts, use-mcp-tool.test.ts, and write-to-file.test.ts to reflect reverted changes.
  • Test Execution:
    • Skips tests in execute-command.test.ts, list-files.test.ts, read-file.test.ts, search-files.test.ts, use-mcp-tool.test.ts, and write-to-file.test.ts by default.
    • Simplifies test logic by removing complex setup and execution paths.
  • Miscellaneous:
    • Cleans up temporary files and directories after test execution.
    • Ensures tests are non-recursive and handle errors gracefully.

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

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Jan 15, 2026
@roomote
Copy link
Contributor

roomote bot commented Jan 15, 2026

Oroocle Clock   See task on Roo Cloud

Reviewed. No issues flagged; this looks like a clean revert of #10720.

  • No issues found

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants