Skip to content

fix(scaffold): ship web projects test-ready (bun test + @types/bun) #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
agjs merged 2 commits into from
Jun 18, 2026
Merged

fix(scaffold): ship web projects test-ready (bun test + @types/bun) #12

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1a3bb2b
fix(scaffold): ship web projects test-ready (bun test + @types/bun)
agjs Jun 18, 2026
35576c6
docs(scaffold): keep the bun:test import on one line in guidance
agjs Jun 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 24 additions & 9 deletions packages/core/src/web-templates.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,8 @@ const REACT_PACKAGE_JSON = `{
"scripts": {
"dev": "vite",
"build": "vite build",
"preview": "vite preview"
"preview": "vite preview",
"test": "bun test"
},
"dependencies": {
"@dnd-kit/core": "^6.3.1",
Expand All @@ -71,6 +72,7 @@ const REACT_PACKAGE_JSON = `{
"devDependencies": {
"@tailwindcss/vite": "^4.0.0",
"@tanstack/router-plugin": "^1.95.0",
"@types/bun": "^1.3.14",
Comment thread
agjs marked this conversation as resolved.
"@types/react": "^19.0.0",
"@types/react-dom": "^19.0.0",
"@vitejs/plugin-react": "^4.3.4",
Expand Down Expand Up @@ -617,10 +619,15 @@ const REACT_GUIDANCE = [
" drop? `@dnd-kit/core` + `@dnd-kit/sortable` are installed. Do NOT add other",
" deps (only these + the scaffold's are installed; the build can't fetch more).",
"Imports use the @/ alias (e.g. @/views/<Feature>/<feature>.types, @/components/ui/button).",
"Do NOT write a checks.json or any browser interaction test. The gate already",
"builds the app with Vite and renders it in a real browser, FAILING on any",
"runtime/console error — that IS the acceptance. Spend your effort on a working,",
"clean app that renders without errors, not on test assertions.",
"Do NOT write a checks.json or browser-interaction/DOM test, and do NOT set up a",
"test runner — `bun test` is ALREADY wired (the `test` script + @types/bun ship in",
"package.json). The gate builds with Vite and renders in a real browser, FAILING on",
"any runtime/console error — that IS the app's acceptance. SEPARATELY the harness",
"enforces TDD on LOGIC: every `.ts` module that exports a function/class needs a",
"co-located `<name>.test.ts` using bun:test —",
' `import { test, expect } from "bun:test"`.',
"Presentational `.tsx` components need NO test. So put real logic (rules, reducers,",
"derived state, formatting) in `.ts` modules and test those.",
"Do NOT run `tsc`, `eslint`, `vite build`, or the gate command yourself to check",
"your work — the harness type-checks each file the moment you write it and runs",
"the full gate automatically, feeding back the exact errors concisely. Running",
Expand Down Expand Up @@ -649,10 +656,12 @@ const VANILLA_PACKAGE_JSON = `{
"scripts": {
"dev": "vite",
"build": "vite build",
"preview": "vite preview"
"preview": "vite preview",
"test": "bun test"
},
"devDependencies": {
"@tailwindcss/vite": "^4.0.0",
"@types/bun": "^1.3.14",
Comment thread
agjs marked this conversation as resolved.
"tailwindcss": "^4.0.0",
"typescript": "^5.7.0",
"vite": "^6.0.0"
Expand Down Expand Up @@ -734,9 +743,15 @@ const VANILLA_GUIDANCE = [
" • src/view.ts — DOM rendering (build/update elements with createElement)",
" • src/main.ts — the entry that wires store + view + events into #app",
"Style with Tailwind utility classes. Keep functions small and single-purpose.",
"Do NOT write a checks.json or browser interaction test. The gate builds with",
"Vite and renders the app in a real browser, failing on any runtime error — that",
"is the acceptance. Focus on a working app that renders cleanly.",
"Do NOT write a checks.json or browser-interaction/DOM test, and do NOT set up a",
"test runner — `bun test` is ALREADY wired (the `test` script + @types/bun ship in",
"package.json). The gate builds with Vite and renders in a real browser, failing on",
"any runtime error — that IS the app's acceptance. SEPARATELY the harness enforces",
"TDD on LOGIC: src/store.ts (pure, DOM-free) and any other `.ts` module that exports",
"a function needs a co-located `<name>.test.ts` using bun:test —",
' `import { test, expect } from "bun:test"`.',
"That is exactly why store logic is kept pure and out of view.ts — so it is",
"unit-testable. Test the store; the entry/view need no test.",
].join("\n");

// ── Harness SDK primitives (vendored): the toolkit the model COMPOSES with
Expand Down