Skip to content

fix(skills): resolve skills info name mismatches#57

Open
BingqingLyu wants to merge 4 commits into
mainfrom
fork-pr-38713-fix-issue-38546-skills-info-lookup
Open

fix(skills): resolve skills info name mismatches#57
BingqingLyu wants to merge 4 commits into
mainfrom
fork-pr-38713-fix-issue-38546-skills-info-lookup

Conversation

@BingqingLyu

@BingqingLyu BingqingLyu commented Apr 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • make openclaw skills info <name> resolve skills using exact, case-insensitive, and separator-normalized matching
  • normalize lookup tokens across whitespace, underscores, slashes, and dashes so list-visible names and skill keys resolve consistently
  • add a true case-insensitive test case so the lowercase branch is explicitly covered
  • trim the requested name in not-found JSON/text responses for cleaner UX

Why

Fixes openclaw#38546 where skills can show as ready in skills list but fail lookup in skills info due strict identifier matching.

Supersedes openclaw#38701, which was auto-closed by queue-limit automation while active PR count was over threshold.

Testing

  • pnpm test src/cli/skills-cli.test.ts
  • bash /Users/newdldewdl/.openclaw/workspace/skills/openclaw-autonomous-contributor/scripts/quality_gate.sh /Users/newdldewdl/.openclaw/workspace/memory/openclaw-contrib/worktrees/issue-38686

AI-assisted disclosure

  • AI-assisted: yes

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.

Title: Skills show as "ready" in list but "not found" in info, and can't be used by agent Steps to reproduce:

2 participants