Skip to content

Conversation

@alexr00
Copy link
Member

@alexr00 alexr00 commented Jan 9, 2026

No description provided.

@alexr00 alexr00 requested a review from osortega January 9, 2026 11:21
@alexr00 alexr00 self-assigned this Jan 9, 2026
@alexr00 alexr00 removed the request for review from osortega January 14, 2026 10:04
@alexr00 alexr00 marked this pull request as ready for review January 14, 2026 10:35
Copilot AI review requested due to automatic review settings January 14, 2026 10:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes chat session functionality from the GitHub Pull Requests extension, specifically removing the integration with GitHub Copilot's coding agent chat sessions.

Changes:

  • Removed CopilotRemoteAgentManager chat session provider functionality and related APIs
  • Cleaned up test files and commands associated with chat sessions
  • Removed UI components and settings related to coding agent chat sessions

Reviewed changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/view/treeNodes/workspaceFolderNode.ts Removed CopilotRemoteAgentManager parameter from constructor and related method calls
src/view/treeNodes/categoryNode.ts Removed CopilotRemoteAgentManager parameter from constructor
src/view/reviewsManager.ts Removed CopilotRemoteAgentManager parameter from PullRequestsTreeDataProvider instantiation
src/view/prsTreeDataProvider.ts Removed CopilotRemoteAgentManager dependency and related event handlers
src/view/prStatusDecorationProvider.ts Removed CopilotRemoteAgentManager parameter and updated to use PrsTreeModel events
src/test/view/reviewCommentController.test.ts Removed CopilotRemoteAgentManager instantiation from tests
src/test/view/prsTree.test.ts Removed CopilotRemoteAgentManager and related test setup
src/test/lm/tools/copilotRemoteAgentTool.test.ts Deleted entire test file for removed tool
src/test/issues/issueTodoProvider.test.ts Removed tests for coding agent CodeLens functionality
src/test/github/copilotRemoteAgentUtils.test.ts Deleted entire test file for removed utilities
src/test/github/copilotRemoteAgent.test.ts Deleted entire test file for removed manager
src/lm/tools/tools.ts Removed coding agent tool registration and related parameters
src/lm/tools/copilotRemoteAgentTool.ts Deleted entire tool implementation file
src/lm/tools/activePullRequestTool.ts Removed coding agent session logic from pull request tool
src/issues/issueTodoProvider.ts Removed CodeLens provider implementation
src/issues/issueFeatureRegistrar.ts Removed coding agent TODO command registration
src/github/copilotRemoteAgentUtils.ts Deleted entire utilities file
src/github/copilotRemoteAgent/gitOperationsManager.ts Deleted git operations manager file
src/github/copilotRemoteAgent/chatSessionContentBuilder.ts Deleted chat session content builder file
src/github/copilotRemoteAgent.ts Removed chat session provider implementation, keeping only basic functionality
src/github/copilotApi.ts Removed remote agent job APIs and session management methods
src/github/common.ts Removed chat session related types and utility functions
src/extension.ts Removed chat session provider registration
src/common/settingKeys.ts Removed SHOW_CODE_LENS setting key
src/commands.ts Removed chat session related commands
package.nls.json Removed localization strings for removed features
package.json Removed chat sessions contribution point and related commands

@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 14, 2026
@alexr00 alexr00 enabled auto-merge (squash) January 14, 2026 10:37
@alexr00 alexr00 merged commit e0381e0 into main Jan 14, 2026
6 checks passed
@alexr00 alexr00 deleted the alexr00/like-crane branch January 14, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants