Remove most of CopilotRemoteAgentManager

[alexr00]

No description provided.

[Copilot]

Pull request overview

This PR trims down CopilotRemoteAgentManager so it primarily acts as a thin container for wiring up Copilot PR watching, and removes now-unneeded test scaffolding.

Changes:

• Removed most of the Copilot remote agent manager logic (availability/assignability/repo selection/Copilot API init), leaving watcher registration.
• Cleaned up a test file by removing an unused CopilotRemoteAgentManager import and mock.

Show a summary per file

File	Description
src/test/issues/issueTodoProvider.test.ts	Removes unused Copilot manager test mock/import.
src/github/copilotRemoteAgent.ts	Removes most CopilotRemoteAgentManager logic and related imports, retaining PR watcher wiring + SessionIdForPr.

Copilot's findings

Comments suppressed due to low confidence (1)

src/test/issues/issueTodoProvider.test.ts:12

• This test file currently only patches issueUtil.isComment in before/after, but contains no it(...) test cases or assertions. That means it doesn’t actually verify any behavior and can be misleading (and may mask regressions if the setup starts throwing). Consider either adding at least one concrete test that exercises IssueTodoProvider behavior, or removing this file/setup entirely if it’s no longer needed.

describe('IssueTodoProvider', function () {
	// Mock isComment
	// We don't have a real 'vscode.TextDocument' in these tests, which
	// causes 'vscode.languages.getTokenInformationAtPosition' to throw.

• Files reviewed: 2/2 changed files
• Comments generated: 1