fix: look up existing release by tag instead of creating a new one (#6 )

tea creates the release before the pipeline runs; POST was failing with conflict, leaving RELEASE_ID empty and skipping the asset upload. Now GETs the release by tag, PATCHes its body, then uploads the binary. 💘 Generated with Crush Assisted-by: Claude Sonnet 4.6 via Crush <crush@charm.land> Reviewed-on: #6
fix: surface release API errors in woodpecker pipeline (#5 )
2026-03-26 13:17:25 +11:00 · 2026-03-26 12:48:54 +11:00
1 changed files with 11 additions and 4 deletions
@@ -28,12 +28,19 @@ steps:
          NOTES=$(git log --merges --pretty=format:"- %s")
        fi
        BODY=$(printf '%s' "$NOTES" | sed 's/"/\\"/g; s/$/\\n/' | tr -d '\n')
-        RELEASE_ID=$(curl -sf -X POST "https://git.unkin.net/api/v1/repos/${CI_REPO}/releases" \
+        GET_RESPONSE=$(curl -s "https://git.unkin.net/api/v1/repos/${CI_REPO}/releases/tags/${CI_COMMIT_TAG}")
+        echo "GET response: ${GET_RESPONSE}"
+        RELEASE_ID=$(echo "${GET_RESPONSE}" | grep -o '"id":[0-9]*' | head -1 | cut -d: -f2)
+        if [ -z "$RELEASE_ID" ]; then
+          echo "ERROR: failed to find release for tag ${CI_COMMIT_TAG}" >&2
+          exit 1
+        fi
+        echo "Release ID: ${RELEASE_ID}"
+        curl -s -X PATCH "https://git.unkin.net/api/v1/repos/${CI_REPO}/releases/${RELEASE_ID}" \
          -u "droneci:$DRONECI_PASSWORD" \
          -H "Content-Type: application/json" \
-          -d "{\"tag_name\":\"${CI_COMMIT_TAG}\",\"name\":\"${CI_COMMIT_TAG}\",\"body\":\"${BODY}\"}" \
-          | grep -o '"id":[0-9]*' | head -1 | cut -d: -f2)
-        curl -sf -X POST "https://git.unkin.net/api/v1/repos/${CI_REPO}/releases/${RELEASE_ID}/assets" \
+          -d "{\"body\":\"${BODY}\"}"
+        curl -s -X POST "https://git.unkin.net/api/v1/repos/${CI_REPO}/releases/${RELEASE_ID}/assets" \
          -u "droneci:$DRONECI_PASSWORD" \
          -F "attachment=@node-lookup"
    backend_options: