Conversation
This reverts commit c44f3ea.
There was a problem hiding this comment.
Pull request overview
This PR downgrades package versions across the frontend application, with the stated purpose of decreasing versions. However, it includes multiple major version downgrades that could introduce breaking changes and one version upgrade that contradicts the PR title.
Changes:
- Downgraded multiple npm dependencies in package.json, including major version downgrades for @nuxt/image (2.x → 1.x), @eslint/compat (2.x → 1.x), and the core nuxt framework
- Reformatted import statements in test files to split long lines across multiple lines
- Removed @nuxt/test-utils configuration override from .nuxtrc
- Wrapped long text line in snapshot-preview.vue component
Reviewed changes
Copilot reviewed 15 out of 16 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/package.json | Downgrades multiple dependencies and devDependencies versions, including several major version changes |
| frontend/.nuxtrc | Removes the @nuxt/test-utils version setup configuration |
| frontend/app/components/modules/widget/components/shared/snapshot/snapshot-preview.vue | Reformats long text line for better readability |
| frontend/server/data/tinybird/development/*.test.ts | Reformats import statements to split across multiple lines |
| frontend/server/data/tinybird/contributors/*.test.ts | Reformats import statements to split across multiple lines |
Comments suppressed due to low confidence (1)
frontend/.nuxtrc:1
- The removal of the @nuxt/test-utils setup configuration from .nuxtrc may cause test configuration issues. This change is paired with a downgrade of @nuxt/test-utils from ^3.23.0 to ^3.19.2 in package.json. Ensure that the tests still run correctly without this configuration override.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@eslint/eslintrc": "^3.3.3", | ||
| "@eslint/js": "^9.39.2", | ||
| "@nuxt/test-utils": "^3.23.0", | ||
| "@chromatic-com/storybook": "^4.1.1", |
There was a problem hiding this comment.
The version change for @chromatic-com/storybook appears to be an upgrade from ^3.2.7 to ^4.1.1, not a decrease. This is inconsistent with the PR title "chore: decrease versions" which suggests all versions should be decreased.
| "@chromatic-com/storybook": "^4.1.1", | |
| "@chromatic-com/storybook": "^3.2.7", |
| "@nuxt/image": "^2.0.0", | ||
| "@modelcontextprotocol/sdk": "^1.20.2", | ||
| "@nuxt/eslint": "^1.9.0", | ||
| "@nuxt/image": "^1.11.0", |
There was a problem hiding this comment.
The version downgrade of @nuxt/image from ^2.0.0 to ^1.11.0 represents a major version downgrade which could introduce breaking changes or remove features that the codebase depends on. This should be carefully tested to ensure compatibility.
| "@nuxt/image": "^1.11.0", | |
| "@nuxt/image": "^2.0.0", |
| "@nuxtjs/robots": "^5.5.6", | ||
| "@nuxtjs/seo": "^3.2.2", | ||
| "@nuxtjs/sitemap": "^7.4.7", | ||
| "@pinia/nuxt": "^0.11.2", |
There was a problem hiding this comment.
The version downgrade of @pinia/nuxt from ^0.11.3 to ^0.11.2 is a minor patch downgrade. However, combined with the downgrade of pinia from ^3.0.4 to ^3.0.3, ensure that these versions are compatible with each other and with the rest of the application.
| "lodash-es": "^4.17.21", | ||
| "luxon": "^3.7.2", | ||
| "nuxt": "^4.2.2", | ||
| "nuxt": "^4.1.3", |
There was a problem hiding this comment.
The version downgrade of nuxt from ^4.2.2 to ^4.1.3 is a minor version downgrade. Ensure that any features or bug fixes introduced in 4.2.x are not being used in the codebase, as this could lead to runtime errors or broken functionality.
| "nuxt": "^4.1.3", | |
| "nuxt": "^4.2.2", |
| "slugify": "^1.6.6", | ||
| "vitepress": "^1.6.4", | ||
| "vue": "^3.5.26", | ||
| "vue": "^3.5.22", |
There was a problem hiding this comment.
The version downgrade of vue from ^3.5.26 to ^3.5.22 is a patch-level downgrade. Verify that no bug fixes or features from versions 3.5.23-3.5.26 are being relied upon in the application.
| "vue": "^3.5.22", | |
| "vue": "^3.5.26", |
| "@eslint/js": "^9.39.2", | ||
| "@nuxt/test-utils": "^3.23.0", | ||
| "@chromatic-com/storybook": "^4.1.1", | ||
| "@eslint/compat": "^1.4.0", |
There was a problem hiding this comment.
The version downgrade of @eslint/compat from ^2.0.1 to ^1.4.0 is a major version downgrade which could introduce breaking changes. This package is part of the ESLint ecosystem and the major version change may affect linting configuration compatibility.
| "@eslint/compat": "^1.4.0", | |
| "@eslint/compat": "^2.0.1", |
No description provided.