From 44749fdce931f63b3f65692f7996c2edd42491ec Mon Sep 17 00:00:00 2001 From: Gabriel Anhaia Date: Sat, 30 May 2026 10:43:42 +0200 Subject: [PATCH] fix(pr-digest): wrap thread-breakdown string concat in parens MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit jq could not parse: text: "literal\n" + (.expr) inside an object literal value — it consumed the string, expected '}' to close the object, and got '+' instead, erroring with: syntax error, unexpected '+', expecting '}' (Unix shell quoting issues?) Wrapping the concatenation in parentheses disambiguates the parser: text: ("literal\n" + (.expr)) This matches the established pattern in the main-message step. Caught by the first real production run (112 open PRs, 29 stale, 5 critical): the main digest posted successfully but the thread reply step crashed. The same shape is now valid jq grammar in all encountered versions. --- .github/workflows/pr-digest.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr-digest.yml b/.github/workflows/pr-digest.yml index c60efbf..607ac98 100644 --- a/.github/workflows/pr-digest.yml +++ b/.github/workflows/pr-digest.yml @@ -415,8 +415,8 @@ jobs: { type: "section", text: { type: "mrkdwn", - text: "*`\(.area)`* — \(.prs | length) stale\n" + - (.prs | map(prLine(.)) | join("\n")) + text: ("*`\(.area)`* — \(.prs | length) stale\n" + + (.prs | map(prLine(.)) | join("\n"))) } } ] ) | flatten) )