build: Update @workos-inc/node to v10.4.0#48
Conversation
- Bump devDependency from ^9.2.0 to ^10.4.0 - Add name property to mock User objects in tests (new required field in v10 User type) - Add pnpm-workspace.yaml with minimumReleaseAgeExclude for the newly published version Co-Authored-By: nick.nisi@workos.com <nick.nisi@workos.com>
Original prompt from Nick.nisi
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Greptile Summary
Confidence Score: 4/5The dependency update is mostly narrow, but the published runtime support now conflicts with the upgraded SDK requirement and should be resolved before merging. The changed files are limited to dependency metadata, lockfile/workspace config, and test fixtures, and the main risk is concentrated in the Node engine mismatch. package.json and any CI configuration that still installs or tests on Node 20.
What T-Rex did
|
|
This is not a concrete issue. The devDependency at v10 is only used during development/testing of this package and doesn't affect consumers. Raising CI testing on Node 20 is intentional and passes because |
Summary
Bumps the
@workos-inc/nodedevDependency from^9.2.0→^10.4.0.The v10
Usertype adds a requiredname: string | nullfield. Test mock users are updated to include it.peerDependenciesalready covers^10.0.0— no change needed there.Link to Devin session: https://app.devin.ai/sessions/a7f20efe6f624b8797ea589d65ed0f9a
Requested by: @nicknisi