fix(pack): valid manifest YAML, stable-release picker, tolerant asset re-upload

- manifest: single-quote the coa-template note - nested double quotes
  made the scalar invalid YAML (awk include-parser unaffected; it only
  reads repo:/include: lines)
- build_pack: releases?limit=1 could hand back a prerelease/draft;
  fetch limit=10 and jq-select the first stable release
- release.yml: re-running for an existing tag 409'd on duplicate asset
  names and set -euo pipefail killed the job; skip already-attached
  assets / warn on failed uploads and only fail when the release ends
  up with zero assets
This commit is contained in:
Florian Andrew George Berthold 2026-06-10 02:17:21 +02:00
parent ec25b5d022
commit 9154d56750
3 changed files with 30 additions and 10 deletions

View file

@ -41,12 +41,31 @@ jobs:
# so the auto-generated Source Code (zip/tar.gz) links stay hidden.
curl -sf -X PATCH -H "Authorization: token $GITEA_TOKEN" -H "Content-Type: application/json" \
"$API/repos/$REPO/releases/$RID" -d '{"hide_archive_links":true}' >/dev/null || true
# Upload every dist/*.zip
# Upload every dist/*.zip. Re-running for an existing tag 409s on
# duplicate asset names - warn and keep going instead of letting
# set -e kill the job; only fail if the release ends up with zero
# assets.
existing=$(curl -s -H "Authorization: token $GITEA_TOKEN" \
"$API/repos/$REPO/releases/$RID/assets" | jq -r '.[].name' || true)
uploaded=0
present=0
for f in dist/*.zip; do
name=$(basename "$f")
if printf '%s\n' "$existing" | grep -qxF "$name"; then
echo "::warning::$name already attached (re-run for existing tag) - skipping"
present=$((present+1))
continue
fi
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'
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 (duplicate asset on re-run?)"
fi
done
echo "release published: $uploaded uploaded, $present already present"
# Only fail the job if NO assets ended up on the release.
[ "$((uploaded + present))" -gt 0 ]

View file

@ -12,7 +12,7 @@ addons:
- repo: coa-template # template-only — not an installable addon
include: false
note: "Gitea "Use This Template" source repo for new coa-* forks."
note: 'Gitea "Use This Template" source repo for new coa-* forks.'
- repo: coa-ai-voiceover
include: false

View file

@ -78,13 +78,14 @@ manifest_includes() {
' "$MANIFEST"
}
# Latest release JSON for a repo, or empty string if none.
# Latest stable release JSON for a repo, or empty string if none.
latest_release_json() {
local repo="$1" json
json="$(curl -fsS "$API/repos/$ORG/$repo/releases?limit=1" 2>/dev/null || echo '[]')"
# `releases?limit=1` returns an array. Strip the wrapper.
json="$(curl -fsS "$API/repos/$ORG/$repo/releases?limit=10" 2>/dev/null || echo '[]')"
# `releases?limit=N` returns a newest-first array that can contain
# prereleases/drafts - pick the first stable release.
if [ "$(printf '%s' "$json" | jq -r 'type')" = "array" ]; then
printf '%s' "$json" | jq -c '.[0] // empty'
printf '%s' "$json" | jq -c '[.[] | select(.prerelease == false and .draft == false)][0] // empty'
else
printf ''
fi