Merge branch 'symfony:2.x' into svelte-5-support

Author: ChqThomas

.github/CODEOWNERS

@@ -0,0 +1,3 @@
+src/LazyImage @Kocal
+src/Map @Kocal
+src/Translator @Kocal

.github/ISSUE_TEMPLATE/1-bug_report.md

@@ -2,7 +2,31 @@
 name: '🐞 Bug Report'
 about: Report a bug in existing features
 title: ''
-labels: 'bug'
+labels: ['Bug']
 assignees: ''
 
 ---
+
+<!-- ======================== Guidelines ============================
+
+Thank you for taking the time to report a bug! 🙏
+
+Please follow these guidelines to help us understand & fix the issue:
+
+Describe Your Problem 🎯
+- Clearly explain the problem you're facing;
+- Describe what you expected to happen versus what actually occurred;
+- List the steps to reproduce the bug.
+	
+Provide Detailed Information 📋
+- Share relevant details: Component version, errors, screenshots;
+- If possible, provide a minimal reproducer in a GitHub repository.
+
+Be Kind and Respectful 🙂
+- Stay patient & open to feedback, and act with kindness and respect;
+- Remember that this is a volunteer-driven project.
+
+============================= Guidelines ======================== -->
+
+
+

.github/ISSUE_TEMPLATE/2-feature_request.md

@@ -2,7 +2,7 @@
 name: '🚀 Feature Request'
 about: Suggest ideas for new features or enhancements
 title: ''
-labels: 'RFC'
+labels: ['RFC']
 assignees: ''
 
 ---

.github/generate-dist-files-size-diff.mjs

@@ -0,0 +1,177 @@
+/**
+ * Generate a markdown table with the difference in size of the dist files between the base and the PR.
+ */
+
+/*
+Usage:
+```shell
+BASE_DIST_FILES='{"src/Autocomplete/assets/dist/controller.js":{"size":15382,"size_gz":3716},"src/Chartjs/assets/dist/controller.js":{"size":2281,"size_gz":771},"src/Cropperjs/assets/dist/controller.js":{"size":1044,"size_gz":475}}' \
+HEAD_DIST_FILES='{"src/Chartjs/assets/dist/controller.js":{"size":1281,"size_gz":171},"src/Cropperjs/assets/dist/controller.js":{"size":1044,"size_gz":475},"src/Cropperjs/assets/dist/style.min.css":{"size":32,"size_gz":66},"src/Dropzone/assets/dist/controller.js":{"size":3199,"size_gz":816},"src/Map/src/Bridge/Google/assets/dist/foo.js":{"size":3199,"size_gz":816}}' \
+HEAD_REPO_NAME='kocal/symfony-ux' \
+HEAD_REF='my-branch-name' \
+  node .github/generate-dist-files-size-diff.mjs
+```
+ */
+
+if (!process.env.BASE_DIST_FILES) {
+    throw new Error('Missing or invalid "BASE_DIST_FILES" env variable.');
+}
+
+if (!process.env.HEAD_DIST_FILES) {
+    throw new Error('Missing or invalid "HEAD_DIST_FILES" env variable.');
+}
+
+if (!process.env.HEAD_REPO_NAME) {
+    throw new Error('Missing or invalid "HEAD_REPO_NAME" env variable.');
+}
+
+if (!process.env.HEAD_REF) {
+    throw new Error('Missing or invalid "HEAD_REF" env variable.');
+}
+
+/**
+ * Adapted from https://gist.github.com/zentala/1e6f72438796d74531803cc3833c039c?permalink_comment_id=4455218#gistcomment-4455218
+ * @param {number} bytes
+ * @param {number} digits
+ * @returns {string}
+ */
+function formatBytes(bytes, digits = 2) {
+    if (bytes === 0) {
+        return '0 B';
+    }
+    const sizes = [`B`, 'kB', 'MB'];
+    const i = Math.floor(Math.log(bytes) / Math.log(1024));
+
+    return parseFloat((bytes / Math.pow(1024, i)).toFixed(digits)) + ' ' + sizes[i];
+}
+
+/**
+ * @param {number} from
+ * @param {number} to
+ * @returns {number}
+ */
+function computeDiffPercent(from, to) {
+    if (from === to) {
+        return 0;
+    }
+
+    return Math.round((from - to) / from * -100);
+}
+
+/**
+ * @param {number} percent
+ * @returns {string}
+ */
+function formatDiffPercent(percent) {
+    return percent > 0 ? `+${percent}% 📈` : percent < 0 ? `${percent}% 📉` : `${percent}%`;
+}
+
+export function main() {
+    const repoUrl = `https://github.com/${process.env.HEAD_REPO_NAME}`;
+    /** @type {Record<string, {size: number, size_gz: number}>} */
+    const base = JSON.parse(process.env.BASE_DIST_FILES);
+    /** @type {Record<string, {size: number, size_gz: number}>} */
+    const pr = JSON.parse(process.env.HEAD_DIST_FILES);
+    let output = '<h1>📊 Packages dist files size difference</h1>\n\n';
+
+    /**
+     * @type {Map<string, {
+     *     meta: {
+     *         packageName: string,
+     *         bridgeName: string,
+     *         url: string,
+     *     },
+     *     files: Set<{
+     *         state: 'added' | 'removed' | 'changed',
+     *         before: {size: number, sizeGz: number},
+     *         after: {size: number, sizeGz: number},
+     *         diffPercent: {size: number, sizeGz: number},
+     *         meta: {fileNameShort: string, fileNameUrl: string}
+     *     }>
+     * }>}
+     */
+    const packagesFiles = [...new Set([...Object.keys(pr), ...Object.keys(base)])]
+        .sort()
+        .reduce((acc, file) => {
+            const beforeSize = base[file]?.size || 0;
+            const afterSize = pr[file]?.size || 0;
+            const beforeSizeGz = base[file]?.size_gz || 0;
+            const afterSizeGz = pr[file]?.size_gz || 0;
+
+            if (beforeSize !== afterSize) {
+                const isBridge = file.includes('src/Bridge'); // we assume that's enough for now
+                const packageName = file.split('/')[1];
+                const bridgeName = isBridge ? file.split('/')[4] : '';
+                const key = isBridge ? `${packageName} (Bridge ${bridgeName})` : packageName;
+                if (!acc.has(key)) {
+                    acc.set(key, {
+                        meta: {
+                            packageName,
+                            bridgeName,
+                            url: isBridge ? `${repoUrl}/tree/${process.env.HEAD_REF}/src/${packageName}/src/Bridge/${bridgeName}/assets/dist` : `${repoUrl}/tree/${process.env.HEAD_REF}/src/${packageName}/assets/dist`,
+                        }, files: new Set(),
+                    });
+                }
+
+                const added = !base[file] && pr[file];
+                const removed = base[file] && !pr[file];
+
+                acc.get(key).files.add({
+                    state: added ? 'added' : (removed ? 'removed' : 'changed'),
+                    before: { size: beforeSize, sizeGz: beforeSizeGz },
+                    after: { size: afterSize, sizeGz: afterSizeGz },
+                    diffPercent: {
+                        size: removed ? -100 : (added ? 100 : computeDiffPercent(beforeSize, afterSize)),
+                        sizeGz: removed ? -100 : (added ? 100 : computeDiffPercent(beforeSizeGz, afterSizeGz)),
+                    },
+                    meta: {
+                        fileNameShort: file.replace(isBridge ? `src/${file.split('/')[1]}/src/Bridge/${file.split('/')[4]}/assets/dist/` : `src/${file.split('/')[1]}/assets/dist/`, ''),
+                        fileNameUrl: `${repoUrl}/blob/${process.env.HEAD_REF}/${file}`,
+                    },
+                });
+            }
+
+            return acc;
+        }, new Map);
+
+    if (packagesFiles.size === 0) {
+        output += 'ℹ️ No difference in dist packagesFiles.\n';
+        return output;
+    }
+
+    output += 'Thanks for the PR! Here is the difference in size of the packages dist files between the base branch and the PR.\n';
+    output += 'Please review the changes and make sure they are expected.\n\n';
+    output += `<table>
+    <thead><tr><th>File</th><th>Before (Size / Gzip)</th><th>After (Size / Gzip)</th></tr></thead>
+    <tbody>`;
+    for (const [pkgKey, pkg] of packagesFiles.entries()) {
+        output += `<tr><td colspan="3"><a href="${pkg.meta.url}"><b>${pkgKey}</b></a></td></tr>`;
+        for (const file of pkg.files) {
+            output += `<tr>
+                <td><a href="${file.meta.fileNameUrl}"><code>${file.meta.fileNameShort}</code></a></td>
+            `;
+            output += file.state === 'added' 
+                ? `<td><em>Added</em></td>`
+                : `<td>
+                    <code>${formatBytes(file.before.size)}</code>
+                    / <code>${formatBytes(file.before.sizeGz)}</code> 
+                </td>`; 
+            output += file.state === 'removed'
+                ? `<td><em>Removed</em></td>`
+                : `<td>
+                    <code>${formatBytes(file.after.size)}</code>${file.state === 'changed' ? `<sup>${formatDiffPercent(file.diffPercent.size)}</sup>` : ''}
+                    / <code>${formatBytes(file.after.sizeGz)}</code>${file.state === 'changed' ? `<sup>${formatDiffPercent(file.diffPercent.sizeGz)}</sup>` : ''}
+                </td>`;
+            output += `</tr>`;
+        }
+    }
+    output += `</tbody>
+</table>
+`;
+
+    return output;
+}
+
+if (!process.env.CI) {
+    console.log(main());
+}

.github/workflows/dist-files-size-diff-comment.yaml

@@ -0,0 +1,29 @@
+name: Dist Files Size Diff (Comment)
+
+on:
+  workflow_run:
+    workflows: ["Dist Files Size Diff"]
+    types:
+      - completed
+
+jobs:
+    dist-files-size-diff:
+        runs-on: ubuntu-latest
+        steps:
+          - name: Download artifacts
+            uses: actions/download-artifact@v4
+            with:
+              name: dist-size-diff
+              run-id: ${{ github.event.workflow_run.id }}
+              github-token: ${{ secrets.GITHUB_TOKEN }}
+              
+          - name: Read pr-number artifact to env var
+            id: read-pr-number
+            run: |
+              echo "pr-number=$(cat ./pr-number)" >> $GITHUB_OUTPUT
+              
+          - name: Comment on the pull request (if success)
+            uses: marocchino/sticky-pull-request-comment@v2
+            with:
+              number: ${{ steps.read-pr-number.outputs.pr-number }}
+              path: ./diff.md

.github/workflows/dist-files-size-diff.yaml

@@ -0,0 +1,76 @@
+name: Dist Files Size Diff
+
+on:
+  pull_request:
+    types: [opened, synchronize]
+    paths:
+      - 'src/*/assets/dist/**'
+      - 'src/*/src/Bridge/*/assets/dist/**'
+
+jobs:
+    dist-files-size-diff:
+        runs-on: ubuntu-latest
+        steps:
+          - name: Configure git
+            run: |
+              git config --global user.email ""
+              git config --global user.name "github-action[bot]"
+              
+          - uses: actions/checkout@v4
+            with:
+              ref: ${{ github.base_ref }}
+
+          - name: Get dist files size (from base branch)
+            id: base-dist-files
+            run: |
+              set -e
+              
+              FILES=$(find src -mindepth 2 -type f -path '*/assets/dist/*' -not \( -path '*/tests/*' -o -path '*/public/*' -o -path '*/vendor/*' \) | sort | while read -r file; do
+                  echo "{\"$file\": {\"size\": $(wc -c < "$file"), \"size_gz\": $(gzip -c "$file" | wc -c)}}"
+              done | jq -s 'add' -c)
+              
+              echo "files=$FILES" >> $GITHUB_OUTPUT
+
+          - uses: actions/checkout@v4
+              
+          - name: Get dist files size (from pull request)
+            id: pr-dist-files
+            run: |
+              set -e
+              
+              FILES=$(find src -mindepth 2 -type f -path '*/assets/dist/*' -not \( -path '*/tests/*' -o -path '*/public/*' -o -path '*/vendor/*' \) | sort | while read -r file; do
+                  echo "{\"$file\": {\"size\": $(wc -c < "$file"), \"size_gz\": $(gzip -c "$file" | wc -c)}}"
+              done | jq -s 'add' -c)
+              
+              echo "files=$FILES" >> $GITHUB_OUTPUT
+
+          - name: Generate the diff
+            id: diff
+            uses: actions/github-script@v7
+            env:
+                BASE_DIST_FILES: ${{ steps.base-dist-files.outputs.files }}
+                HEAD_DIST_FILES: ${{ steps.pr-dist-files.outputs.files }}
+                HEAD_REPO_NAME: ${{ github.event.pull_request.head.repo.full_name }}
+                HEAD_REF: ${{ github.event.pull_request.head.ref }}
+            with:
+              result-encoding: string
+              script: |
+                const fs = require('fs')
+                const { main } = await import('${{ github.workspace }}/.github/generate-dist-files-size-diff.mjs')
+                
+                const diff = await main()
+                console.log(diff);
+                
+                fs.writeFileSync(process.env.GITHUB_WORKSPACE + '/diff.md', diff)
+
+          - name: Save PR number
+            run: |
+              echo "${{ github.event.number }}" > pr-number
+
+          - name: Upload artifacts
+            uses: actions/upload-artifact@v4
+            with:
+              name: dist-size-diff
+              path: |
+                ./diff.md
+                ./pr-number