From 90762d8de33cd01e52c480031fff2c51accb4fc3 Mon Sep 17 00:00:00 2001 From: openhands Date: Wed, 18 Mar 2026 07:18:57 +0000 Subject: [PATCH] =?UTF-8?q?fix:=20address=20review=20feedback=20=E2=80=94?= =?UTF-8?q?=20CODEBERG=5FWEB=20unbound,=20title=20prefix,=20emoji?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Replace ${CODEBERG_WEB} with inline https://codeberg.org/${CODEBERG_REPO} to avoid unbound variable crash in gardener-poll.sh (set -euo pipefail) - Change sub-issue title prefix from fix: to chore: since it's an investigation task, not a code fix - Add emoji prefix to idle_timeout matrix notification for consistency Co-Authored-By: Claude Opus 4.6 --- gardener/gardener-poll.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gardener/gardener-poll.sh b/gardener/gardener-poll.sh index 4be31d6..ff0bcae 100755 --- a/gardener/gardener-poll.sh +++ b/gardener/gardener-poll.sh @@ -1001,8 +1001,8 @@ if [ -s "$ESCALATION_FILE" ]; then # Handle idle_timeout escalations — no CI steps to inspect, just notify if [[ "$ESC_REASON" == idle_timeout* ]]; then - _issue_url="${CODEBERG_WEB}/issues/${ESC_ISSUE}" - sub_title="fix: investigate idle timeout for issue #${ESC_ISSUE}" + _issue_url="https://codeberg.org/${CODEBERG_REPO}/issues/${ESC_ISSUE}" + sub_title="chore: investigate idle timeout for issue #${ESC_ISSUE}" sub_body="## Dev-agent idle timeout The dev-agent session for issue #${ESC_ISSUE} was idle for 2h without a phase update and was killed.$([ "${ESC_PR:-0}" != "0" ] && printf '\n\nPR #%s may still be open.' "$ESC_PR") @@ -1025,7 +1025,7 @@ The dev-agent session for issue #${ESC_ISSUE} was idle for 2h without a phase up if [ -n "$new_issue" ]; then log "Created idle-timeout sub-issue #${new_issue} for #${ESC_ISSUE}" _esc_total_created=$((_esc_total_created + 1)) - matrix_send "gardener" "Created #${new_issue}: idle timeout on #${ESC_ISSUE}" 2>/dev/null || true + matrix_send "gardener" "⏱ Created #${new_issue}: idle timeout on #${ESC_ISSUE}" 2>/dev/null || true fi echo "$esc_entry" >> "$ESCALATION_DONE"