From 437ddb2ef61b1e1eb09b597674e32892698a148b Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Tue, 14 Jul 2026 19:47:57 +0530
Subject: [PATCH 01/18] feat(notify): redesign Slack card, add hash / duration
 / url / view-run
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Refactor the notify composite action to render a single-column key/value
grid instead of the wide two-column layout. Adds three new fields:

* HASH     — linked short SHA (7 chars)
* DURATION — pipeline elapsed time, auto-fetched from the run API
* URL      — deployed app URL (success only), display strips protocol

On failure a red-dot 'FAILED AT' row is promoted to the top, populated
with the first failing step name (auto-fetched from the run's jobs API).

Card structure:

  header    — icon + label + version
  context   — 'Deploy succeeded' / 'Deploy failed' subtitle
  section   — key/value fields (split into two blocks; Slack caps at 10)
  actions   — 'View run' button in the footer

Duration and failed-step are auto-resolved by the action, so callers only
need to grant `actions: read` on the notify job. No new caller wiring
beyond the one new `deployed-url` input.

Refs AGODEV-1495. Not tagged yet — canary-tested via commerce-backend
against this branch before v3.4.0 is cut and @v3 is moved.
---
 README.md                 |  43 ++++++---
 actions/notify/action.yml | 194 +++++++++++++++++++++++++-------------
 2 files changed, 159 insertions(+), 78 deletions(-)

diff --git a/README.md b/README.md
index d130119..71e5cfa 100644
--- a/README.md
+++ b/README.md
@@ -276,25 +276,33 @@ version:
 
 ## notify — inputs
 
-Posts a rich Slack card for a pipeline result: a green/red header
-(`<label> — succeeded|failed`), a Version field, and the **PR** this deploy
-traces to (title + clickable `owner/repo#NN`). With no PR it falls back to the
-**commit** (subject + clickable short SHA), and with neither to a single-line
-text message. A context line shows the author, the UTC build date
-(`YYYY.MM.DD`), and a link to the run. The build date is stamped by the action
-so the deploy date reads unambiguously rather than being inferred from Slack's
-local-timezone message timestamp.
+Posts a rich Slack card for a pipeline result. Header shows the label + version
+with a green ✅ / red ❌ icon; a subtitle line reads `Deploy succeeded` or
+`Deploy failed`. The body is a single-column key/value grid ordered by
+importance: `COMMIT`, `PR`, `AUTHOR`, `HASH`, `DURATION`, `DATE`, plus `URL`
+(success only) or `FAILED AT` (failure only, promoted to the top row). A
+`View run` button links back to the CI run. With no PR it falls back to the
+**commit** (subject + clickable short SHA); with neither, a single-line text
+message.
+
+Duration and the failing step name are auto-fetched by the action via the
+GitHub Actions API — no caller wiring required. The caller must grant
+`actions: read` on the notify job (in addition to `contents: read` and
+`pull-requests: read` used for the PR/commit lookup).
+
 Best-effort: an empty `webhook-url` is a no-op that still succeeds, and a failed
-post never fails the pipeline.
+post never fails the pipeline. If the run/jobs API is unreachable, the duration
+or failed-step rows are silently omitted.
 
 | Input | Required | Default | Description |
 |-------|----------|---------|-------------|
 | `result` | yes | — | `success` / `failure` — e.g. a deploy job's `result`. Anything other than `success` renders as failed. |
 | `webhook-url` | no | `''` | Slack incoming webhook URL. Empty makes the action a no-op. |
-| `label` | no | `Deployment` | Short label for the card header (app / component name). |
-| `version` | no | `''` | Version string shown in the Version field (`n/a` if empty). |
+| `label` | no | `Deployment` | Short label rendered next to the version in the header. Prefer a bare component name (`"Commerce Backend"`) — the Slack channel already implies environment. |
+| `version` | no | `''` | Version string shown next to the label (`n/a` if empty). |
+| `deployed-url` | no | `''` | Fully-qualified URL of the deployed app; shown only on success. The card strips protocol and trailing slash for display, and links back to the full URL. Caller resolves per-environment. |
 | `channel` | no | `slack` | Chat channel; only `slack` is implemented today. |
-| `github-token` | no | `${{ github.token }}` | Token used to read the PR/commit. The default job token covers a same-repo deploy; for a cross-org deploy pass a token minted in the caller from a GitHub App with read access to `target-repo`. |
+| `github-token` | no | `${{ github.token }}` | Token used to read the PR/commit, the workflow run, and (on failure) the run's jobs. The default job token covers a same-repo deploy; for a cross-org deploy pass a token minted in the caller from a GitHub App with read access to `target-repo`. |
 | `target-repo` | no | `${{ github.repository }}` | `owner/repo` whose PR/commit the card describes. Override for a cross-repo deploy. |
 | `target-sha` | no | `${{ github.sha }}` | Exact built/deployed commit SHA used to resolve the PR/commit. |
 
@@ -305,13 +313,17 @@ Example — same-repo deploy (token/target default to this repo):
     needs: deploy
     if: ${{ always() && github.event_name == 'push' }}
     runs-on: ubuntu-latest
-    permissions: { contents: read, pull-requests: read }
+    permissions:
+      contents: read
+      pull-requests: read
+      actions: read
     steps:
       - uses: nurdsoft/ci-workflows/actions/notify@v3
         with:
           result: ${{ needs.deploy.result }}
-          label: "Backend (${{ github.ref_name == 'prod' && 'Prod' || 'Dev' }})"
+          label: "Backend"
           version: ${{ needs.deploy.outputs.version }}
+          deployed-url: ${{ github.ref_name == 'main' && 'https://app.example.com' || 'https://dev.example.com' }}
           webhook-url: ${{ secrets.SLACK_WEBHOOK_URL }}
 ```
 
@@ -328,8 +340,9 @@ Example — cross-org deploy (card describes a PR/commit in another repo):
       - uses: nurdsoft/ci-workflows/actions/notify@v3
         with:
           result: ${{ needs.deploy.result }}
-          label: "Web (Prod)"
+          label: "Web"
           version: ${{ needs.build.outputs.app_version }}
+          deployed-url: https://app.other-org.com
           webhook-url: ${{ secrets.SLACK_WEBHOOK_URL }}
           github-token: ${{ steps.app-token.outputs.token }}
           target-repo: other-org/other-repo
diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index b6f7c36..cab77c0 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -1,11 +1,11 @@
 name: Notify
 description: >-
   Post a pipeline result to a chat channel as a rich card. Resolves the deploy's
-  PR (or commit) to show title, version, author and a run link. Best-effort:
-  with no webhook the action is a no-op and succeeds, so a missing webhook never
-  fails the pipeline. The PR/commit lookup needs `pull-requests: read` and
-  `contents: read` on the calling job; without them the card silently falls back
-  to commit-only details.
+  PR (or commit) to show title, version, author, hash, duration, and a run link.
+  Auto-fetches pipeline duration and (on failure) the failing step name via the
+  GitHub Actions API — caller must grant `actions: read` on the notify job.
+  Best-effort: with no webhook the action is a no-op and succeeds, so a missing
+  webhook never fails the pipeline.
 
 inputs:
   result:
@@ -16,11 +16,21 @@ inputs:
     required: false
     default: ""
   label:
-    description: Short label for the message (app / component name).
+    description: >-
+      Short label for the message header (app / component name). Rendered
+      alongside `version`. Prefer a bare name ("Commerce Backend") — the Slack
+      channel already implies the environment.
     required: false
     default: Deployment
   version:
-    description: Version string shown in the message.
+    description: Version string shown in the header, next to the label (e.g. `v2.1.0`).
+    required: false
+    default: ""
+  deployed-url:
+    description: >-
+      Fully-qualified URL of the deployed app (shown only on success). The card
+      strips protocol and trailing slash for display, and links back to the full
+      URL. Caller resolves per-environment.
     required: false
     default: ""
   channel:
@@ -29,10 +39,10 @@ inputs:
     default: slack
   github-token:
     description: >-
-      Token used to read the PR / commit that this deploy traces to. For a
-      same-repo deploy the default job token is enough; for a cross-org deploy
-      (target-repo differs from this repo) pass a token minted in the caller
-      from a GitHub App with read access to target-repo.
+      Token used to read the PR/commit, the workflow run, and (on failure) the
+      run's jobs. Same-repo deploy: the default job token is enough. Cross-org
+      deploy: pass a token minted in the caller from a GitHub App with read
+      access to target-repo. Requires `actions: read` on the notify job.
     required: false
     default: ${{ github.token }}
   target-repo:
@@ -58,9 +68,11 @@ runs:
         RESULT: ${{ inputs.result }}
         LABEL: ${{ inputs.label }}
         VERSION: ${{ inputs.version }}
+        DEPLOYED_URL: ${{ inputs.deployed-url }}
         WEBHOOK: ${{ inputs.webhook-url }}
         TARGET_REPO: ${{ inputs.target-repo }}
         TARGET_SHA: ${{ inputs.target-sha }}
+        RUN_ID: ${{ github.run_id }}
         RUN_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
         GH_TOKEN: ${{ inputs.github-token }}
       run: |
@@ -71,74 +83,130 @@ runs:
           exit 0
         fi
 
-        emoji=":x:"; color="#D32F2F"; status="failed"
+        # Status → icon + subtitle.
+        icon=":x:"; subtitle="Deploy failed"; is_success=0
         if [ "${RESULT:-}" = "success" ]; then
-          emoji=":white_check_mark:"; color="#2E7D32"; status="succeeded"
+          icon=":white_check_mark:"; subtitle="Deploy succeeded"; is_success=1
         fi
+
         version="${VERSION:-}"; [ -z "$version" ] && version="n/a"
         short_sha="${TARGET_SHA:0:7}"
-
-        # Build date in UTC (YYYY.MM.DD), shown on the card so the deploy date is
-        # unambiguous rather than inferred from Slack's local-timezone timestamp.
         built_utc="$(date -u +%Y.%m.%d)"
 
-        # Prefer the PR this deploy traces to (a merge commit on dev/prod resolves
-        # to its PR); fall back to the commit itself, then to a plain line.
+        # -- Duration: pull run_started_at, compute elapsed vs now.
+        # Falls back silently if the API call fails.
+        duration=""
+        run_json="$(gh api "repos/${TARGET_REPO}/actions/runs/${RUN_ID}" 2>/dev/null || echo '{}')"
+        started_at="$(jq -r '.run_started_at // .created_at // empty' <<<"$run_json")"
+        if [ -n "$started_at" ]; then
+          started_epoch="$(date -u -d "$started_at" +%s 2>/dev/null || echo '')"
+          if [ -n "$started_epoch" ]; then
+            now_epoch="$(date -u +%s)"
+            elapsed=$(( now_epoch - started_epoch ))
+            if [ "$elapsed" -lt 0 ]; then elapsed=0; fi
+            if [ "$elapsed" -lt 60 ]; then
+              duration="${elapsed}s"
+            else
+              duration="$((elapsed / 60))m $((elapsed % 60))s"
+            fi
+          fi
+        fi
+
+        # -- Failed step name: first step across all jobs with conclusion=failure.
+        # Only fetched on failure; falls back silently.
+        failed_step=""
+        if [ "$is_success" -eq 0 ]; then
+          jobs_json="$(gh api "repos/${TARGET_REPO}/actions/runs/${RUN_ID}/jobs" 2>/dev/null || echo '{}')"
+          failed_step="$(jq -r '[.jobs[]? | .steps[]?] | map(select(.conclusion == "failure")) | .[0].name // ""' <<<"$jobs_json")"
+        fi
+
+        # -- Resolve PR (preferred) or commit for the header rows.
         pr_json="$(gh api "repos/${TARGET_REPO}/commits/${TARGET_SHA}/pulls" \
           -H "Accept: application/vnd.github+json" 2>/dev/null || echo '[]')"
         sel="$(jq -c 'if (type=="array" and length>0) then ([.[]|select(.merged_at!=null)][0] // .[0]) else {} end' <<<"$pr_json")"
         pr_num="$(jq -r '.number // empty' <<<"$sel")"
 
+        commit_json="$(gh api "repos/${TARGET_REPO}/commits/${TARGET_SHA}" 2>/dev/null || echo '{}')"
+        commit_subject="$(jq -r '.commit.message // "" | split("\n")[0] // ""' <<<"$commit_json")"
+        # Prefer git-metadata display name over the raw GitHub login.
+        commit_author="$(jq -r '.commit.author.name // .author.login // "unknown"' <<<"$commit_json")"
+        commit_url="$(jq -r '.html_url // ""' <<<"$commit_json")"
+
         if [ -n "$pr_num" ]; then
-          # PR card: title + number + link (no commit bullets).
           pr_title="$(jq -r '.title // ""' <<<"$sel")"
           pr_url="$(jq -r '.html_url // ""' <<<"$sel")"
-          pr_user="$(jq -r '.user.login // "unknown"' <<<"$sel")"
+          pr_label="${TARGET_REPO}#${pr_num}"
+          # PR title beats commit subject when available.
+          [ -n "$pr_title" ] && commit_subject="$pr_title"
+        else
+          pr_url=""; pr_label=""
+        fi
+
+        # -- URL display: strip protocol + trailing slash.
+        url_display=""
+        if [ -n "${DEPLOYED_URL:-}" ]; then
+          url_display="$(printf '%s' "$DEPLOYED_URL" | sed -E 's|^https?://||; s|/+$||')"
+        fi
+
+        # Fallback path: neither PR nor commit resolved → single-line message.
+        if [ -z "$commit_subject" ] && [ -z "$pr_num" ]; then
           payload="$(jq -n \
-            --arg color "$color" --arg header "${emoji} ${LABEL} — ${status}" \
-            --arg version "$version" --arg prlabel "${TARGET_REPO}#${pr_num}" --arg title "$pr_title" \
-            --arg user "$pr_user" --arg runurl "$RUN_URL" --arg prurl "$pr_url" \
-            --arg built "$built_utc" '
-            def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
-            { attachments: [{ color: $color, blocks: [
-                { type:"header", text:{ type:"plain_text", text:$header, emoji:true } },
-                { type:"section", fields:[
-                    { type:"mrkdwn", text:("*Version*\n" + $version) },
-                    { type:"mrkdwn", text:("*PR*\n" + (if $prurl != "" then "<" + $prurl + "|" + $prlabel + ">" else $prlabel end)) } ]},
-                { type:"section", text:{ type:"mrkdwn", text:("*" + ($title|esc) + "*") } },
-                { type:"context", elements:[
-                    { type:"mrkdwn", text:("👤 " + ($user|esc) + "   ·   📅 " + $built + " UTC   ·   🔗 <" + $runurl + "|view run>") } ]}
-            ]}]}')"
+            --arg text "${icon} ${LABEL} ${version}: ${subtitle}  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
+            '{ text: $text }')"
         else
-          # No PR -> show the commit that triggered this deploy (target-sha).
-          commit_json="$(gh api "repos/${TARGET_REPO}/commits/${TARGET_SHA}" 2>/dev/null || echo '{}')"
-          c_subject="$(jq -r '.commit.message // "" | split("\n")[0] // ""' <<<"$commit_json")"
-          c_author="$(jq -r '.author.login // .commit.author.name // "unknown"' <<<"$commit_json")"
-          c_url="$(jq -r '.html_url // ""' <<<"$commit_json")"
-
-          if [ -n "$c_subject" ]; then
-            # Commit card: subject + short SHA (linked) + author.
-            payload="$(jq -n \
-              --arg color "$color" --arg header "${emoji} ${LABEL} — ${status}" \
-              --arg version "$version" --arg short "$short_sha" --arg cmurl "$c_url" \
-              --arg subject "$c_subject" --arg user "$c_author" --arg runurl "$RUN_URL" \
-              --arg built "$built_utc" '
-              def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
-              { attachments: [{ color: $color, blocks: [
-                  { type:"header", text:{ type:"plain_text", text:$header, emoji:true } },
-                  { type:"section", fields:[
-                      { type:"mrkdwn", text:("*Version*\n" + $version) },
-                      { type:"mrkdwn", text:("*Commit*\n" + (if $cmurl != "" then "<" + $cmurl + "|" + $short + ">" else "`" + $short + "`" end)) } ]},
-                  { type:"section", text:{ type:"mrkdwn", text:("*" + ($subject|esc) + "*") } },
-                  { type:"context", elements:[
-                      { type:"mrkdwn", text:("👤 " + ($user|esc) + "   ·   📅 " + $built + " UTC   ·   🔗 <" + $runurl + "|view run>") } ]}
-              ]}]}')"
-          else
-            # Fallback - nothing resolved: original single-line text format.
-            payload="$(jq -n \
-              --arg text "${emoji} ${LABEL}: ${status}  |  version \`${version}\`  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
-              '{ text: $text }')"
-          fi
+          # -- Build the field pairs (label, value) in order.
+          # Success: COMMIT, PR, AUTHOR, HASH, DURATION, DATE, URL   (7 rows)
+          # Failure: FAILED AT, COMMIT, PR, AUTHOR, HASH, DURATION, DATE  (7 rows)
+          fields_json="$(jq -n \
+            --arg is_success "$is_success" \
+            --arg failed_step "$failed_step" \
+            --arg subject "$commit_subject" \
+            --arg pr_label "$pr_label" --arg pr_url "$pr_url" \
+            --arg author "$commit_author" \
+            --arg short_sha "$short_sha" --arg commit_url "$commit_url" \
+            --arg duration "$duration" \
+            --arg built "$built_utc" \
+            --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
+            def mrk(t): { type: "mrkdwn", text: t };
+            def row(k; v): [ mrk("*" + k + "*"), mrk(v) ];
+            def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
+
+            ( if $is_success == "0" and $failed_step != "" then
+                row("FAILED AT"; ":red_circle: " + ($failed_step|esc))
+              else [] end )
+            + ( if $subject != "" then row("COMMIT"; ($subject|esc)) else [] end )
+            + ( if $pr_label != "" then
+                  row("PR"; (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end))
+                else [] end )
+            + row("AUTHOR"; ($author|esc))
+            + row("HASH"; (if $commit_url != "" then "<" + $commit_url + "|`" + $short_sha + "`>" else "`" + $short_sha + "`" end))
+            + ( if $duration != "" then row("DURATION"; $duration) else [] end )
+            + row("DATE"; $built + " UTC")
+            + ( if $is_success == "1" and $url_display != "" then
+                  row("URL"; "<" + $url_full + "|" + $url_display + ">")
+                else [] end )
+          ')"
+
+          # Split into chunks of at most 10 fields per section (Block Kit limit).
+          payload="$(jq -n \
+            --arg icon "$icon" --arg label "$LABEL" --arg version "$version" \
+            --arg subtitle "$subtitle" --arg runurl "$RUN_URL" \
+            --argjson fields "$fields_json" '
+            def chunk10:
+              . as $f
+              | [ range(0; ($f|length); 10) as $i | { type:"section", fields: $f[$i:$i+10] } ];
+
+            { blocks:
+              ( [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } },
+                  { type:"context", elements:[ { type:"mrkdwn", text: $subtitle } ] }
+                ]
+                + ($fields | chunk10)
+                + [ { type:"actions", elements:[
+                      { type:"button",
+                        text:{ type:"plain_text", text:"View run" },
+                        url: $runurl } ] } ]
+              )
+            }')"
         fi
 
         # -f makes curl fail on HTTP >=400 so Slack payload rejections surface in

From e8711575c0ef865aa4c1aff121555a8590887ca9 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 16:39:24 +0530
Subject: [PATCH 02/18] feat(notify): render COMMIT/PR as full-width mrkdwn so
 long subjects stay on one line

Slack section.fields is a 2-col grid that wraps values past ~45 chars per
column, which produced ugly multi-line COMMIT/PR rows on the v3.4.0 canary
card. Move both out of the field grid into a dedicated full-width mrkdwn
section above it; the grid keeps the short tabular rows (AUTHOR, HASH,
DURATION, DATE, URL / FAILED AT).
---
 actions/notify/action.yml | 34 ++++++++++++++++++++++++----------
 1 file changed, 24 insertions(+), 10 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index cab77c0..2475b26 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -154,14 +154,27 @@ runs:
             --arg text "${icon} ${LABEL} ${version}: ${subtitle}  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
             '{ text: $text }')"
         else
-          # -- Build the field pairs (label, value) in order.
-          # Success: COMMIT, PR, AUTHOR, HASH, DURATION, DATE, URL   (7 rows)
-          # Failure: FAILED AT, COMMIT, PR, AUTHOR, HASH, DURATION, DATE  (7 rows)
+          # -- COMMIT + PR live as full-width mrkdwn lines above the field grid.
+          # Slack's section.fields is a 2-col grid that wraps values past ~45
+          # chars per column; long commit subjects and PR refs wrap ugly there.
+          # A single mrkdwn block spans the whole card width instead.
+          header_lines="$(jq -n \
+            --arg subject "$commit_subject" \
+            --arg pr_label "$pr_label" --arg pr_url "$pr_url" '
+            def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
+            [ ( if $subject != "" then "*COMMIT*  " + ($subject|esc) else empty end ),
+              ( if $pr_label != "" then
+                  "*PR*  " + (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end)
+                else empty end )
+            ] | join("\n")
+          ')"
+
+          # -- Field grid holds the short, tabular rows only.
+          # Success: AUTHOR, HASH, DURATION, DATE, URL       (5 rows)
+          # Failure: FAILED AT, AUTHOR, HASH, DURATION, DATE (5 rows)
           fields_json="$(jq -n \
             --arg is_success "$is_success" \
             --arg failed_step "$failed_step" \
-            --arg subject "$commit_subject" \
-            --arg pr_label "$pr_label" --arg pr_url "$pr_url" \
             --arg author "$commit_author" \
             --arg short_sha "$short_sha" --arg commit_url "$commit_url" \
             --arg duration "$duration" \
@@ -174,10 +187,6 @@ runs:
             ( if $is_success == "0" and $failed_step != "" then
                 row("FAILED AT"; ":red_circle: " + ($failed_step|esc))
               else [] end )
-            + ( if $subject != "" then row("COMMIT"; ($subject|esc)) else [] end )
-            + ( if $pr_label != "" then
-                  row("PR"; (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end))
-                else [] end )
             + row("AUTHOR"; ($author|esc))
             + row("HASH"; (if $commit_url != "" then "<" + $commit_url + "|`" + $short_sha + "`>" else "`" + $short_sha + "`" end))
             + ( if $duration != "" then row("DURATION"; $duration) else [] end )
@@ -187,10 +196,12 @@ runs:
                 else [] end )
           ')"
 
-          # Split into chunks of at most 10 fields per section (Block Kit limit).
+          # Split fields into chunks of at most 10 per section (Block Kit limit).
+          # header_lines is omitted from the blocks list when empty (no PR, no commit subject).
           payload="$(jq -n \
             --arg icon "$icon" --arg label "$LABEL" --arg version "$version" \
             --arg subtitle "$subtitle" --arg runurl "$RUN_URL" \
+            --arg header_lines "$header_lines" \
             --argjson fields "$fields_json" '
             def chunk10:
               . as $f
@@ -200,6 +211,9 @@ runs:
               ( [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } },
                   { type:"context", elements:[ { type:"mrkdwn", text: $subtitle } ] }
                 ]
+                + ( if $header_lines != "" then
+                      [ { type:"section", text:{ type:"mrkdwn", text: $header_lines } } ]
+                    else [] end )
                 + ($fields | chunk10)
                 + [ { type:"actions", elements:[
                       { type:"button",

From 850c90e71a8942785f6d4ff42aa2058204d4b9f6 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 16:53:37 +0530
Subject: [PATCH 03/18] fix(notify): emit header_lines with jq -r so newline
 separates COMMIT and PR

Without -r, jq JSON-encodes the joined string; the literal \n survives all
the way into the Slack payload where it renders as the two-char sequence
instead of a line break. -r gives raw output; jq re-encodes correctly when
the second jq call rebuilds the payload.
---
 actions/notify/action.yml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 2475b26..75c7656 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -158,7 +158,9 @@ runs:
           # Slack's section.fields is a 2-col grid that wraps values past ~45
           # chars per column; long commit subjects and PR refs wrap ugly there.
           # A single mrkdwn block spans the whole card width instead.
-          header_lines="$(jq -n \
+          # `-r` is required: without it jq emits a JSON-encoded string with a
+          # literal `\n` two-char sequence, which Slack renders verbatim.
+          header_lines="$(jq -n -r \
             --arg subject "$commit_subject" \
             --arg pr_label "$pr_label" --arg pr_url "$pr_url" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");

From 4c86615c8cd9fc500610c67596cdf17e668fead4 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 17:26:23 +0530
Subject: [PATCH 04/18] feat(notify): unify field grid, drop subtitle, surface
 job name for FAILED AT
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Four fixes on the v3.4.0 canary card, all applied together to avoid extra
churn on the feat/slack-notify canary:

1. FAILED AT now shows the failed JOB name ("Build", "Deploy", …), not a
   step name. Inner steps of composite actions are opaque to the /jobs
   endpoint (only the outer `Run owner/repo/actions/name@ref` wrapper is
   visible), so step-level names were misleading in practice.
2. Drop the "Deploy failed" / "Deploy succeeded" subtitle. The header
   emoji already conveys status.
3. Fold COMMIT and PR back into the same 2-col field grid as the other
   rows, so all seven fields share a single uniform label/value block.
   Long commit subjects will wrap; consistency preferred here.
4. Drop the :red_circle: prefix from FAILED AT — the header emoji already
   marks the card as failed.
---
 actions/notify/action.yml | 66 +++++++++++++++++----------------------
 1 file changed, 29 insertions(+), 37 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 75c7656..66892aa 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -83,10 +83,11 @@ runs:
           exit 0
         fi
 
-        # Status → icon + subtitle.
-        icon=":x:"; subtitle="Deploy failed"; is_success=0
+        # Status icon only. Header emoji conveys success/failure — no separate
+        # subtitle line under it.
+        icon=":x:"; is_success=0
         if [ "${RESULT:-}" = "success" ]; then
-          icon=":white_check_mark:"; subtitle="Deploy succeeded"; is_success=1
+          icon=":white_check_mark:"; is_success=1
         fi
 
         version="${VERSION:-}"; [ -z "$version" ] && version="n/a"
@@ -112,12 +113,16 @@ runs:
           fi
         fi
 
-        # -- Failed step name: first step across all jobs with conclusion=failure.
-        # Only fetched on failure; falls back silently.
+        # -- Failed job name: the first pipeline stage (job) with conclusion=failure.
+        # We surface the JOB name — not a step name — because inner steps of
+        # composite actions are opaque to the /jobs endpoint (only the outer
+        # `Run owner/repo/actions/name@ref` wrapper is visible), which makes
+        # step-level names misleading. Job names ("Build", "Deploy", …) are
+        # always human-meaningful and match what users see in the Actions UI.
         failed_step=""
         if [ "$is_success" -eq 0 ]; then
           jobs_json="$(gh api "repos/${TARGET_REPO}/actions/runs/${RUN_ID}/jobs" 2>/dev/null || echo '{}')"
-          failed_step="$(jq -r '[.jobs[]? | .steps[]?] | map(select(.conclusion == "failure")) | .[0].name // ""' <<<"$jobs_json")"
+          failed_step="$(jq -r '[.jobs[]? | select(.conclusion == "failure")] | .[0].name // ""' <<<"$jobs_json")"
         fi
 
         # -- Resolve PR (preferred) or commit for the header rows.
@@ -151,32 +156,21 @@ runs:
         # Fallback path: neither PR nor commit resolved → single-line message.
         if [ -z "$commit_subject" ] && [ -z "$pr_num" ]; then
           payload="$(jq -n \
-            --arg text "${icon} ${LABEL} ${version}: ${subtitle}  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
+            --arg text "${icon} ${LABEL} ${version}  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
             '{ text: $text }')"
         else
-          # -- COMMIT + PR live as full-width mrkdwn lines above the field grid.
-          # Slack's section.fields is a 2-col grid that wraps values past ~45
-          # chars per column; long commit subjects and PR refs wrap ugly there.
-          # A single mrkdwn block spans the whole card width instead.
-          # `-r` is required: without it jq emits a JSON-encoded string with a
-          # literal `\n` two-char sequence, which Slack renders verbatim.
-          header_lines="$(jq -n -r \
-            --arg subject "$commit_subject" \
-            --arg pr_label "$pr_label" --arg pr_url "$pr_url" '
-            def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
-            [ ( if $subject != "" then "*COMMIT*  " + ($subject|esc) else empty end ),
-              ( if $pr_label != "" then
-                  "*PR*  " + (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end)
-                else empty end )
-            ] | join("\n")
-          ')"
-
-          # -- Field grid holds the short, tabular rows only.
-          # Success: AUTHOR, HASH, DURATION, DATE, URL       (5 rows)
-          # Failure: FAILED AT, AUTHOR, HASH, DURATION, DATE (5 rows)
+          # -- All fields share the same 2-col label/value grid for a single
+          # uniform block — same label column, same value-column start.
+          # Order (success): COMMIT, PR, AUTHOR, HASH, DURATION, DATE, URL
+          # Order (failure): FAILED AT, COMMIT, PR, AUTHOR, HASH, DURATION, DATE
+          # Trade-off: Slack section.fields wraps values past ~45 chars per
+          # column, so long commit subjects will span two lines. Uniformity
+          # is preferred over wrap-avoidance here.
           fields_json="$(jq -n \
             --arg is_success "$is_success" \
             --arg failed_step "$failed_step" \
+            --arg subject "$commit_subject" \
+            --arg pr_label "$pr_label" --arg pr_url "$pr_url" \
             --arg author "$commit_author" \
             --arg short_sha "$short_sha" --arg commit_url "$commit_url" \
             --arg duration "$duration" \
@@ -187,8 +181,12 @@ runs:
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
 
             ( if $is_success == "0" and $failed_step != "" then
-                row("FAILED AT"; ":red_circle: " + ($failed_step|esc))
+                row("FAILED AT"; ($failed_step|esc))
               else [] end )
+            + ( if $subject != "" then row("COMMIT"; ($subject|esc)) else [] end )
+            + ( if $pr_label != "" then
+                  row("PR"; (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end))
+                else [] end )
             + row("AUTHOR"; ($author|esc))
             + row("HASH"; (if $commit_url != "" then "<" + $commit_url + "|`" + $short_sha + "`>" else "`" + $short_sha + "`" end))
             + ( if $duration != "" then row("DURATION"; $duration) else [] end )
@@ -199,23 +197,17 @@ runs:
           ')"
 
           # Split fields into chunks of at most 10 per section (Block Kit limit).
-          # header_lines is omitted from the blocks list when empty (no PR, no commit subject).
+          # Header emoji conveys status; no separate context/subtitle block.
           payload="$(jq -n \
             --arg icon "$icon" --arg label "$LABEL" --arg version "$version" \
-            --arg subtitle "$subtitle" --arg runurl "$RUN_URL" \
-            --arg header_lines "$header_lines" \
+            --arg runurl "$RUN_URL" \
             --argjson fields "$fields_json" '
             def chunk10:
               . as $f
               | [ range(0; ($f|length); 10) as $i | { type:"section", fields: $f[$i:$i+10] } ];
 
             { blocks:
-              ( [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } },
-                  { type:"context", elements:[ { type:"mrkdwn", text: $subtitle } ] }
-                ]
-                + ( if $header_lines != "" then
-                      [ { type:"section", text:{ type:"mrkdwn", text: $header_lines } } ]
-                    else [] end )
+              ( [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } } ]
                 + ($fields | chunk10)
                 + [ { type:"actions", elements:[
                       { type:"button",

From bbe86167a36f96b79df8a27a95ebe5e0dd166aea Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 21:28:04 +0530
Subject: [PATCH 05/18] feat(notify): full-width value column with
 monospace-padded labels

Replace the section.fields 2-col grid (which capped values at ~30 chars per
column and wrapped long commit subjects / PR refs) with a single
section.text mrkdwn block. Labels are wrapped in inline code (`) so they
render in monospace, then right-padded to 9 chars ("FAILED AT" is the
longest). Result: labels line up in a fixed left column, values get the
full remaining card width, long commit subjects and PR refs no longer wrap.
---
 actions/notify/action.yml | 58 ++++++++++++++++++---------------------
 1 file changed, 27 insertions(+), 31 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 66892aa..ac1aa93 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -159,14 +159,15 @@ runs:
             --arg text "${icon} ${LABEL} ${version}  |  📅 ${built_utc} UTC  |  <${RUN_URL}|view run>" \
             '{ text: $text }')"
         else
-          # -- All fields share the same 2-col label/value grid for a single
-          # uniform block — same label column, same value-column start.
-          # Order (success): COMMIT, PR, AUTHOR, HASH, DURATION, DATE, URL
-          # Order (failure): FAILED AT, COMMIT, PR, AUTHOR, HASH, DURATION, DATE
-          # Trade-off: Slack section.fields wraps values past ~45 chars per
-          # column, so long commit subjects will span two lines. Uniformity
-          # is preferred over wrap-avoidance here.
-          fields_json="$(jq -n \
+          # -- Single full-width mrkdwn section with monospace-padded labels.
+          # Slack's section.fields is a 2-col grid ~50/50; the value column is
+          # only ~30 chars, which wrapped long commit subjects and PR refs.
+          # Wrapping each label in inline code (`) forces monospace, so we can
+          # right-pad to a fixed 9-char column ("FAILED AT" is the longest at
+          # 9). That preserves the aligned-column look while giving the value
+          # column the full card width — long values no longer wrap.
+          # `-r` is required so the joined newlines survive raw to the outer jq.
+          rows_text="$(jq -n -r \
             --arg is_success "$is_success" \
             --arg failed_step "$failed_step" \
             --arg subject "$commit_subject" \
@@ -176,44 +177,39 @@ runs:
             --arg duration "$duration" \
             --arg built "$built_utc" \
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
-            def mrk(t): { type: "mrkdwn", text: t };
-            def row(k; v): [ mrk("*" + k + "*"), mrk(v) ];
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
+            def label(k): "`" + k + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end)) + "`";
+            def row(k; v): label(k) + " " + v;
 
             ( if $is_success == "0" and $failed_step != "" then
-                row("FAILED AT"; ($failed_step|esc))
+                [ row("FAILED AT"; ($failed_step|esc)) ]
               else [] end )
-            + ( if $subject != "" then row("COMMIT"; ($subject|esc)) else [] end )
+            + ( if $subject != "" then [ row("COMMIT"; ($subject|esc)) ] else [] end )
             + ( if $pr_label != "" then
-                  row("PR"; (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end))
+                  [ row("PR"; (if $pr_url != "" then "<" + $pr_url + "|" + $pr_label + ">" else $pr_label end)) ]
                 else [] end )
-            + row("AUTHOR"; ($author|esc))
-            + row("HASH"; (if $commit_url != "" then "<" + $commit_url + "|`" + $short_sha + "`>" else "`" + $short_sha + "`" end))
-            + ( if $duration != "" then row("DURATION"; $duration) else [] end )
-            + row("DATE"; $built + " UTC")
+            + [ row("AUTHOR"; ($author|esc)) ]
+            + [ row("HASH"; (if $commit_url != "" then "<" + $commit_url + "|`" + $short_sha + "`>" else "`" + $short_sha + "`" end)) ]
+            + ( if $duration != "" then [ row("DURATION"; $duration) ] else [] end )
+            + [ row("DATE"; ($built + " UTC")) ]
             + ( if $is_success == "1" and $url_display != "" then
-                  row("URL"; "<" + $url_full + "|" + $url_display + ">")
+                  [ row("URL"; "<" + $url_full + "|" + $url_display + ">") ]
                 else [] end )
+            | join("\n")
           ')"
 
-          # Split fields into chunks of at most 10 per section (Block Kit limit).
           # Header emoji conveys status; no separate context/subtitle block.
           payload="$(jq -n \
             --arg icon "$icon" --arg label "$LABEL" --arg version "$version" \
             --arg runurl "$RUN_URL" \
-            --argjson fields "$fields_json" '
-            def chunk10:
-              . as $f
-              | [ range(0; ($f|length); 10) as $i | { type:"section", fields: $f[$i:$i+10] } ];
-
+            --arg rows_text "$rows_text" '
             { blocks:
-              ( [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } } ]
-                + ($fields | chunk10)
-                + [ { type:"actions", elements:[
-                      { type:"button",
-                        text:{ type:"plain_text", text:"View run" },
-                        url: $runurl } ] } ]
-              )
+              [ { type:"header", text:{ type:"plain_text", text:($icon + " " + $label + " " + $version), emoji:true } },
+                { type:"section", text:{ type:"mrkdwn", text: $rows_text } },
+                { type:"actions", elements:[
+                    { type:"button",
+                      text:{ type:"plain_text", text:"View run" },
+                      url: $runurl } ] } ]
             }')"
         fi
 

From 364e610b6dbcb983e096a3ab919db52c0166104a Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 21:38:56 +0530
Subject: [PATCH 06/18] fix(notify): rename jq helper from label to padlabel

`label` is a jq reserved keyword (used with `label $out | break`), so
`def label(k): ...` was a compile error. Rename to `padlabel`.
---
 actions/notify/action.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index ac1aa93..ae8eeab 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -178,8 +178,9 @@ runs:
             --arg built "$built_utc" \
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
-            def label(k): "`" + k + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end)) + "`";
-            def row(k; v): label(k) + " " + v;
+            # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
+            def padlabel(k): "`" + k + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end)) + "`";
+            def row(k; v): padlabel(k) + " " + v;
 
             ( if $is_success == "0" and $failed_step != "" then
                 [ row("FAILED AT"; ($failed_step|esc)) ]

From 1f978e48fb1122f3ed4c5511b7af1f7fb801f5fb Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 21:46:10 +0530
Subject: [PATCH 07/18] style(notify): use plain bold for labels, drop
 inline-code background

---
 actions/notify/action.yml | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index ae8eeab..093fe6c 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -179,7 +179,10 @@ runs:
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
             # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
-            def padlabel(k): "`" + k + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end)) + "`";
+            # Bold label + trailing spaces for approximate column alignment;
+            # Slack renders in a variable-width font so alignment is best-effort,
+            # but plain bold (no code background) is the preferred style.
+            def padlabel(k): "*" + k + "*" + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end));
             def row(k; v): padlabel(k) + " " + v;
 
             ( if $is_success == "0" and $failed_step != "" then

From f4e0379087d754684cbb1775757ce959b2c978a5 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 21:54:43 +0530
Subject: [PATCH 08/18] style(notify): pad labels with NBSP so Slack preserves
 the column spacing

Slack collapses runs of ASCII spaces in mrkdwn to a single space, so the
plain-bold + space-padding attempt (1f978e4) lost its alignment. Non-
breaking spaces (U+00A0) are preserved individually; Slack still uses a
variable-width font so alignment is approximate (bold letter width > NBSP
width), but the value column drift is small and consistent. Bumped the
padding target from 9 to 12 chars to compensate for NBSP being narrower
than a bold letter.
---
 actions/notify/action.yml | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 093fe6c..62dc5e1 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -179,11 +179,14 @@ runs:
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
             # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
-            # Bold label + trailing spaces for approximate column alignment;
-            # Slack renders in a variable-width font so alignment is best-effort,
-            # but plain bold (no code background) is the preferred style.
-            def padlabel(k): "*" + k + "*" + (((9 - (k|length)) as $n | if $n > 0 then " " * $n else "" end));
-            def row(k; v): padlabel(k) + " " + v;
+            # Bold label + trailing NBSP ( ) padding for column alignment.
+            # Slack collapses runs of regular ASCII spaces in mrkdwn to a single
+            # space, so plain-space padding is useless; NBSPs render individually
+            # and preserve spacing. Slack's variable-width font still means
+            # alignment is approximate (a bold letter is wider than a space
+            # glyph), but the value column drift is small and consistent.
+            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then " " * $n else " " end));
+            def row(k; v): padlabel(k) + v;
 
             ( if $is_success == "0" and $failed_step != "" then
                 [ row("FAILED AT"; ($failed_step|esc)) ]

From 2e7e8ef7b4f2a0763195280587e5c079956cdb85 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 22:10:29 +0530
Subject: [PATCH 09/18] fix(notify): emit NBSP via jq \u00a0 escape, not a
 literal in YAML

The previous commit put literal U+00A0 bytes in the composite-action's
run: block. GH Actions rendered the run: value to a temp .sh file and
bash choked on it ("syntax error near unexpected token '('", line 110)
- something in the render pipeline mis-tokenises literal NBSPs even when
they sit inside a jq single-quoted expression. Emit the NBSPs from jq's
\u00a0 escape instead so the action.yml source stays pure ASCII; the
final Slack payload still contains real NBSPs for the column padding.
---
 actions/notify/action.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 62dc5e1..42e56b8 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -179,13 +179,13 @@ runs:
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
             # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
-            # Bold label + trailing NBSP ( ) padding for column alignment.
+            # Bold label + trailing NBSP (\u00a0) padding for column alignment.
             # Slack collapses runs of regular ASCII spaces in mrkdwn to a single
             # space, so plain-space padding is useless; NBSPs render individually
             # and preserve spacing. Slack's variable-width font still means
             # alignment is approximate (a bold letter is wider than a space
             # glyph), but the value column drift is small and consistent.
-            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then " " * $n else " " end));
+            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then "\u00a0" * $n else "\u00a0" end));
             def row(k; v): padlabel(k) + v;
 
             ( if $is_success == "0" and $failed_step != "" then

From 681564dc5b8333b05b3f77c416ae7889243e0392 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 22:24:09 +0530
Subject: [PATCH 10/18] fix(notify): remove apostrophe from comment inside jq
 single-quoted block

Slacks (formerly Slacks apostrophe-s) in a comment inside the jq  argument
closed the outer bash single quote, so everything after it was parsed as
shell and hit a syntax error at the padlabel line. Rephrase the comment
apostrophe-free and add a NOTE explaining the shell-quoting gotcha.
---
 actions/notify/action.yml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 42e56b8..d6fa0c6 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -182,9 +182,11 @@ runs:
             # Bold label + trailing NBSP (\u00a0) padding for column alignment.
             # Slack collapses runs of regular ASCII spaces in mrkdwn to a single
             # space, so plain-space padding is useless; NBSPs render individually
-            # and preserve spacing. Slack's variable-width font still means
-            # alignment is approximate (a bold letter is wider than a space
+            # and preserve spacing. The Slack renderer uses a variable-width font
+            # so alignment is approximate (a bold letter is wider than a space
             # glyph), but the value column drift is small and consistent.
+            # NOTE: no apostrophes in these comments  the outer shell context
+            # is a single-quoted jq expression and an apostrophe would close it.
             def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then "\u00a0" * $n else "\u00a0" end));
             def row(k; v): padlabel(k) + v;
 

From 993bb11591e3fa7d090bb0778bc9d0387f32600a Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Wed, 15 Jul 2026 22:50:20 +0530
Subject: [PATCH 11/18] style(notify): swap NBSP padding for em space so
 alignment matches bold letter width

NBSP (\u00a0) padding rendered narrower than bold letters in Slacks Lato font,
so labels with more bold chars (COMMIT, AUTHOR) drifted right of short-label
rows (PR, URL). Em space (\u2003) is defined as the fonts em width, which is
roughly the width of a bold letter. That makes total label+padding width
= approximately em_width * 12 for every row, giving consistent column alignment.
---
 actions/notify/action.yml | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index d6fa0c6..6392e48 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -179,15 +179,18 @@ runs:
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
             # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
-            # Bold label + trailing NBSP (\u00a0) padding for column alignment.
-            # Slack collapses runs of regular ASCII spaces in mrkdwn to a single
-            # space, so plain-space padding is useless; NBSPs render individually
-            # and preserve spacing. The Slack renderer uses a variable-width font
-            # so alignment is approximate (a bold letter is wider than a space
-            # glyph), but the value column drift is small and consistent.
+            # Bold label + trailing EM SPACE (\u2003) padding for column alignment.
+            # An em space is the fonts em width (~font size in px), which in
+            # Slacks Lato font renders at roughly the same visual width as a
+            # bold letter. So `bold_char_width * N_letters + em_width * N_padding`
+            # collapses to ~`em_width * (N_letters + N_padding)` = constant for
+            # all labels padded to 12 chars total. NBSPs were tried first but
+            # render narrower than bold letters, which caused labels with more
+            # bold chars (COMMIT, AUTHOR) to visibly drift right relative to
+            # short-label rows (URL, PR).
             # NOTE: no apostrophes in these comments  the outer shell context
             # is a single-quoted jq expression and an apostrophe would close it.
-            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then "\u00a0" * $n else "\u00a0" end));
+            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then "\u2003" * $n else "\u2003" end));
             def row(k; v): padlabel(k) + v;
 
             ( if $is_success == "0" and $failed_step != "" then

From 21005caf9c5783bf635a27361948129e884b5023 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 11:45:21 +0530
Subject: [PATCH 12/18] style(notify): fix DURATION column drift and switch
 DATE to ISO format

Two fixes on the v3.4.0 canary card:

1. DURATION value drifted slightly left of the PR/AUTHOR/HASH/DATE/URL
   column. Root cause: em-space (1 em) is wider than a bold letter
   (~0.75 em in Slacks Lato), so a longer label paired with fewer
   em-spaces of padding produced a NARROWER total width. Pure em-space
   padding cannot align rows when letter width and space width diverge.
   Fix: keep the em-space padding (pad to 12 chars total) and add N
   NBSPs per label where N = label length. NBSP is ~0.25 em, so the
   NBSP tail compensates for the 0.25 em per-letter shortfall. Total
   width = L*0.75 + (12-L)*1 + L*0.25 = 12 em, constant across labels.
   DURATION (8 chars) now gets the largest NBSP tail (8), PR (2 chars)
   the smallest (2).

2. DATE format: dots to dashes. `2026.07.15` reads as a version number;
   ISO `2026-07-15` is the recognised date format.
---
 actions/notify/action.yml | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 6392e48..be045ff 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -92,7 +92,7 @@ runs:
 
         version="${VERSION:-}"; [ -z "$version" ] && version="n/a"
         short_sha="${TARGET_SHA:0:7}"
-        built_utc="$(date -u +%Y.%m.%d)"
+        built_utc="$(date -u +%Y-%m-%d)"
 
         # -- Duration: pull run_started_at, compute elapsed vs now.
         # Falls back silently if the API call fails.
@@ -179,18 +179,23 @@ runs:
             --arg url_display "$url_display" --arg url_full "$DEPLOYED_URL" '
             def esc: gsub("&";"&amp;") | gsub("<";"&lt;") | gsub(">";"&gt;");
             # `label` is a jq reserved keyword (label/break), so name this `padlabel`.
-            # Bold label + trailing EM SPACE (\u2003) padding for column alignment.
-            # An em space is the fonts em width (~font size in px), which in
-            # Slacks Lato font renders at roughly the same visual width as a
-            # bold letter. So `bold_char_width * N_letters + em_width * N_padding`
-            # collapses to ~`em_width * (N_letters + N_padding)` = constant for
-            # all labels padded to 12 chars total. NBSPs were tried first but
-            # render narrower than bold letters, which caused labels with more
-            # bold chars (COMMIT, AUTHOR) to visibly drift right relative to
-            # short-label rows (URL, PR).
+            # Two-space padding scheme for column alignment in Slack Lato:
+            #   1. Em space (\u2003 = 1 em)     \u2014 pad to 12 chars total.
+            #   2. NBSP     (\u00a0 = ~0.25 em) \u2014 add N per label = label length.
+            # In Slack Lato, bold letter width is ~0.75 em, em space is 1 em, NBSP
+            # is ~0.25 em. Total row width = L*0.75 + (12-L)*1 + L*0.25 = 12 em,
+            # constant across all label lengths. Pure em-space padding drifted
+            # because bold letter (0.75) < em (1), so longer labels came out
+            # narrower \u2014 visible as DURATION (8 chars) drifting left of PR (2).
+            # The per-label NBSP tail scales with L to make up the missing 0.25 em
+            # per bold letter, so DURATION gets the largest NBSP tail.
             # NOTE: no apostrophes in these comments  the outer shell context
             # is a single-quoted jq expression and an apostrophe would close it.
-            def padlabel(k): "*" + k + "*" + (((12 - (k|length)) as $n | if $n > 0 then "\u2003" * $n else "\u2003" end));
+            def padlabel(k):
+              (k | length) as $L
+              | "*" + k + "*"
+                + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
+                + "\u00a0" * $L;
             def row(k; v): padlabel(k) + v;
 
             ( if $is_success == "0" and $failed_step != "" then

From 4186ffe532fc6f99fff1c5fa2c0e00d1ee6fcf5e Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 12:35:25 +0530
Subject: [PATCH 13/18] style(notify): separate rows with a blank line

Change the row join separator from `\n` to `\n\n` so every field is
visually separated by one blank line (COMMIT / PR / AUTHOR / HASH /
DURATION / DATE / URL). Denser vertical rhythm makes long commit
subjects easier to distinguish from the row below when the value wraps.

No structural change: header, actions block, and field order all
untouched.
---
 actions/notify/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index be045ff..7258871 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -212,7 +212,7 @@ runs:
             + ( if $is_success == "1" and $url_display != "" then
                   [ row("URL"; "<" + $url_full + "|" + $url_display + ">") ]
                 else [] end )
-            | join("\n")
+            | join("\n\n")
           ')"
 
           # Header emoji conveys status; no separate context/subtitle block.

From 81e85ee4db78703838d578a4a633f25b286244a9 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 12:55:05 +0530
Subject: [PATCH 14/18] style(notify): shave 2 NBSPs off COMMIT / AUTHOR /
 DURATION

Post-dispatch screenshot of 4186ffe showed COMMIT, AUTHOR, and DURATION
values drifting right of the column formed by PR / HASH / DATE / URL /
FAILED AT. Root cause is that Slack Lato bold letter width varies per
character: the three drifted labels are all-caps single words dominated
by wide letters (M / D / U / R / A / O / N), so under the L*0.75 em
approximation they render wider than the formula expects and the padded
row overshoots the reference column.

Fix: hard-code a 2-NBSP reduction on those three labels only. Others
untouched. This is empirical tuning, not a general formula, and may need
another iteration of +/- 1 NBSP after the next screenshot.

Padding counts after this commit:
  COMMIT    6 em + 4 NBSP  (was 6 + 6)
  AUTHOR    6 em + 4 NBSP  (was 6 + 6)
  DURATION  4 em + 6 NBSP  (was 4 + 8)
  everything else unchanged
---
 actions/notify/action.yml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 7258871..0f301e2 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -193,9 +193,15 @@ runs:
             # is a single-quoted jq expression and an apostrophe would close it.
             def padlabel(k):
               (k | length) as $L
+              # Empirical shave: COMMIT / AUTHOR / DURATION are all-caps single
+              # words dominated by wide bold letters (M, D, U, R, A, O, N), so
+              # under the analytical formula their rendered width overshoots
+              # the reference column formed by the other labels. Drop 2 NBSPs
+              # from those three to pull their values back into the same X.
+              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 else 0 end) as $shave
               | "*" + k + "*"
                 + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
-                + "\u00a0" * $L;
+                + "\u00a0" * ($L - $shave);
             def row(k; v): padlabel(k) + v;
 
             ( if $is_success == "0" and $failed_step != "" then

From 08ee58b68bd8149840a191fd3694af8334e8b062 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 13:04:20 +0530
Subject: [PATCH 15/18] style(notify): add 1 NBSP to PR to close last column
 gap
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Post-shave dispatch of 81e85ee confirmed COMMIT / AUTHOR / HASH /
DURATION / DATE / URL all landed on the reference column. Only PR
still sat slightly left. PR is only 2 letters (P + R) and both are
medium-width, so the label ends at a narrower X than the four-letter
labels — the em-space count is already at ceiling (10) and the letter
contribution is small, leaving the row slightly under target width.

Fix: negative shave on PR only (extends the NBSP tail by 1). No other
label touched.
---
 actions/notify/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 0f301e2..c83bfa9 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -198,7 +198,7 @@ runs:
               # under the analytical formula their rendered width overshoots
               # the reference column formed by the other labels. Drop 2 NBSPs
               # from those three to pull their values back into the same X.
-              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 else 0 end) as $shave
+              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 elif k == "PR" then -1 else 0 end) as $shave
               | "*" + k + "*"
                 + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
                 + "\u00a0" * ($L - $shave);

From 75ab3e333da2117ee50f9f1783be8d559e0ae922 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 13:12:40 +0530
Subject: [PATCH 16/18] Revert "style(notify): add 1 NBSP to PR to close last
 column gap"
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts 08ee58b. The +1 NBSP overshot — PR now sits slightly right
of the reference column. Remove the -1 shave clause, back to PR = 10 em
+ 2 NBSP.

We do not have a fractional-width padding char that fits between 2 and
3 NBSPs (~0.25 em each), so this direction is capped. If PR's residual
drift left is still visible after this revert, the remaining options are
option 3 (monospace code block) or accepting the sub-NBSP miss.
---
 actions/notify/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index c83bfa9..0f301e2 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -198,7 +198,7 @@ runs:
               # under the analytical formula their rendered width overshoots
               # the reference column formed by the other labels. Drop 2 NBSPs
               # from those three to pull their values back into the same X.
-              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 elif k == "PR" then -1 else 0 end) as $shave
+              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 else 0 end) as $shave
               | "*" + k + "*"
                 + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
                 + "\u00a0" * ($L - $shave);

From cf0c7fff194341111f423f86f99c636bc4eb2141 Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 13:25:44 +0530
Subject: [PATCH 17/18] style(notify): add 2 NBSPs to PR row to close residual
 column gap
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

75ab3e3 reverted the earlier PR +1 NBSP (which overshot). Post-revert
dispatch confirmed PR still sits slightly left of the reference column.
+2 NBSPs (10 em + 4 NBSP total on PR) should land closer to the target
than +1 did — the earlier +1 attempt was measured against the pre-shave
formula and may have interacted with the wide-letter-shave landing.
Nothing else touched.
---
 actions/notify/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 0f301e2..8d2538e 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -198,7 +198,7 @@ runs:
               # under the analytical formula their rendered width overshoots
               # the reference column formed by the other labels. Drop 2 NBSPs
               # from those three to pull their values back into the same X.
-              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 else 0 end) as $shave
+              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 elif k == "PR" then -2 else 0 end) as $shave
               | "*" + k + "*"
                 + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
                 + "\u00a0" * ($L - $shave);

From 772fc1aed8d7e59c09a45b645f0993869f8e8caa Mon Sep 17 00:00:00 2001
From: chinmay jain <jchinmay654@gmail.com>
Date: Thu, 16 Jul 2026 14:26:07 +0530
Subject: [PATCH 18/18] style(notify): add 2 NBSPs to FAILED AT row to match PR
 alignment fix

Same drift and same fix as cf0c7ff for PR: the FAILED AT row (present
only on failure cards) sits slightly left of the reference column,
because the analytical formula under-estimates the width shortfall of
labels whose bold-letter content skews narrow (F, I, L, T, E + space).
Extend the shave map so FAILED AT gets the same -2 (i.e. +2 NBSP) that
PR does. FAILED AT now 3 em + 11 NBSP.

No other label touched.
---
 actions/notify/action.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/actions/notify/action.yml b/actions/notify/action.yml
index 8d2538e..0455fbd 100644
--- a/actions/notify/action.yml
+++ b/actions/notify/action.yml
@@ -198,7 +198,7 @@ runs:
               # under the analytical formula their rendered width overshoots
               # the reference column formed by the other labels. Drop 2 NBSPs
               # from those three to pull their values back into the same X.
-              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 elif k == "PR" then -2 else 0 end) as $shave
+              | (if k == "COMMIT" or k == "AUTHOR" or k == "DURATION" then 2 elif k == "PR" or k == "FAILED AT" then -2 else 0 end) as $shave
               | "*" + k + "*"
                 + "\u2003" * (if 12 - $L > 0 then 12 - $L else 0 end)
                 + "\u00a0" * ($L - $shave);