Configure hermes-webui to connect to existing Hermes gateway#47
Merged
Conversation
Add HERMES_WEBUI_GATEWAY_BASE_URL as a required field and hardcode HERMES_WEBUI_CHAT_BACKEND=gateway so the UI shares sessions with the user's already-running Hermes Agent rather than spinning up an isolated in-process agent. https://claude.ai/code/session_01Xwf5nZf6d1npMerNteodq2
There was a problem hiding this comment.
Pull request overview
This PR updates the hermes-webui app definition so the WebUI connects to an already-running Hermes Agent gateway (instead of running an embedded/in-process agent), allowing sessions/history to be shared across interfaces.
Changes:
- Adds a required
HERMES_WEBUI_GATEWAY_BASE_URLconfiguration field and wires it into the container environment. - Forces
HERMES_WEBUI_CHAT_BACKEND=gatewayso the WebUI uses the gateway-backed session store. - Updates the app’s
description.mdto document the gateway-connected setup and persistence expectations.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| apps/hermes-webui/metadata/description.md | Updates end-user documentation to reflect gateway-connected behavior and persistence notes. |
| apps/hermes-webui/docker-compose.json | Adds env wiring for gateway mode (HERMES_WEBUI_CHAT_BACKEND, HERMES_WEBUI_GATEWAY_BASE_URL). |
| apps/hermes-webui/config.json | Bumps tipi_version and adds a required “Gateway URL” form field to collect the gateway base URL. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+27
to
+31
| { | ||
| "type": "text", | ||
| "label": "Gateway URL", | ||
| "hint": "URL of your existing Hermes Agent gateway (e.g. http://192.168.1.10:8642)", | ||
| "required": true, |
| ], | ||
| "description": "Hermes WebUI is a lightweight, dark-themed web interface for Hermes Agent. It provides near-complete parity with the CLI experience via a three-panel layout with session management, chat, and workspace file browsing. Use it in any browser, or pair it with the Hermes Agent Mobile iOS app by setting a password.", | ||
| "tipi_version": 1, | ||
| "tipi_version": 2, |
- updated_at bumped to reflect config changes (tipi_version 2) - description now mentions gateway connection to set correct expectations https://claude.ai/code/session_01Xwf5nZf6d1npMerNteodq2
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
HERMES_WEBUI_GATEWAY_BASE_URLas a required form field — the URL of the user's already-running Hermes Agent gatewayHERMES_WEBUI_CHAT_BACKEND=gatewayso the UI shares sessions/history with the existing agent rather than spinning up an isolated in-process agentWhy
The default single-container mode runs an embedded in-process agent, completely isolated from any existing Hermes Agent instance. This change makes the WebUI act as a second frontend to the existing gateway instead.
Test plan
bun testpasses (54 tests, verified locally)https://claude.ai/code/session_01Xwf5nZf6d1npMerNteodq2
Generated by Claude Code