Copilot CLI - do not open repository to compute worktree statistics

[lszomoru]

• getRepository() waits until the initial repository discovery is complete
• getRepository() now takes an option to open the repository if is not found
• Computing worktree statistics takes advantage of the new option to not open a repository if not found

[Copilot]

Pull request overview

This PR optimizes the Copilot CLI worktree statistics computation by adding an option to avoid unnecessarily opening git repositories. The getRepository() method now accepts an optional forceOpen parameter (defaulting to true for backward compatibility) to control whether repositories should be opened if not already discovered. The worktree statistics computation uses forceOpen: false to avoid the overhead of opening repositories that may not yet be discovered.

Key changes:

• Add optional forceOpen parameter to IGitService.getRepository() with default value of true
• Ensure getRepository() waits for initial repository discovery to complete before checking opened repositories
• Refactor worktree statistics computation to use the git service method instead of directly accessing the git extension API

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/platform/git/common/gitService.ts	Add optional forceOpen parameter to getRepository() interface
src/platform/git/vscode/gitServiceImpl.ts	Implement forceOpen parameter logic and ensure initialization is complete before querying repositories
src/platform/test/node/simulationWorkspaceServices.ts	Update test mock signature to match new interface
src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts	Use forceOpen: false when computing worktree statistics and refactor to use git service methods

[Copilot]

The diffIndexWithHEADShortStats method is being called with change.uri, but the implementation in gitServiceImpl.ts (line 220-227) doesn't pass the file path to the underlying repository method. The repository's diffIndexWithHEADShortStats(path?: string) method expects an optional path parameter (a string), but the service implementation calls it without any arguments.

This means the method will return stats for the entire repository instead of per-file stats. You should either:

1. Update the service method signature to accept a URI and extract uri.fsPath to pass to the repository method, or
2. Call the method differently to get per-file statistics.

The old code called gitRepository.diffIndexWithHEADShortStats(change.uri.fsPath) directly, which was correct.

Suggested change

	const fileStats = await this.gitService.diffIndexWithHEADShortStats(change.uri);
	const fileStats = await this.gitService.diffIndexWithHEADShortStats(change.uri.fsPath);