[fix] Restore the read-only Skills empty text in the agent config card - #6745
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe read-only Skills row now displays “None available” when no skills are configured. The edit-mode label remains “Add skills”. ChangesSkills label update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This restores “None available” for read-only Skills rows while keeping “Add skills” in edit mode. The focused test passes and no merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Railway Preview Environment
Updated at 2026-09-10T11:02:33.418Z |
Context
agentConfigSummaryCard.test.tsxfails onrelease/v0.116.0, so every PR against the release branch inherits a redrun-web-unit-tests. The read-only agent config card shows "None added" for an agent with no skills. #6641 made that text "None available" and pinned it in the test. Therelease/v0.116.0merge intofeat/skill-registry(#6604) then resolved the line to "None added".Changes
The read-only Skills row goes back to "None available". That matches #6641, its test, and the other read-only rows ("None enabled", "None connected"). The editable card still offers "Add skills".
Tests
vitest run tests/unit/agentConfigSummaryCard.test.tsxin@agenta/entity-ui: 13 of 13 pass (1 failed before).KeyError: 'search_skills'). [fix] Give the skill registry ops a build-kit permission so skills load again #6744 fixes those. Once both land, the web and API unit jobs onrelease/v0.116.0should be green.