Skip to content

Conversation

AviVahl
Copy link
Contributor

@AviVahl AviVahl commented Apr 22, 2025

The kind of change this PR does introduce

  • a bug fix
  • a new feature
  • an update to the documentation
  • a code change that improves performance
  • other - deps upgrade

Current behavior

Packages use older minors of ws and debug.

New behavior

Upgraded to latest versions of ws and debug.

Other information (e.g. related issues)

Noticed these cause multiple versions on our end and conflict with openai's peerDep of ws 8.18.x. Had to do an override to get it sorted, so here's a PR.

https://github.com/websockets/ws/releases/tag/8.18.3
https://github.com/debug-js/debug/releases/tag/4.4.1

@AviVahl AviVahl changed the title chore(deps): ws@8.18.1, debug@4.4.0 chore(deps): ws@8.18.2, debug@4.4.0 May 3, 2025
@AviVahl
Copy link
Contributor Author

AviVahl commented May 5, 2025

heya @darrachequesne, any chance you've got time to check this out?
it's small, de-duplicates dependencies, and improves compatibility with projects using the openai package.

@dargmuesli
Copy link

@darrachequesne I'd also like to request this update and a quick patch release. Also I'd like to suggest using the caret symbol instead of the tilde to make requests like this appear less in the future. Thank you for your work!

@AviVahl AviVahl force-pushed the upgrade-ws-and-debug branch from 658c3bc to 40114a8 Compare September 5, 2025 12:41
@AviVahl AviVahl force-pushed the upgrade-ws-and-debug branch from 40114a8 to bf1e01a Compare September 5, 2025 12:41
@AviVahl AviVahl changed the title chore(deps): ws@8.18.2, debug@4.4.0 chore(deps): ws@8.18.3, debug@4.4.1 Sep 5, 2025
@AviVahl
Copy link
Contributor Author

AviVahl commented Sep 5, 2025

rebased to latest ws and debug versions and fixed package-lock.json conflicts

@darrachequesne darrachequesne merged commit 6f9b198 into socketio:main Sep 11, 2025
3 checks passed
@darrachequesne
Copy link
Member

Thanks!

@AviVahl AviVahl deleted the upgrade-ws-and-debug branch September 11, 2025 06:18
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