Skip to content

Conversation

@jamiew
Copy link

@jamiew jamiew commented Dec 22, 2025

Basic CI workflow that runs on PRs and pushes to main. typechecks, lint, build

dpeends on #4 and other prior PRs

- Enable strict mode, noUnusedLocals, noUnusedParameters, noFallthroughCasesInSwitch
- Remove unused React imports (JSX transform handles it)
- Remove/prefix unused variables with underscore
- Remove unused type imports (Settings, TileEffect, GradientSettings)
- Clean up unused local variables in customGeometries.ts
- Simplify tsconfig.json (defer to tsconfig.app.json for strict settings)
- VisualizerCanvas3D: move useEffect before conditional return to fix hooks rules violation
- RegionSelector: fix stale closure by using ref for localRegion in mouseup handler
- Index: clarify source removal logic with explicit isLastSource check
- useRegionRandomizer/usePlayMode: use ReturnType<typeof setInterval> for browser compat
- Remove 24 unused npm packages (react-query, recharts, date-fns, vaul, etc.)
- Remove 36 unused shadcn/ui component files
- Remove QueryClientProvider wrapper from App.tsx (no queries used)
- Simplify sonner.tsx to remove next-themes dependency
- CSS bundle reduced from 62KB to 32KB
# Conflicts:
#	package-lock.json
#	package.json
#	src/components/ui/calendar.tsx
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