Skip to content

Remove unused packages#58

Merged
pamelafox merged 2 commits into
mainfrom
cleanupfrontend
Oct 25, 2025
Merged

Remove unused packages#58
pamelafox merged 2 commits into
mainfrom
cleanupfrontend

Conversation

@pamelafox

Copy link
Copy Markdown
Contributor

This pull request updates the frontend dependencies in package.json by removing several unused packages from both the dependencies and devDependencies sections. This helps streamline the project, reduce bundle size, and minimize maintenance overhead.

Dependency cleanup:

  • Removed @azure/msal-browser and @azure/msal-react authentication libraries from dependencies.
  • Removed @react-spring/web, dompurify, ndjson-readablestream, react-syntax-highlighter, and scheduler from dependencies, indicating these libraries are no longer used in the codebase.
  • Removed @types/dompurify and @types/react-syntax-highlighter from devDependencies, reflecting the removal of their corresponding libraries.

@pamelafox pamelafox requested a review from Copilot October 25, 2025 06:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes unused frontend dependencies from package.json to reduce bundle size and maintenance overhead. The changes include removing authentication libraries, animation/rendering utilities, and their corresponding TypeScript type definitions.

  • Removed 7 unused packages from dependencies
  • Removed 2 unused TypeScript type definition packages from devDependencies
Files not reviewed (1)
  • app/frontend/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pamelafox pamelafox merged commit 9333813 into main Oct 25, 2025
6 checks passed
@pamelafox pamelafox deleted the cleanupfrontend branch October 25, 2025 06:32
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