Skip to content

Conversation

@marcodejongh
Copy link
Owner

Summary

  • Removed 7 unused dependencies to reduce bundle size and maintenance overhead
  • Added missing dayjs dependency that was being imported but not declared
  • Verified all changes with successful build and lint tests

Dependencies Removed

  • react-router-dom - Next.js provides built-in routing
  • react-swipeable - No usage found in codebase
  • react-intersection-observer - No usage found in codebase
  • d3-scale-chromatic - No usage found in codebase
  • cookies - No usage found in codebase
  • @types/jest - Project uses Vitest, not Jest
  • @types/d3-interpolate - No direct d3-interpolate usage found

Dependencies Added

  • dayjs - Was being imported in multiple files but missing from package.json

Test Plan

  • Build passes (npm run build)
  • Linting passes (npm run lint)
  • No TypeScript errors
  • All imports resolve correctly

🤖 Generated with Claude Code

- Remove unused packages: react-router-dom, react-swipeable, react-intersection-observer, d3-scale-chromatic, cookies
- Remove unused dev dependencies: @types/jest, @types/d3-interpolate
- Add missing dayjs dependency that was being imported
- Keep @types/cookie as it's needed by session.ts
- Build and lint tests pass

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Aug 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
boardsesh ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2025 0:08am

@marcodejongh marcodejongh merged commit 8f0b014 into main Aug 10, 2025
4 checks passed
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