chore: Update package dependencies in package.json #36
Conversation
- Downgraded @eslint/js from version 9.27.0 to 8.57.1. - Updated typescript-eslint from version 8.32.1 to 8.33.0. - Added new dependencies: @slack/web-api (7.9.2), axios (1.9.0), axios-rate-limit (1.4.0), axios-retry (4.5.0), and mongoose (8.15.1). These changes ensure compatibility and introduce new functionalities for the project.
📝 WalkthroughWalkthroughThe Changes
Possibly related PRs
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
🔍 Vulnerabilities of
|
| digest | sha256:da30eca7154c7c4cb2616e31f7411021ea9146d5b15c68adf2a57df4c52b164c |
| vulnerabilities |     |
| platform | linux/amd64 |
| size | 218 MB |
| packages | 358 |
📦 Base Image alpine:3
| also known as |
|
| digest | sha256:1c4eef651f65e2f7daee7ee785882ac164b02b78fb74503052a26dc061c90474 |
| vulnerabilities |   |
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description | ||||||||||||
Description
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
workers/main/package.json (1)
33-41: Evaluate new runtime dependencies
You’ve added@slack/web-api,axios,axios-rate-limit,axios-retry, andmongooseas runtime deps. Please verify:
- Each is actually imported/used in the codebase.
- No known vulnerabilities exist (
npm audit).- The Node.js version in CI/production meets the requirements of Mongoose v8.15.1 and TypeScript v5.8.3.
Consider adding an
"engines": { "node": ">=18.0.0" }field to enforce compatibility.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
workers/main/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
workers/main/package.json(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: Docker Security Scanning (n8n, Dockerfile.n8n, n8n-test:latest)
- GitHub Check: Service Availability Check
🔇 Additional comments (2)
workers/main/package.json (2)
27-27: typescript-eslint patch update looks good
Upgrading from v8.32.1 to v8.33.0 is a non-breaking, patch-level change. No further action required.
11-11:❓ Verification inconclusive
Potential ESLint plugin/core version mismatch
The@eslint/jspackage has been downgraded to v8.57.1 while the coreeslintremains at v9.27.0. A major‐version mismatch between plugin and core can break linting.Run this to inspect the peerDependencies for compatibility:
🏁 Script executed:
#!/bin/bash # Check peerDependencies for @eslint/js v8.57.1 npm view @eslint/[email protected] peerDependenciesLength of output: 45
🏁 Script executed:
#!/bin/bash # Inspect peerDependencies for @eslint/js v8.57.1 in JSON format npm view @eslint/[email protected] peerDependencies --jsonLength of output: 52
Verify ESLint plugin/core version alignment
The@eslint/jspackage is pinned at v8.57.1 while the coreeslintremains at v9.27.0. A major‐version mismatch between plugin and core can break your linting setup—please confirm compatibility by inspecting the plugin’s peerDependencies or official docs.• File:
workers/main/package.json, line 11
• Run locally to double-check:npm view @eslint/[email protected] peerDependencies• Or verify on the npm registry: https://www.npmjs.com/package/@eslint/js
killev
left a comment
killev
left a comment
There was a problem hiding this comment.
Why did you reduce ECLint. I don't mind, just need to document
…ge-lock.json - Updated @eslint/js from version 8.57.1 to 9.27.0 to incorporate the latest features and improvements. - Adjusted node engine compatibility in package-lock.json to support newer versions. These changes enhance code quality and maintainability by ensuring the use of the latest ESLint version.
|
|
The version downgrade was due to a compatibility issue with one of the plugins that is no longer in use. https://typescript-eslint.io/users/dependency-versions#eslint |
These changes ensure compatibility and introduce new functionalities for the project.