Skip to content

fix: resolve recent activity hosts by ID - #3

Merged
LukeGus merged 2 commits into
Termix-SSH:dev-1.1.2from
ZacharyZcR:fix/recent-activity-host-id
Nov 7, 2026
Merged

LukeGus merged 2 commits into
Termix-SSH:dev-1.1.2from
ZacharyZcR:fix/recent-activity-host-id

Conversation

@ZacharyZcR

@ZacharyZcR ZacharyZcR commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Recent activity in the command palette resolves its historical host name against the current host list. Renaming a host, reusing its old name or having duplicate names can therefore open a different host. A deleted host can also redirect to a same-name replacement.

Resolve activity by host ID and pass the selected host through to tab opening. Missing hosts do not open a replacement, and singleton tab routing is preserved.

Related: Termix-SSH/Plugin-SSH-Terminal#53. This fixes a reproduced command-palette defect in Classic Layout; the dashboard card already resolves by ID, so it does not establish that every interaction described in that report is resolved.

Validation: regressions cover renamed, duplicate-name and deleted hosts. All 444 tests, typecheck, build, manifest validation and formatting passed.

The SDK lockfile is synchronized. A fresh npm ci followed by tests, typecheck, build, validation, pack and formatting passes.

@termix-triage-bot

termix-triage-bot Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

AI review | Looks safe to merge

The change updates recent activity host resolution to use IDs instead of names, preventing incorrect host selection when names are reused or changed. It includes tests for the new behavior and maintains backward compatibility, with no security or stability risks.

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

@LukeGus
LukeGus merged commit bceecd1 into Termix-SSH:dev-1.1.2 Nov 7, 2026
2 checks passed
LukeGus added a commit that referenced this pull request Nov 7, 2026
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