Copilot CLI - pass in the branch name when creating a worktree #2600
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the Copilot CLI worktree creation functionality by automatically assigning a branch name to newly created worktrees. The change ensures that isolated worktrees for CLI sessions have unique, timestamped branch names that respect the user's Git branch prefix configuration.
Key Changes
- Added automatic branch name generation when creating worktrees for CLI session isolation
- Branch names follow the format:
<git.branchPrefix>worktree-<timestamp>where timestamp is a filesystem-safe ISO string
| const branchPrefix = workspace.getConfiguration('git').get<string>('branchPrefix') ?? ''; | ||
| const branch = `${branchPrefix}worktree-${new Date().toISOString().replace(/[:.]/g, '-').slice(0, 19)}`; |
There was a problem hiding this comment.
The branch name generation uses new Date().toISOString() which uses UTC time. This could potentially create confusion when multiple worktrees are created in quick succession or when debugging issues, as the timestamp won't reflect the user's local time zone. Consider using a more unique identifier or adding additional entropy (like a random suffix) to ensure uniqueness even if multiple worktrees are created within the same second.
No description provided.