Skip to content

CL-5790: Skills page K1 shell (toolbar, empty state, create dialog) - #40

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-5790-skills-k1-shell
Aug 10, 2026
Merged

CL-5790: Skills page K1 shell (toolbar, empty state, create dialog)#40
TheGreatAxios merged 4 commits into
mainfrom
cl-5790-skills-k1-shell

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

What

The Skills page is no longer a stub. It renders the agents-style chrome: a toolbar (search + view toggle), an honest empty state with a primary Create skill action, and a CreateSkillDialog that collects name, description, and body.

Why

CL-5790 K1 — the Skills list/detail/create UI shells. The page no longer says "Skills aren't built yet"; it shows the real product chrome with an empty list and a working create dialog (the dialog never POSTs yet — no skill registry in the hub).

Changes

  • skills-page.tsx — toolbar (search + view toggle), empty state with Create action, listens for workbench:skills:create window event
  • create-skill-dialog.tsx (new) — IntakeForm-based dialog collecting name/description/body with validation
  • Tests updated for the new empty-state copy and Create action

Closes CL-5790 (K1 milestone)

The Skills page is no longer a stub — it renders the agents-style chrome
(toolbar with search + view toggle), an honest empty state with a primary
Create skill action, and a CreateSkillDialog that collects name, body,
and description. The dialog never POSTs (no skill registry yet) but
collects the values a future registry will expect.
@TheGreatAxios
TheGreatAxios force-pushed the cl-5790-skills-k1-shell branch from 68c010b to d3f94a8 Compare August 9, 2026 22:30
@TheGreatAxios
TheGreatAxios force-pushed the cl-5790-skills-k1-shell branch from 15068bc to c17ed77 Compare August 10, 2026 00:21
@TheGreatAxios
TheGreatAxios merged commit fe50bcc into main Aug 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant