Skip to content

feat: skills system end-to-end — ClawHub fetch + error feedback#39

Merged
rogerchappel merged 3 commits into
mainfrom
feat/skills-end-to-end
Apr 3, 2026
Merged

feat: skills system end-to-end — ClawHub fetch + error feedback#39
rogerchappel merged 3 commits into
mainfrom
feat/skills-end-to-end

Conversation

@rogerchappel
Copy link
Copy Markdown
Owner

Changes

ClawHub Marketplace Fetch (browse/route.ts)

  • Fetches real skills from https://clawhub.com/api/skills instead of hardcoded mocks
  • 5s timeout, 5-min in-memory cache, falls back to hardcoded list on failure

Error Feedback (skills/page.tsx)

  • Added toast notifications for all skill operations (install, create, edit, delete, assign)
  • Error toasts on failure, success toasts on completion
  • Auto-dismiss after 3s

Already Working (verified)

  • PATCH/DELETE endpoints exist with cascade delete
  • Installed badge in browse tab works via installedSlugs cross-reference
  • Built-in skill import before agent assignment works

Closes TSK-391

rogerchappel and others added 3 commits April 3, 2026 16:35
…ilt-in import

- Add batch agent-skills endpoint: GET/POST /api/skills/[id]/agents
- Add PATCH to agent skill route for enable/disable toggle
- Skills detail panel shows all agents with assign/unassign toggles
- Assigned agents get separate enable/disable control
- Built-in skills auto-import to DB before agent assignment
- Replace N+1 agent-skill queries with single batch fetch

Closes TSK-391
Replace hardcoded marketplace data with live fetch from
https://clawhub.com/api/skills. Adds 5s timeout, 5-minute in-memory
cache, and falls back to the hardcoded list if the API is unreachable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace all silent catch blocks with user-facing toast notifications
for install, create, edit, delete, and agent assignment failures.
Adds success toasts for installs and edits. Auto-dismisses after 3s.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rogerchappel rogerchappel merged commit 34dd362 into main Apr 3, 2026
1 check 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