Skip to content

Comments

chore: Dedupe packages and fix types#9667

Open
snowystinger wants to merge 59 commits intomainfrom
dedupe-packages-and-fix-types
Open

chore: Dedupe packages and fix types#9667
snowystinger wants to merge 59 commits intomainfrom
dedupe-packages-and-fix-types

Conversation

@snowystinger
Copy link
Member

@snowystinger snowystinger commented Feb 16, 2026

Closes

Fixes the issue we have where one of our patches applies multiple times as more yarn install's are run.
Also fixes other typescript errors we have in our node_modules, usually by updating to a newer version where it's been fixed.
Finally, dedupes our packages.

This is partially follow up to #9066 and I just got tired of seeing the ts errors

Should wait to go in until the Storybook 9 PR has been merged

✅ 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:

Make sure to check the docs and generated api tables, that would be the place most likely to be affected

🧢 Your Project:

# Conflicts:
#	.storybook-s2/docs/Illustrations.jsx
#	.storybook-s2/docs/Intro.jsx
#	.storybook-s2/preview.tsx
#	package.json
#	packages/@react-spectrum/badge/chromatic-fc/Badge.stories.tsx
#	packages/@react-spectrum/badge/chromatic/Badge.stories.tsx
#	packages/@react-spectrum/calendar/stories/Calendar.stories.tsx
#	packages/@react-spectrum/calendar/stories/RangeCalendar.stories.tsx
#	packages/@react-spectrum/illustratedmessage/chromatic/IllustratedMessage.Languages.stories.tsx
#	packages/@react-spectrum/illustratedmessage/chromatic/IllustratedMessage.stories.tsx
#	yarn.lock
This reverts commit 225c07b.
This reverts commit a420e84.
snowystinger and others added 15 commits July 28, 2025 11:46
# Conflicts:
#	.circleci/comment.js
#	examples/s2-next-macros/package.json
#	package.json
#	packages/@react-spectrum/s2/chromatic/Picker.stories.tsx
#	packages/@react-spectrum/s2/chromatic/Tabs.stories.tsx
#	packages/@react-spectrum/s2/stories/Breadcrumbs.stories.tsx
#	packages/@react-spectrum/s2/stories/DatePicker.stories.tsx
#	packages/@react-spectrum/s2/stories/DateRangePicker.stories.tsx
#	packages/@react-spectrum/s2/stories/Toast.stories.tsx
#	packages/dev/storybook-builder-parcel/package.json
#	packages/dev/storybook-react-parcel/package.json
#	packages/react-aria-components/stories/Autocomplete.stories.tsx
#	packages/react-aria-components/stories/Button.stories.tsx
#	packages/react-aria-components/stories/Dropzone.stories.tsx
#	packages/react-aria-components/stories/ListBox.stories.tsx
#	packages/react-aria-components/stories/Menu.stories.tsx
#	plop-templates/@react-spectrum/stories/{{ componentName }}.stories.tsx.hbs
#	starters/docs/package.json
#	starters/docs/stories/Tabs.stories.tsx
#	starters/tailwind/package.json
#	yarn.lock
@snowystinger snowystinger changed the base branch from main to upgrade-sb-9 February 16, 2026 22:48
@rspbot
Copy link

rspbot commented Feb 16, 2026

This reverts commit d263826.
@snowystinger snowystinger marked this pull request as ready for review February 16, 2026 23:08
@rspbot
Copy link

rspbot commented Feb 16, 2026

Base automatically changed from upgrade-sb-9 to main February 20, 2026 16:27
"@types/node@npm:*": "^24",
"@types/node@npm:^18.0.0": "^24",
"@types/node@npm:>= 8": "^24",
"unist-util-is": "6.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

not sure where this one came from, do we actually need it?

@rspbot
Copy link

rspbot commented Feb 20, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants