Skip to content

chore: remove unused devDependencies - #10405

Open
reidbarber wants to merge 2 commits into
mainfrom
chore-remove-unused-devdeps
Open

chore: remove unused devDependencies#10405
reidbarber wants to merge 2 commits into
mainfrom
chore-remove-unused-devdeps

Conversation

@reidbarber

Copy link
Copy Markdown
Member

Removes unused dev dependencies across the repo. No runtime code changes.

Root package.json:

Package Reason
is-mingw, walk-object No references anywhere in the repo.
@vitest/browser-preview browser config uses @vitest/browser-playwright.
@storybook/addon-themes Not registered in any .storybook*/main addons list.
@parcel/reporter-bundle-analyzer Not referenced in any .parcelrc, script, or CI workflow.
eslint-plugin-jest Repo now lints with oxlint (native jest plugin), no ESLint config exists.
regenerator-runtime Nothing imports it and no babel config references it.

Workspace packages:

Package Removed Reason
@parcel/transformer-s2-icon prettier Redundant devDep, already a real dependencies entry.
@react-spectrum/ai @storybook/jest, @react-aria/test-utils, @testing-library/dom, @testing-library/react Unreferenced; tests use @react-spectrum/test-utils-internal + vitest-browser-react + @testing-library/user-event (kept).
@react-spectrum/toast, @react-stately/toast @types/use-sync-external-store use-sync-external-store is neither imported nor a dependency.
@react-spectrum/contextualhelp @react-spectrum/link, @react-spectrum/view No references in the package.
react-aria-components @tailwindcss/postcss No tailwind/postcss usage; the plugin lives in tailwindcss-react-aria-components.

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Smoke test

🧢 Your Project:

@rspbot

rspbot commented Jul 31, 2026

Copy link
Copy Markdown

@rspbot

rspbot commented Jul 31, 2026

Copy link
Copy Markdown

Agent Skills Changes

Modified (2)
Install

React Spectrum S2:

npx skills add https://d1pzu54gtk2aed.cloudfront.net/pr/b49d06235bfb39c20fc2b7c73442c39d2175abfd/

React Aria:

npx skills add https://d5iwopk28bdhl.cloudfront.net/pr/b49d06235bfb39c20fc2b7c73442c39d2175abfd/

"devDependencies": {
"@react-aria/test-utils": "^1.0.0-alpha.8",
"@storybook/jest": "^0.2.3",
"@testing-library/dom": "^10.1.0",

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.

removed a little too much, yarn install has a new line item now

@react-spectrum/ai@workspace:packages/@react-spectrum/ai doesn't provide @testing-library/dom (p837d16), requested by @testing-library/user-event.

Comment thread package.json
"@parcel/optimizer-terser": "^2.16.3",
"@parcel/packager-react-static": "^2.16.3",
"@parcel/packager-ts": "^2.16.3",
"@parcel/reporter-bundle-analyzer": "^2.16.3",

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.

we should have a script target for running this one, it can be useful

Comment thread yarn.lock
languageName: node
linkType: hard

"@testing-library/dom@npm:^10.4.1":

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.

this one should be deduped, the last time i tried to run yarn dedupe though things broke, so maybe just target this one, we can do the full dedupe later

Comment thread yarn.lock
languageName: node
linkType: hard

"prettier@npm:^3.6.2":

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.

how'd we end up with a newer prettier? i guess the devDep was pulling it down, we should probably just use the 3.8.1 version we already have or dedupe to the higher

@adobe adobe deleted a comment from anotherben Aug 1, 2026
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.

3 participants