Addition of the Desjardins Group logo

.github/workflows/pr-svg-normalize.yml

@@ -57,7 +57,7 @@ jobs:
         with:
           ref: main
           path: base
-          fetch-depth: 1  # intentional: we only need HEAD:<path> existence checks
+          fetch-depth: 1
 
       # -- 2. Install deps from main's lockfile (never from the PR tree) -----
       - name: Install dependencies
@@ -71,7 +71,7 @@ jobs:
           repository: ${{ steps.pr.outputs.head_repo }}
           ref: ${{ steps.pr.outputs.head_ref }}
           path: pr-branch
-          fetch-depth: 1
+          fetch-depth: 0   # required: shallow clone breaks fork-PR push and fetch ref patterns
           submodules: false
           # keep default persist-credentials behavior (clearer intent since we may push)
       - run: git -C base fetch --depth=1 origin dev:refs/remotes/origin/dev
@@ -101,20 +101,33 @@ jobs:
             const owner = context.repo.owner;
             const repo  = context.repo.repo;
             const pr    = Number(process.env.PR_NUMBER);
-
-            const files = [];
+            const BOT_EMAIL = "41898282+github-actions[bot]@users.noreply.github.com";
+
+            // Get all commits on this PR
+            const commits = [];
             for (let page = 1; ; page++) {
-              const { data } = await github.rest.pulls.listFiles({
+              const { data } = await github.rest.pulls.listCommits({
                 owner, repo, pull_number: pr, per_page: 100, page
               });
               if (!data.length) break;
-              for (const f of data) files.push(f.filename);
+              commits.push(...data);
               if (data.length < 100) break;
             }
-
+
+            // Collect files only from non-bot commits
+            const nonBotFiles = new Set();
+            for (const c of commits) {
+              if (c.commit.author?.email === BOT_EMAIL) continue;
+              const { data: detail } = await github.rest.repos.getCommit({
+                owner, repo, ref: c.sha
+              });
+              for (const f of (detail.files ?? [])) nonBotFiles.add(f.filename);
+            }
+
             const iconRoot = "icon-svg/";
-            const svgFiles = files.filter(f => f.toLowerCase().endsWith(".svg"));
-            const iconSvgs = svgFiles.filter(f => f.startsWith(iconRoot));
+            const iconSvgs = [...nonBotFiles].filter(f =>
+              f.startsWith(iconRoot) && f.toLowerCase().endsWith(".svg")
+            );
 
             // Policy: no subfolders under icon-svg (only iconRoot + basename.svg)
             const bad = iconSvgs.filter(f => f.slice(iconRoot.length).includes("/"));
@@ -334,13 +347,15 @@ jobs:
               echo "> After enabling **close** and then **re-open** the PR. This will rerun the action."
               echo ">"
               echo "> Please also review the normalized icons as errors might occur."
+              echo "> Note that differences in the displayed size are expected (\`width\` / \`height\` attributes are deleted)."
               echo ">"
               echo "> Thank you for your contribution!"
               echo ""
             else
               echo "> ✔ Normalization **successful**."
               echo ">"
               echo "> Please review the normalized icons as errors might occur."
+              echo "> Note that differences in the displayed size are expected (\`width\` / \`height\` attributes are deleted)."
               echo ">"
               echo "> If needed, **manually update** the PR with a fixed version."
               echo ">"
@@ -382,9 +397,9 @@ jobs:
 
             # New icon detection against BASE_REF
             if git -C base cat-file -e "refs/remotes/origin/dev:${file}" 2>/dev/null; then
-              printf '| ![](%s) | ![](%s) | %s |\\n' "$url_before" "$url_after" "$result" >> "$RUNNER_TEMP/report.md"
+              printf '| ![](%s) | ![](%s) | %s |\n' "$url_before" "$url_after" "$result" >> "$RUNNER_TEMP/report.md"
             else
-              printf '| ![](%s) | ![](%s) | 🆕 New · %s |\\n' "$url_before" "$url_after" "$result" >> "$RUNNER_TEMP/report.md"
+              printf '| ![](%s) | ![](%s) | 🆕 New · %s |\n' "$url_before" "$url_after" "$result" >> "$RUNNER_TEMP/report.md"
             fi
           done < "$RUNNER_TEMP/changed_svgs.txt"