Skip to content

switch to copilot icon for cli#308831

Merged
justschen merged 1 commit into
mainfrom
justin/litleo
Apr 9, 2026
Merged

switch to copilot icon for cli#308831
justschen merged 1 commit into
mainfrom
justin/litleo

Conversation

@justschen

@justschen justschen commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings April 9, 2026 17:09

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.

Pull request overview

This PR updates the iconography used to represent the Copilot CLI / background agent provider so it uses the Copilot icon consistently across UI surfaces and tests.

Changes:

  • Switch the Background/Copilot CLI provider icon from worktree to copilot in the agent sessions provider icon mapping.
  • Update the Copilot CLI harness descriptor icon to use Codicon.copilot.
  • Update the Copilot extension’s provider contribution icon and adjust related unit tests.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/test/browser/agentSessions/agentSessionViewModel.test.ts Updates expectations so Background provider uses Codicon.copilot.
src/vs/workbench/contrib/chat/common/customizationHarnessService.ts Changes the CLI harness icon to Codicon.copilot.
src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.ts Maps Background (copilotcli) provider icon to Codicon.copilot.
extensions/copilot/package.json Updates the Copilot CLI provider contribution icon to $(copilot).

Copilot's findings

  • Files reviewed: 4/4 changed files
  • Comments generated: 0

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 1ec46de1 Current: 511f71c1

Changed (2)

chat/aiCustomizations/aiCustomizationManagementEditor/CliHarness/Dark
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/CliHarness/Light
Before After
before after

@justschen justschen enabled auto-merge (squash) April 9, 2026 17:14
@justschen justschen merged commit 35a23cd into main Apr 9, 2026
30 checks passed
@justschen justschen deleted the justin/litleo branch April 9, 2026 17:33
@vs-code-engineering vs-code-engineering Bot added this to the 1.116.0 milestone Apr 9, 2026
ElromEvedElElyon added a commit to ElromEvedElElyon/claude-builders-bounty-1 that referenced this pull request Apr 9, 2026
…uilders-bounty#4)

Claude Code sub-agent that reviews GitHub PRs with structured output:
- CLI: claude-review --pr <url> [--post] [--output file]
- GitHub Action: auto-reviews on PR open/sync
- Output: summary, risks, suggestions, confidence score
- Zero npm deps: pure bash + gh + claude CLI
- Tested on 2 real PRs: react/react#36243, microsoft/vscode#308831

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@justschen justschen restored the justin/litleo branch April 9, 2026 18:36
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