stacklok / toolhive-studio / 25044124219 / 1

70%

main: 70%

push

github

feat(skills): nice empty state when search returns no results (#2127)

* feat(skills): use shared empty state on registry tab no-results

Swap the bare "No skills found matching the current filter" line on
the Registry tab (table + grid views) for the shared `EmptyState`
component with `IllustrationNoSearchResults`, matching the pattern
already used on the skill / build / server detail "not found" pages.

* feat(skills): use shared empty state on installed tab no-results

Same treatment as the registry tab: the filter-empty placeholder on
the Installed tab (table + grid) now renders the shared `EmptyState`
with `IllustrationNoSearchResults`. The existing "No skills installed"
empty state in `skills-page.tsx` is unchanged.

* feat(skills): use shared empty state on builds tab no-results

Replace the filter-empty placeholder on the Local Builds tab (table +
grid) with the shared `EmptyState` + `IllustrationNoSearchResults`.
The "No local builds" empty state shown when there are zero builds at
all is intentionally untouched.

* refactor(empty-state): make children optional with no actions

Address Copilot review on #2127. The action-less branch of
`EmptyStateProps` required `children`, which forced every search
empty state on the Skills page to pass a meaningless `<></>` to
satisfy the type. Relax the union to `children?: ReactNode` and drop
the placeholder fragments from all six skills components.

Rendered DOM is unchanged (the empty fragment never produced a node),
so no test updates are needed.

4153 of 6825 branches covered (60.85%)

6319 of 9536 relevant lines covered (66.26%)

125.7 hits per line