Skip to content

feat(skillhub): fix desktop packaging crash and skill catalog corrections#323

Merged
alchemistklk merged 1 commit intomainfrom
feat/skill-integration-expansion
Mar 22, 2026
Merged

feat(skillhub): fix desktop packaging crash and skill catalog corrections#323
alchemistklk merged 1 commit intomainfrom
feat/skill-integration-expansion

Conversation

@alchemistklk
Copy link
Copy Markdown
Contributor

Summary

  • Fix desktop app crash on startup: controller sidecar packaging now copies static/runtime-plugins/ directory; OpenClawRuntimePluginWriter tolerates missing directory gracefully instead of crashing with ENOENT
  • Fix broken ClawHub catalog slugs: SLUG_CORRECTIONS mapping (find-skillsfind-skill) applied in install, uninstall, and catalog read paths
  • Curated skill list cleanup: moved find-skill from static to curated with correct slug; removed delisted baoyu-xhs-images; added deep-research, research-to-diagram, qiaomu-mondo-poster-design as static bundled skills
  • Fix reconcile logic: preserves user's explicit curated uninstall choices; only purges records for slugs retired from the curated list
  • Added curated-skills-slugs test: validates slug format, uniqueness, and no overlap between curated and static lists

Test plan

  • pnpm typecheck passes
  • pnpm test — all 50 tests pass (including new curated-skills-slugs test)
  • Build desktop package (pnpm dist:mac:unsigned) — verify controller starts without ENOENT crash
  • Verify find-skill shows as installed if already on disk
  • Verify uninstalling a curated skill persists across app restarts

…s, and reconcile improvements

- Fix controller crash on startup by copying static/runtime-plugins to sidecar
  and adding ENOENT tolerance in OpenClawRuntimePluginWriter
- Add SLUG_CORRECTIONS mapping to fix broken ClawHub catalog slugs (find-skills → find-skill)
  applied in installSkill, uninstallSkill, and readCachedSkills
- Move find-skill from STATIC to CURATED list with correct slug
- Remove delisted baoyu-xhs-images from curated list
- Add deep-research, research-to-diagram, qiaomu-mondo-poster-design as static bundled skills
- Fix reconcile to preserve user's curated uninstall choices while allowing
  retired slugs to be purged
- Add curated-skills-slugs test for validation, uniqueness, and overlap checks
@alchemistklk
Copy link
Copy Markdown
Contributor Author

/cr

@slack-code-review-channel
Copy link
Copy Markdown

✅ CR topic created in Feishu topic group Refly CR.

@alchemistklk alchemistklk merged commit cf78462 into main Mar 22, 2026
7 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.

2 participants