Update non-major dependencies#120
Conversation
|
Updated the AI SDK and linting tooling to their latest minor releases across both the PR review and summary services for consistency and to incorporate recent fixes and improvements. Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 672 | Completion Tokens: 404 |
There was a problem hiding this comment.
Enjoy this friendly AI-assisted code review. Some suggestions come from predictions and may not always be perfect. Feel free to choose the ideas that fit your style. Your choices guide the process, and AI is here to help.
Model: o4-mini-2025-04-16 | Prompt Tokens: 1204 | Completion Tokens: 1921
| "@sap-ai-sdk/ai-api": "2.10.0", | ||
| "@sap-ai-sdk/orchestration": "2.10.0", | ||
| "@sap-ai-sdk/ai-api": "2.11.0", | ||
| "@sap-ai-sdk/orchestration": "2.11.0", |
There was a problem hiding this comment.
Consider using a caret (^) range for SDK dependencies so you automatically receive compatible patch updates without manually bumping each time. For example:
"@sap-ai-sdk/ai-api": "^2.11.0",
"@sap-ai-sdk/orchestration": "^2.11.0",| "@sap-ai-sdk/ai-api": "2.10.0", | ||
| "@sap-ai-sdk/orchestration": "2.10.0", | ||
| "@sap-ai-sdk/ai-api": "2.11.0", | ||
| "@sap-ai-sdk/orchestration": "2.11.0", |
There was a problem hiding this comment.
Before upgrading to 2.11.0, verify the upstream changelog for any breaking changes or required migration steps. You could add a comment or link to the changelog in your release notes:
# Upgrade @sap-ai-sdk to 2.11.0
- See https://github.com/SAP/ai-sdk/releases/tag/v2.11.0 for breaking changes| "tsx": "4.22.3", | ||
| "typescript": "6.0.3", | ||
| "typescript-eslint": "8.59.4", | ||
| "typescript-eslint": "8.60.0", |
There was a problem hiding this comment.
Similar to the SDK bump, consider using a caret range for the ESLint plugin dependency to get minor fixes automatically:
"typescript-eslint": "^8.60.0",| "@sap-ai-sdk/ai-api": "2.10.0", | ||
| "@sap-ai-sdk/orchestration": "2.10.0", | ||
| "@sap-ai-sdk/ai-api": "2.11.0", | ||
| "@sap-ai-sdk/orchestration": "2.11.0", |
There was a problem hiding this comment.
This workspace also upgrades the SAP AI SDK. To keep versioning consistent across workspaces and simplify future bumps, you might extract these shared dependencies into the root package.json or a shared configs file.
| "tsx": "4.22.3", | ||
| "typescript": "6.0.3", | ||
| "typescript-eslint": "8.59.4", | ||
| "typescript-eslint": "8.60.0", |
There was a problem hiding this comment.
Ensure that bumping typescript-eslint here aligns with your overall ESLint config. If you have a centralized ESLint setup, consider updating it in one place instead of per-package.
a9b07a7 to
ab6d417
Compare
|
This update standardizes dependency versions across both the pr-review and pr-summary packages, ensuring that core AI SDK modules, tooling runtimes, and linting plugins are aligned to their latest patch releases for improved stability and compatibility. Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 756 | Completion Tokens: 949 | Diff Range: a9b07a7...ab6d417 |
There was a problem hiding this comment.
We’ve assembled a friendly, AI-powered review of your code with practical suggestions. Treat each point as a flexible idea rather than a strict rule. Choose what aligns with your goals and style. You’re in control of the final decisions, and AI is here to support you every step of the way.
Model: o4-mini-2025-04-16 | Prompt Tokens: 924 | Completion Tokens: 2148 | Diff Range: a9b07a7...ab6d417
| "eslint-plugin-unicorn": "64.0.0", | ||
| "prettier": "3.8.3", | ||
| "tsx": "4.22.3", | ||
| "tsx": "4.22.4", |
There was a problem hiding this comment.
It looks like you’re bumping the tsx version explicitly rather than using a semver range. For better flexibility and to automatically pick up compatible patch/minor updates, consider using a caret prefix. This way you’ll avoid manual bumps for every small release.
- "tsx": "4.22.4",
+ "tsx": "^4.22.4",| "eslint-plugin-sonarjs": "4.0.3", | ||
| "eslint-plugin-unicorn": "64.0.0", | ||
| "prettier": "3.8.3", | ||
| "tsx": "4.22.3", | ||
| "tsx": "4.22.4", | ||
| "typescript": "6.0.3", | ||
| "typescript-eslint": "8.59.4", | ||
| "typescript-eslint": "8.60.0", | ||
| "yaml": "2.9.0" |
There was a problem hiding this comment.
I see the same set of ESLint and Prettier dependencies repeated in both pr-review and pr-summary packages. You could centralize shared devDependencies in a workspace root to avoid duplication and ensure consistency across all sub-packages.
For example, in your monorepo root package.json:
{
"name": "my-monorepo",
"private": true,
"workspaces": [
"pr-review",
"pr-summary"
],
"devDependencies": {
"eslint-plugin-sonarjs": "^4.0.3",
"eslint-plugin-unicorn": "^64.0.0",
"prettier": "^3.8.3",
"tsx": "^4.22.4",
"typescript": "^6.0.3",
"@typescript-eslint/eslint-plugin": "^8.60.0",
"yaml": "^2.9.0"
}
}Then remove these entries from each sub-package’s devDependencies. This reduces maintenance overhead and keeps versions in sync.
This PR contains the following updates:
2.10.0→2.11.02.10.0→2.11.04.22.3→4.22.48.59.4→8.60.0Release Notes
SAP/ai-sdk-js (@sap-ai-sdk/ai-api)
v2.11.0Compare Source
Patch Changes
c8c0e41]75bb9a9]SAP/ai-sdk-js (@sap-ai-sdk/orchestration)
v2.11.0Compare Source
Minor Changes
f72bb58: [feat] Update orchestration specification to v0.131.3Patch Changes
cbcefb9: [fix] Route messages tomessages_historywhen using an orchestration config reference or prompt template reference.7db7237: [Fixed Issue] Disallow providing both orchestration config reference and config object at the same time.c8c0e41]75bb9a9]privatenumber/tsx (tsx)
v4.22.4Compare Source
Bug Fixes
This release is also available on:
typescript-eslint/typescript-eslint (typescript-eslint)
v8.60.0Compare Source
This was a version bump only for typescript-eslint to align it with other projects, there were no code changes.
See GitHub Releases for more information.
You can read about our versioning strategy and releases on our website.
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.