[2.x] Only compile app scripts if it has contents

[emmadesilva]

Abstract

• Closes Consider removing the app.js file #2027
• Targets HydePHP v2.x Development #1565 via [2.x] New asset system #2006

Comparison

Before

PS H:\monorepo> npm run build

> build
> vite build

vite v5.4.10 building for production...
✓ 2 modules transformed.
Generated an empty chunk: "app".
_media/app.css  38.78 kB │ gzip: 7.51 kB
_media/app.js    0.00 kB │ gzip: 0.02 kB
✓ built in 1.09s

After

Without contents

PS H:\monorepo> npm run build

> build
> vite build

vite v5.4.10 building for production...
✓ 1 modules transformed.
_media/app.css  37.77 kB │ gzip: 7.27 kB
✓ built in 1.44s

With contents

PS H:\monorepo> npm run build

> build
> vite build

vite v5.4.10 building for production...
✓ 2 modules transformed.
_media/app.css  37.77 kB │ gzip: 7.27 kB
_media/app.js    0.02 kB │ gzip: 0.04 kB
✓ built in 986ms

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (c42d8fe) to head (64b33c5).
Report is 3 commits behind head on new-asset-system.

Additional details and impacted files

@@                 Coverage Diff                  @@
##             new-asset-system     #2028   +/-   ##
====================================================
  Coverage              100.00%   100.00%           
  Complexity               1906      1906           
====================================================
  Files                     195       195           
  Lines                    5078      5078           
====================================================
  Hits                     5078      5078           

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