Conversation
added 3 commits
April 23, 2026 16:08
The sidebar request was a bust-like identity closer to the sculpted logo reference, not the existing hive-node glyph. This swaps the mirrored Activity Bar SVG to a small monochrome bust silhouette and bumps the extension manifests so reinstall and reload flows can pick up the asset change. The numeric Activity Bar badge stays unchanged because the current VS Code view-badge surface only exposes a count and tooltip, not per-view badge color or severity control. Constraint: VS Code ViewBadge exposes only value and tooltip for tree views Rejected: Force a red badge through theme colors | would require global workbench theme changes instead of a per-view override Confidence: high Scope-risk: narrow Reversibility: clean Directive: Keep contributed Activity Bar icons monochrome and readable at 16-24px; avoid detailed artwork that collapses at sidebar size Tested: node --test test/vscode-active-agents-session-state.test.js Not-tested: Live VS Code render in an already-open window before reinstall/reload
…ow-up
The Active Agents lane already ships its custom Activity Bar icon and mirrored manifest updates.
This follow-up records the scope change in the T1 note and sets the workspace file icon default to material-icon-theme without changing extension-owned glyph logic.
Constraint: Keep Active Agents custom glyphs independent from the workbench file icon theme
Rejected: Commit .vscode/extensions.json recommendation | ignored local-only helper, not repo state
Confidence: high
Scope-risk: narrow
Directive: If workspace icon defaults change again, keep extension-owned icons independent from the workbench file icon theme
Tested: node --test test/vscode-active-agents-session-state.test.js; node -e "JSON.parse(require('fs').readFileSync('.vscode/settings.json', 'utf8')); console.log('settings-json-ok')"
Not-tested: Manual VS Code visual confirmation with Material Icon Theme installed
The prior glyph read as a hive graph and competed with the numeric badge at Activity Bar size. This keeps the icon simpler and monochrome, syncs the shipped/template manifests at 0.0.15, and removes unrelated workspace icon-theme drift from this lane before finish. Constraint: VS Code TreeView.badge only exposes value and tooltip; the extension cannot restyle the badge or replace Explorer's icon when the view is docked there Rejected: Fake a custom red badge treatment | unsupported by VS Code API Rejected: Ship the material-icon-theme workspace default | unrelated workspace drift Confidence: high Scope-risk: narrow Directive: Keep Activity Bar glyphs simple solid silhouettes; tiny surfaces lose detail fast Tested: JSON manifest parse, mirrored SVG diff, .vscode/settings.json vs main, rg checks for icon path/version/badge API Not-tested: Manual VS Code render of the updated Activity Bar icon
NagyVikt
force-pushed
the
agent/codex/active-agents-bust-icon-2026-04-23-14-52
branch
from
1eabc48 to
f0b2a91
Compare
NagyVikt
deleted the
agent/codex/active-agents-bust-icon-2026-04-23-14-52
branch
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Testing