[object Object]

← back to Cli Printing Press

fix(ci): close jq if-then-else with end so >10-thread path doesn't crash

f7365c2602d828bae4352c9f316da496d8d36ce1 · 2026-05-16 16:09:25 -0700 · Trevin Chow

Mirrors printing-press-library#623: outer if in the listing jq filter
was missing 'end'. jq emits a parse error and exits non-zero; set -e
bails the shell before the step summary and ::error:: annotation can
write, so authors see a cryptic jq error instead of the thread list.

Files touched

Diff

commit f7365c2602d828bae4352c9f316da496d8d36ce1
Author: Trevin Chow <trevin@trevinchow.com>
Date:   Sat May 16 16:09:25 2026 -0700

    fix(ci): close jq if-then-else with end so >10-thread path doesn't crash
    
    Mirrors printing-press-library#623: outer if in the listing jq filter
    was missing 'end'. jq emits a parse error and exits non-zero; set -e
    bails the shell before the step summary and ::error:: annotation can
    write, so authors see a cryptic jq error instead of the thread list.
---
 .github/workflows/conversation-resolution-check.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/conversation-resolution-check.yml b/.github/workflows/conversation-resolution-check.yml
index 7d234f4a..a2c5f8d4 100644
--- a/.github/workflows/conversation-resolution-check.yml
+++ b/.github/workflows/conversation-resolution-check.yml
@@ -165,7 +165,7 @@ jobs:
               (.comments.nodes[0].author.login // "unknown") + "`" +
               (if .isOutdated then " *(outdated — line no longer exists; likely addressed by a later commit)*" else "" end)
             ] | join("\n")) +
-            (if $count > 10 then "\n\n_… and " + ($count - 10 | tostring) + " more._" else "")
+            (if $count > 10 then "\n\n_… and " + ($count - 10 | tostring) + " more._" else "" end)
           ' "${threads_json}")"
 
           if [ "${unresolved_active}" -gt 0 ] && [ "${unresolved_outdated}" -gt 0 ]; then

← 94332417 fix(ci): use job exit-code instead of POST/PATCH for check_r  ·  back to Cli Printing Press  ·  fix(skills): split SQL example out of go-fenced block in NUL 63f745b8 →