feat(agents): framework label pill in the Agents app#568
Conversation
The agent row only showed a framework-derived emoji, which is ambiguous (you can't tell openclaw from hermes from smolagents at a glance). Add a small framework-name pill next to the agent name (desktop) and in the detail row (mobile). Hidden for none/generic agents. Test asserts the labelled pill renders per agent.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAgentRow now displays each agent's framework as an accessible labeled pill component in both mobile and desktop layouts. The framework label is derived from the agent's framework property, filtering out generic values. A new test verifies the framework labels are rendered as accessible elements. ChangesFramework Label Display
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
What
Adds a small framework-name pill to each agent in the Agents app, so you can tell at a glance which framework an agent runs on (openclaw, hermes, smolagents, …). Previously the row showed only a framework-derived emoji, which is ambiguous.
none/genericagents (no meaningful framework).aria-label/title=Framework: <name>for a11y + tooltips.Test
AgentsApp.test.tsxasserts the labelled pill renders per agent (openclaw + smolagents fixtures). Full AgentsApp suite green; tsc clean.Context
Requested while validating an end-to-end OpenClaw deploy (kilo provider +
stepfun/step-3.7-flash:free) on the Pi.Summary by CodeRabbit
New Features
Tests