Skip to content

chore: remove unused dependencies#697

Merged
nikohoffren merged 2 commits into
fork-commit-merge:mainfrom
vansh2408:chore/remove-unused-deps-694
Oct 1, 2025
Merged

chore: remove unused dependencies#697
nikohoffren merged 2 commits into
fork-commit-merge:mainfrom
vansh2408:chore/remove-unused-deps-694

Conversation

@vansh2408

Copy link
Copy Markdown
Contributor

Pull Request Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe: Dependency cleanup

What is the current behavior?

The package.json contained unused and misplaced dependencies:

  • Several dev-only tools (TypeScript, ESLint, Prettier, Tailwind, PostCSS, Autoprefixer, @types/*) were under dependencies instead of devDependencies.
  • Extra libraries (e.g., react-loader-spinner, react-spinners, axios, next-auth, etc.) were included but unused.

Issue Number: [resolves #694 ]

What is the new behavior?

✅ Removed unused dependencies

  • @mui/icons-material
  • @upstash/redis
  • axios
  • formidable
  • md5
  • multer
  • next-auth
  • react-loader-spinner
  • react-spinners
  • remark-gfm
  • sharp
  • … and other unused packages flagged by depcheck

✅ Moved dev-only packages to devDependencies

  • @types/node
  • @types/react
  • @types/react-dom
  • typescript
  • eslint, eslint-config-next
  • prettier, prettier-plugin-tailwindcss
  • tailwindcss, postcss, autoprefixer

✅ Final dependencies (runtime only)

  • @ant-design/icons
  • @clerk/nextjs
  • @clerk/themes
  • @heroicons/react
  • framer-motion
  • next
  • react
  • react-dom
  • react-intersection-observer

✅ Verified

  • next build && next start works
  • Lint/formatting checks run successfully
  • No runtime functionality is broken

Does this introduce a breaking change?

  • Yes
  • No

Other information

This cleanup reduces production bundle size, ensures only runtime packages are shipped, and keeps development tools in the correct section.

@vercel

vercel Bot commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

@vansh2408 is attempting to deploy a commit to the Niko Hoffrén's Team Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
fork-commit-merge-web Ready Ready Preview Comment Oct 1, 2025 6:07am

@nikohoffren nikohoffren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@nikohoffren nikohoffren merged commit 7e3041e into fork-commit-merge:main Oct 1, 2025
5 checks passed
@nikohoffren

Copy link
Copy Markdown
Member

Pull request merged

This is an automated message from Fork, Commit, Merge - web [BOT].

Thank you for your contribution! Your pull request has been merged and your name has been added to the List of Contributors in README.md.
Please leave a star if you enjoyed working on this project.

What's next?

If you're looking for more ways to contribute, I invite you to check out my other projects here.

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.

Remove unused dependencies

2 participants