Skip to content

fix: prefix skill names with olostep- to avoid conflicts#13

Closed
ZeeshanAdilButt wants to merge 1 commit into
mainfrom
fix/skill-names
Closed

fix: prefix skill names with olostep- to avoid conflicts#13
ZeeshanAdilButt wants to merge 1 commit into
mainfrom
fix/skill-names

Conversation

@ZeeshanAdilButt

Copy link
Copy Markdown
Collaborator

Summary

  • Skill names in SKILL.md frontmatter were bare (search, scrape, map, etc.) — when users have multiple skill providers installed (e.g. ours + another tool's skill pack), the agent sees name conflicts and can't disambiguate
  • Prefixed all 13 skill name: fields with olostep-: searcholostep-search, scrapeolostep-scrape, etc.
  • Updated runInstall filter logic to normalize both sides so --skill search and --skill olostep-search both work (backward compat preserved)
  • Installed folder names were already olostep-<name> — this change makes the frontmatter name: field consistent with the folder name

Affected skills

olostep-answers, olostep-batch, olostep-crawl, olostep-debug-error, olostep-docs-to-code, olostep-extract-schema, olostep-integrate, olostep-map, olostep-migrate-code, olostep-research, olostep-scrape, olostep-search, olostep-setup

Test plan

  • npm test — all 193 tests pass
  • olostep add skills --skill search still works after the change
  • olostep add skills --skill olostep-search also works
  • After olostep skills update, agent no longer shows skill name conflicts

@ZeeshanAdilButt

Copy link
Copy Markdown
Collaborator Author

consolidating into single PR

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