Fix dependency vulnerabilities reported by Dependabot by max-programming · Pull Request #259 · max-programming/hacktoberfest-projects · GitHub
Skip to content

Fix dependency vulnerabilities reported by Dependabot - #259

Merged
max-programming merged 1 commit into
mainfrom
claude/beautiful-tesla-dt9y4h
Oct 12, 2026
Merged

max-programming merged 1 commit into
mainfrom
claude/beautiful-tesla-dt9y4h

Conversation

@max-programming

Copy link
Copy Markdown
Owner

Summary

pnpm audit checks the same GitHub Advisory Database that Dependabot uses. It reported 130 advisories (5 critical, 70 high, 55 moderate, 4 low); after this PR, 1 is left, and that one has no patched release yet.

Changes

  • Remove 24 unused packages. The live site is now static and no longer imports any of them: axios, next-auth, @auth/*, drizzle-orm, drizzle-kit, postgres, zod, daisyui, framer-motion, react-icons, next-cloudinary, next-seo, jotai, react-hook-form, react-hot-toast and others. These caused most of the advisories, including the critical Auth.js ones. The archived code in archive/ isn't built, and the old dependency list is in git history.
  • Upgrade Next.js from 15.5.9 to 15.5.27, with eslint-config-next to match. This fixes the critical remote-code-execution advisories and the other Next.js ones. It stays on the 15.5 line, so no migration is needed. Everything else is updated within its existing version range.
  • pnpm overrides for vulnerable dependencies-of-dependencies:
    • postcss ^8.5.29: Next.js pins 8.4.31.
    • yaml ^2.8.3: now not installed at all, since it was only an optional peer.
    • postcss-selector-parser ^7.1.6: used by Tailwind 3.
  • Move the now-unused src/lib/utils.ts (clsx/tailwind-merge) to archive/.

Remaining

  • braces <=3.0.3 (high) has no fixed version published. It's only reached through ESLint tooling (eslint-config-next → fast-glob → micromatch) and never ships to visitors.

Testing

  • pnpm build passes, and every route is still static (○).
  • The generated CSS is byte-identical with and without the overrides (compared SHA-256 hashes).
  • pnpm lint and prettier --check pass.
  • Ran the production server: / and /contributors return 200, old routes return 307 to /, and /github returns 308. Screenshots at desktop and mobile widths show no broken images and no horizontal scroll.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RUmCecArL3A392pphy2xZN


Generated by Claude Code

pnpm audit (same GitHub Advisory Database as Dependabot) reported 130
advisories (5 critical, 70 high). Now down to 1 with no patched release.

- Remove 24 packages the static site no longer imports (axios,
  next-auth, @auth/*, drizzle-orm, drizzle-kit, postgres, zod, daisyui,
  framer-motion, react-icons, next-cloudinary, ...). The archived code in
  archive/ is not built; its old package.json is in git history.
- Upgrade Next.js 15.5.9 -> 15.5.27 (fixes the critical RCE advisories)
  and eslint-config-next to match; update the rest within their ranges.
- Override transitive postcss (^8.5.29, Next pins 8.4.31), yaml and
  postcss-selector-parser (^7.1.6, used by Tailwind 3). The generated
  CSS is byte-identical before and after.
- Move the now-unused src/lib/utils.ts (clsx/tailwind-merge) to archive.

Remaining: braces <=3.0.3 (high) has no fixed version yet; it is only
used by ESLint tooling via eslint-config-next and never ships to users.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RUmCecArL3A392pphy2xZN
@vercel

vercel Bot commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

@max-programming
max-programming merged commit b1a0886 into main Oct 12, 2026
5 checks passed

This branch was successfully deployed

1 active deployment
Preview — 8c9cdedb Deployed Oct 12, 2026 by vercel[bot]
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