Repository navigation
Add remote CLIProxyAPI connection settings - #2
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 44 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (8)
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. Comment |
# Conflicts: # BarWidget.qml # scripts/omaproxy.py
soojy
left a comment
soojy
left a comment
There was a problem hiding this comment.
Reviewed remote connection URL validation, redirect/proxy refusal, credential storage and output allowlists, authentication-failure backoff, and per-command connection binding. Resolved conflicts with current main while preserving native controls and the existing backend pin. Added isolation for named-key registries and quota-alert state, cleared cached panels/forms on connection changes, guarded stale clipboard receipts, blocked local repair in remote mode, and made saved client-key removal explicit. Regression tests cover connection-specific registries and alerts.
The final tree independently passes all 178 Python tests with zero skips against each of CLIProxyAPI 7.2.154 and 8.0.13 using Codex 0.160.0. JavaScript, manifest and QML gates pass. Native smoke validation passes with fake accounts, including remote/management-only switching, provider edits, named keys, diagnostics, alerts and concealed reopening. Bandit findings were manually reviewed; no additional confirmed source-code blocker was identified. The separate backend dependency advisory findings remain unresolved under #5; this PR does not upgrade the backend.
3 participants
Add Settings → Connection → Remote so users can connect to an existing CLIProxyAPI server without installing a local backend.
Closes #1.
Configure and test the server URL, management key, and optional client API key in Settings.
Display remote accounts, quotas, and models, with API connection health and controls appropriate to remote mode. New-account sign-in opens the server's management panel.
Preserve local mode, store keys privately, support verified HTTPS or loopback HTTP, and isolate cached results when switching connections.
Use the standard neutral bar icon color without a status dot.
Validation: 60 Python tests pass; six optional real-backend integration tests also passed before removing the local test backend. JavaScript checks, manifest validation, QML formatting, and diff checks pass. Tested the Settings form, connection submission, account/quota display, and offline behavior in the actual Omarchy shell.