Skip to content

chore: remove unused frontend dependencies and fix build - #206

Merged
JamesEjembi merged 1 commit into
VeriNode-Labs:mainfrom
zinodict121:remove-unused-dependencies
Aug 22, 2026
Merged

chore: remove unused frontend dependencies and fix build#206
JamesEjembi merged 1 commit into
VeriNode-Labs:mainfrom
zinodict121:remove-unused-dependencies

Conversation

@zinodict121

Copy link
Copy Markdown
Contributor

Pull Request: Audit and Remove Unused Frontend Dependencies

Fixes #187

Description

This PR addresses issue #187 by auditing the frontend project and removing unused dependencies to help reduce bloat, optimize the workspace, and keep our lockfiles clean. It also includes a few minor syntax, type, and linting fixes that were discovered while ensuring a clean build.

Changes Made

  • Removed Unused Dependencies:

    • Uninstalled @axe-core/playwright and fast-check which were flagged by depcheck as unused.
    • Automatically regenerated package.json and package-lock.json.
  • Fixed Pre-existing Build/Lint Errors:

    • Syntax Error: Fixed Nullish coalescing operator(??) requires parens error in src/components/governance/ProposalCreator.tsx.
    • Type Error: Removed an invalid canceled case from the switch statement handling ProposalStatus in src/components/governance/ProposalCard.tsx.
    • Lint Error: Added a necessary @typescript-eslint/no-explicit-any disable comment for an intentional type cast in src/__tests__/hexDecoder.test.ts.

Testing Instructions

  1. Checkout the branch remove-unused-dependencies.
  2. Run npm install to ensure your node_modules are synced with the updated package-lock.json.
  3. Run npm run build — the build should complete successfully without any Turbopack or TypeScript errors.
  4. Run npm run lint — the linter should pass without the Unexpected any error.

Checklist

  • Audited dependencies using depcheck
  • Unused dependencies successfully removed
  • npm run build completes successfully
  • npm run lint completes successfully
  • Tested locally

@JamesEjembi
JamesEjembi merged commit 367878e into VeriNode-Labs:main Aug 22, 2026
2 of 7 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.

Remove Unused NPM Packages

2 participants