From 42e21fab5d241ab41475efc925f1f41c833b90c5 Mon Sep 17 00:00:00 2001
From: Florian Berthold <florian.berthold@sub-net.at>
Date: Mon, 25 May 2026 12:16:59 +0200
Subject: [PATCH] ci: respect GITHUB_REPOSITORY + tolerate per-asset upload
 failures

---
 .gitea/workflows/release.yml | 34 ++++++++++++++++++++++++++++------
 tools/build_zip.sh           | 13 ++++++++++++-
 2 files changed, 40 insertions(+), 7 deletions(-)

diff --git a/.gitea/workflows/release.yml b/.gitea/workflows/release.yml
index 7c1ffe1..709a7cd 100644
--- a/.gitea/workflows/release.yml
+++ b/.gitea/workflows/release.yml
@@ -24,6 +24,9 @@ jobs:
           REPO: ${{ github.repository }}
           TAG:  ${{ github.ref_name }}
           API:  ${{ github.server_url }}/api/v1
+          # Gitea attachment ceiling is 200 MiB (see roles/gitea config).
+          # Skip anything larger so one oversized asset doesn't fail the job.
+          MAX_BYTES: 209715200
         run: |
           set -euo pipefail
           # Create the release (or reuse if it already exists for this tag).
@@ -38,12 +41,31 @@ jobs:
               | jq -r '.id')
           fi
           echo "release id: $RID"
-          # Upload every dist/*.zip
+          # Upload every dist/*.zip. Per-asset failures don't fail the job —
+          # we want partial releases to still publish rather than block the
+          # whole pipeline on one big file.
+          failed=0
+          uploaded=0
           for f in dist/*.zip; do
             name=$(basename "$f")
-            echo "uploading $name"
-            curl -sf -X POST -H "Authorization: token $GITEA_TOKEN" \
-              -F "attachment=@$f" \
-              "$API/repos/$REPO/releases/$RID/assets?name=$name" \
-              | jq -r '"  -> " + .browser_download_url'
+            size=$(stat -c '%s' "$f")
+            if [ "$size" -gt "$MAX_BYTES" ]; then
+              echo "::warning::skip $name (${size} B > ${MAX_BYTES} B Gitea limit; host on CDN instead)"
+              failed=$((failed+1))
+              continue
+            fi
+            echo "uploading $name ($(numfmt --to=iec "$size"))"
+            if curl -sf -X POST -H "Authorization: token $GITEA_TOKEN" \
+                -F "attachment=@$f" \
+                "$API/repos/$REPO/releases/$RID/assets?name=$name" \
+                | jq -r '"  -> " + .browser_download_url'; then
+              uploaded=$((uploaded+1))
+            else
+              echo "::warning::upload failed for $name"
+              failed=$((failed+1))
+            fi
           done
+          echo "release published: $uploaded uploaded, $failed skipped/failed"
+          # Only fail the job if NO assets uploaded — a release with zero
+          # attachments isn't useful to anyone.
+          [ "$uploaded" -gt 0 ]
diff --git a/tools/build_zip.sh b/tools/build_zip.sh
index f9c2d68..45d1400 100755
--- a/tools/build_zip.sh
+++ b/tools/build_zip.sh
@@ -6,12 +6,23 @@
 # - Always archives HEAD, so the working tree state is irrelevant.
 # - If more than one addon folder is present, also emits <repo>-all.zip
 #   with every addon folder side-by-side at the zip root.
+# - When run inside Gitea Actions the working tree lives under a
+#   per-job dir like /var/lib/act_runner/work/.../hostexecutor, so the
+#   repo name comes from $GITHUB_REPOSITORY (set by the runner) and
+#   only falls back to the toplevel basename for local invocations.
 set -euo pipefail
 
 root=$(git rev-parse --show-toplevel)
 cd "$root"
 
-repo_name=$(basename "$root")
+# Gitea Actions sets GITHUB_REPOSITORY=owner/repo. The basename of
+# `git rev-parse --show-toplevel` inside the runner is the worker dir
+# (e.g. `hostexecutor`), which would name the bundle wrong.
+if [ -n "${GITHUB_REPOSITORY:-}" ]; then
+    repo_name="${GITHUB_REPOSITORY##*/}"
+else
+    repo_name=$(basename "$root")
+fi
 dist="$root/dist"
 
 # Find Foo/Foo.toc pairs at depth 1; ignore libs nested deeper.