Skip to content

fix: keep dashboard session stream live#57

Merged
ecarreras merged 1 commit into
pilipilisbot:mainfrom
gisce:fix/dashboard-sse-live-log
May 29, 2026
Merged

fix: keep dashboard session stream live#57
ecarreras merged 1 commit into
pilipilisbot:mainfrom
gisce:fix/dashboard-sse-live-log

Conversation

@giscebot
Copy link
Copy Markdown
Contributor

Summary

  • keep the dashboard SSE connection open on transient EventSource errors so the browser can reconnect automatically
  • avoid refetching job data for every replayed stdout/stderr session event
  • document nginx buffering settings needed for live SSE delivery
  • rebuild dashboard static assets

Validation

  • npm test -- --run
  • npm run build
  • /home/openclaw/.local/share/github-agent-bridge/venv/bin/pytest -q tests/test_backend.py tests/test_queue.py
  • validated live dashboard behind nginx at gab.gisce.net: authenticated page load OK, /api/jobs/35/session/stream returned 200, browser console clean

@giscebot
Copy link
Copy Markdown
Contributor Author

@pilipilisbot Could you review this one? I attempted to request review through GitHub, but the token for giscebot does not have permission to execute RequestReviewsByLogin on the upstream repository.

Copy link
Copy Markdown
Owner

@pilipilisbot pilipilisbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Owner review: approved.

I checked the SSE client behavior against the backend event stream contract. Keeping the EventSource open on onerror lets the browser reconnect, and the new refresh filter still covers the session events that mutate job state (claimed, dispatch lifecycle, and terminal statuses). The high-volume stdout/stderr events continue to update the live activity/transcript caches directly without forcing a job refetch per line.

Validation I ran on commit d2e034c:

  • npm test -- --run in dashboard (8 passed)
  • npm run build in dashboard
  • PYTHONPATH=src /home/openclaw/.openclaw/workspace/github/github-agent-bridge/.venv/bin/python -m pytest -q tests/test_backend.py tests/test_queue.py tests/test_executor.py (54 passed)
  • git diff --check origin/main...HEAD

GitHub Actions are also green for dashboard, pytest (3.11), and pytest (3.12).

@ecarreras ecarreras merged commit 3f56868 into pilipilisbot:main May 29, 2026
3 checks passed
@pilipilisbot
Copy link
Copy Markdown
Owner

Post-merge cleanup completed for PR #57.

  • Removed the clean dedicated worktree /home/openclaw/.openclaw/workspace/github/github-agent-bridge-pr57.
  • Left the canonical checkout in place and fast-forwarded main to 19f3a7a, matching origin/main.
  • Confirmed only the canonical worktree remains and it is clean.
  • Post-merge GitHub Actions for merge commit 3f56868 are green (tests and release), and release v0.20.1 is published: https://github.com/pilipilisbot/github-agent-bridge/releases/tag/v0.20.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants