ci(release): improve release workflow script and notification [need ci]

- Remove check for [skip android] flag and related conditional logic
- Add check to skip DockerHub login if credentials are missing
- Exit early if no changed projects detected
- Only run npm version bump if bump is not "none"
- Handle build failure with warning message instead of silent failure
- Condition Docker build and push on DOCKERHUB_USER presence and allow failures
- Simplify Android build step, remove skip android condition
- Enhance Telegram notification with pipeline URL and better formatting
- Replace curl JSON payload with form data for Telegram message sending
- Add download artifacts button to Telegram notification
- Refactor comments and improve script readability and robustness

Author: EndyKaufman

.github/workflows/release.yml

@@ -23,7 +23,7 @@ jobs:
         with:
           node-version: 23.11.0
 
-      # 🔍 Определяем, есть ли флаг [need ci]
+      # 🔍 Проверяем наличие [need ci]
       - name: Check if [need ci] present
         id: need_ci
         run: |
@@ -35,18 +35,6 @@ jobs:
             echo "ℹ️ No [need ci] found"
           fi
 
-      # 🔍 Проверяем, есть ли флаг [skip android]
-      - name: Check if [skip android] present
-        id: skip_android
-        run: |
-          if git log ${{ github.event.before }}..${{ github.sha }} --pretty=format:"%s%b" | grep -q "\[skip android\]"; then
-            echo "skip_build=true" >> $GITHUB_OUTPUT
-            echo "🚫 Found [skip android] — skipping Android build"
-          else
-            echo "skip_build=false" >> $GITHUB_OUTPUT
-            echo "✅ No [skip android] found — proceeding with Android build"
-          fi
-
       # 🔍 Определяем изменённые проекты
       - name: Detect changed projects
         id: detect
@@ -59,7 +47,7 @@ jobs:
           echo "changed_projects=$CHANGED_PROJECTS" >> $GITHUB_OUTPUT
           echo "Changed projects: $CHANGED_PROJECTS"
 
-      # 🧩 Определяем тип версии
+      # 🧩 Определяем тип bump'а
       - name: Determine bump type
         id: bump_type
         run: |
@@ -86,12 +74,17 @@ jobs:
         run: |
           sudo apt-get install -y jq
           npm install -g conventional-changelog-cli
-          echo "${DOCKERHUB_TOKEN}" | docker login -u "${DOCKERHUB_USER}" --password-stdin
+
+          if [ -n "${DOCKERHUB_USER}" ] && [ -n "${DOCKERHUB_TOKEN}" ]; then
+            echo "${DOCKERHUB_TOKEN}" | docker login -u "${DOCKERHUB_USER}" --password-stdin
+          else
+            echo "⚠️ DockerHub credentials missing — skipping Docker publish"
+          fi
 
           BUMP=${{ steps.bump_type.outputs.bump }}
           TAGS=""
 
-          # Если [need ci], прогоняем все проекты
+          # Если [need ci] — прогоняем все проекты
           if [ "${{ steps.need_ci.outputs.force_run }}" = "true" ]; then
             CHANGED_PROJECTS=$(find . -maxdepth 1 -type d ! -path . | while read d; do
               if [ -f "$d/package.json" ]; then echo "$(basename "$d")"; fi
@@ -100,40 +93,44 @@ jobs:
             CHANGED_PROJECTS=${{ steps.detect.outputs.changed_projects }}
           fi
 
+          if [ -z "$CHANGED_PROJECTS" ]; then
+            echo "ℹ️ No changed projects detected. Exiting."
+            exit 0
+          fi
+
           for project in $CHANGED_PROJECTS; do
             if [ -f "$project/package.json" ]; then
               cd $project
               NAME=$(jq -r '.name' package.json)
               DESC=$(jq -r '.description // "No description"' package.json)
               OLD_VERSION=$(jq -r '.version' package.json)
+
               if [ "$BUMP" != "none" ]; then
                 npm version $BUMP --no-git-tag-version
               fi
-              NEW_VERSION=$(jq -r '.version' package.json)
 
-              conventional-changelog -p angular -i CHANGELOG.md -s -r 1 || echo "No changelog generated"
+              NEW_VERSION=$(jq -r '.version' package.json)
+              conventional-changelog -p angular -i CHANGELOG.md -s -r 1 || true
 
               if jq -e '.scripts.build' package.json >/dev/null; then
                 npm ci
-                npm run build || echo "Build failed"
+                npm run build || echo "⚠️ Build failed for $project"
               fi
 
-              if [ -f "Dockerfile" ]; then
+              if [ -f "Dockerfile" ] && [ -n "${DOCKERHUB_USER}" ]; then
                 IMAGE_NAME="${DOCKERHUB_USER}/${NAME}"
                 docker build -t "$IMAGE_NAME:$NEW_VERSION" -t "$IMAGE_NAME:latest" \
                   --label "org.opencontainers.image.title=$NAME" \
                   --label "org.opencontainers.image.description=$DESC" \
                   --label "org.opencontainers.image.version=$NEW_VERSION" \
                   --label "org.opencontainers.image.source=https://github.com/${{ github.repository }}" \
-                  .
-                docker push "$IMAGE_NAME:$NEW_VERSION"
-                docker push "$IMAGE_NAME:latest"
+                  . || true
+                docker push "$IMAGE_NAME:$NEW_VERSION" || true
+                docker push "$IMAGE_NAME:latest" || true
               fi
 
-              # Skip Android build if [skip android] flag is present
-              if [ -f "capacitor.config.ts" ] && [ "${{ steps.skip_android.outputs.skip_build }}" = "false" ]; then
+              if [ -f "capacitor.config.ts" ]; then
                 echo "${{ secrets.KEYSTORE }}" | base64 --decode > "${{ github.workspace }}/$project/my-dashboard.jks"
-                ls "${{ github.workspace }}/$project"
 
                 docker run --rm \
                   -e KEYSTORE_PASSWORD="${{ secrets.KEYSTORE_PASSWORD }}" \
@@ -143,11 +140,8 @@ jobs:
                   -v "${{ github.workspace }}/$project:/app" \
                   endykaufman/ionic-capacitor:latest
 
-                ls -la ${{ github.workspace }}/$project/android/app/build/outputs/apk/release/
                 mkdir -p ${{ github.workspace }}/$project/artifacts
                 cp -R ${{ github.workspace }}/$project/android/app/build/outputs/apk/release/* ${{ github.workspace }}/$project/artifacts/
-              elif [ -f "capacitor.config.ts" ] && [ "${{ steps.skip_android.outputs.skip_build }}" = "true" ]; then
-                echo "⏭️ Skipping Android build for $project due to [skip android] flag"
               fi
 
               cd -
@@ -156,6 +150,7 @@ jobs:
               git tag "$TAG"
             fi
           done
+
           echo "tags=$TAGS" >> $GITHUB_OUTPUT
 
       # 💾 Commit and push changes
@@ -212,11 +207,14 @@ jobs:
 
             RELEASE_URL="https://github.com/${{ github.repository }}/releases/tag/${TAG}"
             PROJECT_URL="https://github.com/${{ github.repository }}/tree/main/${PROJECT}"
+            PIPELINE_URL="https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"
+
+            NOTES="📢 *${NAME} v${VERSION} released!*%0A%0A_${DESC}_%0A%0A"
 
-            NOTES="📢 *${NAME} v${VERSION} released!*\n\n_${DESC}_\n\n"
             if [ -f "$CHANGELOG_PATH" ]; then
               BODY=$(awk '/^##/{i++}i==1' "$CHANGELOG_PATH" | head -n 20)
-              NOTES+="*Changelog:*\n\`\`\`\n$BODY\n\`\`\`\n"
+              BODY_ESCAPED=$(echo "$BODY" | sed ':a;N;$!ba;s/\n/%0A/g')
+              NOTES+="*Changelog:*%0A$BODY_ESCAPED%0A%0A"
             fi
 
             if [ -d "$ARTIFACTS_PATH" ]; then
@@ -228,6 +226,7 @@ jobs:
 
             if [ "$SKIP_TELEGRAM" -eq 0 ]; then
               BUTTONS="["
+              BUTTONS="${BUTTONS}[{\"text\":\"💾 Download all artifacts\",\"url\":\"${PIPELINE_URL}\"}],"
               if [ -f "$PROJECT/Dockerfile" ]; then
                 BUTTONS="${BUTTONS}[{\"text\":\"🐳 Docker image\",\"url\":\"https://hub.docker.com/r/${DOCKERHUB_USER}/${NAME}\"}],"
               fi
@@ -237,12 +236,10 @@ jobs:
               BUTTONS=$(echo "$BUTTONS" | sed 's/,]/]/')
 
               curl -s -X POST "https://api.telegram.org/bot${TELEGRAM_BOT_TOKEN}/sendMessage" \
-                -H "Content-Type: application/json" \
-                -d "{
-                  \"chat_id\": \"${TELEGRAM_CHAT_ID}\",
-                  \"parse_mode\": \"Markdown\",
-                  \"text\": \"${NOTES}\",
-                  \"reply_markup\": {\"inline_keyboard\": ${BUTTONS}}
-                }" >/dev/null 2>&1
+                -d "chat_id=${TELEGRAM_CHAT_ID}" \
+                -d "parse_mode=Markdown" \
+                -d "disable_web_page_preview=true" \
+                -d "text=${NOTES}" \
+                -d "reply_markup={\"inline_keyboard\":${BUTTONS}}" >/dev/null 2>&1
             fi
-          done
+          done