fix(ui): simplify workspace context cards - #134
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughWorkspace cards now expand when their content is expandable. Workspace agents and providers display logos and metadata. Workspace instructions and skills use visible, styled rows with optional instruction expansion. ChangesWorkspace UI updates
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant WorkspaceCard
participant WorkspaceApp
participant ProviderLogoLookup
participant WorkspaceChip
WorkspaceCard->>WorkspaceApp: render workspace metadata
WorkspaceApp->>ProviderLogoLookup: normalize provider name
ProviderLogoLookup-->>WorkspaceApp: logo URL or undefined
WorkspaceApp->>WorkspaceChip: render provider and agent metadata
WorkspaceApp->>WorkspaceChip: render instructions and skills
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryWorkspace context cards now open useful details by default and simplify skills, instructions, agents, and provider presentation.
Confidence Score: 5/5The PR appears safe to merge, with no concrete blocking or independently actionable non-blocking defects identified. The updated expansion, instruction visibility, provider deduplication, and wrapped-list behavior remain consistent with the card data lifecycle and bounded workspace layout.
|
| Filename | Overview |
|---|---|
| src/ui/card-types.ts | Makes expandable workspace cards open by default without changing expandability criteria. |
| src/ui/card-types.test.ts | Adds regression coverage confirming that workspace cards containing details are initially expanded. |
| src/ui/icons.ts | Adds normalized provider-name lookup for bundled logo assets. |
| src/ui/workspace-app.tsx | Simplifies workspace detail rendering, preserves instruction preview state, and adds branded agent/provider chips. |
| src/ui/workspace-app.css | Replaces disclosure styling with wrapped detail lists, instruction controls, and provider-logo presentation. |
Reviews (1): Last reviewed commit: "feat(ui): show provider icons on agent p..." | Re-trigger Greptile
Workspace cards now open their useful context by default instead of making users step through repeated View all and Show less controls. Skills render directly with path and description metadata in their tooltips, while the instructions row keeps loaded AGENTS.md files visible and reveals additional discovered files through one View all action.
Agent profiles now show the profile name alongside the provider logo used underneath. Raw providers remain compact icon-only entries, with bundled dark-surface logos for the supported providers. The change also adds a focused regression test for the default workspace-card state.
Screenshots
Summary by CodeRabbit
New Features
UI Improvements