fix: address review — phase signal quoting, issue count limits, reply comment

- Fix critical: use double quotes for $PHASE_FILE in formula phase signal
- Fix low: use limit=50 for backlog/in-progress/blocked issue counts
- Fix nit: correct misleading comment about escalation reply timing

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-21 00:39:58 +00:00
parent bfdc01202c
commit 52f7c4973e
2 changed files with 5 additions and 5 deletions

View file

@ -232,6 +232,6 @@ IMPORTANT: Do NOT commit or push the journal — it is a local working file.
The journal directory is committed to git periodically by other agents. The journal directory is committed to git periodically by other agents.
After writing the journal, write the phase signal: After writing the journal, write the phase signal:
echo 'PHASE:done' > '$PHASE_FILE' echo 'PHASE:done' > "$PHASE_FILE"
""" """
needs = ["report"] needs = ["report"]

View file

@ -139,9 +139,9 @@ echo ""
# ── Backlog + In-Progress ───────────────────────────────────────────────── # ── Backlog + In-Progress ─────────────────────────────────────────────────
echo "## Issue Status (${PROJECT_NAME})" echo "## Issue Status (${PROJECT_NAME})"
_backlog_count=$(codeberg_api GET "/issues?state=open&labels=backlog&type=issues&limit=1" 2>/dev/null | jq 'length' 2>/dev/null || echo "?") _backlog_count=$(codeberg_api GET "/issues?state=open&labels=backlog&type=issues&limit=50" 2>/dev/null | jq 'length' 2>/dev/null || echo "?")
_in_progress_count=$(codeberg_api GET "/issues?state=open&labels=in-progress&type=issues&limit=1" 2>/dev/null | jq 'length' 2>/dev/null || echo "?") _in_progress_count=$(codeberg_api GET "/issues?state=open&labels=in-progress&type=issues&limit=50" 2>/dev/null | jq 'length' 2>/dev/null || echo "?")
_blocked_count=$(codeberg_api GET "/issues?state=open&labels=blocked&type=issues&limit=1" 2>/dev/null | jq 'length' 2>/dev/null || echo "?") _blocked_count=$(codeberg_api GET "/issues?state=open&labels=blocked&type=issues&limit=50" 2>/dev/null | jq 'length' 2>/dev/null || echo "?")
echo "Backlog: ${_backlog_count}, In-progress: ${_in_progress_count}, Blocked: ${_blocked_count}" echo "Backlog: ${_backlog_count}, In-progress: ${_in_progress_count}, Blocked: ${_blocked_count}"
echo "" echo ""
@ -181,7 +181,7 @@ echo "## Escalation Replies (from Matrix)"
if [ -s /tmp/supervisor-escalation-reply ]; then if [ -s /tmp/supervisor-escalation-reply ]; then
cat /tmp/supervisor-escalation-reply cat /tmp/supervisor-escalation-reply
echo "" echo ""
echo "(This reply will be consumed after this run)" echo "(Reply already consumed by supervisor-run.sh before this session)"
else else
echo " None" echo " None"
fi fi