symfony · Kocal · Sep 20, 2025 · Sep 20, 2025
diff --git a/.github/workflows/code-quality.yaml b/.github/workflows/code-quality.yaml
@@ -19,6 +19,114 @@ concurrency:
     cancel-in-progress: true
 
 jobs:
+    validate-packages:
+        name: Validate packages definition
+        runs-on: ubuntu-latest
+        steps:
+            -   uses: actions/checkout@v4
+
+            -   name: Check all composer.json have label "symfony-ux"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name composer.json); do
+                        if ! jq -e '.keywords | index("symfony-ux")' "$file" > /dev/null; then
+                            echo "File $file does not have the keyword 'symfony-ux' in its composer.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all composer.json have license "MIT"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name composer.json); do
+                        if ! jq -e '.license == "MIT"' "$file" > /dev/null; then
+                            echo "File $file does not have the license 'MIT' in its composer.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all composer.json have minimum-stability "dev"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name composer.json); do
+                        if ! jq -e '."minimum-stability" == "dev"' "$file" > /dev/null; then
+                            echo "File $file does not have the minimum-stability 'dev' in its composer.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all composer.json have dependency to "php" >=8.1
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name composer.json); do
+                        if ! jq -e '.require.php | test(">=8.1")' "$file" > /dev/null; then
+                            echo "File $file does not have the dependency 'php' >=8.1 in its composer.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all package.json have license "MIT"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name package.json -not -path "*/tests/*"); do
+                        if ! jq -e '.license == "MIT"' "$file" > /dev/null; then
+                            echo "File $file does not have the license 'MIT' in its package.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all package.json have keywords including "symfony-ux"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name package.json -not -path "*/tests/*"); do
+                        if ! jq -e '.keywords | index("symfony-ux")' "$file" > /dev/null; then
+                            echo "File $file does not have the keyword 'symfony-ux' in its package.json";
+                            exit 1;
+                        fi
+                    done
+
+
+            -   name: Check all package.json have type "module"
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name package.json -not -path "*/tests/*"); do
+                        if ! jq -e '.type == "module"' "$file" > /dev/null; then
+                            echo "File $file does not have the type 'module' in its package.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all package.json have files '["dist"]'
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name package.json -not -path "*/tests/*"); do
+                        if ! jq -e '.files | index("dist")' "$file" > /dev/null; then
+                            echo "File $file does not have the files 'dist' in its package.json";
+                            exit 1;
+                        fi
+                    done
+
+            -   name: Check all package.json peerDependencies are present in devDependencies and importmap to the exact same version
+                if: always()
+                run: |
+                    for file in $(find src/ -mindepth 2 -type f -name package.json -not -path "*/tests/*"); do
+                        peerDependencies=$(jq -r '.peerDependencies | keys[]' "$file")
+                        for peerDependency in $peerDependencies; do
+                            peerDependencyVersion=$(jq -r --arg dep "$peerDependency" '.peerDependencies[$dep]' "$file")
+                            importmapVersion=$(jq -r ".symfony.importmap.\"$peerDependency\" | if type == \"string\" then . else .version end" "$file")
+
+                            if [ "$importmapVersion" == null ]; then
+                                echo "File $file does not have the peerDependency '$peerDependency' in its symfony.importmap, skipping version check";
+                                continue
+                            fi
+
+                            if [ "$peerDependencyVersion" != "$importmapVersion" ]; then
+                                echo "File $file has a mismatch for $peerDependency: peerDependency version is '$peerDependencyVersion' but symfony.importmap version is '$importmapVersion'";
+                                exit 1;
+                            fi
+                        done
+                    done
+
     coding-style-js:
         name: JavaScript Coding Style
         runs-on: ubuntu-latest
@@ -68,7 +176,7 @@ jobs:
 
                   # TODO: Only Turbo has PHPStan configuration, let's improve this later :)
                   PACKAGES=Turbo
-                  #PACKAGES=$(find src/ -mindepth 2 -type f -name composer.json -not -path "*/vendor/*" -printf '%h\n' | sed 's/^src\///' | sort |  tr '\n' ' ')
+                  #PACKAGES=$(find src/ -mindepth 2 -type f -name composer.json -printf '%h\n' | sed 's/^src\///' | sort |  tr '\n' ' ')
                   echo "Packages: $PACKAGES"
                   echo "PACKAGES=$PACKAGES" >> $GITHUB_ENV
 

diff --git a/bin/test_package.sh → bin/unit_test_package.sh b/bin/test_package.sh → bin/unit_test_package.sh
@@ -1,6 +1,6 @@
 #!/bin/bash
 
-# This script is used to test an UX package.
+# This script is used to unit test assets from an UX package.
 # It also handle the case where a package has multiple versions of a peerDependency defined.
 
 SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
@@ -21,13 +21,6 @@ if [ ! -d "$location" ]; then
     exit 1
 fi
 
-shift
-case "$1" in
-  --unit) testType="unit" ;;
-  --browser) testType="browser" ;;
-  *) echo "Unknown test type: $2. Please use --unit or --browser."; exit 1 ;;
-esac
-
 shift
 args=("$@")
 
@@ -38,15 +31,9 @@ if ! command -v jq &> /dev/null; then
 fi
 
 runTestSuite() {
-    if [ "$testType" == "unit" ]; then
-      echo -e "🧪  Running unit tests for $workspace...\n"
-      # Using "cd" instead of "pnpm --filter" prevent rendering issues in GitHub Actions that does not support nested groups
-      (cd "$location"; pnpm exec vitest --run "${args[@]}") || { all_tests_passed=false; }
-    elif [ "$testType" == "browser" ]; then
-      echo -e "🧪  Running browser tests for $workspace...\n"
-      # Using "cd" instead of "pnpm --filter" prevent rendering issues in GitHub Actions that does not support nested groups
-      (cd "$location"; pnpm exec playwright test "${args[@]}") || { all_tests_passed=false; }
-    fi
+  echo -e "🧪  Running unit tests for $workspace...\n"
+  # Using "cd" instead of "pnpm --filter" prevent rendering issues in GitHub Actions that does not support nested groups
+  (cd "$location"; pnpm exec vitest --run "${args[@]}") || { all_tests_passed=false; }
 }
 
 processWorkspace() {
@@ -69,30 +56,6 @@ processWorkspace() {
 
     echo -e "⏳  Processing workspace $workspace at location $location...\n"
 
-    # TODO: Refactor the `test_package.sh` script to its simple form:  only run unit tests for a given UX package,
-    # The rest will be moved to `pnpm` scripts or other CI checks
-    if [ "$testType" == "browser" ]; then
-      runTestSuite
-    fi
-
-    echo "⚙️  Checking '$package_json_path' for peerDependencies and importmap dependencies to have the same version"
-    deps=$(jq -r '.peerDependencies | keys[]' "$package_json_path")
-    for library in $deps; do
-        version=$(jq -r ".peerDependencies.\"$library\"" "$package_json_path")
-        importmap_version=$(jq -r ".symfony.importmap.\"$library\" | if type == \"string\" then . else .version end" "$package_json_path")
-
-        if [ "$importmap_version" == null ]; then
-            echo " ⚠  No importmap version found for $library in $package_json_path, skipping..."
-            continue
-        fi
-
-        if [ "$version" != "$importmap_version" ]; then
-            echo " ⚠  Version mismatch for $library: $version (peerDependencies) vs $importmap_version (importmap)"
-            echo " ⚠  You need to match the version of the \"peerDependency\" with the version in the \"importmap\""
-            exit 1
-        fi
-    done
-
     echo "⚙️  Checking '$package_json_path' for peerDependencies with multiple versions defined"
     deps_with_multiple_versions=$(jq -r '.peerDependencies | to_entries[] | select(.value | contains("||")) | .key' "$package_json_path")
 

diff --git a/src/Autocomplete/assets/package.json b/src/Autocomplete/assets/package.json
@@ -18,9 +18,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Chartjs/assets/package.json b/src/Chartjs/assets/package.json
@@ -18,9 +18,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Cropperjs/assets/package.json b/src/Cropperjs/assets/package.json
@@ -21,9 +21,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Dropzone/assets/package.json b/src/Dropzone/assets/package.json
@@ -21,9 +21,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

@@ -18,7 +18,7 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit",
-        "test:unit": "../../../bin/test_package.sh . --unit",
+        "test:unit": "../../../bin/unit_test_package.sh .",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/LiveComponent/assets/package.json b/src/LiveComponent/assets/package.json
@@ -23,9 +23,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

@@ -22,7 +22,7 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit",
-        "test:unit": "../../../bin/test_package.sh . --unit",
+        "test:unit": "../../../bin/unit_test_package.sh .",
         "check": "biome check",
         "ci": "biome ci"
     },

@@ -20,8 +20,8 @@
         "build": "tsx ../../../../../../bin/build_package.ts .",
         "watch": "tsx ../../../../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:browser",
-        "test:browser": "../../../../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../../../../bin/test_package.sh . --browser --ui",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

@@ -20,8 +20,8 @@
         "build": "tsx ../../../../../../bin/build_package.ts .",
         "watch": "tsx ../../../../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:browser",
-        "test:browser": "../../../../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../../../../bin/test_package.sh . --browser --ui",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Notify/assets/package.json b/src/Notify/assets/package.json
@@ -18,9 +18,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/React/assets/package.json b/src/React/assets/package.json
@@ -18,9 +18,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/StimulusBundle/assets/package.json b/src/StimulusBundle/assets/package.json
@@ -18,7 +18,7 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
+        "test:unit": "../../../bin/unit_test_package.sh .",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Svelte/assets/package.json b/src/Svelte/assets/package.json
@@ -17,9 +17,9 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit && pnpm run test:browser",
-        "test:unit": "../../../bin/test_package.sh . --unit",
-        "test:browser": "../../../bin/test_package.sh . --browser",
-        "test:browser:ui": "../../../bin/test_package.sh . --browser --ui",
+        "test:unit": "../../../bin/unit_test_package.sh .",
+        "test:browser": "playwright test",
+        "test:browser:ui": "playwright test --ui",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Swup/assets/package.json b/src/Swup/assets/package.json
@@ -18,7 +18,7 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit",
-        "test:unit": "../../../bin/test_package.sh . --unit",
+        "test:unit": "../../../bin/unit_test_package.sh .",
         "check": "biome check",
         "ci": "biome ci"
     },

diff --git a/src/Swup/composer.json b/src/Swup/composer.json
@@ -17,6 +17,9 @@
             "homepage": "https://symfony.com/contributors"
         }
     ],
+    "require": {
+        "php": ">=8.1"
+    },
     "autoload": {
         "psr-4": {
             "Symfony\\UX\\Swup\\": "src/"

diff --git a/src/TogglePassword/assets/package.json b/src/TogglePassword/assets/package.json
@@ -21,7 +21,7 @@
         "build": "tsx ../../../bin/build_package.ts .",
         "watch": "tsx ../../../bin/build_package.ts . --watch",
         "test": "pnpm run test:unit",
-        "test:unit": "../../../bin/test_package.sh . --unit",
+        "test:unit": "../../../bin/unit_test_package.sh .",
         "check": "biome check",
         "ci": "biome ci"
     },