Skip to content

fix: preserve remote connection state before status refresh - #7

Open
Marlos001 wants to merge 1 commit into
soojy:mainfrom
Marlos001:fix/remote-receipts-20261005
Open

Marlos001 wants to merge 1 commit into
soojy:mainfrom
Marlos001:fix/remote-receipts-20261005

Conversation

@Marlos001

@Marlos001 Marlos001 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

After saving a remote connection, the UI cleared its snapshot before the next status poll and temporarily showed local mode and setup controls. Connection receipts now include an explicit mode, URL and client-key availability, and the QML applies them immediately while preserving stale-receipt checks and per-connection cache resets.

Both local setup controls remain hidden in remote mode. Switching back to local retains the remote URL for editing. Receipts use an explicit nonsecret field allowlist and never contain management or client keys.

Validation: all 178 tests passed without skips in each isolated v7.2.154 and v8.0.13 lane with Codex 0.160.0, plus JavaScript checks, plugin manifest validation, Qt 6 QML parsing and native fixture smoke. The focused 19 remote tests assert transition metadata, exact allowed receipt fields and absence of credential values. Rendered remote-connected and management-only fixtures were inspected. Both hosted compatibility lanes passed on commit 0fb9ffd860de7c4643b18093457bbc6ed8d25634: https://github.com/soojy/omaproxy/actions/runs/37306953471. Live installation was unchanged.

This is the remaining UI slice from DigitalPals#1 after the broader remote support and security fixes were merged via #2. It targets current upstream main and does not resubmit those merged changes.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e98e6c46-83b9-4a68-b5e1-4e68854cfaf2
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant