refactor agent command center and projection code#288163
Merged
joshspicer merged 7 commits intomainfrom Jan 15, 2026
Merged
Conversation
joshspicer
changed the title
Contributor
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: @bpaseroMatched files:
|
…entSessionsProjection2 # Conflicts: # .vscode/settings.json # src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionProjectionService.ts # src/vs/workbench/contrib/chat/browser/agentSessions/agentStatusWidget.ts # src/vs/workbench/contrib/chat/browser/agentSessions/media/agentStatusWidget.css
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request refactors the "Focus View" feature to be renamed as "Agent Session Projection" and introduces a separate "Agent Status" widget. The changes split the previous "Agents Control" implementation into two distinct features with separate configuration options and clearer naming.
Changes:
- Renamed "Focus View" terminology to "Agent Session Projection" throughout the codebase
- Introduced a new
AgentStatusServiceto manage the agent status widget state independently - Split configuration into
AgentStatusEnabledandAgentSessionProjectionEnabledsettings - Updated CSS class names from
agents-control-*toagent-status-*for consistency - Created separate CSS files for agent status widget and projection mode styling
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/vs/workbench/contrib/chat/common/constants.ts | Added new AgentStatusEnabled configuration constant |
| src/vs/workbench/contrib/chat/common/actions/chatContextKeys.ts | Renamed inFocusViewMode to inAgentSessionProjection and added agentStatusEnabled context key |
| src/vs/workbench/contrib/chat/browser/chat.contribution.ts | Added configuration registration for AgentStatusEnabled setting |
| src/vs/workbench/contrib/chat/browser/agentSessions/media/agentStatusWidget.css | Renamed CSS classes from agents-control-* to agent-status-*, removed focus view border styling |
| src/vs/workbench/contrib/chat/browser/agentSessions/media/agentSessionProjection.css | New file containing agent session projection visual styling (borders, tabs, glow animation) |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentStatusWidget.ts | Renamed from AgentsControlViewItem to AgentStatusWidget, refactored to use IAgentStatusService |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentStatusService.ts | New service to manage agent status widget state and modes |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionsOpener.ts | Updated to use renamed IAgentSessionProjectionService |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.contribution.ts | Updated service registrations and configuration references |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionProjectionService.ts | Renamed from FocusViewService to AgentSessionProjectionService |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionProjectionActions.ts | Renamed actions from FocusView to AgentSessionProjection |
| src/vs/workbench/contrib/chat/browser/actions/chatNewActions.ts | Updated to use renamed projection service |
| src/vs/workbench/contrib/chat/browser/actions/chatActions.ts | Updated configuration reference in menu registration |
| .vscode/settings.json | Changed chat.tools.terminal.outputLocation setting (unrelated to main feature) |
Comments suppressed due to low confidence (1)
src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionProjectionActions.ts:157
- The action title 'Agent Session Projection' is somewhat technical and may not be immediately clear to users. Consider a more user-friendly name like 'Focused Session Review' or 'Session Review Mode' that better describes what the feature does for end users.
DonJayamanne
previously approved these changes
Jan 15, 2026
justschen
approved these changes
Jan 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.