Skip to content

feat: js-web skill#272

Open
sarahxsanders wants to merge 5 commits intomainfrom
js-skll
Open

feat: js-web skill#272
sarahxsanders wants to merge 5 commits intomainfrom
js-skll

Conversation

@sarahxsanders
Copy link
Contributor

@sarahxsanders sarahxsanders commented Feb 12, 2026

pass at adding a fallback JS skill. catches any browser-side JS project that doesn't match a specific framework. vanilla JS, react w/o router, etc.

it works by checking for package.json:

  • excludes projects w/ known framework packages
  • confirms JS by checking for a lockfile or ACTUAL dependencies in package.json
  • empty package.json files w/o dependencies return false

@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 laravel
  • /wizard-ci next-js
  • /wizard-ci nuxt
  • /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 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 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.

@sarahxsanders sarahxsanders changed the title draft: js skill feat: js skill Feb 12, 2026
@sarahxsanders sarahxsanders marked this pull request as ready for review February 12, 2026 20:11
@sarahxsanders sarahxsanders requested a review from a team February 12, 2026 20:12
`posthog.capture('$pageview')`,
`Or use capture_pageview: 'history_change' in the init options for History API based routing.`,
``,
`IMPORTANT: These best practices are MANDATORY. The implementation will fail review if they are not followed.`,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This extra context is SO MEATY. Should it live in the context-mill/skill itself? Maybe we enhance the skill definition system to accept additional local files as framework guidance?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im gonna test this approach bc it feels like the right call!! just make sure it doesn't break anything... and fix Python bc I did something similar there :]

@sarahxsanders sarahxsanders changed the title feat: js skill feat: js-web skill Feb 13, 2026
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