Skip to content

feat(agents): framework label pill in the Agents app#568

Merged
jaylfc merged 1 commit into
devfrom
fix/agents-framework-label
Jun 4, 2026
Merged

feat(agents): framework label pill in the Agents app#568
jaylfc merged 1 commit into
devfrom
fix/agents-framework-label

Conversation

@jaylfc
Copy link
Copy Markdown
Owner

@jaylfc jaylfc commented Jun 4, 2026

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.

  • Desktop list row: pill next to the agent name.
  • Mobile card: pill in the detail row (alongside host/vectors).
  • Hidden for none/generic agents (no meaningful framework).
  • aria-label/title = Framework: <name> for a11y + tooltips.

Test

AgentsApp.test.tsx asserts 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

    • Framework information is now displayed on agent rows as accessible labeled indicators in mobile and desktop layouts, helping users quickly identify agent types.
  • Tests

    • Added test coverage to verify framework labels are properly rendered and accessible across different agent implementations.

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.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 4, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a931bf6f-14d3-4b9f-a1a6-3024e39e9751

📥 Commits

Reviewing files that changed from the base of the PR and between 435ad81 and eea5218.

📒 Files selected for processing (2)
  • desktop/src/apps/__tests__/AgentsApp.test.tsx
  • desktop/src/apps/agents/AgentRow.tsx

📝 Walkthrough

Walkthrough

AgentRow 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.

Changes

Framework Label Display

Layer / File(s) Summary
Framework pill component and layout rendering
desktop/src/apps/agents/AgentRow.tsx
FrameworkPill component is defined with frameworkLabel derived from agent.framework (excluding "none"/"generic"), with accessible title and aria-label attributes. The pill is rendered in both mobile (after host row) and desktop (after emoji/identity block) layouts.
Framework label test coverage
desktop/src/apps/__tests__/AgentsApp.test.tsx
AgentsApp test suite verifies that agent frameworks are accessible labeled elements via getByLabelText/findByLabelText for Framework: openclaw and Framework: smolagents.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • jaylfc/taOS#558: Introduces/refactors the agents UI row component that this PR builds upon with the framework-label pill feature.

Poem

🐰 A framework label hops into view,
Accessible, labeled—clear and true!
Mobile and desktop both get the pill,
Tests verify each agent's skill.
Framework pride on display!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: adding a framework label pill to the Agents app. It directly reflects the primary purpose of the PR.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/agents-framework-label

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 and usage tips.

@jaylfc jaylfc merged commit 9700776 into dev Jun 4, 2026
7 checks passed
@jaylfc jaylfc deleted the fix/agents-framework-label branch June 4, 2026 12:01
@github-project-automation github-project-automation Bot moved this from Todo to Done in TinyAgentOS Roadmap Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant