coder
diff --git a/‎.github/workflows/aigateway-prices-refresh.yaml‎
Lines changed: 15 additions & 20 deletions b/‎.github/workflows/aigateway-prices-refresh.yaml‎
Lines changed: 15 additions & 20 deletions
@@ -153,37 +153,32 @@ jobs:
               --jq '.html_url'
           fi
 
-      - name: Send Slack notification when price book is current
-        if: steps.detect.outputs.changed == 'false'
+      - name: Send Slack notification
+        if: failure() || steps.detect.outputs.changed == 'false'
         env:
+          REFRESH_STATUS: ${{ job.status }}
+          PRICE_BOOK_CHANGED: ${{ steps.detect.outputs.changed }}
           SLACK_WEBHOOK: ${{ secrets.AIGATEWAY_PRICES_SLACK_WEBHOOK }}
           RUN_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
         run: |
           set -euo pipefail
           if [ -z "${SLACK_WEBHOOK}" ]; then
-            echo "::error::AIGATEWAY_PRICES_SLACK_WEBHOOK is not set; the no-change notification could not be sent."
+            echo "::error::AIGATEWAY_PRICES_SLACK_WEBHOOK is not set; the notification could not be sent."
             exit 1
           fi
-          text=":white_check_mark: *AI Gateway price book refresh completed.* No generated changes were found. Run: ${RUN_URL}"
-          payload="$(jq -nc --arg text "${text}" '{text: $text}')"
-          curl -fsSL -X POST -H 'Content-type: application/json' -d "${payload}" "${SLACK_WEBHOOK}"
-          echo "Sent Slack notification"
 
-      - name: Send Slack notification on failure
-        if: failure()
-        env:
-          SLACK_WEBHOOK: ${{ secrets.AIGATEWAY_PRICES_SLACK_WEBHOOK }}
-          RUN_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
-        run: |
-          set -euo pipefail
-          if [ -z "${SLACK_WEBHOOK}" ]; then
-            echo "::error::AIGATEWAY_PRICES_SLACK_WEBHOOK is not set; the failure alert could not be sent."
+          if [ "${REFRESH_STATUS}" = "failure" ]; then
+            # printf, not a double-quoted literal: bash leaves \n as two
+            # characters, and jq --arg then escapes the backslash, so Slack
+            # would print \n as text instead of breaking the line.
+            text="$(printf ':warning: *AI model price book refresh failed.*\nThe generator fails by design when upstream drops a model pinned in scripts/aibridgepricesgen/overrides.jq or curated in curation.json. Logs: %s' "${RUN_URL}")"
+          elif [ "${PRICE_BOOK_CHANGED}" = "false" ]; then
+            text=":white_check_mark: *AI Gateway price book refresh completed.* No generated changes were found. Run: ${RUN_URL}"
+          else
+            echo "::error::Unexpected notification state: status=${REFRESH_STATUS}, changed=${PRICE_BOOK_CHANGED}"
             exit 1
           fi
-          # printf, not a double-quoted literal: bash leaves \n as two
-          # characters, and jq --arg then escapes the backslash, so Slack
-          # would print \n as text instead of breaking the line.
-          text="$(printf ':warning: *AI model price book refresh failed.*\nThe generator fails by design when upstream drops a model pinned in scripts/aibridgepricesgen/overrides.jq or curated in curation.json. Logs: %s' "${RUN_URL}")"
+
           payload="$(jq -nc --arg text "${text}" '{text: $text}')"
           curl -fsSL -X POST -H 'Content-type: application/json' -d "${payload}" "${SLACK_WEBHOOK}"
           echo "Sent Slack notification"