Skip to content

fix(ui): add memory and communicate service proxy routes#930

Open
geoffjay wants to merge 2 commits intomainfrom
issue-929
Open

fix(ui): add memory and communicate service proxy routes#930
geoffjay wants to merge 2 commits intomainfrom
issue-929

Conversation

@geoffjay
Copy link
Copy Markdown
Owner

@geoffjay geoffjay commented Apr 2, 2026

Add proxy routes for the memory and communicate services to the UI server, and update the React frontend to use relative proxy paths instead of direct localhost URLs. This fixes the /memories page silently failing due to port mismatch (17008 vs 7008).

Closes #929

@geoffjay geoffjay added the review-agent Used to invoke a review by an agent tracking this label label Apr 2, 2026
@geoffjay
Copy link
Copy Markdown
Owner Author

geoffjay commented Apr 2, 2026

This change is part of the following stack:

Change managed by git-spice.

- Add memory_service_url and communicate_service_url fields to UiConfig
- Load AGENTD_MEMORY_SERVICE_URL (default http://localhost:7008) and
  AGENTD_COMMUNICATE_SERVICE_URL (default http://localhost:7010) from env
- Add memory_url and communicate_url fields to ProxyState
- Add proxy_memory and proxy_communicate handler functions
- Register /api/memory/{*path} and /api/communicate/{*path} routes
- Update React frontend serviceConfig to use relative proxy paths
- Fix base.ts buildUrl and openWebSocket to handle relative /api/... URLs
  via window.location.origin (fixes 'Invalid URL' error)
- Fix inline WS URL helpers in hooks/components for relative base URLs
- Add memory and communicate dev proxy entries to vite.config.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-agent Used to invoke a review by an agent tracking this label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(ui): Add memory and communicate service proxy routes to UI server

1 participant