Skip to content

Conversation

@TylerLeonhardt
Copy link
Member

We don't want to step on toes.

We don't want to step on toes.
Copilot AI review requested due to automatic review settings January 28, 2026 21:54
@TylerLeonhardt TylerLeonhardt 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 PR temporarily disables the Claude terminal feature (/terminal slash command) pending legal review. The feature allowed users to create a terminal session with the Claude CLI configured to use GitHub Copilot Chat endpoints.

Changes:

  • Commented out the self-registration of the terminal slash command
  • Removed the command from package.json contributions
  • Updated documentation to indicate the feature is temporarily disabled
  • Removed associated localization strings

Reviewed changes

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

Show a summary per file
File Description
src/extension/agents/claude/vscode-node/slashCommands/terminalCommand.ts Commented out registerClaudeSlashCommand() call and removed import of registration function; added TODO comment explaining why
src/extension/agents/claude/vscode-node/slashCommands/index.ts Commented out import of terminalCommand module with TODO comment
src/extension/agents/claude/AGENTS.md Updated documentation to note the /terminal command is temporarily disabled pending legal review
package.nls.json Removed localized string for the terminal command
package.json Removed command contribution and slash command declaration from VS Code package manifest

@TylerLeonhardt TylerLeonhardt added this pull request to the merge queue Jan 28, 2026
Merged via the queue into main with commit c82171a Jan 28, 2026
25 checks passed
@TylerLeonhardt TylerLeonhardt deleted the tyler/curious-lemming branch January 28, 2026 22:44
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.

4 participants