Skip to content

fix: Skills allow underscore#342

Merged
gewenyu99 merged 1 commit intomainfrom
underscores
Mar 16, 2026
Merged

fix: Skills allow underscore#342
gewenyu99 merged 1 commit intomainfrom
underscores

Conversation

@gewenyu99
Copy link
Contributor

@gewenyu99 gewenyu99 commented Mar 16, 2026

We have underscores in some skill names. This causes some errors

  • Fix skill ID validation regex to allow underscores, unblocking integration-javascript_node and
    integration-javascript_web skills

@github-actions
Copy link

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci android
  • /wizard-ci angular
  • /wizard-ci astro
  • /wizard-ci django
  • /wizard-ci fastapi
  • /wizard-ci flask
  • /wizard-ci javascript-node
  • /wizard-ci javascript-web
  • /wizard-ci laravel
  • /wizard-ci next-js
  • /wizard-ci nuxt
  • /wizard-ci python
  • /wizard-ci rails
  • /wizard-ci react-native
  • /wizard-ci react-router
  • /wizard-ci sveltekit
  • /wizard-ci swift
  • /wizard-ci tanstack-router
  • /wizard-ci tanstack-start
  • /wizard-ci vue

Test an individual app:

  • /wizard-ci android/Jetchat
  • /wizard-ci angular/angular-saas
  • /wizard-ci astro/astro-hybrid-marketing
Show more apps
  • /wizard-ci astro/astro-ssr-docs
  • /wizard-ci astro/astro-static-marketing
  • /wizard-ci astro/astro-view-transitions-marketing
  • /wizard-ci django/django3-saas
  • /wizard-ci fastapi/fastapi3-ai-saas
  • /wizard-ci flask/flask3-social-media
  • /wizard-ci javascript-node/express-todo
  • /wizard-ci javascript-node/fastify-blog
  • /wizard-ci javascript-node/hono-links
  • /wizard-ci javascript-node/koa-notes
  • /wizard-ci javascript-node/native-http-contacts
  • /wizard-ci javascript-web/saas-dashboard
  • /wizard-ci laravel/laravel12-saas
  • /wizard-ci next-js/15-app-router-saas
  • /wizard-ci next-js/15-app-router-todo
  • /wizard-ci next-js/15-pages-router-saas
  • /wizard-ci next-js/15-pages-router-todo
  • /wizard-ci nuxt/movies-nuxt-3-6
  • /wizard-ci nuxt/movies-nuxt-4
  • /wizard-ci python/meeting-summarizer
  • /wizard-ci rails/fizzy
  • /wizard-ci react-native/expo-react-native-hacker-news
  • /wizard-ci react-native/react-native-saas
  • /wizard-ci react-router/react-router-v7-project
  • /wizard-ci react-router/rrv7-starter
  • /wizard-ci react-router/saas-template
  • /wizard-ci react-router/shopper
  • /wizard-ci sveltekit/CMSaasStarter
  • /wizard-ci swift/hackers-ios
  • /wizard-ci tanstack-router/tanstack-router-code-based-saas
  • /wizard-ci tanstack-router/tanstack-router-file-based-saas
  • /wizard-ci tanstack-start/tanstack-start-saas
  • /wizard-ci vue/movies

Results will be posted here when complete.

@gewenyu99 gewenyu99 changed the title Fix Skills allow underscore Fix: Skills allow underscore Mar 16, 2026
@gewenyu99 gewenyu99 requested a review from a team March 16, 2026 22:11
@gewenyu99
Copy link
Contributor Author

From the log, the agent tried these in order:

  1. Exact ID (rejected by regex):
    Error: skillId must be lowercase alphanumeric with hyphens.
  2. Guessed integration-javascript-node (not found):
    Error: skill "integration-javascript-node" not found. Use load_skill_menu to see available skills.
  3. Guessed integration-javascriptnode (not found):
    Error: skill "integration-javascriptnode" not found. Use load_skill_menu to see available skills.
  4. Guessed integration-node (not found):
    Error: skill "integration-node" not found. Use load_skill_menu to see available skills.
  5. Fell back to MCP resource:
    MCP error -32602: Resource posthog://docs/frameworks/javascript_node not found

@gewenyu99 gewenyu99 changed the title Fix: Skills allow underscore fix: Skills allow underscore Mar 16, 2026
@gewenyu99 gewenyu99 merged commit 10a291b into main Mar 16, 2026
18 of 21 checks passed
@gewenyu99 gewenyu99 deleted the underscores branch March 16, 2026 22:16
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