show cloud backend check results - #420
Conversation
mcthesw
changed the title
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 443cfbccdb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ) -> Result<(), String> { | ||
| ) -> Result<CloudBackendCheckReport, String> { | ||
| info!(target:"rgsm::ipc", "Checking cloud backend: {:?}", session.backend.clone().sanitize()); | ||
| match svc(&app_handle).check_cloud_backend(&session).await { |
There was a problem hiding this comment.
AGENTS.md specifically says apps/rgsm-gui/src-tauri/src/ipc_handler.rs is a thin export layer and commands should stay 1-3 lines while delegating to services/domain modules. This change moves outcome-specific control flow and logging into the IPC command, so future backend-check behavior is now split between the service and GUI export layer; please move this reporting/logging decision into a service/domain helper and keep the command as a small delegation wrapper.
Useful? React with 👍 / 👎.
1 participant
Fixes #417.
Adds structured backend check results and shows them in Sync settings.