lhumina_code/hero_browser_mcp
26
0
Fork 0
Code Issues 2 Pull requests 1 Projects Releases Packages Wiki Activity Actions

development_user_session_remote_view #8

Open
peter wants to merge 2 commits from development_user_session_remote_view into development
pull from: development_user_session_remote_view
merge into: lhumina_code:development
Conversation 0 Commits 2 Files changed 7 +726 -9
peter commented 2026-02-25 09:04:04 +00:00
Member
Copy link

full multi-user session support, a real-time screenshot streaming endpoint (SSE), session visibility APIs, and skill compliance improvements required for hero ecosystem integration.
The implementation has been fully tested (unit + integration + live multi-user validation on Ubuntu 24.04).

full multi-user session support, a real-time screenshot streaming endpoint (SSE), session visibility APIs, and skill compliance improvements required for hero ecosystem integration. The implementation has been fully tested (unit + integration + live multi-user validation on Ubuntu 24.04).
- Add GET /health endpoint (hero_sockets skill requirement)
   - Add CORS via tower-http CorsLayer::permissive() (web_embed skill requirement)
   - Add GET /api/sessions/{browser_id}/pages/{page_id}/stream SSE endpoint
     - query param fps (default 2, max 10)
     - streams base64-encoded PNG frames as text/event-stream
     - auto-stops when browser/page is gone or client disconnects
   - Detect missing DISPLAY/WAYLAND_DISPLAY on Linux and fall back to
     headless mode with a tracing::warn instead of crashing
   - Update admin UI with Stream button per page and stream-modal viewer
Add multi-user session test to rhaiexamples
All checks were successful
Test / test (pull_request) Successful in 1m20s
Details
Build and Test / build (pull_request) Successful in 3m2s
Details
dd92f2150e 
- New rhaiexamples/multi_user_session_test.rhai: 20 checks across 8 phases
  testing multi-user session support (browser isolation, cross-session
  protection, user independence, concurrent screenshots)
- Update rhaiexamples/README.md with documentation and expected output

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
All checks were successful
Test / test (pull_request) Successful in 1m20s
Details
Build and Test / build (pull_request) Successful in 3m2s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin development_user_session_remote_view:development_user_session_remote_view
git switch development_user_session_remote_view

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch development
git merge --no-ff development_user_session_remote_view
git switch development_user_session_remote_view
git rebase development
git switch development
git merge --ff-only development_user_session_remote_view
git switch development_user_session_remote_view
git rebase development
git switch development
git merge --no-ff development_user_session_remote_view
git switch development
git merge --squash development_user_session_remote_view
git switch development
git merge --ff-only development_user_session_remote_view
git switch development
git merge development_user_session_remote_view
git push origin development
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_browser_mcp!8
No description provided.