Skip to content

add more tips, make all of them contextual#294237

Merged
meganrogge merged 1 commit intomainfrom
merogge/more-contextual-tips
Feb 10, 2026
Merged

add more tips, make all of them contextual#294237
meganrogge merged 1 commit intomainfrom
merogge/more-contextual-tips

Conversation

@meganrogge
Copy link
Collaborator

@meganrogge meganrogge commented Feb 10, 2026

fixes #290019
fixes #293536

Added 10 new tips: custom agents, skills, message queueing, yolo mode, mermaid, GitHub repo, subagents, context usage, send to new chat.

Introduced two new exclusion mechanisms on tip definitions:

  • excludeWhenToolsInvoked suppresses a tip after a specific tool has been used
  • excludeWhenPromptFilesExist suppresses a tip when prompt files of a given type are found in the workspace

Also adds a new command to focus the Context Window Usage widget, so we can allow users to open that from the hint.

Copilot AI review requested due to automatic review settings February 10, 2026 21:52
@meganrogge meganrogge self-assigned this Feb 10, 2026
@meganrogge meganrogge added this to the February 2026 milestone Feb 10, 2026
@meganrogge meganrogge enabled auto-merge (squash) February 10, 2026 21:52
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 expands the chat “tips” system to be more context-aware, adding new tips and new exclusion mechanisms so tips are suppressed once the user has already discovered/used the referenced features (commands, tools, or workspace prompt files). It also introduces a command to open/focus the Context Window Usage details UI directly from a hint.

Changes:

  • Added new tip definitions and made existing tips more contextual via new exclusion conditions (tool-invoked + prompt-file-exists).
  • Enhanced TipEligibilityTracker to persist and evaluate tool invocations and prompt file presence for tip suppression.
  • Added a command + widget plumbing to show/focus “Context Window Usage” details and track whether it has been opened.

Reviewed changes

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

Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/chatTipService.ts Adds new tips and implements new exclusion mechanisms (tool invocation + prompt file existence), with persistence.
src/vs/workbench/contrib/chat/test/browser/chatTipService.test.ts Extends unit tests for new exclusion behavior (tools + prompt files + agent change event).
src/vs/workbench/contrib/chat/common/actions/chatContextKeys.ts Adds a new context key tracking whether context usage details have been opened.
src/vs/workbench/contrib/chat/browser/widgetHosts/viewPane/chatContextUsageWidget.ts Persists/sets the “opened” state and exposes showDetails() to programmatically open the sticky hover.
src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts Adds showContextUsageDetails() to bridge commands to the widget.
src/vs/workbench/contrib/chat/browser/actions/chatActions.ts Registers the new workbench.action.chat.showContextUsage command to open the details UI.

@meganrogge meganrogge merged commit 9f6e415 into main Feb 10, 2026
23 of 24 checks passed
@meganrogge meganrogge deleted the merogge/more-contextual-tips branch February 10, 2026 22:20
Copy link
Member

@cwebster-99 cwebster-99 left a comment

Choose a reason for hiding this comment

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

Added some small tweaks to tips. Should we also add one for steering?

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.

Message queueing should be one of the tips, esp if a user hasn't done it yet Indicate what's new in chat in a context aware way

3 participants