Skip to content

fix: polish demo favicon and replay navigation#393

Merged
tobytkershaw merged 5 commits intomainfrom
codex/demo-favicon-sse
Mar 12, 2026
Merged

fix: polish demo favicon and replay navigation#393
tobytkershaw merged 5 commits intomainfrom
codex/demo-favicon-sse

Conversation

@tobytkershaw
Copy link
Contributor

Summary

  • use the existing demo favicon on both live and replay pages instead of relying on the browser fallback request
  • close the live EventSource intentionally when leaving for replay so navigation does not log an SSE reconnect warning

Testing

  • cd packages/agentvault-demo-ui && npm run build
  • browser verification with Playwright CLI against ./run-demo.sh:
    • favicon.svg served with 200
    • navigating from live view to replay produced 0 console warnings

Closes #391
Closes #392

@tobytkershaw tobytkershaw merged commit ad64655 into main Mar 12, 2026
0 of 2 checks passed
@tobytkershaw tobytkershaw deleted the codex/demo-favicon-sse branch March 12, 2026 10:10
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.

Demo UI logs SSE reconnect warning when navigating from live view to replay Demo UI serves no favicon.ico

1 participant