Skip to content

Conversation

@yi-nuo426
Copy link
Contributor

Signed-off-by: Yi Nuo [email protected]

Copilot AI review requested due to automatic review settings January 16, 2026 14:44
Signed-off-by: Yi Nuo <[email protected]>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the @meshery/schemas package from version 0.8.107 to 0.8.118 and also upgrades xstate from 5.19.2 to 5.25.0.

Changes:

  • Updated @meshery/schemas to ^0.8.118 in both devDependencies and peerDependencies
  • Updated xstate to ^5.25.0 in both devDependencies and peerDependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"@eslint/eslintrc": "^3.3.1",
"@eslint/js": "^9.39.2",
"@meshery/schemas": "^0.8.107",
"@meshery/schemas": "^0.8.118",
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The package-lock.json file has not been updated to reflect this version change. It currently references version 0.8.116 in node_modules/@meshery/schemas and still contains references to ^0.8.107 in the packages section (lines 32, 71). You need to run npm install to update the lock file to ensure the correct version (0.8.118) is installed.

Copilot uses AI. Check for mistakes.
"tsup": "^8.4.0",
"typescript": "^5.8.2",
"xstate": "^5.19.2"
"xstate": "^5.25.0"
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The package-lock.json file still references ^5.19.2 for xstate in the packages section (lines 68, 75). While the resolved version in node_modules is 5.25.1 (which satisfies ^5.25.0), the packages section needs to be updated by running npm install to properly reflect the new version constraint.

Copilot uses AI. Check for mistakes.
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.

2 participants