Skip to content

feat: allow disabling automatic RDP reconnection - #35

Merged
LukeGus merged 2 commits into
Termix-SSH:dev-1.4.0from
ZacharyZcR:feat/disable-auto-reconnect-30
Nov 7, 2026
Merged

LukeGus merged 2 commits into
Termix-SSH:dev-1.4.0from
ZacharyZcR:feat/disable-auto-reconnect-30

Conversation

@ZacharyZcR

Copy link
Copy Markdown
Member

An unattended Termix tab can reconnect after its RDP session is lost and interrupt a session opened from another client. Add an "Automatically reconnect RDP" switch in the host editor's Connection section, enabled by default to preserve existing behavior.

When disabled, session loss stays on the disconnected screen until the user reconnects manually. Persist the per-host setting, retain explicit false on import, and leave other protocols unchanged. The setting controls reconnection after session loss; it does not infer server disconnect reasons.

Closes #30.

Validation: 450 tests passed, 1 skipped; clean npm ci, typecheck, build, manifest validation, pack and formatting passed. Component regressions verify no new token after session loss when disabled, manual retry, default-enabled behavior and VNC isolation. Host editor/form tests cover the actual toggle, saved settings and import. The opt-out regression fails against the original component. Real Windows RDP server acceptance remains unverified.

@termix-triage-bot

Copy link
Copy Markdown

AI review | Looks safe to merge

Adds a per-host toggle to disable automatic RDP reconnection, preserving existing behavior by default. The implementation correctly scopes the setting to RDP, handles form state and imports safely, and includes comprehensive tests.

Reviewed dfc73a8. AI can be wrong, a maintainer still decides. Comment /termix review to run it again.

# Conflicts:
#	locales/en.json
@LukeGus
LukeGus merged commit 4d38bf3 into Termix-SSH:dev-1.4.0 Nov 7, 2026
1 check failed
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