Skip to content

7.0 vite - #467

Open
GeorgeCatalinStan wants to merge 6 commits into
7.0from
7.0-vite
Open

7.0 vite#467
GeorgeCatalinStan wants to merge 6 commits into
7.0from
7.0-vite

Conversation

@GeorgeCatalinStan

Copy link
Copy Markdown
Member

transition from Webpack to VITE

George added 3 commits June 17, 2026 10:59
# Conflicts:
#	src/App/assets/js/components/_main.js
#	src/App/assets/js/components/_table_settings.js
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@arhimede
arhimede requested a review from alexmerlin August 10, 2026 10:39
Comment thread src/Admin/src/Handler/Account/PostLoginAccountHandler.php
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.53%. Comparing base (17aa6a5) to head (8405af6).

Additional details and impacted files
@@            Coverage Diff            @@
##                7.0     #467   +/-   ##
=========================================
  Coverage     38.53%   38.53%           
  Complexity      925      925           
=========================================
  Files           169      169           
  Lines          3768     3768           
=========================================
  Hits           1452     1452           
  Misses         2316     2316           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alexmerlin alexmerlin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see a couple of issues with the JS files generated by the build:

  • they are not minified+uglified
  • they seem to contain both the JS content and a base64 dump of the JS content plaed at the end of each file (it looks similar to the JS .map files generated by other libraries)
//# sourceMappingURL=data:application/json;charset=utf-8;base64,...

Similar issue with the CSS file generated by the build:

  • it is not minified

@alexmerlin

Copy link
Copy Markdown
Member

Everything seems to work as expected in the Admin.
But, executing one of the npm run * commands will also display some deprecations:

For example, this is the output of the npm run prod command:

> admin@2.0.0 prod
> vite build

vite v8.2.1 building client environment for production...
Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
1 │ @import 'bootstrap/scss/bootstrap';
  │         ^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
    src/App/assets/scss/index.scss 1:9  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
2 │ @import "settings";
  │         ^^^^^^^^^^
  ╵
    src/App/assets/scss/index.scss 2:9  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
3 │ @import "tools";
  │         ^^^^^^^
  ╵
    src/App/assets/scss/index.scss 3:9  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
4 │ @import "utils";
  │         ^^^^^^^
  ╵
    src/App/assets/scss/index.scss 4:9  root stylesheet

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
5 │ @import "components";
  │         ^^^^^^^^^^^^
  ╵
    src/App/assets/scss/index.scss 5:9  root stylesheet

Deprecation Warning [global-builtin]: Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
Use color.adjust instead.

More info and automated migrator: https://sass-lang.com/d/import

   ╷
74 │ $inverse-danger       : lighten($default-danger, 35%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 74:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [color-functions]: lighten() is deprecated. Suggestions:

color.scale($color, $lightness: 91.5384615385%)
color.adjust($color, $lightness: 35%)

More info: https://sass-lang.com/d/color-functions

   ╷
74 │ $inverse-danger       : lighten($default-danger, 35%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 74:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [global-builtin]: Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
Use color.adjust instead.

More info and automated migrator: https://sass-lang.com/d/import

   ╷
75 │ $inverse-info         : lighten($default-info, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 75:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [color-functions]: lighten() is deprecated. Suggestions:

color.scale($color, $lightness: 89.2996108949%)
color.adjust($color, $lightness: 45%)

More info: https://sass-lang.com/d/color-functions

   ╷
75 │ $inverse-info         : lighten($default-info, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 75:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [global-builtin]: Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
Use color.adjust instead.

More info and automated migrator: https://sass-lang.com/d/import

   ╷
76 │ $inverse-primary      : lighten($default-primary, 30%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 76:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [color-functions]: lighten() is deprecated. Suggestions:

color.scale($color, $lightness: 93.8650306748%)
color.adjust($color, $lightness: 30%)

More info: https://sass-lang.com/d/color-functions

   ╷
76 │ $inverse-primary      : lighten($default-primary, 30%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 76:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [global-builtin]: Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
Use color.adjust instead.

More info and automated migrator: https://sass-lang.com/d/import

   ╷
77 │ $inverse-success      : lighten($default-success, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 77:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [color-functions]: lighten() is deprecated. Suggestions:

color.scale($color, $lightness: 90%)
color.adjust($color, $lightness: 45%)

More info: https://sass-lang.com/d/color-functions

   ╷
77 │ $inverse-success      : lighten($default-success, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 77:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [global-builtin]: Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
Use color.adjust instead.

More info and automated migrator: https://sass-lang.com/d/import

   ╷
78 │ $inverse-warning      : lighten($default-warning, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 78:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Deprecation Warning [color-functions]: lighten() is deprecated. Suggestions:

color.scale($color, $lightness: 90%)
color.adjust($color, $lightness: 45%)

More info: https://sass-lang.com/d/color-functions

   ╷
78 │ $inverse-warning      : lighten($default-warning, 45%);
   │                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    src/App/assets/scss/settings/baseColors.scss 78:25  @import
    src/App/assets/scss/settings/index.scss 4:9         @import
    src/App/assets/scss/index.scss 2:9                  root stylesheet

Warning: 59 repetitive deprecation warnings omitted.
Run in verbose mode to see all warnings.

✓ 80 modules transformed.
[vite-plugin-static-copy] Copied 312 items.
computing gzip size...
public/fonts/themify.woff                56.10 kB
public/fonts/fontawesome-webfont.woff2   66.62 kB
public/fonts/fontawesome-webfont.eot     70.80 kB
public/fonts/themify.ttf                 78.55 kB
public/fonts/themify.eot                 78.72 kB
public/fonts/fontawesome-webfont.woff    83.58 kB
public/fonts/fontawesome-webfont.ttf    142.04 kB
public/fonts/themify.svg                234.26 kB │ gzip:  74.41 kB
public/fonts/fontawesome-webfont.svg    365.61 kB │ gzip: 110.52 kB
public/css/app.css                      982.19 kB │ gzip: 112.10 kB
public/js/admin.js                        1.87 kB │ gzip:   0.55 kB
public/js/table_settings.js               2.26 kB │ gzip:   0.87 kB
public/js/user.js                         2.53 kB │ gzip:   0.74 kB
public/js/jquery.module.js               78.43 kB │ gzip:  27.78 kB
public/js/app.js                        424.35 kB │ gzip: 139.90 kB

✓ built in 4.33s
[PLUGIN_TIMINGS] Your build spent 96% of 4.3s inside plugin hooks (4.2s).
Measured inside the callback, so queue time is excluded and time the callback itself awaited is not:
  - vite:css transform (90%, 3.9s, 1 call)
See https://rolldown.rs/reference/InputOptions.checks#plugintimings for more details.

@arhimede Should this PR address those deprecations as well?

@arhimede

Copy link
Copy Markdown
Member

@GeorgeCatalinStan
let's do this modifications related to import and global built in functions now, in this PR

that way we have few years until the next major upgrade
this is quite major, so we target to have a clean npm run

@arhimede
arhimede requested a review from alexmerlin August 13, 2026 19:15
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.

3 participants