Skip to content

Enable all the commenting commands - #8923

Merged
Alex Ross (alexr00) merged 1 commit into
mainfrom
alexr00/valuable-bonobo
Sep 3, 2026
Merged

Enable all the commenting commands#8923
Alex Ross (alexr00) merged 1 commit into
mainfrom
alexr00/valuable-bonobo

Conversation

@alexr00

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 3, 2026 15:15
@alexr00
Alex Ross (alexr00) enabled auto-merge (squash) September 3, 2026 15:15
@alexr00 Alex Ross (alexr00) self-assigned this Sep 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The change is a straightforward set of when-clause updates in package.json with no apparent correctness or schema issues.

Review tier: Lite
Findings: None

What changed in this PR

This PR updates the extension’s contribution points so GitHub PR comment-related commands (keybindings and comment UI context actions) are no longer suppressed in VS Code “Sessions” windows, effectively enabling the full commenting workflow there.

Changes:

  • Removes && !isSessionsWindow gating from the pr.makeSuggestion keybinding.
  • Removes && !isSessionsWindow gating from comment thread/title/comment context menu actions (resolve/unresolve, diff outdated, apply suggestion, apply with Copilot, etc.).
  • Removes && !isSessionsWindow gating from the editor title action for adding a file comment.
File Description
package.json Enables PR commenting-related keybindings and menus in Sessions windows by dropping !isSessionsWindow from relevant when clauses.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@alexr00
Alex Ross (alexr00) merged commit b6cd72a into main Sep 3, 2026
7 checks passed
@alexr00
Alex Ross (alexr00) deleted the alexr00/valuable-bonobo branch September 3, 2026 15:31
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