Skip to content

Sessions - improve merge skill + a minor fix#307657

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/lovely-rhinoceros
Apr 3, 2026
Merged

Sessions - improve merge skill + a minor fix#307657
lszomoru merged 1 commit intomainfrom
lszomoru/lovely-rhinoceros

Conversation

@lszomoru
Copy link
Copy Markdown
Member

@lszomoru lszomoru commented Apr 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 15:23
@lszomoru lszomoru enabled auto-merge (squash) April 3, 2026 15:23
@lszomoru lszomoru self-assigned this Apr 3, 2026
Copy link
Copy Markdown
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 updates the Sessions built-in merge skill documentation to provide a more explicit, step-by-step merge workflow (including conflict handling and validation), and makes a small observable implementation change in the Sessions Changes view model.

Changes:

  • Expanded merge skill instructions with explicit guidelines, merge/conflict steps, and post-merge validation commands.
  • Updated ChangesViewModel to compute activeSessionRepositoryStateObs using derived(...) instead of derivedOpts(...).

Reviewed changes

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

File Description
src/vs/sessions/skills/merge/SKILL.md Rewrites/expands the merge skill workflow, adding detailed steps for merging via the main worktree and validating the result.
src/vs/sessions/contrib/changes/browser/changesView.ts Adjusts how the active session repository state observable is derived.

@lszomoru lszomoru merged commit 3d70aab into main Apr 3, 2026
23 checks passed
@lszomoru lszomoru deleted the lszomoru/lovely-rhinoceros branch April 3, 2026 16:58
@vs-code-engineering vs-code-engineering bot added this to the 1.115.0 milestone Apr 3, 2026
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