Skip to content

Background - include metadata for each session#3231

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/certain-jackal
Jan 28, 2026
Merged

Background - include metadata for each session#3231
lszomoru merged 1 commit intomainfrom
lszomoru/certain-jackal

Conversation

@lszomoru
Copy link
Member

Copilot AI review requested due to automatic review settings January 28, 2026 17:14
@lszomoru lszomoru enabled auto-merge January 28, 2026 17:14
@lszomoru lszomoru self-assigned this Jan 28, 2026
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 28, 2026
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 pull request adds a metadata field to the ChatSessionItem interface to support storing arbitrary session metadata. The implementation populates this field with worktree properties for Copilot CLI sessions.

Changes:

  • Added an optional metadata property to the ChatSessionItem interface in the proposed API
  • Implemented metadata population in the Copilot CLI session provider with branch name, repository path, and worktree path

Reviewed changes

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

File Description
src/extension/vscode.proposed.chatSessionsProvider.d.ts Added optional metadata field to ChatSessionItem interface with JSDoc documentation
src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts Implemented metadata creation with worktree properties in _toChatSessionItem method

@lszomoru lszomoru added this pull request to the merge queue Jan 28, 2026
Merged via the queue into main with commit fe21b9c Jan 28, 2026
25 checks passed
@lszomoru lszomoru deleted the lszomoru/certain-jackal branch January 28, 2026 17:50
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.

2 participants