Skip to content

Create a skill file that captures the wizard architecture principles#438

Open
daniloc wants to merge 4 commits intomainfrom
wizard-dev-skill
Open

Create a skill file that captures the wizard architecture principles#438
daniloc wants to merge 4 commits intomainfrom
wizard-dev-skill

Conversation

@daniloc
Copy link
Copy Markdown
Collaborator

@daniloc daniloc commented May 5, 2026

Let's make it easy for other teams to extend the wizard, with pull requests that are easy to approve.

This skill is a collaboration between myself and Opus 4.7:

  • Core wizard architecture loaded into context
  • Adversarial example code loaded into context
  • Design principles exposed and refined
  • Implicit things we've been building into the project over two quarters made explicit

After extended discussion, the model took a swing at SKILL.md, which I edited closely. I think it's pretty good!

ARCHITECTURE.md cues future agents with a low-cost survey of the project, sparing it around a dozen tool calls of reading. ANTI-PATTERNS.md illuminates the failure modes of breaking the architecture bargains.

Questions

  • Seem about right? Anything you meaningfully take issue with?
  • Do we need ANTI-PATTERNS? It certainly follows the redirect strategy we know works well at a micro level (instead of useEffect, do…)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

🧙 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 basic-integration
  • /wizard-ci misc
  • /wizard-ci revenue

Test an individual app:

  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
Show more apps
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci misc/quack-quack
  • /wizard-ci revenue/stripe

Results will be posted here when complete.

@daniloc daniloc requested a review from a team May 5, 2026 21:01
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