sanitize filenames for artifact upload

- Add step to sanitize screenshot filenames before upload
- Remove invalid characters (: * ? < > |) from app names
- Fixes error with apps like 'fre:ac' containing colons
- Update publish workflow to handle flat file structure

Author: AshBuk

.github/workflows/publish-pr-screenshot.yml

@@ -48,14 +48,13 @@ jobs:
           SHORT="${HEAD_SHA::8}"
           COUNT=0
           FALLBACK="run-${{ github.event.workflow_run.id }}"
-          for f in out/database/*/screenshot.png; do
+          for f in out/*.png; do
             [ -f "$f" ] || continue
             base="$(basename "$f")"
-            ext="${base##*.}"
-            appname="$(basename "$(dirname "$f")")"
+            appname="${base%.png}"  # Remove .png extension to get app name
             # Produce deterministic upload name with app name
             PN="${PR_NUMBER:-$FALLBACK}"
-            name="pr-${PN}-${SHORT}-${appname}.${ext}"
+            name="pr-${PN}-${SHORT}-${appname}.png"
             cp "$f" "upload/${name}"
             echo "${name}" >> upload/list.txt
             COUNT=$((COUNT+1))

.github/workflows/test.yml

@@ -86,13 +86,26 @@ jobs:
           # xpra stop :99
           killall Xvfb
           # bundle exec jekyll build # https://help.github.com/en/articles/viewing-jekyll-build-error-messages#configuring-a-third-party-service-to-display-jekyll-build-error-messages
+      - name: Prepare screenshots for upload
+        if: github.event_name == 'pull_request'
+        shell: bash
+        run: |
+          mkdir -p screenshots-upload
+          for screenshot in database/*/screenshot.png; do
+            if [ -f "$screenshot" ]; then
+              appname=$(basename "$(dirname "$screenshot")")
+              # Sanitize filename: remove invalid characters for artifacts
+              safe_name=$(echo "$appname" | tr -d '":*?<>|' | tr ' ' '_')
+              cp "$screenshot" "screenshots-upload/${safe_name}.png"
+            fi
+          done
+          ls -la screenshots-upload/ || true
       - name: Upload screenshot artifact
         if: github.event_name == 'pull_request'
         uses: actions/upload-artifact@v4
         with:
           name: pr-screenshots
-          path: |
-            database/*/screenshot.png
+          path: screenshots-upload/*.png
           if-no-files-found: ignore
           retention-days: 7
       - name: Check log