Skip to content

fix(skills): distinguish external skill install paths and open real location#463

Merged
vcfgv merged 1 commit intomainfrom
bugfix/distinguish-externally-installed-skills
Mar 13, 2026
Merged

fix(skills): distinguish external skill install paths and open real location#463
vcfgv merged 1 commit intomainfrom
bugfix/distinguish-externally-installed-skills

Conversation

@vcfgv
Copy link
Contributor

@vcfgv vcfgv commented Mar 13, 2026

Summary

Fixes a path mismatch in Skills:

  • Skills list could show items discovered from non-managed locations
  • But "Open Folder"/README lookup still assumed ~/.openclaw/skills

This caused "visible but cannot open" behavior, especially on Windows or when skills were installed via external clawhub flows.

- `source`, `baseDir`, `filePath` in skills state model
  • Prefer Gateway skills.status path fields when available
  • Extended ClawHub list result to include managed location metadata
  • Added new host API route:
    • POST /api/clawhub/open-path to open the actual skill directory
  • Updated README open logic:
    • open-readme now accepts and prefers baseDir when provided
  • Skills UI updates:
    • Show source badge + resolved path in list and detail
    • Add "Open Actual Folder" action per-skill
    • Add "Copy path" action
    • Keep global "Open Skills Folder" behavior unchanged (managed dir)
  • i18n updates (en/zh/ja) for new labels/toasts
  • README updates (en/zh/ja) to document multi-source discovery and actual path visibility

Why

Align display/discovery and folder-open behavior to the same source-of-truth so users can reliably locate externally installed skills.

Validation

  • Targeted test passed:
    • npm test -- tests/unit/skills-errors.test.ts
  • npm run typecheck still reports an existing unrelated baseline issue in this environment:
    • src/hooks/use-stick-to-bottom-instant.ts missing module use-stick-to-bottom
  • Full test suite still has pre-existing unrelated failures (localStorage/test env), not introduced by this change.

@vcfgv vcfgv merged commit 4cfb552 into main Mar 13, 2026
2 checks passed
@vcfgv vcfgv deleted the bugfix/distinguish-externally-installed-skills branch March 13, 2026 06:40
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