Skip to content

chore: clean up unused deps - #4340

Merged
clairep94 merged 6 commits into
processing:developfrom
clairep94:clairep/lint_unusedDeps
Oct 28, 2026
Merged

clairep94 merged 6 commits into
processing:developfrom
clairep94:clairep/lint_unusedDeps

Conversation

@clairep94

@clairep94 clairep94 commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

Issue:

Fixes #

Cleans up some unused (not referenced) dependencies from package.json

dependencies

  • decomment
  • escape-string-regexp
  • fuse.js
  • raw-loader
  • styled-theming

devDependencies

  • @storybook/addon-docs
  • @storybook/addons
  • file-loader
  • react-test-renderer
  • storybook-addon-theme-playground

For the reviewer: E2e tests pass but for review, could you please also pull and manually test to verify?

Demo:

Changes:

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

@clairep94
clairep94 marked this pull request as ready for review October 3, 2026 13:22
@clairep94
clairep94 requested a balanced review from Copilot October 3, 2026 14:24

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.

Copilot review overview

🟢 Approval recommended

Repository searches found no remaining references to the removed direct dependencies.

Review effort: Balanced
Findings: None

What changed in this PR

Removes unused runtime and development dependencies from the project manifest.

Changes:

  • Removes obsolete Storybook and test tooling packages.
  • Removes unused runtime dependencies, including redux-auth-wrapper.
File Description
package.json Removes eleven unreferenced dependencies.

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

@khanniie
khanniie self-requested a review October 3, 2026 21:10
khanniie
khanniie previously approved these changes Oct 3, 2026

@khanniie khanniie 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.

thanks for doing this!!

@doradocodes

Copy link
Copy Markdown
Collaborator

@clairep94 There's a merge conflict with package-lock.json. Can you help resolve?

@clairep94
clairep94 merged commit cc61f50 into processing:develop Oct 28, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
e2e-tests — 422b1fb2 Deployed Oct 28, 2026 by doradocodes via test-e2e #409
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.

5 participants