Skip to content

fix: generate solution artifacts during workshop setup - #10

Merged
kody-bot merged 6 commits into
mainfrom
cursor/ws-react-e2e-playwright-e6cb
Jul 29, 2026
Merged

fix: generate solution artifacts during workshop setup#10
kody-bot merged 6 commits into
mainfrom
cursor/ws-react-e2e-playwright-e6cb

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

  • make the durable repository-level artifact generator target solution apps only
  • generate React Router route types and migrate/generate TypedSQL Prisma clients for every solution app
  • scope root typecheck to that generator and root lint to deploy tooling, avoiding intentionally incomplete problem apps and stale test fixtures
  • leave validate.yml untouched so Update Workshops cannot remove the fix

Verification

  • npm run typecheck
  • npm run lint
  • generated route types and Prisma clients for all 12 solution apps only
  • Setup passed on Ubuntu, Windows, and macOS
  • Test and Cursor Bugbot passed

CI: https://github.com/epicweb-dev/react-e2e-testing-with-playwright/actions/runs/30446826649

Open in Web Open in Cursor 

cursoragent and others added 4 commits July 29, 2026 01:45
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
…laywright-e6cb

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot marked this pull request as ready for review July 29, 2026 02:53
cursoragent and others added 2 commits July 29, 2026 03:03
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot merged commit 050e79b into main Jul 29, 2026
6 checks passed
@kody-bot
kody-bot deleted the cursor/ws-react-e2e-playwright-e6cb branch July 29, 2026 11:34
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.

3 participants