Skip to content

feat(web): project favicon and workspace icons in command subtitles - #6330

Merged
maria-rcks merged 3 commits into
pingdotgg:mainfrom
gsimone:t3code/add-project-icons-to-commands
Aug 12, 2026
Merged

feat(web): project favicon and workspace icons in command subtitles#6330
maria-rcks merged 3 commits into
pingdotgg:mainfrom
gsimone:t3code/add-project-icons-to-commands

Conversation

@gsimone

@gsimone gsimone commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Adds project favicon and model icon to threads in the command palette

image image

Summary

  • Command palette thread rows no longer use plain project · #branch text for the subtitle
  • Subtitles now show the real project favicon, local vs worktree icons (Folder / FolderGit2), branch label, and harness/provider icon
  • Layout is flipable via THREAD_COMMAND_SUBTITLE_VARIANT while iterating (favicon-workspace-harness default)

Test plan

  • Open command palette and confirm recent threads show project favicons
  • Compare a local/main thread vs a worktree thread icon
  • Confirm harness icon matches the thread provider
  • Confirm “Current thread” still appears on the active thread
  • Search threads and verify the same subtitle treatment in results

Made with Cursor

Note

Add project favicons and provider icons to command palette thread subtitles

  • Introduces ThreadCommandSubtitle, a new component that renders thread items in the command palette with project favicon, workspace/branch icon, provider icon, and a 'Current thread' marker.
  • Extends buildThreadActionItems in CommandPalette.logic.ts with an optional renderDescription callback so callers can supply rich React subtitles instead of plain text.
  • Builds provider instance and favicon maps in CommandPalette.tsx and passes them to ThreadCommandSubtitle via the new callback.
  • Changes the description container class in CommandPaletteResults.tsx from truncate to min-w-0 so nested rich content can manage its own truncation.

Macroscope summarized 0c9200d.

Replace the plain project · #branch text in command palette thread rows with the real project favicon, local/worktree icons, and harness icon so threads scan like the sidebar.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e0b95f49-8439-46c8-b676-559c73d41000

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 12, 2026
Comment thread apps/web/src/components/CommandPalette.tsx
@macroscopeapp

macroscopeapp Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs human review

1 blocking correctness issue found. This PR adds visual icons to command palette thread subtitles - a presentational feature. However, there is an unresolved Medium-severity finding about provider entries potentially resolving incorrectly for remote threads, showing wrong or missing provider icons.

You can customize Macroscope's approvability policy. Learn more.

gsimone and others added 2 commits August 12, 2026 17:24
Look up provider instances from each thread's own environment config so remote threads no longer inherit the primary server's harness icon (or show none).

Co-authored-by: Cursor <cursoragent@cursor.com>
@maria-rcks
maria-rcks merged commit ac1264e into pingdotgg:main Aug 12, 2026
15 checks passed
juliusmarminge added a commit that referenced this pull request Aug 12, 2026
- PullRequestDetailPanel takes main's #6039 rework wholesale (reactions,
  update-branch, auto-merge, in-place editing); the thread-details action
  hooks stay in usePullRequestActions with label maps extended for the
  new action variants
- CommandPalette #6330 provider subtitles adapted to the v2 shell
  (session -> runtime for provider instance and display name)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 13, 2026
## What's Changed
* fix(web): cmd+click sidebar PR numbers open in the browser by @t3dotgg in pingdotgg/t3code#6378
* feat(web): project favicon and workspace icons in command subtitles by @gsimone in pingdotgg/t3code#6330
* web/settings: fix source control scan on relay environments by @dominic-r in pingdotgg/t3code#6230
* fix(web): make reset zoom hover visible by @t3-code[bot] in pingdotgg/t3code#6385
* fix(web): keep the typed prompt when a draft changes repo by @t3dotgg in pingdotgg/t3code#6393

## New Contributors
* @gsimone made their first contribution in pingdotgg/t3code#6330
* @dominic-r made their first contribution in pingdotgg/t3code#6230

**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260812.1079...v0.0.34-nightly.20260813.1081

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260813.1081
0veek added a commit to 0veek/Ronin that referenced this pull request Aug 13, 2026
Ported from t3code ac1264e (pingdotgg#6330).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
juliusmarminge added a commit that referenced this pull request Aug 14, 2026
- PullRequestDetailPanel takes main's #6039 rework wholesale (reactions,
  update-branch, auto-merge, in-place editing); the thread-details action
  hooks stay in usePullRequestActions with label maps extended for the
  new action variants
- CommandPalette #6330 provider subtitles adapted to the v2 shell
  (session -> runtime for provider instance and display name)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
juliusmarminge added a commit that referenced this pull request Aug 14, 2026
- PullRequestDetailPanel takes main's #6039 rework wholesale (reactions,
  update-branch, auto-merge, in-place editing); the thread-details action
  hooks stay in usePullRequestActions with label maps extended for the
  new action variants
- CommandPalette #6330 provider subtitles adapted to the v2 shell
  (session -> runtime for provider instance and display name)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
juliusmarminge added a commit that referenced this pull request Aug 15, 2026
- PullRequestDetailPanel takes main's #6039 rework wholesale (reactions,
  update-branch, auto-merge, in-place editing); the thread-details action
  hooks stay in usePullRequestActions with label maps extended for the
  new action variants
- CommandPalette #6330 provider subtitles adapted to the v2 shell
  (session -> runtime for provider instance and display name)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants