chore: Update agent skills - #3400
Conversation
|
View your CI Pipeline Execution ↗ for commit 4d5968d ☁️ Nx Cloud last updated this comment at |
|
|
📬 Published Alpha Packages:
|
|
🚀 Styleguide deploy preview ready! Preview URL: https://6a691143c62c9b6a8116aec3--gamut-preview.netlify.app |
| **/code-connect/** | ||
| packages/code-connect | ||
| .nx | ||
| packages/gamut/agent-tools/skills/*-workspace |
There was a problem hiding this comment.
If this is too broad, and we'd rather add ignore comments for the examples created, I can do that instead.
dreamwasp
left a comment
There was a problem hiding this comment.
looks awesome minus the tiniest nit, wanna talk more about iteration tracking but we'll do that not on GH 🔥
| | Progress bar / radial/circular progress | `ProgressBar`, `RadialProgress` | Storybook — no dedicated skill yet | | ||
| | Loading state / skeleton placeholder | `Loading`, `FeatureShimmer` | Storybook — no dedicated skill yet | | ||
| | Bulk-comparison table | `DataTable` | [`gamut-datatable`](../gamut-datatable/SKILL.md) | | ||
| | Item list with expand/select | `DataList` | [`gamut-datalist`](../gamut-datalist/SKILL.md) | |
There was a problem hiding this comment.
this could be list or datalist, bulk comparison i'd say DataTable and DataList
Overview
Improves the Gamut agent-tools skills (the
SKILL.mdguides that drive AI-assisted code review and development against the Gamut design system) — adding new skills, fixing stale documentation, and closing gaps found while eval-testing the existing skills against real PR feedback.styled(GamutComponent)bypassing system props) togamut-review, including Step 4 (detectspadding/margin/etc. used instead of their scale-aware alias likep/minsidecss(), which silently skips token/rem conversion) and Step 5 (<Box display="flex">/<Box display="grid">→ recommendsFlexBox/GridBoxinstead)gamut-review— a heuristic, always-⚠ check for hand-rolled widgets that duplicate an existing Gamut componentgamut-component-first/SKILL.md(new) — a decision table and signals for checking existing Gamut components before building bespoke onesgamut-modal/SKILL.md(new) — Modal vs Dialog composition guidance, dismiss/focus-trap patterns, and stacking ordergamut-z-index/SKILL.md(new) — z-index token scale and a stacking-component reference, cross-referenced withgamut-modalgamut-system-props/SKILL.mdandgamut-style-utilities/SKILL.mdwith thestyled()-wrapper anti-pattern, correctbackgroundgradient-prop usage, semantic vs. fixed color-token guidance, and a scale-verification tableRadio/RadioGroupsection togamut-forms/SKILL.mdDESIGN.LXStudio.mdandDESIGN.Percipio.md— both themes actually inherit dark mode unmodified from Core since their theme definitions only overridelightgamut-modal/gamut-z-indexskills fromgamut-accessibility,gamut-datalist,gamut-datatable, andgamut-menuevals/evals.jsontest cases forgamut-review,gamut-system-props,gamut-style-utilities, andgamut-component-firstused to benchmark these skillsPR Checklist
Testing Instructions
Don't make me tap the sign.
PR Links and Envs
N/A