Skip to content

Clean-up and remove some packages - #598

Merged
AlexJSully merged 1 commit into
mainfrom
alexjsully-260824-update-package-json
Aug 24, 2026
Merged

Clean-up and remove some packages#598
AlexJSully merged 1 commit into
mainfrom
alexjsully-260824-update-package-json

Conversation

@AlexJSully

Copy link
Copy Markdown
Owner

Out of caution of constant supply chain attacks, clean-up and remove some unused or unnecessary packages from the codebase.

@AlexJSully
AlexJSully requested a lite review from Copilot August 24, 2026 19:45
@AlexJSully AlexJSully self-assigned this Aug 24, 2026
@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
portfolio Ready Ready Preview Aug 24, 2026 7:46pm

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No final review comments identify unresolved issues blocking approval.

Pull request overview

This pull request removes unused packages and refreshes dependency and tooling configuration.

Changes:

  • Removes unused dependencies and obsolete configuration.
  • Updates dependencies and lockfile.
  • Simplifies Jest, ESLint, and Prettier settings.
File summaries
File Summary
package.json Updates and removes dependencies.
package-lock.json Refreshes the dependency graph.
next-env.d.ts Adds generated Next.js type references.
jest.config.js Removes obsolete mappings.
eslint.config.js Simplifies ESLint configuration.
.prettierrc Removes obsolete import-order settings.
.github/copilot-instructions.md Updates documented linting rules.
Review details

Suppressed comments (1)

package.json:48

  • Removing the root webpack entry does not remove webpack from the installed graph: package-lock.json still installs node_modules/webpack as the non-optional peer of the retained @sentry/webpack-plugin. If reducing the supply-chain surface is the goal, this change does not achieve it; either keep the required peer explicitly declared, or remove/replace the Sentry Webpack integration and regenerate the lockfile before removing webpack.
		"sharp": "^0.35.3"
  • Files reviewed: 4/7 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@AlexJSully
AlexJSully merged commit b5525ba into main Aug 24, 2026
3 checks passed
@AlexJSully
AlexJSully deleted the alexjsully-260824-update-package-json branch August 24, 2026 19:53
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