Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

@ammar-agent ammar-agent commented Dec 1, 2025

Status URLs now persist through page reloads and compactions via localStorage.

Changes

  • StreamingMessageAggregator takes optional workspaceId in constructor
  • Saves lastStatusUrl to localStorage keyed by workspace when set
  • Loads on construction so URL survives history rewrites

Regression introduced in cc64299.

Generated with mux

@ammar-agent ammar-agent force-pushed the fix-status-url-persistence branch 3 times, most recently from 3379cf8 to 964f87f Compare December 1, 2025 19:22
@ammar-agent ammar-agent changed the title 🤖 fix: persist status_url when loading historical messages 🤖 fix: persist status URL across sessions via localStorage Dec 1, 2025
@ammar-agent ammar-agent force-pushed the fix-status-url-persistence branch from 964f87f to dca6e77 Compare December 1, 2025 19:46
The loadHistoricalMessages function was only processing the most recent
assistant message, which meant that lastStatusUrl wasn't populated from
earlier messages. This caused the URL to be lost when:

1. User sends a message with status_set that includes a URL
2. User sends another message with status_set WITHOUT a URL
3. Page reloads or workspace switches

Now we scan ALL historical messages in chronological order to build up
lastStatusUrl before processing the final state.

Additionally, status URLs now persist through compaction via localStorage:
- Frontend persists lastStatusUrl to localStorage keyed by workspaceId
- Loaded on aggregator construction, survives history rewrite
- Backend has no knowledge of status URLs (clean separation)

Regression introduced in cc64299 by ammar-agent.

_Generated with `mux`_
@ammar-agent ammar-agent force-pushed the fix-status-url-persistence branch from dca6e77 to 1f9e13f Compare December 1, 2025 19:50
@ammario ammario merged commit 99d3070 into main Dec 1, 2025
13 checks passed
@ammario ammario deleted the fix-status-url-persistence branch December 1, 2025 20:02
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.

2 participants