Skip to content

Conversation

@charlesvien
Copy link
Contributor

@charlesvien charlesvien commented Jan 7, 2026

https://www.electronjs.org/docs/latest/tutorial/online-offline-events#overview

  • Trust net.isOnline() for offline detection, only HTTP verify when transitioning from offline to online
  • Allow the agent to auto reconnect and continue when connectivity comes back up but we need to handle session state properly here
  • When the users internet cuts out we will display a connectivity error page through App.tsx, when connectivity is restored it will re-render the app and all state. We may want to still allow the user to navigate around the app while they are offline, which is more of an undertaking than what I have prototyped here (so far, still thinking, draft)
    • E.g. I imagine they might be able to get some work done locally but may want the conversational history in Array to reference while they are offline? Something like when you're on a plane but have no wifi because Delta sucks and want to carry on working.

Related: #309

@charlesvien charlesvien changed the title feat: add concurrency controls and consistent formatting to CI workflows feat: Offline/online connectivity service and support Jan 7, 2026
@charlesvien charlesvien changed the title feat: Offline/online connectivity service and support feat: Offline/online connectivity service and handling Jan 7, 2026
@charlesvien charlesvien changed the title feat: Offline/online connectivity service and handling feat: Connectivity service and handling + Agent subprocess handling improvements Jan 7, 2026
@charlesvien charlesvien force-pushed the 01-06-offline_online_connectivity_check_and_page branch 5 times, most recently from d2f5166 to e0b95fc Compare January 7, 2026 21:57
@charlesvien charlesvien changed the base branch from main to graphite-base/407 January 7, 2026 21:57
@charlesvien charlesvien changed the base branch from graphite-base/407 to 01-07-minor_ci_changes January 7, 2026 21:57
Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@charlesvien charlesvien force-pushed the 01-06-offline_online_connectivity_check_and_page branch from e0b95fc to 8a95c57 Compare January 8, 2026 04:57
@charlesvien charlesvien changed the base branch from 01-07-minor_ci_changes to 01-07-improve_agent_subprocess_error_handling January 8, 2026 04:58
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